Speed up Pester tests by using -NoStatus

HowardWolosky · web-flow · commit b9dc12f63d19 · 2020-05-31T09:58:36.000-07:00
Suppressing Status (e.g. using `-NoStatus`) reduces the overall runtime of the UT's across all 3 platforms (Windows, Linux, Mac) by 25-30 min. It also has the side-effect of improving the readability of the raw logs on CI for Linux and Mac. [Most recent run](https://dev.azure.com/ms/PowerShellForGitHub/_build/results?buildId=84315) without this change: 1h 19m [Run with this change](https://dev.azure.com/ms/PowerShellForGitHub/_build/results?buildId=84357) without this change: 54m Resolves #206
diff --git a/Tests/Common.ps1 b/Tests/Common.ps1
@@ -89,6 +89,7 @@ function Initialize-CommonTestSetup
     Reset-GitHubConfiguration
     Set-GitHubConfiguration -DisableTelemetry # We don't want UT's to impact telemetry
     Set-GitHubConfiguration -LogRequestBody # Make it easier to debug UT failures
+    Set-GitHubConfiguration -DefaultNoStatus # Status corrupts the raw CI logs for Linux and Mac, and makes runs take slightly longer.
 }
 
 Initialize-CommonTestSetup

Original file line number	Diff line number	Diff line change
`@@ -89,6 +89,7 @@ function Initialize-CommonTestSetup`
`89`	`89`	`Reset-GitHubConfiguration`
`90`	`90`	`Set-GitHubConfiguration -DisableTelemetry # We don't want UT's to impact telemetry`
`91`	`91`	`Set-GitHubConfiguration -LogRequestBody # Make it easier to debug UT failures`
	`92`	`+ Set-GitHubConfiguration -DefaultNoStatus # Status corrupts the raw CI logs for Linux and Mac, and makes runs take slightly longer.`
`92`	`93`	`}`
`93`	`94`
`94`	`95`	`Initialize-CommonTestSetup`