Fix crash when 'run selected tests' but no tests are selected

TestCentric · Oct 13, 2024 · a5ed7e9 · a5ed7e9
1 parent 6971c79
commit a5ed7e9
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 3 deletions.
diff --git a/src/TestCentric/testcentric.gui/Presenters/TestCentricPresenter.cs b/src/TestCentric/testcentric.gui/Presenters/TestCentricPresenter.cs
@@ -954,7 +954,8 @@ private void RunAllTests()
         private void RunSelectedTests()
         {
             var testSelection = _model.SelectedTests;
-            _model.RunTests(testSelection);
+            if (testSelection != null)
+                _model.RunTests(testSelection);
         }
 
         private void RunFailedTests()

diff --git a/src/TestCentric/tests/Presenters/Main/CommandTests.cs b/src/TestCentric/tests/Presenters/Main/CommandTests.cs
@@ -250,9 +250,17 @@ public void RunAllButton_RunsAllTests()
         [Test]
         public void RunButton_RunsSelectedTests()
         {
-            // TODO: Specify Results and test with specific argument
+            var testSelection = new TestSelection();
+            _model.SelectedTests = testSelection;
             _view.RunSelectedButton.Execute += Raise.Event<CommandHandler>();
-            _model.Received().RunTests(Arg.Any<TestSelection>());
+            _model.Received().RunTests(testSelection);
+        }
+
+        [Test]
+        public void RunButton_NoTestSelected_DidnotRunTests()
+        {
+            _view.RunSelectedButton.Execute += Raise.Event<CommandHandler>();
+            _model.DidNotReceiveWithAnyArgs().RunTests(Arg.Any<TestSelection>());
         }
 
         [Test]