Skyline: Fix intermittent failure in TestRetentionTimeFilter

ProteoWizard · Oct 10, 2024 · bc93c28 · bc93c28
1 parent 2f9a56f
commit bc93c28
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 5 deletions.
diff --git a/pwiz_tools/Skyline/SettingsUI/PeptideSettingsUI.cs b/pwiz_tools/Skyline/SettingsUI/PeptideSettingsUI.cs
@@ -1206,6 +1206,12 @@ public void ChooseRegression(string name)
             comboRetentionTime.SelectedItem = name;
         }
 
+        public string RetentionTimeRegressionName
+        {
+            get { return comboRetentionTime.SelectedItem.ToString(); }
+            set { comboRetentionTime.SelectedItem = value; }
+        }
+
         public void UseMeasuredRT(bool use)
         {
             cbUseMeasuredRT.Checked = use;

diff --git a/pwiz_tools/Skyline/TestFunctional/RetentionTimeFilterTest.cs b/pwiz_tools/Skyline/TestFunctional/RetentionTimeFilterTest.cs
@@ -162,6 +162,7 @@ protected void TestUsePredictedTime()
                 const string calcName = "TestCalculator";
                 const string regressionName = "TestCalculatorAutoCalcRegression";
                 var peptideSettingsDlg = ShowDialog<PeptideSettingsUI>(SkylineWindow.ShowPeptideSettingsUI);
+                RunUI(() => peptideSettingsDlg.SelectedTab = PeptideSettingsUI.TABS.Prediction);
                 var editIrtDlg = ShowDialog<EditIrtCalcDlg>(peptideSettingsDlg.AddCalculator);
                 RunUI(() =>
                 {
@@ -180,11 +181,9 @@ protected void TestUsePredictedTime()
                     editRtDlg.SetTimeWindow(1.0);
                 });
                 OkDialog(editRtDlg, editRtDlg.OkDialog);
-                RunUI(() =>
-                {
-                    peptideSettingsDlg.ChooseRegression(regressionName);
-                    peptideSettingsDlg.UseMeasuredRT(false);
-                });
+                WaitForConditionUI(() => Equals(regressionName, peptideSettingsDlg.RetentionTimeRegressionName),
+                    () => string.Format("Expected regression name '{0}' not set. Found '{1}' instead.", regressionName, peptideSettingsDlg.RetentionTimeRegressionName));
+                RunUI(() => peptideSettingsDlg.UseMeasuredRT(false));
                 OkDialog(peptideSettingsDlg, peptideSettingsDlg.OkDialog);
                 docBeforeImport = WaitForDocumentChange(docBeforeSettingsChange);
                 AssertEx.AreEqual(calcName, SkylineWindow.Document.Settings.PeptideSettings.Prediction.RetentionTime?.Calculator.Name);