dan-s1 commented on code in PR #11361:
URL: https://github.com/apache/nifi/pull/11361#discussion_r3452842976
##########
nifi-extension-bundles/nifi-standard-bundle/nifi-standard-processors/src/test/java/org/apache/nifi/processors/standard/TestFlattenJson.java:
##########
@@ -65,7 +65,7 @@ void testFlatten() throws JsonProcessingException {
}
}""";
final Map parsed = (Map) baseTest(testRunner, json, 2);
- assertEquals(parsed.get("test.msg"), "Hello, world", "test.msg should
exist, but doesn't");
+ assertEquals("Hello, world", parsed.get("test.msg"), "test.msg should
exist, but doesn't");
Review Comment:
As described in the ticket, I discovered these using Intellij's Analyze tool
with the inspection name `Misordered 'assertEqual' arguments`. I ran this
before submitting this PR and now again after the changes you had requested and
its not seeing any other violations.
--
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]
