@empiricalrun/test-run 0.1.0 → 0.1.1

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # @empiricalrun/test-run
 
+## 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,CAmDD"}

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,8 +24,8 @@ 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)({
         filePath: matchingFilePath,
@@ -42,6 +41,7 @@ async function runTest({ name, dir, pwOptions }) {
     if (!isFileMarkedSerial) {
         await (0, utils_1.markTestAsOnly)(matchingFilePath, testNode.getText(), parentDescribe?.getText() || "");
     }
+    let hasTestPassed = true;
     try {
         const env = Object({ ...process.env });
         const pwRunCmd = `npx playwright test ${matchingFilePath} ${pwOptions}`;
@@ -51,13 +51,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/package.json

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