Re: Review of java.hints/test/unit/data/

2017-10-23 Thread Geertjan Wielenga
Right. OK. Would be great to update the Rat report for the unit test files in schema2beans and java.hints. Are the tests in java.hints.declarative also impacted? https://github.com/apache/incubator-netbeans/pull/180/commits/15b8f859cd3b2dccba017ae7b13b542666579953 Gj On Mon, Oct 23, 2017 at

Re: Review of java.hints/test/unit/data/

2017-10-23 Thread Jan Lahoda
Some golden files include line numbers, like: java.hints/test/unit/data/goldenfiles/org/netbeans/modules/java/hints/infrastructure/ErrorHintsProviderTest/testShortErrors2.pass Other contain a modified copy of an input file, like:

Re: Review of java.hints/test/unit/data/

2017-10-23 Thread Geertjan Wielenga
What's the reason the test will fail if headers are added? Gj On Mon, Oct 23, 2017 at 5:59 PM, Geertjan Wielenga < geertjan.wiele...@googlemail.com> wrote: > It seems to me that files in java.hints/test/unit/data/ are all "test data > for which the addition of a source header would cause the

Re: Review of java.hints/test/unit/data/

2017-10-23 Thread Geertjan Wielenga
It seems to me that files in java.hints/test/unit/data/ are all "test data for which the addition of a source header would cause the tests to fail" -- i.e., these do not require a license header ( https://www.apache.org/legal/src-headers.html). I think these should be included in the Rat report

Re: Review of java.hints/test/unit/data/

2017-10-23 Thread Jan Lahoda
On Mon, Oct 23, 2017 at 12:41 PM, Geertjan Wielenga < geertjan.wiele...@googlemail.com> wrote: > Hi all, > > There's about 230 unlicensed Java source files in > java.hints/test/unit/data. > > Before I manually add Apache licenses to all of them manually, is there a > reason why these weren't

Review of java.hints/test/unit/data/

2017-10-23 Thread Geertjan Wielenga
Hi all, There's about 230 unlicensed Java source files in java.hints/test/unit/data. Before I manually add Apache licenses to all of them manually, is there a reason why these weren't licensed to begin with, i.e., will tests fail if a license is added to the Java source files? Thanks, Gj