Filter toolbar button is only visible for NUNIT tree format

TestCentric · Dec 17, 2024 · e86d46a · e86d46a
1 parent 721f4a2
commit e86d46a
Show file tree

Hide file tree

Showing 2 changed files with 32 additions and 0 deletions.
diff --git a/src/TestCentric/testcentric.gui/Presenters/TestCentricPresenter.cs b/src/TestCentric/testcentric.gui/Presenters/TestCentricPresenter.cs
@@ -773,6 +773,7 @@ private void UpdateViewCommands(bool testLoading = false)
             _view.DisplayFormatButton.Enabled =
             _view.RunParametersButton.Enabled = testLoaded && !testRunning;
             _view.ShowHideFilterButton.Enabled = testLoaded && _view.DisplayFormat.SelectedItem == "NUNIT_TREE";
+            _view.ShowHideFilterButton.Visible = testLoaded && _view.DisplayFormat.SelectedItem == "NUNIT_TREE";
 
             _view.RunSelectedButton.Enabled = testLoaded && !testRunning && _model.SelectedTests != null && _model.SelectedTests.Any();
 

diff --git a/src/TestCentric/tests/Presenters/Main/CommandTests.cs b/src/TestCentric/tests/Presenters/Main/CommandTests.cs
@@ -16,6 +16,7 @@ namespace TestCentric.Gui.Presenters.Main
     using Views;
     using Model;
     using NSubstitute.Core.Arguments;
+    using System.Windows.Forms;
 
     public class CommandTests : MainPresenterTestBase
     {
@@ -377,5 +378,35 @@ public void ShowFilterChanged_ChangesModelSetting(bool show)
             // Assert
             Assert.That(_model.Settings.Gui.TestTree.ShowFilter, Is.EqualTo(show));
         }
+
+        [TestCase("NUNIT_TREE", true)]
+        [TestCase("FIXTURE_List", false)]
+        public void ShowFilterIsEnabled_ForDisplayFormat(string displayFormat, bool expectedIsEnabled)
+        {
+            // Arrange
+            _model.HasTests.Returns(true);
+            _view.DisplayFormat.SelectedItem = displayFormat;
+
+            // Act
+            _model.Events.SelectedTestsChanged += Raise.Event<TestSelectionEventHandler>(new TestSelectionEventArgs(null));
+
+            // Assert
+            Assert.That(_view.ShowHideFilterButton.Enabled, Is.EqualTo(expectedIsEnabled));
+        }
+
+        [TestCase("NUNIT_TREE", true)]
+        [TestCase("FIXTURE_List", false)]
+        public void ShowFilterIsVisible_ForDisplayFormat(string displayFormat, bool expectedIsVisible)
+        {
+            // Arrange
+            _model.HasTests.Returns(true);
+            _view.DisplayFormat.SelectedItem = displayFormat;
+
+            // Act
+            _model.Events.SelectedTestsChanged += Raise.Event<TestSelectionEventHandler>(new TestSelectionEventArgs(null));
+
+            // Assert
+            Assert.That(_view.ShowHideFilterButton.Visible, Is.EqualTo(expectedIsVisible));
+        }
     }
 }