ThreeMammals#177 - optimise the build scripts (ThreeMammals#178)

* Remove explicit restore, and don't rebuild during tests.

This currently fails because the release config doesn't contain symbols needed by opencover.

* Build unit tests in debug

Turns out that for test coverage we need to have debug symbols.

build.cake

@@ -102,17 +102,10 @@ Task("Version")
 		}
 	});
 
-Task("Restore")
+Task("Compile")
 	.IsDependentOn("Clean")
 	.IsDependentOn("Version")
 	.Does(() =>
-	{	
-		DotNetCoreRestore(slnFile);
-	});
-
-Task("Compile")
-	.IsDependentOn("Restore")
-	.Does(() =>
 	{	
 		var settings = new DotNetCoreBuildSettings
 		{
@@ -199,6 +192,9 @@ Task("RunAcceptanceTests")
 		var settings = new DotNetCoreTestSettings
 		{
 			Configuration = compileConfig,
+			ArgumentCustomization = args => args
+				.Append("--no-restore")
+				.Append("--no-build")
 		};
 
 		EnsureDirectoryExists(artifactsForAcceptanceTestsDir);
@@ -212,6 +208,9 @@ Task("RunIntegrationTests")
 		var settings = new DotNetCoreTestSettings
 		{
 			Configuration = compileConfig,
+			ArgumentCustomization = args => args
+				.Append("--no-restore")
+				.Append("--no-build")
 		};
 
 		EnsureDirectoryExists(artifactsForIntegrationTestsDir);
@@ -474,4 +473,4 @@ private bool ShouldPublishToUnstableFeed(string filter, string branchName)
 		Information("Branch " + branchName + " will not be published to the unstable feed");
 	}
 	return publish;	
-}
+}