Tests failing on Slicer factory that do not fail locally

Development
1

I noticed a while ago that about a dozen tests keep failing on the Slicer dashboard both for Windows and Mac. I ran the tests that I think I’d be responsible for fixing (py_SubjectHierarchyGenericSelfTest, qSlicerModelsModuleWidgetTest*), and all of these pass on my computer. The dashboard does not show any output for these tests for neither platform.

All of these tests actually start Slicer and show the GUI. Can the reason for the failing tests be that the GUI cannot be oponed on the factory? Or any other ideas? Thanks! @jcfr @Sam_Horvath

2

For py_SubjectHierarchyGenericSelfTest I see some test output on the cdash dashboard such as:

Test directory: /var/folders/6y/zp9dkw5d5wg35y8qvn29hpww0000gn/T/Slicer-tmp-kitware/SubjectHierarchyGenericSelfTest
Clear scene
Switching to temporary DICOM database: /var/folders/6y/zp9dkw5d5wg35y8qvn29hpww0000gn/T/Slicer-tmp-kitware/SubjectHierarchyGenericSelfTest/CtkDicomDatabase
QSqlDatabasePrivate::removeDatabase: connection 'SLICER' is still in use, all queries will cease to work.
QSqlDatabasePrivate::addDatabase: duplicate connection name 'SLICER', old connection removed.

Do you not see this output for Mac tests?
I’m also not sure why the Windows test output is usually less descriptive on cdash dashboard. That’s why I always look at the Mac output.

The test also appears to have been failing since April 16th. You seemed to have a discussion with JC about a DICOM browser commit but I can’t tell if anything was acted upon.

3

Same here. Not sure why I didn’t see this.
The log does not show much though, these messages are normal. It still seems to me that the GUI is not constructed.

4

I suspect it if related to remaining files in temporary directory. I will confirm this tomorrow.

5

Just to clarify, this is not normal, it is an error, most probably because multiple database connections are created with the same name. The easiest solution is to not provide a connection name to ctkDICOMDatabase.openDatabase (and then a random connection name is generated).

6

I remember getting this error message regularly in passing tests (an example). So in this sense it is normal.

The tests do not fail because of this, especially that most of these failing tests do not even use DICOM.

7

The test timeout because of the message asking to update the database, as discussed here I suggest test automatically choose one or the other option.

image
image1204×369 105 KB

8

Right, thanks for the reminder. I’ll fix this asap.

Some other tests that fail on the dashboard and pass on my computer (like the Models tests) don’t use DICOM, so they must fail for a different reason.