frida-test 0.3.1 → 0.3.3

package/README.md

@@ -1,5 +1,7 @@
 # frida-test Documentation
 
+[![Test frida-test](https://github.com/bernhste/frida-test/actions/workflows/test.yml/badge.svg)](https://github.com/bernhste/frida-test/actions/workflows/test.yml)
+
 This is a small test framework which runs on the target. It is used to unit test Frida code running on actual devices. It was originally developed to test the Frida agent code used in [frooky](https://github.com/cpholguera/frooky).
 
 The following chapters explain how to write and run tests.
@@ -10,7 +12,7 @@ The following chapters explain how to write and run tests.
 npm install --save-dev frida-test
 ```
 
-After the installation, import the type `frida-test` into you project by adding the following configuration to the `tsconfig.json`:
+After the installation, import the type `frida-test` into your project by adding the following configuration to the `tsconfig.json`:
 
 ```json
 {
@@ -42,7 +44,7 @@ describe('Classloader', () => {
 
 Tests can be nested to any depth and can be synchronous or asynchronous.
 
-### Test discovery
+### Test Discovery
 
 The framework collects every file matching `*.test.ts` in the directories passed on the command line, recursively.
 
@@ -84,7 +86,7 @@ myProject/
 | `.not.<matcher>` | Inverts the assertion result |
 
 > [!NOTE]
-> `frida-test` test itself. So for examples for all Matches and more, have a look a the `*.test.ts` located in the [test folder](./tests/)
+> `frida-test` tests itself. So for examples of all matchers and more, have a look at the `*.test.ts` files located in the [test folder](./tests/).
 
 ### Setup and Teardown
 
@@ -169,7 +171,7 @@ frida-test -U -N org.owasp.mastestapp.MASTestApp-iOS ./tests/ios ./tests/shared
 frida-test -H 192.168.1.10:27042 --token secret -p 4926 ./tests/shared
 ```
 
-## Compile Agent
+## Compiling the Agent
 
 `frida-test` automatically bundles the test suites and compiles them together with the testing framework into a Frida agent.
 

package/dist/reporter/console.js

@@ -24,10 +24,10 @@ function printTestResult(node, depth = 0) {
 }
 export function printTestSuiteResult(suite) {
     if (suite.status == "failed") {
-        logger.warn(`Test suite "${suite.name}" failed:`);
+        console.log(`❌ Test suite "${suite.name}" failed:`);
     }
     else if (suite.status == "passed") {
-        logger.success(`Test suite "${suite.name}" passed:`);
+        console.log(`✅ Test suite "${suite.name}" passed:`);
     }
     if (suite.testResult) {
         printTestResult(suite.testResult, 1);

package/dist/reporter/summary.js

@@ -21,7 +21,7 @@ export function printSummary(runSummary) {
     console.log(chalk.bold("Test Suites"));
     for (const suite of testSuitesResults) {
         const isPassed = suite.status === "passed";
-        console.log(`  ${isPassed ? chalk.green("✓") : chalk.red("✗")} ${suite.name}`);
+        console.log(`  ${isPassed ? "✅" : "❌"} ${suite.name}`);
         if (!isPassed && suite.testResult?.name) {
             for (const line of suite.testResult.name.split("\n")) {
                 console.log(chalk.red(`      ${line}`));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "frida-test",
-  "version": "0.3.1",
+  "version": "0.3.3",
   "type": "module",
   "license": "GPL-3.0-only",
   "homepage": "https://github.com/bernhste/frida-test/blob/main/README.md",

package/src/agent-runtime/registry.ts

@@ -252,35 +252,29 @@ export async function runTests(nodes: TestSuiteNode[], emit: (message: AgentMess
 
   const parentHooks: EachHooks = { beforeEach: rootHooks.beforeEach, afterEach: rootHooks.afterEach };
 
-  const settled = await Promise.allSettled(
-    nodes.map(async (node) => {
-      emit({ type: "test-suite-started", name: node.name });
-      const { result: testResult, counts } = await runTestSuiteNode(node, verbose, parentHooks);
-      const suiteResult: TestSuiteResult = { name: node.name, testResult, status: testResult.status };
-      emit({ type: "test-suite-finished", name: node.name, result: suiteResult });
-      return { suiteResult, counts };
-    }),
-  );
-
   const testSuitesResults: TestSuiteResult[] = [];
   let counts = ZERO_COUNTS;
 
-  settled.forEach((outcome, i) => {
-    if (outcome.status === "fulfilled") {
-      testSuitesResults.push(outcome.value.suiteResult);
-      counts = addCounts(counts, outcome.value.counts);
-      return;
+  for (const node of nodes) {
+    emit({ type: "test-suite-started", name: node.name });
+    try {
+      const { result: testResult, counts: nodeCounts } = await runTestSuiteNode(node, verbose, parentHooks);
+      const suiteResult: TestSuiteResult = { name: node.name, testResult, status: testResult.status };
+      emit({ type: "test-suite-finished", name: node.name, result: suiteResult });
+      testSuitesResults.push(suiteResult);
+      counts = addCounts(counts, nodeCounts);
+    } catch (err) {
+      const error = serializeError(err, verbose);
+      const suiteResult: TestSuiteResult = {
+        name: node.name,
+        testResult: { name: node.name, status: "failed", durationMs: 0, error },
+        status: "failed",
+      };
+      emit({ type: "test-suite-finished", name: node.name, result: suiteResult });
+      testSuitesResults.push(suiteResult);
+      counts = addCounts(counts, { total: 1, passed: 0, failed: 1 });
     }
-
-    const error = serializeError(outcome.reason, verbose);
-    const testSuiteResult: TestSuiteResult = {
-      name: nodes[i].name,
-      testResult: { name: nodes[i].name, status: "failed", durationMs: 0, error },
-      status: "failed",
-    };
-    testSuitesResults.push(testSuiteResult);
-    counts = addCounts(counts, { total: 1, passed: 0, failed: 1 });
-  });
+  }
 
   const afterAllError = await runTeardownHooks(rootHooks.afterAll, verbose);
   if (afterAllError) {