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.
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.
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.
I suspect it if related to remaining files in temporary directory. I will confirm this tomorrow.
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).
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.
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.
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.