Github user pvillard31 commented on the issue: https://github.com/apache/nifi/pull/2560 @MikeThomsen - can you have a look at this commit: https://github.com/pvillard31/nifi/commit/5a67aa152bf2844de6a9192bfb75a073e80a595b I reworked a bit the tests to avoid "removing" a property while executing the tests. Basically, instead of having a shared instance of the TestRunner for all the test suite, I initialize one eache time (to be sure there is no conflict between the tests and to avoid issues in cases tests are not executed in the same order each time). Regarding your comment about the validation failing, it's because we're providing a default value "_id". Because of that the property will always return true to isSet(). Since we're adding a new property and the user needs to use one of the two, I suggest removing the default value: it does not break anything, and it "forces" the user to understand the choice the user is doing. Does that make sense?
---