@empiricalrun/test-run 0.1.0 → 0.1.2

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # @empiricalrun/test-run
 
+## 0.1.2
+
+### Patch Changes
+
+- 177ad09: fix: test-run fails due to a syntax error
+
+## 0.1.1
+
+### Patch Changes
+
+- cb2ec31: fix: test-run marking failed tests as pass
+
 ## 0.1.0
 
 ### Minor Changes

package/dist/bin/index.js

@@ -31,9 +31,18 @@ const __1 = require("..");
         console.error("forbid-only option is not supported");
         process.exit(1);
     }
-    await (0, __1.runTest)({
-        name: options.name,
-        dir: options.dir || "tests",
-        pwOptions: pwOptions.join(" "),
-    });
+    try {
+        const { hasTestPassed } = await (0, __1.runTest)({
+            name: options.name,
+            dir: options.dir || "tests",
+            pwOptions: pwOptions.join(" "),
+        });
+        if (!hasTestPassed) {
+            process.exit(1);
+        }
+    }
+    catch (error) {
+        console.error("Error while running playwright test:", error.message);
+        process.exit(1);
+    }
 })();

package/dist/index.d.ts

@@ -4,5 +4,7 @@ import { TestRunParameters } from "./types";
  * @export
  * @param {TestRunParameters} { name, dir, pwOptions }
  */
-export declare function runTest({ name, dir, pwOptions }: TestRunParameters): Promise<void>;
+export declare function runTest({ name, dir, pwOptions, }: TestRunParameters): Promise<{
+    hasTestPassed: boolean;
+}>;
 //# sourceMappingURL=index.d.ts.map

package/dist/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAEA,OAAO,EAAE,iBAAiB,EAAE,MAAM,SAAS,CAAC;AAW5C;;;;GAIG;AACH,wBAAsB,OAAO,CAAC,EAAE,IAAI,EAAE,GAAG,EAAE,SAAS,EAAE,EAAE,iBAAiB,iBAoDxE"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAEA,OAAO,EAAE,iBAAiB,EAAE,MAAM,SAAS,CAAC;AAW5C;;;;GAIG;AACH,wBAAsB,OAAO,CAAC,EAC5B,IAAI,EACJ,GAAG,EACH,SAAS,GACV,EAAE,iBAAiB,GAAG,OAAO,CAAC;IAC7B,aAAa,EAAE,OAAO,CAAC;CACxB,CAAC,CAoDD"}

package/dist/index.js

@@ -11,8 +11,7 @@ const utils_1 = require("./utils");
  * @export
  * @param {TestRunParameters} { name, dir, pwOptions }
  */
-async function runTest({ name, dir, pwOptions }) {
-    console.log("getting all files in directory:", dir);
+async function runTest({ name, dir, pwOptions, }) {
     const files = await (0, utils_1.getAllFilePaths)(dir);
     let matchingFilePath = "";
     // find the first file that contains the test block
@@ -25,23 +24,29 @@ async function runTest({ name, dir, pwOptions }) {
         }
     }
     if (!matchingFilePath) {
-        console.error("No test block found for the given test name");
-        return;
+        const message = `No test block found for the given test name: ${name}`;
+        throw Error(message);
     }
-    const testNode = await (0, utils_1.getTestCaseNode)({
+    const { testCaseNode, sourceFile } = await (0, utils_1.getTestCaseNode)({
         filePath: matchingFilePath,
         scenarioName: name,
     });
-    const parentDescribe = (0, utils_1.findFirstSerialDescribeBlock)(testNode);
+    const parentDescribe = (0, utils_1.findFirstSerialDescribeBlock)(testCaseNode);
     const isFileMarkedSerial = await (0, utils_1.hasTopLevelDescribeConfigureWithSerialMode)(matchingFilePath);
-    console.log("Identified test block:", !!testNode);
+    console.log("Identified test block:", !!testCaseNode);
     console.log("Is parent describe block marked serial:", !!parentDescribe);
     console.log("Is file marked serial:", isFileMarkedSerial);
     const currentFileContent = await fs_extra_1.default.readFile(matchingFilePath, "utf-8");
     // if the file is not marked serial, we need to mark the test or describe block as only
-    if (!isFileMarkedSerial) {
-        await (0, utils_1.markTestAsOnly)(matchingFilePath, testNode.getText(), parentDescribe?.getText() || "");
+    if (!isFileMarkedSerial && testCaseNode) {
+        await (0, utils_1.markTestAsOnly)({
+            sourceFile,
+            parentDescribeNode: parentDescribe,
+            testCaseNode: testCaseNode,
+            filePath: matchingFilePath,
+        });
     }
+    let hasTestPassed = true;
     try {
         const env = Object({ ...process.env });
         const pwRunCmd = `npx playwright test ${matchingFilePath} ${pwOptions}`;
@@ -51,13 +56,12 @@ async function runTest({ name, dir, pwOptions }) {
         });
     }
     catch (e) {
-        console.error("Error executing playwright test", e);
+        hasTestPassed = false;
     }
     // revert the changes made to the file to mark tests as only
     await fs_extra_1.default.writeFile(matchingFilePath, currentFileContent);
-    if (!matchingFilePath) {
-        console.error("No test block found for the given test name");
-        return;
-    }
+    return {
+        hasTestPassed,
+    };
 }
 exports.runTest = runTest;

package/dist/utils/index.d.ts

@@ -1,4 +1,4 @@
-import { Node } from "ts-morph";
+import { Node, SourceFile } from "ts-morph";
 export declare function cmd(command: string[], options: {
     env?: Record<string, string>;
 }): Promise<number>;
@@ -12,12 +12,20 @@ export declare function getAllFilePaths(directoryPath?: string): Promise<string[
 export declare function getTestCaseNode({ filePath, scenarioName, }: {
     filePath: string;
     scenarioName: string;
-}): Promise<Node | undefined>;
+}): Promise<{
+    testCaseNode: Node | undefined;
+    sourceFile: SourceFile;
+}>;
 export declare function hasTestBlock({ filePath, scenarioName, }: {
     filePath: string;
     scenarioName: string;
 }): Promise<boolean>;
 export declare function findFirstSerialDescribeBlock(node: Node | undefined): Node | undefined;
 export declare function hasTopLevelDescribeConfigureWithSerialMode(filePath: string): Promise<boolean>;
-export declare function markTestAsOnly(filePath: string, testBlock: string, parentDescribeBlock?: string): Promise<void>;
+export declare function markTestAsOnly({ sourceFile, parentDescribeNode, testCaseNode, filePath, }: {
+    sourceFile: SourceFile;
+    parentDescribeNode?: Node | undefined;
+    testCaseNode: Node;
+    filePath: string;
+}): Promise<void>;
 //# sourceMappingURL=index.d.ts.map

package/dist/utils/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/utils/index.ts"],"names":[],"mappings":"AAGA,OAAO,EAAE,IAAI,EAAuB,MAAM,UAAU,CAAC;AAErD,wBAAgB,GAAG,CACjB,OAAO,EAAE,MAAM,EAAE,EACjB,OAAO,EAAE;IAAE,GAAG,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAA;CAAE,GACxC,OAAO,CAAC,MAAM,CAAC,CA2BjB;AAED;;;;;GAKG;AACH,wBAAsB,eAAe,CACnC,aAAa,GAAE,MAAW,GACzB,OAAO,CAAC,MAAM,EAAE,CAAC,CAqBnB;AAED,wBAAsB,eAAe,CAAC,EACpC,QAAQ,EACR,YAAY,GACb,EAAE;IACD,QAAQ,EAAE,MAAM,CAAC;IACjB,YAAY,EAAE,MAAM,CAAC;CACtB,GAAG,OAAO,CAAC,IAAI,GAAG,SAAS,CAAC,CAc5B;AAED,wBAAsB,YAAY,CAAC,EACjC,QAAQ,EACR,YAAY,GACb,EAAE;IACD,QAAQ,EAAE,MAAM,CAAC;IACjB,YAAY,EAAE,MAAM,CAAC;CACtB,GAAG,OAAO,CAAC,OAAO,CAAC,CAMnB;AAED,wBAAgB,4BAA4B,CAC1C,IAAI,EAAE,IAAI,GAAG,SAAS,GACrB,IAAI,GAAG,SAAS,CA2BlB;AAED,wBAAsB,0CAA0C,CAC9D,QAAQ,EAAE,MAAM,oBA+BjB;AAED,wBAAsB,cAAc,CAClC,QAAQ,EAAE,MAAM,EAChB,SAAS,EAAE,MAAM,EACjB,mBAAmB,CAAC,EAAE,MAAM,iBAkB7B"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/utils/index.ts"],"names":[],"mappings":"AAGA,OAAO,EAAE,IAAI,EAAW,UAAU,EAAc,MAAM,UAAU,CAAC;AAEjE,wBAAgB,GAAG,CACjB,OAAO,EAAE,MAAM,EAAE,EACjB,OAAO,EAAE;IAAE,GAAG,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAA;CAAE,GACxC,OAAO,CAAC,MAAM,CAAC,CA2BjB;AAED;;;;;GAKG;AACH,wBAAsB,eAAe,CACnC,aAAa,GAAE,MAAW,GACzB,OAAO,CAAC,MAAM,EAAE,CAAC,CAqBnB;AAED,wBAAsB,eAAe,CAAC,EACpC,QAAQ,EACR,YAAY,GACb,EAAE;IACD,QAAQ,EAAE,MAAM,CAAC;IACjB,YAAY,EAAE,MAAM,CAAC;CACtB,GAAG,OAAO,CAAC;IAAE,YAAY,EAAE,IAAI,GAAG,SAAS,CAAC;IAAC,UAAU,EAAE,UAAU,CAAA;CAAE,CAAC,CAatE;AAED,wBAAsB,YAAY,CAAC,EACjC,QAAQ,EACR,YAAY,GACb,EAAE;IACD,QAAQ,EAAE,MAAM,CAAC;IACjB,YAAY,EAAE,MAAM,CAAC;CACtB,GAAG,OAAO,CAAC,OAAO,CAAC,CAMnB;AAED,wBAAgB,4BAA4B,CAC1C,IAAI,EAAE,IAAI,GAAG,SAAS,GACrB,IAAI,GAAG,SAAS,CA2BlB;AAED,wBAAsB,0CAA0C,CAC9D,QAAQ,EAAE,MAAM,oBA+BjB;AAED,wBAAsB,cAAc,CAAC,EACnC,UAAU,EACV,kBAAkB,EAClB,YAAY,EACZ,QAAQ,GACT,EAAE;IACD,UAAU,EAAE,UAAU,CAAC;IACvB,kBAAkB,CAAC,EAAE,IAAI,GAAG,SAAS,CAAC;IACtC,YAAY,EAAE,IAAI,CAAC;IACnB,QAAQ,EAAE,MAAM,CAAC;CAClB,iBAgBA"}

package/dist/utils/index.js

@@ -72,19 +72,18 @@ async function getTestCaseNode({ filePath, scenarioName, }) {
     const project = new ts_morph_1.Project();
     const content = await fs_extra_1.default.readFile(filePath, "utf-8");
     const sourceFile = project.createSourceFile("test.ts", content);
-    sourceFile.getDescendants;
-    const testFunctionNode = sourceFile.getFirstDescendant((node) => !!(node.isKind(ts_morph_1.SyntaxKind.CallExpression) &&
+    const testCaseNode = sourceFile.getFirstDescendant((node) => !!(node.isKind(ts_morph_1.SyntaxKind.CallExpression) &&
         node.getExpression().getText() === "test" &&
         node.getArguments()[0]?.getText().includes(scenarioName)));
-    return testFunctionNode;
+    return { testCaseNode, sourceFile };
 }
 exports.getTestCaseNode = getTestCaseNode;
 async function hasTestBlock({ filePath, scenarioName, }) {
-    const testNode = await getTestCaseNode({
+    const { testCaseNode } = await getTestCaseNode({
         filePath,
         scenarioName,
     });
-    return !!testNode;
+    return !!testCaseNode;
 }
 exports.hasTestBlock = hasTestBlock;
 function findFirstSerialDescribeBlock(node) {
@@ -139,17 +138,22 @@ async function hasTopLevelDescribeConfigureWithSerialMode(filePath) {
     return false;
 }
 exports.hasTopLevelDescribeConfigureWithSerialMode = hasTopLevelDescribeConfigureWithSerialMode;
-async function markTestAsOnly(filePath, testBlock, parentDescribeBlock) {
-    const fileContent = await fs_extra_1.default.readFile(filePath, "utf-8");
-    let updatedTestFileContent = fileContent;
-    if (!parentDescribeBlock) {
-        const updatedTestBlock = testBlock.replace("test(", "test.only(");
-        updatedTestFileContent = fileContent.replace(testBlock, updatedTestBlock);
+async function markTestAsOnly({ sourceFile, parentDescribeNode, testCaseNode, filePath, }) {
+    let updatedTestFileContent = sourceFile.getFullText();
+    if (!parentDescribeNode) {
+        const updatedTestBlock = testCaseNode
+            .getText()
+            .replace("test(", "test.only(");
+        testCaseNode?.replaceWithText(updatedTestBlock);
+        updatedTestFileContent = sourceFile.getFullText();
     }
     else {
-        const describeMarkedAsOnly = parentDescribeBlock.replace("test.describe(", "test.describe.only(");
-        updatedTestFileContent = fileContent.replace(parentDescribeBlock, describeMarkedAsOnly);
+        const describeMarkedAsOnly = parentDescribeNode
+            .getText()
+            .replace("test.describe(", "test.describe.only(");
+        parentDescribeNode.replaceWithText(describeMarkedAsOnly);
+        updatedTestFileContent = sourceFile.getFullText();
     }
-    await fs_extra_1.default.writeFile(filePath, updatedTestFileContent);
+    await fs_extra_1.default.writeFile(filePath, updatedTestFileContent, "utf-8");
 }
 exports.markTestAsOnly = markTestAsOnly;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@empiricalrun/test-run",
-  "version": "0.1.0",
+  "version": "0.1.2",
   "publishConfig": {
     "registry": "https://registry.npmjs.org/",
     "access": "public"