Skip UpdatePowerShell tests in CI on macOS (#2479)

Author: TylerLeonhardt

src/features/UpdatePowerShell.ts

@@ -45,7 +45,7 @@ export class GitHubReleaseInformation {
 
         if (!response.ok) {
             const json = await response.json();
-            throw json.message || json || "response was not ok.";
+            throw new Error(json.message || json || "response was not ok.");
         }
 
         // For preview, we grab all the releases and then grab the first prerelease.

test/features/UpdatePowerShell.test.ts

@@ -5,18 +5,24 @@
 import * as assert from "assert";
 import { GitHubReleaseInformation } from "../../src/features/UpdatePowerShell";
 
-suite("UpdatePowerShell tests", () => {
-    test("Can get the latest version", async () => {
-        const release: GitHubReleaseInformation = await GitHubReleaseInformation.FetchLatestRelease(false);
-        assert.strictEqual(release.isPreview, false, "expected to not be preview.");
-        assert.strictEqual(release.version.prerelease.length === 0, true, "expected to not have preview in version.");
-        assert.strictEqual(release.assets.length > 0, true, "expected to have assets.");
-    });
+// Due to Azure DevOps using the same macOS instances, the macOS builds hit
+// the GitHub API rate limit often. Let's skip these tests on macOS until
+// they are hooked up to only run on release.
+if (process.env.TF_BUILD && process.platform === "win32") {
+    suite("UpdatePowerShell tests", () => {
+        test("Can get the latest version", async () => {
+            const release: GitHubReleaseInformation = await GitHubReleaseInformation.FetchLatestRelease(false);
+            assert.strictEqual(release.isPreview, false, "expected to not be preview.");
+            assert.strictEqual(
+                release.version.prerelease.length === 0, true, "expected to not have preview in version.");
+            assert.strictEqual(release.assets.length > 0, true, "expected to have assets.");
+        });
 
-    test("Can get the latest preview version", async () => {
-        const release: GitHubReleaseInformation = await GitHubReleaseInformation.FetchLatestRelease(true);
-        assert.strictEqual(release.isPreview, true, "expected to be preview.");
-        assert.strictEqual(release.version.prerelease.length > 0, true, "expected to have preview in version.");
-        assert.strictEqual(release.assets.length > 0, true, "expected to have assets.");
+        test("Can get the latest preview version", async () => {
+            const release: GitHubReleaseInformation = await GitHubReleaseInformation.FetchLatestRelease(true);
+            assert.strictEqual(release.isPreview, true, "expected to be preview.");
+            assert.strictEqual(release.version.prerelease.length > 0, true, "expected to have preview in version.");
+            assert.strictEqual(release.assets.length > 0, true, "expected to have assets.");
+        });
     });
-});
+}