Page MenuHomePhabricator

mitkSceneIOTest fails randomly
Closed, ResolvedPublic

Description

This test fails randomly due to a (sometimes) corrupted zip file. Unzipping it fails.

Event Timeline

zelzer added a subscriber: zelzer.

[SVN revision 21306]
COMP (#3336): temporarly disabling mitkSceneIOTest

[SVN revision 21308]
COMP (#3336): temporarly disabling mitkPropertySerializationTest because it fails too. Jochen or Daniel, can you please have a look at it?

Hannes and Diana implemented this test during the last weeks. Could you two please look into this?

@Sascha
On which machines does the test fail. For us on Windows 7 64bit everything went fine so far.

The two tests failed randomly on the VC8 continuous dartclient dartxp_vm_continuous_vc8, but also on mbits (continuous dartclient).

See http://mbits/cdash/index.php?project=MITK&date=2010-02-12

You could try to enable the tests again and see what happens.

Assigning to Daniel. A temporary solution is to check if the file is present after storing. If not do not continue with the test...

[SVN revision 21696]
CHG (#3336): re-enable test to see what happens

[SVN revision 21771]
FIX (#3336): attempt to remove random failures: delete zip file before using it

[SVN revision 21777]
FIX (#3336): attempt to remove random failures: make sure lib is loaded

I hope this is fixed now. We can reopen it again if still something is wrong..

At mbits/~maleike/mbits-SceneIO-cannot-zip.tar.gz is a directory that was created on mbits as /tmp/tmp14528aaaaaa/

The test driver fails with a message:

  1. ERROR: Could not create ZIP file from /tmp/tmp14528aaaaaa

[SVN revision 22497]
FIX (#3336): scene IO and test work now as expected

[SVN revision 22501]
FIX (#3336): reactivate test now that it seems to pass

Test still fails:

http://mbits/cdash/viewTest.php?onlyfailed&buildid=29032

I think this happens when multiple dartclients from different users run on the same machine, since they all use "/tmp/scene.zip" as output file. I suggest using MITK_TEST_OUTPUT_DIR instead, which is defined in mitkTestingConfig.h .

(In reply to comment #16)

Test still fails:

http://mbits/cdash/viewTest.php?onlyfailed&buildid=29032

I think this happens when multiple dartclients from different users run on the
same machine, since they all use "/tmp/scene.zip" as output file. I suggest
using MITK_TEST_OUTPUT_DIR instead, which is defined in mitkTestingConfig.h .

The name issue was fixed in T4121, the cleanup in T4340.

This bug still occures. Take a look at this failed test: http://mbits/cdash/viewTest.php?onlyfailed&buildid=38009

After the failed test I started another run of the dart client without any changes of the code and the test succeeded. So I assume this is still the bug which was described here.

I also took a look at the dart client if the harddisc was full which may also cause such an error but there was enough space left.

Settings this to ASSIGNED to make Bugzilla quiet. However, I cannot look into this. The problem here is some temporary file creation. This code could be cleaned up and filled with some additional tests for the temp file:

  • does the directory exist?
  • is the directory writable?
  • can a simple test file be created and deleted inside the directory?

If the test gives this information as additional output, it would be easier to find the reason for the failure.

Testing bugs are part of their respective modules -> changing component to other, please assign appropriate component.