Hi, I have created NETBEANS-2652 <https://issues.apache.org/jira/browse/NETBEANS-2652> and the corresponding pull request <https://github.com/apache/netbeans/pull/1286>.
Mark On Thu, Jun 6, 2019 at 7:36 PM Geertjan Wielenga <[email protected]> wrote: > We agree and the pull request would be awesome! :-) > > Gj > > On Thu, 6 Jun 2019 at 19:04, Mark Herkrath <[email protected] > .invalid> > wrote: > > > Hi community. > > > > I have written the original mail as per instructions on the NB site. > > JaCoCo is effectively not working with NB 11 unless configured in a > > non-standard way. I have spent some time in analyzing and fixing this > issue > > (for me). I am happy to contribute the fix to everyone, but somebody > > entitled needs to agree. Then I provide a pull request. > > > > Thanks, > > Mark > > > > > Am 23.05.2019 um 18:34 schrieb Mark Herkrath <[email protected] > >: > > > > > > Hi, > > > > > > I am running into issues with JaCoCo in NB 11. It was working well in > NB > > 10. > > > > > > The problem is the following, it got introduced by NETBEANS-1768 - > > "CodeCoverage doesn't work when custom outputDirectory is used in a > > PluginExecution": > > > > > > In the change done, it is tested if the predetermined JaCoCo output > > directory exists. If it doesn't exist, the outputDirectory variable is > > overwritten by the outputDirectory configuration of the JaCoCo report > > goal. However, if this is not defined, the outputDirectory variable > becomes > > null. As a result, the file is being watched via FileChangeAdapter > becomes > > C:\Program Files\NetBeans 11.0\bin\jacoco.xml, or similar. > > > > > > Now, often the Maven target directory, or at least the JaCoCo output > > directory, doesn't exist yet when the Code Coverage report is opened. Due > > to the issue described above, NB doesn't detect when jacoco.xml becomes > > available and claims that there is no data yet. > > > > > > As per my understanding NETBEANS-1768 was fixed incorrectly. The fix > > should have been only to change line 128: > > > outputDirectory = PluginPropertyUtils.getPluginProperty(p, > GROUP_JOCOCO, > > ARTIFACT_JOCOCO, "outputDirectory", null, null); > > > to > > > outputDirectory = PluginPropertyUtils.getPluginProperty(p, > GROUP_JOCOCO, > > ARTIFACT_JOCOCO, "outputDirectory", "report", null); > > > > > > As a result, the outputDirectory setting will be taken from the report > > goal level but it will still picked up correctly if the plug > configuration > > is done at plugin level for example. > > > > > > I have commented about this issue at NETBEANS-1997, which was reopened > > by someone else. Possibly it would be better to create a new ticket. > JaCoCo > > 0.8.3 is working well for me after applying the above change. > > > > > > Cheers, > > > Mark > > >
