This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 222811 - Failing org.netbeans.modules.ide.ergonomics tests on Windows and JDK7u10
Summary: Failing org.netbeans.modules.ide.ergonomics tests on Windows and JDK7u10
Status: VERIFIED FIXED
Alias: None
Product: ide
Classification: Unclassified
Component: Extbrowser (show other bugs)
Version: 7.3
Hardware: PC Windows 7
: P1 normal (vote)
Assignee: Martin Entlicher
URL:
Keywords: JDK_SPECIFIC, TEST
: 223014 (view as bug list)
Depends on:
Blocks:
 
Reported: 2012-11-27 11:06 UTC by Jiri Skrivanek
Modified: 2013-02-27 08:12 UTC (History)
2 users (show)

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments
testEnabledWindows - screen shot. (19.79 KB, image/png)
2012-11-27 11:06 UTC, Jiri Skrivanek
Details
testCheckLibrariesReal thread dump. (10.09 KB, text/plain)
2012-11-27 11:07 UTC, Jiri Skrivanek
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Jiri Skrivanek 2012-11-27 11:06:53 UTC
Created attachment 128441 [details]
testEnabledWindows - screen shot.

Tests in org.netbeans.modules.ide.ergonomics fails on Windows and JDK7u10. Test testEnabledWindows opens some strange window and testCheckLibrariesReal is waiting for something (see attachments).

http://hudson4qe.cz.oracle.com/view/Trunk-QE/job/CV-multi/

org.netbeans.modules.ide.ergonomics.CachingPreventsLoadingOfModuleManifestsTest.testEnabledWindows
org.netbeans.modules.ide.ergonomics.DynamicVerifyTest.testCheckLibrariesReal

ant -k -f ide.ergonomics test test-generate-html -Dtest.includes=**/CachingPreventsLoadingOfModuleManifestsTest.class

ant -k -f ide.ergonomics test test-generate-html -Dtest.includes=**/DynamicVerifyTest.class
Comment 1 Jiri Skrivanek 2012-11-27 11:07:30 UTC
Created attachment 128442 [details]
testCheckLibrariesReal thread dump.
Comment 2 Jiri Skrivanek 2012-11-27 11:16:43 UTC
The same thread dump is also for ValidateModulesTest.testPluginDisplay 

ant -k -f o.n.core test test-generate-html -Dtest.includes=**/ValidateModulesTest.class
Comment 3 Jaroslav Tulach 2012-11-27 11:55:06 UTC
Let's start with the dialog. Imho, it should not be shown when -Dnetbeans.fullhack=true - e.g. in tests. Maybe that will solve part of the problem. 

I don't know who shows the dialog, however.
Comment 4 Jaroslav Tulach 2012-11-28 12:53:49 UTC
Need to be re-assigned to the dialog owner, I think.
Comment 5 Petr Jiricka 2012-11-28 14:53:28 UTC
Denis, can you please investigate? This refers to the internal websocket server.
Comment 6 Denis Anisimov 2012-11-28 16:00:10 UTC
Dialog notifies about occupied port which is being tried to use by
WebSocket server. The notification is not a bug.
But there is no need to start server at all in this specific case. ExternalBrowserPlugin had not been used and had not been instantiated
 before it has been called in the Shutdown class as I see from the stacktrace.

The exception comes from Shutdown class whose logic is not needed at all if 
ExternalBrowserPlugin wasn't instantiated previously.
So this is introduced by Martin via this commit :
http://hg.netbeans.org/web-main/rev/2f4580af5554
Comment 7 Martin Entlicher 2012-11-29 10:32:11 UTC
IMHO it's quite unexpected, that initialization of a static field starts the server!

Fixed by changeset:   240890:48bd1c546514
http://hg.netbeans.org/web-main/rev/48bd1c546514
Comment 8 Quality Engineering 2012-11-30 02:39:12 UTC
Integrated into 'main-golden', will be available in build *201211300001* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress)
Changeset: http://hg.netbeans.org/main-golden/rev/48bd1c546514
User: mentlicher@netbeans.org
Log: #222811: Do not use @OnStop, so that we do not start the server.
Comment 9 Jiri Skrivanek 2012-11-30 07:17:06 UTC
*** Bug 223014 has been marked as a duplicate of this bug. ***