From 8246fd8c357e2b54b6b459d7ce9f2367c3f6c688 Mon Sep 17 00:00:00 2001 From: Andrew Schwartzmeyer Date: Fri, 5 Nov 2021 09:11:57 -0700 Subject: [PATCH] Skip `UpdatePowerShell` tests in CI The rate limiting is in fact hit, though not on macOS as the previous comment indicated, but on Windows, as the previous code indicated. To be honest, it's not a super important test, but it's now failing CI. --- test/features/UpdatePowerShell.test.ts | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/test/features/UpdatePowerShell.test.ts b/test/features/UpdatePowerShell.test.ts index 705cfcbeee..3dacaf1793 100644 --- a/test/features/UpdatePowerShell.test.ts +++ b/test/features/UpdatePowerShell.test.ts @@ -5,6 +5,11 @@ import * as assert from "assert"; import { GitHubReleaseInformation } from "../../src/features/UpdatePowerShell"; describe("UpdatePowerShell feature", function () { + before(function () { + // NOTE: GitHub API is rate limited in CI + if (process.env.TF_BUILD) { this.skip(); } + }); + it("Gets the latest version", async function () { const release: GitHubReleaseInformation = await GitHubReleaseInformation.FetchLatestRelease(false); assert.strictEqual(release.isPreview, false, "expected to not be preview.");