Skip to content

Commit

Permalink
Fix all the compile warnings and detekt issues (opensearch-project#603)
Browse files Browse the repository at this point in the history
* Fix all the compile warnings and detekt issues

Signed-off-by: bowenlan-amzn <[email protected]>

* Fix time capture is 0

Signed-off-by: bowenlan-amzn <[email protected]>

Signed-off-by: bowenlan-amzn <[email protected]>
(cherry picked from commit a21e4a6)
  • Loading branch information
bowenlan-amzn authored and Angie Zhang committed Dec 9, 2022
1 parent 2a5552f commit cca682b
Showing 1 changed file with 3 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -50,15 +50,13 @@ abstract class ODFERestTestCase : OpenSearchRestTestCase() {
// create adminDN (super-admin) client
val uri = javaClass.classLoader.getResource("security/sample.pem").toURI()
val configPath = PathUtils.get(uri).parent.toAbsolutePath()
// TODO once common utils is updated in maven, we can use this method to define hosts
// SecureRestClientBuilder(settings, configPath, hosts).setSocketTimeout(5000).build()
SecureRestClientBuilder(settings, configPath).setSocketTimeout(5000).build()
SecureRestClientBuilder(settings, configPath).setSocketTimeout(60000).build()
}
false -> {
// create client with passed user
val userName = System.getProperty("user")
val password = System.getProperty("password")
SecureRestClientBuilder(hosts, isHttps(), userName, password).setSocketTimeout(5000).build()
SecureRestClientBuilder(hosts, isHttps(), userName, password).setSocketTimeout(60000).build()
}
}
} else {
Expand All @@ -68,4 +66,4 @@ abstract class ODFERestTestCase : OpenSearchRestTestCase() {
return builder.build()
}
}
}
}

0 comments on commit cca682b

Please sign in to comment.