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?
---
