From d783fd50ed406d5a203ad03ca08e09c10195a2fd Mon Sep 17 00:00:00 2001 From: Howard Wolosky Date: Mon, 29 Jun 2020 15:41:47 -0700 Subject: [PATCH] Improve CI speed by changing repo targeted by Forks tests Pull #251 changed the repo targeted by the forks tests from `microsoft/PowerShellForGitHub` to `octocat/Hello-World` in order to prevent the accidental deletion of real forks when running the UT's locally against your own account. In practice, this almost doubled the execution time of the UT's, because the execution time of the Forks API's took so much longer against a repo with 1400+ forks. Changing the test over to use a repo that currently only has 39 forks (several orders of magnitude fewer). --- Tests/GitHubRepositoryForks.tests.ps1 | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Tests/GitHubRepositoryForks.tests.ps1 b/Tests/GitHubRepositoryForks.tests.ps1 index a44b1466..7ea08eb1 100644 --- a/Tests/GitHubRepositoryForks.tests.ps1 +++ b/Tests/GitHubRepositoryForks.tests.ps1 @@ -20,7 +20,7 @@ try # Define Script-scoped, readonly, hidden variables. @{ upstreamOwnerName = 'octocat' - upstreamRepositoryName = 'Hello-World' + upstreamRepositoryName = 'git-consortium' }.GetEnumerator() | ForEach-Object { Set-Variable -Force -Scope Script -Option ReadOnly -Visibility Private -Name $_.Key -Value $_.Value }