All of the current ExecuteStreamCommand tests rely on those jars. - - - - - - Joseph Percivall linkedin.com/in/Percivall e: joeperciv...@yahoo.com
On Thursday, November 12, 2015 5:34 PM, Joe Witt <joe.w...@gmail.com> wrote: i think we should kill those tests which depend on the build of those jars personally. But if the view is to undo the removal of those three classes i can do that. Thanks Joe On Thu, Nov 12, 2015 at 5:32 PM, Tony Kurc <trk...@gmail.com> wrote: > Do you plan to undo the removal? > On Nov 12, 2015 4:46 PM, "Joe Witt" <joe.w...@gmail.com> wrote: > >> well that explains these goofball classes I deleted the other day >> >> https://issues.apache.org/jira/browse/NIFI-1134 >> >> These classes were used to make those Jars. Those jars are used to >> test execute command. We've now removed the source that was floating >> randomly. We need the built to automatically create whatever we >> execute against if we're going to do this. Those tests should be >> replaced by something else. >> >> >> On Thu, Nov 12, 2015 at 3:02 PM, Joe Percivall >> <joeperciv...@yahoo.com.invalid> wrote: >> > Tony, >> > >> > I did a bit of digging through the history and the jars were a part of >> the initial code import so unless if Joe or someone else knows where they >> came from then we may be out of luck. >> > >> > Joe >> > >> > - - - - - - >> > Joseph Percivall >> > linkedin.com/in/Percivall >> > e: joeperciv...@yahoo.com >> > >> > >> > >> > >> > On Wednesday, November 11, 2015 6:23 PM, Tony Kurc <trk...@gmail.com> >> wrote: >> > >> > >> > >> > All, I was code reviewing and something occurred to me. This raised my >> > eyebrow: >> > >> https://github.com/apache/nifi/blob/master/nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/test/java/org/apache/nifi/processors/standard/TestExecuteStreamCommand.java#L63 >> > >> > If I'm reading it right, the test "runs" a jar that we've got in our >> source >> > tree >> > >> > What code made those jars in src/test/resources? >>