krusty-jasmine-reporter - npm Package Compare versions

Comparing version 0.0.1 to 0.0.2

bin/models.js

@@ -131,6 +131,8 @@ /* jshint node: true */
    * @param {Number} time The time it took to run the test. An example would be, "0.800"
+   * @param {String} packageName The package name to be prefixed on the classname
    * @constructor
    */
-  function TestCase(result, time) {
-    this.classname = result.fullName.substr(0, result.fullName.indexOf(result.description) - 1) || result.fullName;
+  function TestCase(result, time, packageName) {
+    this.classname = packageName + "." + (result.fullName.substr(0, result.fullName.indexOf(result.description) - 1) ||
+      result.fullName);
     this.name = result.description;
@@ -137,0 +139,0 @@ this.time = time;

bin/reporter.js

@@ -81,3 +81,3 @@ /* jshint node: true */
       var seconds = specTimer.elapsed() / 1000;
-      var testCase = new models.TestCase(result, seconds);
+      var testCase = new models.TestCase(result, seconds, options.JUnitReportPackageName);
       testSuite.addTestCase(testCase);
@@ -84,0 +84,0 @@ };

package.json

 {
   "name": "krusty-jasmine-reporter",
-  "version": "0.0.1",
+  "version": "0.0.2",
   "description": "A reporter used to generate jasmine to JUnit results that can be interpreted by Jenkins",
@@ -5,0 +5,0 @@ "main": "index.js",

test/unit/models-spec.js

@@ -25,3 +25,3 @@ /* jshint node: true */
       'timestamp="2014-07-31" id="0" hostname="localhost" tests="1" errors="0" failures="0" skipped="0" time="0.008">' +
-      '<testcase classname="test-case-full-name" name="test-case-name" time="0.008"> </testcase></testsuite>' +
+      '<testcase classname="package-name.test-case-full-name" name="test-case-name" time="0.008"> </testcase></testsuite>' +
       '</testsuites>'
@@ -50,3 +50,3 @@ },
       'timestamp="2014-07-31" id="0" hostname="localhost" tests="1" errors="1" failures="0" skipped="0" ' +
-      'time="0.008"><testcase classname="test-case-full-name" name="test-case-name" time="0.008"> ' +
+      'time="0.008"><testcase classname="package-name.test-case-full-name" name="test-case-name" time="0.008"> ' +
       '<error message="test failure"><![CDATA[backtrace]]></error> </testcase></testsuite></testsuites>'
@@ -75,3 +75,3 @@ },
       'timestamp="2014-07-31" id="0" hostname="localhost" tests="1" errors="0" failures="1" skipped="0" time="0.008">' +
-      '<testcase classname="test-case-full-name" name="test-case-name" time="0.008"> <failure message="test failure">' +
+      '<testcase classname="package-name.test-case-full-name" name="test-case-name" time="0.008"> <failure message="test failure">' +
       '<![CDATA[backtrace]]></failure> </testcase></testsuite></testsuites>'
@@ -96,3 +96,3 @@ },
       'timestamp="2014-07-31" id="0" hostname="localhost" tests="1" errors="0" failures="0" skipped="1" time="0.008">' +
-      '<testcase classname="test-case-full-name" name="test-case-name" time="0.008"> <skipped></skipped> </testcase>' +
+      '<testcase classname="package-name.test-case-full-name" name="test-case-name" time="0.008"> <skipped></skipped> </testcase>' +
       '</testsuite></testsuites>'
@@ -153,3 +153,3 @@ }
 
-  testCase = new models.TestCase(data.testCaseResult, data.testCaseTime);
+  testCase = new models.TestCase(data.testCaseResult, data.testCaseTime, data.testSuiteData.packageName);
 
@@ -156,0 +156,0 @@ // Add the test case to the test suite

test/unit/reporter-spec.js

@@ -102,3 +102,3 @@ /* jshint node: true */
     it('should instantiate TestCase', function () {
-      expect(models.TestCase).toHaveBeenCalledWith(result, 2);
+      expect(models.TestCase).toHaveBeenCalledWith(result, 2, options.JUnitReportPackageName);
     });
@@ -105,0 +105,0 @@

No alert changes

Improved metrics

Total package byte prevSize

29302

0.93%

Lines of code

650

0.31%

No dependency changes