Add method ITestTreeView.EnableTestFilter(bool enable) to enable/disable all filter all at once

Author: rowo360

src/TestCentric/testcentric.gui/Presenters/DisplayStrategy.cs

@@ -75,8 +75,7 @@ public bool HasResults
         public void OnTestUnloaded()
         {
             ClearTree();
-            _view.OutcomeFilter.Enabled = false;
-            _view.TextFilter.Enabled = false;
+            _view.EnableTestFilter(false);
         }
 
         public virtual void OnTestFinished(ResultNode result)

src/TestCentric/testcentric.gui/Presenters/NUnitTreeDisplayStrategy.cs

@@ -49,8 +49,7 @@ public override void OnTestLoaded(TestNode testNode, VisualState visualState)
             else
                 SetDefaultInitialExpansion();
 
-            _view.OutcomeFilter.Enabled = true;
-            _view.TextFilter.Enabled = true;
+            _view.EnableTestFilter(true);
         }
 
         protected override VisualState CreateVisualState() => new VisualState("NUNIT_TREE", _settings.Gui.TestTree.ShowNamespace).LoadFrom(_view.TreeView);

src/TestCentric/testcentric.gui/Views/ITestTreeView.cs

@@ -57,6 +57,8 @@ public interface ITestTreeView : IView
 
         void SetTestFilterVisibility(bool visible);
 
+        void EnableTestFilter(bool enable);
+
         // Tree-related Methods
         void Clear();
         void Add(TreeNode treeNode);

src/TestCentric/testcentric.gui/Views/TestTreeView.cs

@@ -211,6 +211,12 @@ public void SetTestFilterVisibility(bool isVisible)
             filterTextToolStrip.Visible = isVisible;
         }
 
+        public void EnableTestFilter(bool enable)
+        {
+            filterToolStrip.Enabled = enable;
+            filterTextToolStrip.Enabled = enable;
+        }
+
         public void LoadAlternateImages(string imageSet)
         {
             string[] imageNames = { "Skipped", "Inconclusive", "Success", "Ignored", "Failure" };

src/TestCentric/tests/Presenters/NUnitTreeDisplayStrategyTests.cs

@@ -360,7 +360,7 @@ public void OnTestLoaded_SetupFixtureNode_ContainingTwoNamespaces_AreNotFolded()
         }
 
         [Test]
-        public void OnTestLoaded_OutcomeFilter_IsEnabled()
+        public void OnTestLoaded_TestFilters_AreEnabled()
         {
             // Arrange
             string xml =
@@ -371,42 +371,17 @@ public void OnTestLoaded_OutcomeFilter_IsEnabled()
             _strategy.OnTestLoaded(new TestNode(xml), null);
 
             // Assert
-            _view.OutcomeFilter.Received().Enabled = true;
+            _view.Received().EnableTestFilter(true);
         }
 
         [Test]
-        public void OnTestUnloaded_OutcomeFilter_IsDisabled()
+        public void OnTestUnloaded_TestFilters_AreDisabled()
         {
             // Arrange + Act
             _strategy.OnTestUnloaded();
 
             // Assert
-            _view.OutcomeFilter.Received().Enabled = false;
-        }
-
-        [Test]
-        public void OnTestLoaded_TextFilter_IsEnabled()
-        {
-            // Arrange
-            string xml =
-                "<test-suite type='Assembly' id='1-1030' name='Library.Test.dll'>" +
-                "</test-suite>";
-
-            // Act
-            _strategy.OnTestLoaded(new TestNode(xml), null);
-
-            // Assert
-            _view.TextFilter.Received().Enabled = true;
-        }
-
-        [Test]
-        public void OnTestUnloaded_TextFilter_IsDisabled()
-        {
-            // Arrange + Act
-            _strategy.OnTestUnloaded();
-
-            // Assert
-            _view.TextFilter.Received().Enabled = false;
+            _view.Received().EnableTestFilter(false);
         }
     }