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 221478 - recent changes broke subversion tests - http://vcs-test.cz.oracle.com:8080/job/vesioning/
Summary: recent changes broke subversion tests - http://vcs-test.cz.oracle.com:8080/jo...
Status: RESOLVED FIXED
Alias: None
Product: platform
Classification: Unclassified
Component: Data Systems (show other bugs)
Version: 7.3
Hardware: PC Linux
: P2 normal (vote)
Assignee: Jaroslav Tulach
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-11-05 11:47 UTC by Ondrej Vrabec
Modified: 2012-11-07 03:43 UTC (History)
0 users

See Also:
Issue Type: DEFECT
Exception Reporter:

Attachments
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 Ondrej Vrabec 2012-11-05 11:47:37 UTC 
Product Version = NetBeans IDE Dev (Build 20121105-3d7ccfa4f0f2)
Operating System = Linux version 3.1.10-1.16-desktop running on amd64
Java; VM; Vendor = 1.7.0_06
Runtime = Java HotSpot(TM) 64-Bit Server VM 23.2-b09

Our tests started to fail about 3 days ago: http://vcs-test.cz.oracle.com:8080/job/vesioning/. The following test-case does not work now :
1) Create a file at ./FILE
2) Move the file (via DataObject.move) to ./folder/FILE
3) Move the file (via DataObject.move) back to ./FILE
The test no fails with an assertion at:
> at org.openide.loaders.DataObjectPool.changePrimaryFile(DataObjectPool.java:868)
> at org.openide.loaders.DataObjectPool.access$1000(DataObjectPool.java:60)
> at org.openide.loaders.DataObjectPool$Item.changePrimaryFile(DataObjectPool.java:992)
> at org.openide.loaders.DataObject$2Op.run(DataObject.java:762)
> at org.openide.loaders.DataObject$1WrapRun.run(DataObject.java:907)
> at org.openide.loaders.DataObjectPool$1WrapAtomicAction.run(DataObjectPool.java:261)
> at org.openide.filesystems.EventControl.runAtomicAction(EventControl.java:127)
> at org.openide.filesystems.FileSystem.runAtomicAction(FileSystem.java:607)
> at org.openide.loaders.DataObjectPool.runAtomicAction(DataObjectPool.java:273)
> at org.openide.loaders.DataObject.invokeAtomicAction(DataObject.java:927)
> at org.openide.loaders.DataObject.move(DataObject.java:767)
> at org.netbeans.modules.subversion.InteceptorTest.moveDO(InteceptorTest.java:4350)
> at org.netbeans.modules.subversion.InteceptorTest.moveA2B2A_DO(InteceptorTest.java:2509)

I have no idea what #125633c664ad is supposed to do, but maybe the assertion should be changed to: assert prev == null || prev == item; ??
Comment 1 Jaroslav Tulach 2012-11-05 13:03:31 UTC 
Get the fix as ergonomics#2b1d5c897acc
Comment 2 Quality Engineering 2012-11-07 03:43:12 UTC 
Integrated into 'main-golden', will be available in build *201211070001* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress)
Changeset: http://hg.netbeans.org/main-golden/rev/2b1d5c897acc
User: Jaroslav Tulach <jtulach@netbeans.org>
Log: #221478: Less strict assert