fix(test): Don't run all detectors when blessing a single file

nowarp · Nov 29, 2024 · db3d24a · db3d24a
1 parent b106ae0
commit db3d24a
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 14 deletions.
diff --git a/test/builtinDetectors.spec.ts b/test/builtinDetectors.spec.ts
@@ -72,7 +72,7 @@ function processSingleDetectorFile(filePath: string) {
   runTestForFile(contractPath, nameBase, detectorName, detectorName);
 }
 
-const filePathArg = getFilePathArg();
+const filePathArg = getFilePathArg(DETECTORS_DIR);
 if (filePathArg) {
   // Run test for a single file
   const fullPath = path.resolve(filePathArg);
@@ -105,16 +105,18 @@ if (filePathArg) {
   });
 }
 
-describe("Built-in Detectors Tests", () => {
-  it("should have test contracts for all built-in detectors", () => {
-    const allDetectors = getAllDetectors();
-    const testFiles = fs
-      .readdirSync(DETECTORS_DIR)
-      .filter((file) => file.endsWith(".tact"));
-    const missingTests = allDetectors.filter((detector) => {
-      const expectedTestFile = `${detector}.tact`;
-      return !testFiles.includes(expectedTestFile);
+if (!filePathArg) {
+  describe("Built-in Detectors Tests", () => {
+    it("should have test contracts for all built-in detectors", () => {
+      const allDetectors = getAllDetectors();
+      const testFiles = fs
+        .readdirSync(DETECTORS_DIR)
+        .filter((file) => file.endsWith(".tact"));
+      const missingTests = allDetectors.filter((detector) => {
+        const expectedTestFile = `${detector}.tact`;
+        return !testFiles.includes(expectedTestFile);
+      });
+      expect(missingTests).toEqual([]);
     });
-    expect(missingTests).toEqual([]);
   });
-});
+}
diff --git a/test/testUtil.ts b/test/testUtil.ts
@@ -110,6 +110,6 @@ export function resetIds(): void {
  *
  * Example usage: yarn test tactIR.spec.ts <filepath>
  */
-export function getFilePathArg(): string | undefined {
-  return process.argv.find((arg) => path.resolve(arg).includes(ALL_DIR));
+export function getFilePathArg(dir = ALL_DIR): string | undefined {
+  return process.argv.find((arg) => path.resolve(arg).includes(dir));
 }