Improve test coverage

visualeyes · Oct 22, 2015 · 0d56fb5 · 0d56fb5
1 parent 8a61384
commit 0d56fb5
Show file tree

Hide file tree

Showing 3 changed files with 28 additions and 6 deletions.
diff --git a/src/Budgerigar.Tests/Budgetting/PerformanceBudgetExtensionFacts.cs b/src/Budgerigar.Tests/Budgetting/PerformanceBudgetExtensionFacts.cs
@@ -66,7 +66,7 @@ public async Task StepAsync_Func_Creates_Step(bool hasMonitor) {
         public void Step_Action_Performs_Work_Value_If_Budget_Null() {
             PerformanceBudget budget = null;
             bool workDone = false;
-            budget.Step("step", () => workDone = true);
+            budget.Step("step", () => { workDone = true; });
             Assert.True(workDone);
         }
 
@@ -79,7 +79,7 @@ public void Step_Action_Creates_Step(bool hasMonitor) {
 
             var budget = GetBudget(hasMonitor, (r) => result = r);
             bool workDone = false;
-            budget.Step(stepName, () => workDone = true);
+            budget.Step(stepName, () => { workDone = true; });
             budget.Dispose();
 
             Assert.True(workDone);
@@ -92,7 +92,7 @@ public async Task StepAsync_Action_Performs_Work_Value_If_Budget_Null() {
             bool workDone = false;
             await budget.StepAsync("step", () => {
                 workDone = true;
-                return Task.FromResult(0);
+                return (Task)Task.FromResult(0);
             });
             Assert.True(workDone);
         }
@@ -108,7 +108,7 @@ public async Task StepAsync_Action_Creates_Step(bool hasMonitor) {
             bool workDone = false;
             await budget.StepAsync(stepName, () => {
                 workDone = true;
-                return Task.FromResult(0);
+                return (Task)Task.FromResult(0);
             });
             budget.Dispose();
 

diff --git a/src/Budgerigar.Tests/Budgetting/PerformanceBudgetterExtensionFacts.cs b/src/Budgerigar.Tests/Budgetting/PerformanceBudgetterExtensionFacts.cs
@@ -68,7 +68,7 @@ public async Task RunWithBudgetAsync_Action_Does_Work() {
             PerformanceBudget budget = null;
             IPerformanceBudgetter budgetter = GetBudgetter();
 
-            await budgetter.RunWithBudgetAsync("test", 1.0M, (b) => { workDone = true; budget = b; return Task.FromResult(0); }, null);
+            await budgetter.RunWithBudgetAsync("test", 1.0M, (b) => { workDone = true; budget = b; return (Task)Task.FromResult(0); }, null);
 
             Assert.True(workDone);
             Assert.NotNull(budget);
@@ -80,7 +80,7 @@ public async Task RunWithBudgetAsync_Action_Does_Work_If_Budgetter_Null() {
             PerformanceBudget budget = null;
             IPerformanceBudgetter budgetter = null;
 
-            await budgetter.RunWithBudgetAsync("test", 1.0M, (b) => { workDone = true; budget = b; return Task.FromResult(0); }, null);
+            await budgetter.RunWithBudgetAsync("test", 1.0M, (b) => { workDone = true; budget = b; return (Task)Task.FromResult(0); }, null);
 
             Assert.True(workDone);
             Assert.Null(budget);

diff --git a/src/Budgerigar.Tests/Timing/StopWatch/StopwatchPerformanceMonitorStepFacts.cs b/src/Budgerigar.Tests/Timing/StopWatch/StopwatchPerformanceMonitorStepFacts.cs
@@ -0,0 +1,22 @@
+using Budgerigar.Timing.StopWatch;
+using System;
+using System.Collections.Generic;
+using System.Diagnostics;
+using System.Linq;
+using System.Text;
+using System.Threading.Tasks;
+using Xunit;
+
+namespace Budgerigar.Tests.Timing.StopWatch {
+    public class StopwatchPerformanceMonitorStepFacts {
+
+        [Fact]
+        public void Throws_If_Stopwatch_Not_Running() {
+            var stopwatch = new Stopwatch();
+
+            Assert.Throws<ArgumentException>(() => new StopwatchPerformanceMonitorStep("test", stopwatch));
+        }
+
+
+    }
+}