Fix some tests that should be serial

Author: henrymercer

src/config-utils.test.ts

@@ -139,7 +139,7 @@ function mockListLanguages(languages: string[]) {
   sinon.stub(api, "getApiClient").value(() => client);
 }
 
-test("load empty config", async (t) => {
+test.serial("load empty config", async (t) => {
   return await withTmpDir(async (tempDir) => {
     const logger = getRunnerLogger(true);
     const languages = "javascript,python";
@@ -180,7 +180,7 @@ test("load empty config", async (t) => {
   });
 });
 
-test("load code quality config", async (t) => {
+test.serial("load code quality config", async (t) => {
   return await withTmpDir(async (tempDir) => {
     const logger = getRunnerLogger(true);
     const languages = "actions";
@@ -230,7 +230,7 @@ test("load code quality config", async (t) => {
   });
 });
 
-test("initActionState doesn't throw if there are queries configured in the repository properties", async (t) => {
+test.serial("initActionState doesn't throw if there are queries configured in the repository properties", async (t) => {
   return await withTmpDir(async (tempDir) => {
     const logger = getRunnerLogger(true);
     const languages = "javascript";
@@ -288,7 +288,7 @@ test("initActionState doesn't throw if there are queries configured in the repos
   });
 });
 
-test("loading a saved config produces the same config", async (t) => {
+test.serial("loading a saved config produces the same config", async (t) => {
   return await withTmpDir(async (tempDir) => {
     const logger = getRunnerLogger(true);
 
@@ -335,7 +335,7 @@ test("loading a saved config produces the same config", async (t) => {
   });
 });
 
-test("loading config with version mismatch throws", async (t) => {
+test.serial("loading config with version mismatch throws", async (t) => {
   return await withTmpDir(async (tempDir) => {
     const logger = getRunnerLogger(true);
 
@@ -387,7 +387,7 @@ test("loading config with version mismatch throws", async (t) => {
   });
 });
 
-test("load input outside of workspace", async (t) => {
+test.serial("load input outside of workspace", async (t) => {
   return await withTmpDir(async (tempDir) => {
     try {
       await configUtils.initConfig(
@@ -412,7 +412,7 @@ test("load input outside of workspace", async (t) => {
   });
 });
 
-test("load non-local input with invalid repo syntax", async (t) => {
+test.serial("load non-local input with invalid repo syntax", async (t) => {
   return await withTmpDir(async (tempDir) => {
     // no filename given, just a repo
     const configFile = "octo-org/codeql-config@main";
@@ -440,7 +440,7 @@ test("load non-local input with invalid repo syntax", async (t) => {
   });
 });
 
-test("load non-existent input", async (t) => {
+test.serial("load non-existent input", async (t) => {
   return await withTmpDir(async (tempDir) => {
     const languagesInput = "javascript";
     const configFile = "input";
@@ -470,7 +470,7 @@ test("load non-existent input", async (t) => {
   });
 });
 
-test("load non-empty input", async (t) => {
+test.serial("load non-empty input", async (t) => {
   return await withTmpDir(async (tempDir) => {
     const codeql = createStubCodeQL({
       async betterResolveLanguages() {
@@ -541,7 +541,7 @@ test("load non-empty input", async (t) => {
   });
 });
 
-test("Using config input and file together, config input should be used.", async (t) => {
+test.serial("Using config input and file together, config input should be used.", async (t) => {
   return await withTmpDir(async (tempDir) => {
     process.env["RUNNER_TEMP"] = tempDir;
     process.env["GITHUB_WORKSPACE"] = tempDir;
@@ -595,7 +595,7 @@ test("Using config input and file together, config input should be used.", async
   });
 });
 
-test("API client used when reading remote config", async (t) => {
+test.serial("API client used when reading remote config", async (t) => {
   return await withTmpDir(async (tempDir) => {
     const codeql = createStubCodeQL({
       async betterResolveLanguages() {
@@ -644,7 +644,7 @@ test("API client used when reading remote config", async (t) => {
   });
 });
 
-test("Remote config handles the case where a directory is provided", async (t) => {
+test.serial("Remote config handles the case where a directory is provided", async (t) => {
   return await withTmpDir(async (tempDir) => {
     const dummyResponse = []; // directories are returned as arrays
     mockGetContents(dummyResponse);
@@ -671,7 +671,7 @@ test("Remote config handles the case where a directory is provided", async (t) =
   });
 });
 
-test("Invalid format of remote config handled correctly", async (t) => {
+test.serial("Invalid format of remote config handled correctly", async (t) => {
   return await withTmpDir(async (tempDir) => {
     const dummyResponse = {
       // note no "content" property here
@@ -700,7 +700,7 @@ test("Invalid format of remote config handled correctly", async (t) => {
   });
 });
 
-test("No detected languages", async (t) => {
+test.serial("No detected languages", async (t) => {
   return await withTmpDir(async (tempDir) => {
     mockListLanguages([]);
     const codeql = createStubCodeQL({
@@ -728,7 +728,7 @@ test("No detected languages", async (t) => {
   });
 });
 
-test("Unknown languages", async (t) => {
+test.serial("Unknown languages", async (t) => {
   return await withTmpDir(async (tempDir) => {
     const languagesInput = "rubbish,english";
 
@@ -862,7 +862,7 @@ const mockRepositoryNwo = parseRepositoryNwo("owner/repo");
     expectedLanguages: ["javascript"],
   },
 ].forEach((args) => {
-  test(`getLanguages: ${args.name}`, async (t) => {
+  test.serial(`getLanguages: ${args.name}`, async (t) => {
     const mockRequest = mockLanguagesInRepo(args.languagesInRepository);
     const stubExtractorEntry = {
       extractor_root: "",