Skip to content

Commit c702b10

Browse files
authored
Merge pull request #4514 from github/copilot/investigate-cli-test-failure
Pin tutorial pack in CLI authentication tests
2 parents 4a208a2 + 58e29b1 commit c702b10

1 file changed

Lines changed: 4 additions & 2 deletions

File tree

extensions/ql-vscode/test/vscode-tests/cli-integration/run-cli.test.ts

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -130,6 +130,8 @@ describe("Use cli", () => {
130130
},
131131
);
132132

133+
// Pin this pack because these tests exercise authentication, not registry compatibility.
134+
// Remove the pin once every supported CLI can download the latest tutorial pack.
133135
describe("github authentication", () => {
134136
itWithCodeQL()(
135137
"should not use authentication if there are no credentials",
@@ -138,7 +140,7 @@ describe("Use cli", () => {
138140
.spyOn(authentication, "getSession")
139141
.mockResolvedValue(undefined);
140142

141-
await cli.packDownload(["codeql/tutorial"]);
143+
await cli.packDownload(["codeql/tutorial@0.0.11"]);
142144
expect(getSession).toHaveBeenCalledTimes(1);
143145
expect(getSession).toHaveBeenCalledWith(
144146
"github",
@@ -165,7 +167,7 @@ describe("Use cli", () => {
165167
scopes: ["read:packages"],
166168
});
167169

168-
await cli.packDownload(["codeql/tutorial"]);
170+
await cli.packDownload(["codeql/tutorial@0.0.11"]);
169171
expect(getSession).toHaveBeenCalledTimes(2);
170172
expect(getSession).toHaveBeenCalledWith(
171173
"github",

0 commit comments

Comments
 (0)