Skip to content

Conversation

@KacerCZ
Copy link
Contributor

@KacerCZ KacerCZ commented Jan 3, 2021

https://issues.apache.org/jira/browse/NETBEANS-5210

Inspection settings (from Code > Inspect) can be exported from Tools > Options.
Settings are registered under Miscellaneous > Code Inspections.

@KacerCZ
Copy link
Contributor Author

KacerCZ commented Jan 3, 2021

Travis failed becaue it run too long.

@matthiasblaesing
Copy link
Contributor

There are unittest failures. The failing test can be run with ant commit-validation. The results can be then found in <nb-checkout>/nbbuild/build/test/commit-validation/results/html/index.html. I find:

ValidateLayerConsistencyTest#testFolderOrdering

No warnings relating to folder ordering; cf: http://deadlock.netbeans.org/job/nbms-and-javadoc/lastSuccessfulBuild/artifact/nbbuild/build/generated/layers.txt Not all children in OptionsExport/Advanced/ marked with the position attribute: [CssPreprocessors, Databases, Debugger, Diff, Favorites, Files, HTMLPalette, J2EE, JSPPalette, JavaPlatform, ProjectLibraries, Toolbars, outputOptions], but some are: [Inspections]

junit.framework.AssertionFailedError: No warnings relating to folder ordering; cf: http://deadlock.netbeans.org/job/nbms-and-javadoc/lastSuccessfulBuild/artifact/nbbuild/build/generated/layers.txt
Not all children in OptionsExport/Advanced/ marked with the position attribute: [CssPreprocessors, Databases, Debugger, Diff, Favorites, Files, HTMLPalette, J2EE, JSPPalette, JavaPlatform, ProjectLibraries, Toolbars, outputOptions], but some are: [Inspections]
at org.netbeans.core.validation.ValidateLayerConsistencyTest.assertNoErrors(ValidateLayerConsistencyTest.java:157)
at org.netbeans.core.validation.ValidateLayerConsistencyTest.testFolderOrdering(ValidateLayerConsistencyTest.java:806)
at org.netbeans.junit.NbTestCase.access$200(NbTestCase.java:77)
at org.netbeans.junit.NbTestCase$2.doSomething(NbTestCase.java:476)
at org.netbeans.junit.NbTestCase$1Guard.run(NbTestCase.java:402)

@KacerCZ KacerCZ force-pushed the netbeans-5210-export-inspection-settings branch from bd7af1e to abf0264 Compare January 3, 2021 19:20
@KacerCZ
Copy link
Contributor Author

KacerCZ commented Jan 3, 2021

I've fixed layer.xml so test passed locally.
Thank you.

@matthiasblaesing
Copy link
Contributor

This was not tested - I build from source, opened Options, went to Export and directly got:

    [exec] Caused: java.util.MissingResourceException: No such bundle org.netbeans.modules.analysis.ui.resources.Bundle
     [exec] 	at org.openide.util.NbBundle.getBundle(NbBundle.java:426)
     [exec] 	at org.openide.util.NbBundle.getBundle(NbBundle.java:358)
     [exec] [catch] at org.netbeans.core.startup.layers.BinaryFS$AttrImpl.getValue(BinaryFS.java:676)
     [exec] 	at org.netbeans.core.startup.layers.BinaryFS$BFSBase.getAttribute(BinaryFS.java:506)
     [exec] 	at org.openide.filesystems.MultiFileObject.getAttribute(MultiFileObject.java:918)
     [exec] 	at org.openide.filesystems.MultiFileObject.getAttribute(MultiFileObject.java:851)
     [exec] 	at org.openide.filesystems.MultiFileObject.getAttribute(MultiFileObject.java:914)
     [exec] 	at org.openide.filesystems.MultiFileObject.getAttribute(MultiFileObject.java:851)
     [exec] 	at org.openide.filesystems.MultiFileObject.getAttribute(MultiFileObject.java:772)
     [exec] 	at org.netbeans.modules.options.export.OptionsExportModel$Category.getItems(OptionsExportModel.java:626)
     [exec] 	at org.netbeans.modules.options.export.OptionsChooserPanel.createOptionsTreeModel(OptionsChooserPanel.java:431)
     [exec] 	at org.netbeans.modules.options.export.OptionsChooserPanel.access$700(OptionsChooserPanel.java:72)
     [exec] 	at org.netbeans.modules.options.export.OptionsChooserPanel$6.run(OptionsChooserPanel.java:339)
     [exec] ALL [null]: Offending classloader: SystemClassLoader[445 modules]
     [exec] CONFIG [null]: Missing resource from class: org.netbeans.modules.analysis.ui.resources.Bundle
     [exec] SEVERE [global]
     [exec] java.lang.AssertionError: Display name of export option item not defined in layer.
     [exec] 	at org.netbeans.modules.options.export.OptionsExportModel$Category.getItems(OptionsExportModel.java:627)
     [exec] 	at org.netbeans.modules.options.export.OptionsChooserPanel.createOptionsTreeModel(OptionsChooserPanel.java:431)
     [exec] 	at org.netbeans.modules.options.export.OptionsChooserPanel.access$700(OptionsChooserPanel.java:72)
     [exec] [catch] at org.netbeans.modules.options.export.OptionsChooserPanel$6.run(OptionsChooserPanel.java:339)

From a quick look the display name was defined in the wrong bundle.

https://issues.apache.org/jira/browse/NETBEANS-5210

Inspection settings (from Code > Inspect) can be exported from Tools > Options.
Settings are registered under Miscellaneous > Code Inspections.
@KacerCZ KacerCZ force-pushed the netbeans-5210-export-inspection-settings branch from abf0264 to 0d935a1 Compare January 5, 2021 20:43
@KacerCZ
Copy link
Contributor Author

KacerCZ commented Jan 5, 2021

Sorry, that's embarassing :(
I tried export/import but somehow I made error in commiting changes.

@matthiasblaesing
Copy link
Contributor

Looks good to me. Thank you.

@matthiasblaesing matthiasblaesing merged commit d80a1dc into apache:master Jan 6, 2021
@KacerCZ KacerCZ deleted the netbeans-5210-export-inspection-settings branch January 6, 2021 20:01
@KacerCZ
Copy link
Contributor Author

KacerCZ commented Jan 6, 2021

Thanks for review.
Please set milestone to 12.3.

@matthiasblaesing matthiasblaesing modified the milestones: 12.0-u2, 12.3 Jan 6, 2021
@matthiasblaesing
Copy link
Contributor

Done - sorry, did not think about it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants