BUG: Fix ctkDICOMBrowserTester checking for use of native file dialog

Follow-up of c4f1eee ("ENH: Update ctkFileDialog to use native dialog by default", 2021-06-10). Co-authored-by: Jean-Christophe Fillion-Robin <jchris.fillionr@kitware.com>
commontk · Jan 13, 2024 · a30c3be · a30c3be
1 parent 7b00d52
commit a30c3be
Showing 1 changed file with 13 additions and 5 deletions.
diff --git a/Libs/DICOM/Widgets/Testing/Cpp/ctkDICOMBrowserTest.cpp b/Libs/DICOM/Widgets/Testing/Cpp/ctkDICOMBrowserTest.cpp
@@ -149,13 +149,21 @@ void ctkDICOMBrowserTester::testImportDirectoryMode()
   browser.setImportDirectoryMode(ctkDICOMBrowser::ImportDirectoryAddLink);
   QCOMPARE(browser.importDirectoryMode(), ctkDICOMBrowser::ImportDirectoryAddLink);
 
-  QComboBox* comboBox = browser.importDialog()->bottomWidget()->findChild<QComboBox*>();
+  bool usingNativeDialog = !browser.importDialog()->testOption(QFileDialog::DontUseNativeDialog);
+  if (!usingNativeDialog)
+    {
+    QComboBox* comboBox = browser.importDialog()->bottomWidget()->findChild<QComboBox*>();
 
-  comboBox->setCurrentIndex(comboBox->findData(static_cast<int>(ctkDICOMBrowser::ImportDirectoryCopy)));
-  QCOMPARE(browser.importDirectoryMode(), ctkDICOMBrowser::ImportDirectoryCopy);
+    comboBox->setCurrentIndex(comboBox->findData(static_cast<int>(ctkDICOMBrowser::ImportDirectoryCopy)));
+    QCOMPARE(browser.importDirectoryMode(), ctkDICOMBrowser::ImportDirectoryCopy);
 
-  comboBox->setCurrentIndex(comboBox->findData(static_cast<int>(ctkDICOMBrowser::ImportDirectoryAddLink)));
-  QCOMPARE(browser.importDirectoryMode(), ctkDICOMBrowser::ImportDirectoryAddLink);
+    comboBox->setCurrentIndex(comboBox->findData(static_cast<int>(ctkDICOMBrowser::ImportDirectoryAddLink)));
+    QCOMPARE(browser.importDirectoryMode(), ctkDICOMBrowser::ImportDirectoryAddLink);
+    }
+  else
+    {
+    QCOMPARE(browser.importDialog()->bottomWidget(), nullptr);
+    }
 }
 
 // ----------------------------------------------------------------------------