fix: Tests that call this.skip() are marked as failed instead of skip…

…ped fix #77
karma-runner · Nov 12, 2015 · 73e554f · 73e554f
1 parent 61f57df
commit 73e554f
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 0 deletions.
diff --git a/src/adapter.js b/src/adapter.js
@@ -50,6 +50,7 @@ var createMochaReporterConstructor = function (tc, pathname) {
     // - test end
     // - pass
     // - fail
+    // - pending
 
     runner.on('start', function () {
       tc.info({total: runner.total})
@@ -65,6 +66,10 @@ var createMochaReporterConstructor = function (tc, pathname) {
       test.$errors = []
     })
 
+    runner.on('pending', function (test) {
+      test.pending = true
+    })
+
     runner.on('fail', function (test, error) {
       if (test.type === 'hook') {
         test.$errors = [formatError(error)]

diff --git a/test/adapter.spec.js b/test/adapter.spec.js
@@ -113,6 +113,21 @@ describe('adapter mocha', function () {
         expect(tc.result.called).to.eq(true)
       })
 
+      it('should report skipped result', function () {
+        sandbox.stub(tc, 'result', function (result) {
+          expect(result.skipped).to.eq(true)
+        })
+
+        var mockMochaResult = {
+          parent: {root: true}
+        }
+
+        runner.emit('pending', mockMochaResult)
+        runner.emit('test end', mockMochaResult)
+
+        expect(tc.result.called).to.eq(true)
+      })
+
       it('should report time 0 for skipped tests', function () {
         sandbox.stub(tc, 'result', function (result) {
           expect(result.skipped).to.eq(true)