@travetto/test - npm Package Compare versions

Comparing version 0.0.30 to 0.0.31

package.json

@@ -31,3 +31,3 @@ {
   }, 
-  "version": "0.0.30"
+  "version": "0.0.31"
 }

src/model/test.ts

@@ -16,2 +16,4 @@ import { Class } from '@travetto/registry/src/model/types';
 export interface Assertion {
+  className: string;
+  methodName: string;
   actual?: any;
@@ -18,0 +20,0 @@ expected?: any;

src/runner/assert.ts

 import { AppEnv } from '@travetto/base';
 import * as assert from 'assert';
-import { Assertion } from '../model';
+import { Assertion, TestConfig } from '../model';
 import * as _ from 'lodash';
@@ -54,2 +54,3 @@
   static listener?: (a: Assertion) => void;
+  static test: TestConfig;
 
@@ -79,3 +80,4 @@ static readFilePosition(err: Error, filename: string) {
 
-  static start(listener?: (a: Assertion) => void) {
+  static start(test: TestConfig, listener?: (a: Assertion) => void) {
+    this.test = test;
     this.listener = listener;
@@ -88,3 +90,8 @@ this.asserts = [];
 
-    const assertion: Assertion = { file, line, text, operator: ASSERT_FN_OPERATOR[name] };
+    const assertion: Assertion = {
+      className: this.test.className,
+      methodName: this.test.methodName,
+      file, line, text,
+      operator: ASSERT_FN_OPERATOR[name]
+    };
     if (name === 'fail') {
@@ -169,5 +176,5 @@ if (args.length > 1) {
     this.asserts = [];
-    this.listener = undefined;
+    delete this.listener, this.test;
     return ret;
   }
 }

src/runner/execute.ts

@@ -24,3 +24,3 @@ import * as fs from 'fs';
 
-  static async generateSuiteError(consumer: Consumer, suite: SuiteConfig, description: string, error: Error) {
+  static async generateSuiteError(consumer: Consumer, suite: SuiteConfig, methodName: string, error: Error) {
     const { line, file } = AssertUtil.readFilePosition(error, suite.file);
@@ -31,4 +31,6 @@ const badAssert: Assertion = {
       error,
+      className: suite.className,
+      methodName,
       message: error.message,
-      text: '(outer)',
+      text: methodName,
       operator: 'throws'
@@ -39,4 +41,4 @@ };
       className: suite.className,
-      methodName: description,
-      description,
+      methodName,
+      description: methodName,
       lines: { start: line, end: line },
@@ -51,4 +53,4 @@ file,
       class: suite.class,
-      className: badTest.className,
-      file: badTest.file,
+      className: suite.className,
+      file: suite.file,
       lines: badTest.lines,
@@ -74,6 +76,7 @@ methodName: badTest.methodName,
       if (error === TIMEOUT) {
-        error = new Error(`${suite.className}: ${phase} timed out`);;
+        error = new Error(`${suite.className}: ${phase} timed out`);
       }
       const res = await this.generateSuiteError(consumer, suite, phase, error);
       result.tests.push(res);
+      result.fail++;
       throw BREAKOUT;
@@ -148,3 +151,5 @@ }
 
-      AssertUtil.start((a) => consumer.onEvent({ type: 'assertion', phase: 'after', assertion: a }));
+      AssertUtil.start(test, (a) => {
+        consumer.onEvent({ type: 'assertion', phase: 'after', assertion: a });
+      });
 
@@ -175,5 +180,9 @@ const timeout = this.asyncTimeout(test.timeout);
         const { file, line } = AssertUtil.readFilePosition(err, test.file);
-        const assertion: Assertion = { file, line, operator: 'throws', text: '(uncaught)', error: err, message: err.message };
-        // result.output = result.output || {};
-        // result.output['error'] = `${(result.output['error'] || '')}\n${err.stack}`;
+        const assertion: Assertion = {
+          className: test.className,
+          methodName: test.methodName,
+          file, line,
+          operator: 'throws', text: '(uncaught)',
+          error: err, message: err.message
+        };
         result.assertions.push(assertion);
@@ -207,7 +216,6 @@ }
     } catch (e) {
-      if (e === BREAKOUT) {
-        // Done
-      } else {
+      if (e !== BREAKOUT) {
         const res = await this.generateSuiteError(consumer, suite, 'all', e);
         result.tests.push(res);
+        result.fail++;
       }
@@ -246,7 +254,6 @@ }
     } catch (e) {
-      if (e === BREAKOUT) {
-        // Done
-      } else {
+      if (e !== BREAKOUT) {
         const res = await this.generateSuiteError(consumer, suite, 'all', e);
         result.tests.push(res);
+        result.fail++;
       }
@@ -253,0 +260,0 @@ }

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

46228

increased by0.6%

Lines of code

1322

increased by1.23%

No dependency changes