Merge pull request #1198 from TestCentric/issue-1193

TestResultSubView: change image set - fix missing call to WireUpEvents()

Author: rowo360

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

@@ -34,6 +34,7 @@ public TestResultSubViewPresenter(ITestResultSubView view, ITestModel model)
 
             string imageSet = _model.Settings.Gui.TestTree.AlternateImageSet;
             _view.LoadImages(imageSet);
+            WireUpEvents();
         }
 
         /// <summary>

src/TestCentric/tests/Presenters/TestResultSubViewPresenterTests.cs

@@ -93,5 +93,22 @@ public void Update_WithTestSuite_DetailSectionIsVisible()
 
             view.Received().UpdateDetailSection(Arg.Any<TestResultCounts>());
         }
+
+        [Test]
+        public void SettingsChanged_AlternateImageSet_LoadImagesIsCalled()
+        {
+            // 1. Arrange
+            ITestResultSubView view = Substitute.For<ITestResultSubView>();
+            ITestModel model = Substitute.For<ITestModel>();
+            var settings = new FakeUserSettings();
+            model.Settings.Returns(settings);
+
+            // 2. Act
+            TestResultSubViewPresenter presenter = new TestResultSubViewPresenter(view, model);
+            settings.Gui.TestTree.AlternateImageSet = "Classic";
+
+            // 3. Assert
+            view.Received().LoadImages("Classic");
+        }
     }
 }