wzhallright commented on code in PR #5850:
URL: https://github.com/apache/ozone/pull/5850#discussion_r1434642886
##########
hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/om/TestOmMetrics.java:
##########
@@ -324,8 +324,8 @@ public void testKeyOps() throws Exception {
writeClient.commitKey(keyArgs, keySession.getId());
} catch (Exception e) {
//Expected Failure in preExecute due to not enough datanode
- assertTrue(e.getMessage().contains("No enough datanodes to choose"),
- e::getMessage);
+ assertThat(e.getMessage()).withFailMessage(e.getMessage())
+ .contains("No enough datanodes to choose");
Review Comment:
fixed
##########
hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/om/TestKeyManagerImpl.java:
##########
@@ -847,9 +848,8 @@ public void testLookupKeyWithLocation() throws IOException {
// lookup key, random node as client
OmKeyInfo key4 = keyManager.lookupKey(keyArgs, resolvedBucket(),
"/d=default-drack/127.0.0.1");
- assertTrue(
- keyPipeline.getNodes().containsAll(key4.getLatestVersionLocations()
- .getLocationList().get(0).getPipeline().getNodesInOrder()));
+
assertThat(keyPipeline.getNodes()).containsAll(key4.getLatestVersionLocations()
+ .getLocationList().get(0).getPipeline().getNodesInOrder());
Review Comment:
fixed
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
