jest-bamboo-reporter - npm Package Compare versions

Comparing version 1.1.0 to 1.2.0

index.js

@@ -27,14 +27,38 @@ var fs = require('fs');
   results.testResults.forEach(function (suiteResult) {
-    suiteResult.testResults.forEach(function (testResult) {
-      var testFileName = path.basename(suiteResult.testFilePath);
-      var variables = {
-        'firstAncestorTitle': testResult.ancestorTitles[0],
-        'filePath': suiteResult.testFilePath,
-        'fileName': testFileName,
-        'fileNameWithoutExtension': path.parse(testFileName).name
-      }
+    var testFileName = path.basename(suiteResult.testFilePath);
 
-      var suiteName = helpers.replaceCharsNotSupportedByBamboo(helpers.replaceVariables(suiteNameTemplate, variables));
-      var testTitle = helpers.replaceCharsNotSupportedByBamboo(testResult.ancestorTitles.concat([testResult.title]).join(nameSeparator));
+    if (suiteResult.failureMessage) {
+      var suiteName = helpers.replaceCharsNotSupportedByBamboo(
+        helpers.replaceVariables(suiteNameTemplate, {
+          firstAncestorTitle: suiteResult.displayName,
+          filePath: suiteResult.testFilePath,
+          fileName: testFileName,
+          fileNameWithoutExtension: path.parse(testFileName).name
+        })
+      );
+      output.failures.push({
+        title: suiteName,
+        fullTitle: suiteName,
+        duration: suiteResult.perfStats.end - suiteResult.perfStats.start,
+        errorCount: 1,
+        error: suiteResult.failureMessage
+      });
 
+      // Nothing to process if the suite failed
+      return;
+    }
+
+    suiteResult.testResults.forEach(function (testResult) {
+      var suiteName = helpers.replaceCharsNotSupportedByBamboo(
+        helpers.replaceVariables(suiteNameTemplate, {
+          firstAncestorTitle: testResult.ancestorTitles[0],
+          filePath: suiteResult.testFilePath,
+          fileName: testFileName,
+          fileNameWithoutExtension: path.parse(testFileName).name
+        })
+      );
+      var testTitle = helpers.replaceCharsNotSupportedByBamboo(
+        testResult.ancestorTitles.concat([testResult.title]).join(nameSeparator)
+      );
+      
       if (testTitle in existingTestTitles) {
@@ -41,0 +65,0 @@ var newTestTitle;

package.json

 {
   "name": "jest-bamboo-reporter",
-  "version": "1.1.0",
+  "version": "1.2.0",
   "main": "index.js",
@@ -27,5 +27,5 @@ "scripts": {
   "devDependencies": {
-    "jest": "~21.2.1",
-    "sinon": "~4.1.2"
+    "jest": "~22.4.3",
+    "sinon": "~5.0.3"
   }
 }

test-files/expected-result-with-filename.json

@@ -12,2 +12,9 @@ {
     {
+      "title": "SpaceOverview-test",
+      "fullTitle": "SpaceOverview-test",
+      "duration": 0,
+      "errorCount": 1,
+      "error": "  \u001b[1m● \u001b[22mTest suite failed to run\n\n    Cannot find module '../../../../test/enzyme-utils-local' from 'SpaceOverview-test.js'\n\n    \u001b[0m \u001b[90m 12 | \u001b[39mjest\u001b[33m.\u001b[39mmock(\u001b[32m\"../../InlineComments\"\u001b[39m)\u001b[33m;\u001b[39m\u001b[0m\n    \u001b[0m \u001b[90m 13 | \u001b[39mjest\u001b[33m.\u001b[39mmock(\u001b[32m\"../../EditorLoader\"\u001b[39m)\u001b[33m;\u001b[39m\u001b[0m\n    \u001b[0m\u001b[31m\u001b[1m>\u001b[22m\u001b[39m\u001b[90m 14 | \u001b[39mjest\u001b[33m.\u001b[39mmock(\u001b[32m\"../../EditorFlags\"\u001b[39m)\u001b[33m;\u001b[39m\u001b[0m\n    \u001b[0m \u001b[90m    | \u001b[39m                                                             \u001b[31m\u001b[1m^\u001b[22m\u001b[39m\u001b[0m\n    \u001b[0m \u001b[90m 15 | \u001b[39m\u001b[0m\n    \u001b[0m \u001b[90m 16 | \u001b[39mit(\u001b[32m\"should show blank experience if space doesn't have space overview page\"\u001b[39m\u001b[33m,\u001b[39m async () \u001b[33m=>\u001b[39m {\u001b[0m\n    \u001b[0m \u001b[90m 17 | \u001b[39m  \u001b[36mconst\u001b[39m wrapper \u001b[33m=\u001b[39m await mockAndMount(\u001b[33m<\u001b[39m\u001b[33mSpaceOverview\u001b[39m spaceKey\u001b[33m=\u001b[39m\u001b[32m\"DS\"\u001b[39m \u001b[33m/\u001b[39m\u001b[33m>\u001b[39m\u001b[33m,\u001b[39m {\u001b[0m\n\n      \u001b[2mat Resolver.resolveModule (\u001b[22m\u001b[0m\u001b[36m../../node_modules/jest-runner/node_modules/jest-resolve/build/index.js\u001b[39m\u001b[0m\u001b[2m:210:17)\u001b[22m\n      \u001b[2mat Object.<anonymous> (\u001b[22m\u001b[0m\u001b[36msrc/components/SpaceOverview/__tests__/SpaceOverview-test.js\u001b[39m\u001b[0m\u001b[2m:14:376)\u001b[22m\n"
+    },
+    {
       "title": "Saga middleware – it should call \"asyncFetchOpportunities\"",
@@ -14,0 +21,0 @@ "fullTitle": "saga-test",

test-files/expected-result.json

@@ -12,2 +12,9 @@ {
     {
+      "title": "src/components/SpaceOverview/__tests__/SpaceOverview-test_js",
+      "fullTitle": "src/components/SpaceOverview/__tests__/SpaceOverview-test_js",
+      "duration": 0,
+      "errorCount": 1,
+      "error": "  \u001b[1m● \u001b[22mTest suite failed to run\n\n    Cannot find module '../../../../test/enzyme-utils-local' from 'SpaceOverview-test.js'\n\n    \u001b[0m \u001b[90m 12 | \u001b[39mjest\u001b[33m.\u001b[39mmock(\u001b[32m\"../../InlineComments\"\u001b[39m)\u001b[33m;\u001b[39m\u001b[0m\n    \u001b[0m \u001b[90m 13 | \u001b[39mjest\u001b[33m.\u001b[39mmock(\u001b[32m\"../../EditorLoader\"\u001b[39m)\u001b[33m;\u001b[39m\u001b[0m\n    \u001b[0m\u001b[31m\u001b[1m>\u001b[22m\u001b[39m\u001b[90m 14 | \u001b[39mjest\u001b[33m.\u001b[39mmock(\u001b[32m\"../../EditorFlags\"\u001b[39m)\u001b[33m;\u001b[39m\u001b[0m\n    \u001b[0m \u001b[90m    | \u001b[39m                                                             \u001b[31m\u001b[1m^\u001b[22m\u001b[39m\u001b[0m\n    \u001b[0m \u001b[90m 15 | \u001b[39m\u001b[0m\n    \u001b[0m \u001b[90m 16 | \u001b[39mit(\u001b[32m\"should show blank experience if space doesn't have space overview page\"\u001b[39m\u001b[33m,\u001b[39m async () \u001b[33m=>\u001b[39m {\u001b[0m\n    \u001b[0m \u001b[90m 17 | \u001b[39m  \u001b[36mconst\u001b[39m wrapper \u001b[33m=\u001b[39m await mockAndMount(\u001b[33m<\u001b[39m\u001b[33mSpaceOverview\u001b[39m spaceKey\u001b[33m=\u001b[39m\u001b[32m\"DS\"\u001b[39m \u001b[33m/\u001b[39m\u001b[33m>\u001b[39m\u001b[33m,\u001b[39m {\u001b[0m\n\n      \u001b[2mat Resolver.resolveModule (\u001b[22m\u001b[0m\u001b[36m../../node_modules/jest-runner/node_modules/jest-resolve/build/index.js\u001b[39m\u001b[0m\u001b[2m:210:17)\u001b[22m\n      \u001b[2mat Object.<anonymous> (\u001b[22m\u001b[0m\u001b[36msrc/components/SpaceOverview/__tests__/SpaceOverview-test.js\u001b[39m\u001b[0m\u001b[2m:14:376)\u001b[22m\n"
+    },
+    {
       "title": "Saga middleware – it should call \"asyncFetchOpportunities\"",
@@ -14,0 +21,0 @@ "fullTitle": "Saga middleware",

No alert changes

Improved metrics

Total package byte prevSize

783726

increased by0.89%

Lines of code

30612

increased by0.26%

No dependency changes