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 42007 - SelectionModeTest failures
Summary: SelectionModeTest failures
Status: CLOSED FIXED
Alias: None
Product: platform
Classification: Unclassified
Component: Window System (show other bugs)
Version: 4.x
Hardware: PC Linux
: P2 blocker (vote)
Assignee: David Simonek
URL:
Keywords: TEST
: 38110 (view as bug list)
Depends on:
Blocks:
 
Reported: 2004-04-15 08:12 UTC by Jaroslav Tulach
Modified: 2008-12-22 20:39 UTC (History)
1 user (show)

See Also:
Issue Type: DEFECT
Exception Reporter:

Attachments
The failures (7.76 KB, text/html)
2004-04-15 08:12 UTC, Jaroslav Tulach 		Details
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Jaroslav Tulach 2004-04-15 08:12:11 UTC 
testSingleSelectionMode	fail	[MODE:
SINGLE_TREE_SELECTION][NODES: two nodes
contiguous] PropertyVetoException was NOT thrown.	 

testContigousSelection	fail	[MODE:
CONTIGUOUS_TREE_SELECTION][NODES: two nodes
discontiguous] PropertyVetoException was NOT thrown.
Comment 1 Jaroslav Tulach 2004-04-15 08:12:39 UTC 
Created attachment 14409 [details]
The failures
Comment 2 Jiri Rechtacek 2004-04-18 16:20:53 UTC 
Maybe a thread problem if selection is preplanned to AWT thread, will
be investigate.
Comment 3 Jiri Rechtacek 2004-04-23 10:46:10 UTC 
Assigned to new owner.
Comment 4 Petr Nejedly 2004-05-28 21:48:48 UTC 
After long debugging, I can only conclude that the problem is in the
interaction of the test and windows system.
The TreeView's addNotify is not called, although the test does call
ExplorerPanel.open() on an EP containing TreeView.
TreeView needs to get addNotify() to work correctly (to be able to
lookup its EM).
It has to do something with new WS, and the following exception
is logged during the test:
java.lang.IllegalStateException: Creating default editor mode.
It shouldn't happen this way
org.netbeans.core.windows.WindowManagerImpl.getDefaultEditorMode(WindowManagerImpl.java:482)
org.netbeans.core.windows.WindowManagerImpl.topComponentOpen(WindowManagerImpl.java:939)
org.openide.windows.TopComponent.open(TopComponent.java:276)
org.openide.explorer.ExplorerPanel.open(ExplorerPanel.java:172)

Note: The code does correctly veto the selection change if set so.
It does no replanning, so Jirka's idea is wrong.
Comment 5 Petr Nejedly 2004-06-17 14:50:15 UTC 
*** Issue 38110 has been marked as a duplicate of this issue. ***
Comment 6 Jaroslav Tulach 2004-07-28 12:53:46 UTC 
cvs -q ci -m "#42007: The test was failing due to interaction with
core/windows. I've disabled the lookup, so default impl of
WindowSystem is used and everything seems to work"
Checking in unit/src/org/openide/explorer/view/SelectionModeTest.java;
/cvs/openide/test/unit/src/org/openide/explorer/view/SelectionModeTest.java,v
 <--  SelectionModeTest.java
new revision: 1.9
Comment 7 Petr Nejedly 2004-07-28 15:14:10 UTC 
Great, it works OK now.
Thanks, Yarda.