Merge pull request petabridge#103 from Aaronontheweb/TestRunner-cleanup

added some code contracts to TestRunner
alexpantyukhin · Apr 9, 2016 · 85c6c23 · 85c6c23
2 parents 5550935 + be7e14d
commit 85c6c23
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/src/NBench/Sdk/TestRunner.cs b/src/NBench/Sdk/TestRunner.cs
@@ -6,6 +6,7 @@
 using NBench.Sdk.Compiler;
 using System;
 using System.Diagnostics;
+using System.Diagnostics.Contracts;
 using System.Threading;
 
 namespace NBench.Sdk
@@ -46,6 +47,7 @@ public TestRunner(TestPackage package)
         /// <returns></returns>
         public static TestRunner CreateRunner(AppDomain domain, TestPackage package)
         {
+            Contract.Requires(domain != null);
             var runnerType = typeof(TestRunner);
             return domain.CreateInstanceAndUnwrap(runnerType.Assembly.FullName, runnerType.FullName, false, 0, null, new object[] { package }, null, null) as TestRunner;
         }
@@ -58,6 +60,7 @@ public static TestRunner CreateRunner(AppDomain domain, TestPackage package)
         /// <remarks>Creates a new AppDomain and executes the tests.</remarks>
         public static TestRunnerResult Run(TestPackage package)
         {
+            Contract.Requires(package != null);
             // create the test app domain
             var testDomain = DomainManager.CreateDomain(package);