Skip to content

Commit 0959cb8

Browse files
committed
Remove assertion in tests
I am removing these assertions so that our internal integration tests can pass. They are currently failing because the number of dependencies of the `codeql/javascript-all` pack has changed. It no longer makes sense to test this value as newer versions of this pack will have more dependencies and we expect this value will continue to go up.
1 parent 1c705da commit 0959cb8

File tree

1 file changed

+0
-3
lines changed

1 file changed

+0
-3
lines changed

extensions/ql-vscode/test/vscode-tests/cli-integration/remote-queries/remote-queries-manager.test.ts

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -191,7 +191,6 @@ describe("Remote queries", () => {
191191

192192
// check dependencies.
193193
expect(packNames).toContain("javascript-all");
194-
expect(packNames.length).toBeLessThan(3);
195194
});
196195

197196
it("should run a remote query that is not part of a qlpack", async () => {
@@ -248,7 +247,6 @@ describe("Remote queries", () => {
248247

249248
// check dependencies.
250249
expect(packNames).toContain("javascript-all");
251-
expect(packNames.length).toBeLessThan(3);
252250
});
253251

254252
it("should run a remote query that is nested inside a qlpack", async () => {
@@ -308,7 +306,6 @@ describe("Remote queries", () => {
308306

309307
// check dependencies.
310308
expect(packNames).toContain("javascript-all");
311-
expect(packNames.length).toBeLessThan(3);
312309
});
313310

314311
it("should cancel a run before uploading", async () => {

0 commit comments

Comments
 (0)