dan-s1 commented on code in PR #7221:
URL: https://github.com/apache/nifi/pull/7221#discussion_r1183943055


##########
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/test/java/org/apache/nifi/processors/standard/TestValidateJson.java:
##########
@@ -109,6 +117,8 @@ void testPatternSchemaCheck() {
         runner.assertTransferCount(ValidateJson.REL_VALID, 0);
 
         assertValidationErrors(ValidateJson.REL_INVALID, true);
+        assertEquals(1, runner.getProvenanceEvents().size());
+        assertEquals(ProvenanceEventType.ROUTE, 
runner.getProvenanceEvents().get(0).getEventType());

Review Comment:
   > @MikeThomsen Thank you so much for reviewing and merging my code so 
quickly. I did have a question regarding the number of Provenance events with 
the invalid relationship. There is an attribute created when JSON is invalid 
which previously resulted in a Provenance modified attribute event. Now that I 
added the Provenance route event the modified attribute event is no longer 
there as this unit test indicates there is only one resulting Provenance event. 
Why is that so?
   
   @exceptionfactory Can I get your take on this question?



-- 
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: issues-unsubscr...@nifi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to