test: Consistently throw errors in pending test fixtures.

test/integration/fixtures/pending/skip-async-before-nested.fixture.js

@@ -7,14 +7,18 @@ describe('skip in before with nested describes', function () {
     }, 0);
   });
 
-  it('should never run this test', function () { });
+  it('should never run this test', function () {
+    throw new Error('never run this test');
+  });
 
   describe('nested describe', function () {
     before(function () {
       throw new Error('first level before should not run');
     });
 
-    it('should never run this test', function () { });
+    it('should never run this test', function () {
+      throw new Error('never run this test');
+    });
 
     after(function () {
       throw new Error('first level after should not run');
@@ -25,7 +29,9 @@ describe('skip in before with nested describes', function () {
         throw new Error('second level before should not run');
       });
 
-      it('should never run this test', function () { });
+      it('should never run this test', function () {
+        throw new Error('never run this test');
+      });
 
       after(function () {
         throw new Error('second level after should not run');

test/integration/fixtures/pending/skip-async-before.fixture.js

@@ -1,7 +1,6 @@
 'use strict';
 
 describe('outer describe', function () {
-
   it('should run this test', function () {});
 
   describe('skip in before', function () {
@@ -12,10 +11,13 @@ describe('outer describe', function () {
       }, 0);
     });
 
-    it('should never run this test', function () {});
-    it('should never run this test', function () {});
+    it('should never run this test', function () {
+      throw new Error('never run this test');
+    });
+    it('should never run this test', function () {
+      throw new Error('never run this test');
+    });
   });
 
   it('should run this test', function () {});
-
 });

test/integration/fixtures/pending/skip-async-beforeEach.fixture.js

@@ -9,7 +9,13 @@ describe('skip in beforeEach', function () {
     }, 0);
   });
 
-  it('should skip this test', function () {});
-  it('should error before reaching this test', function () {});
-  it('should error before reaching this test', function () {});
+  it('should skip this test', function () {
+    throw new Error('never run this test');
+  });
+  it('should not reach this test', function () {
+    throw new Error('never run this test');
+  });
+  it('should not reach this test', function () {
+    throw new Error('never run this test');
+  });
 });

test/integration/fixtures/pending/skip-async-spec.fixture.js

@@ -8,7 +8,5 @@ describe('skip in test', function () {
     }, 0);
   });
 
-  it('should run other tests in the suite', function () {
-    // Do nothing
-  });
+  it('should run other tests in the suite', function () {});
 });

test/integration/fixtures/pending/skip-sync-before-hooks.fixture.js

@@ -39,7 +39,7 @@ describe('outer suite', function () {
       });
 
       it('should never run this test', function () {
-        throw new Error('skipped suie test should not run');
+        throw new Error('skipped suite test should not run');
       });
 
       after(function () {

test/integration/fixtures/pending/skip-sync-before-nested.fixture.js

@@ -5,14 +5,18 @@ describe('skip in before with nested describes', function () {
     this.skip();
   });
 
-  it('should never run this test', function () { });
+  it('should never run this test', function () {
+    throw new Error('never run this test');
+  });
 
   describe('nested describe', function () {
     before(function () {
       throw new Error('first level before should not run');
     });
 
-    it('should never run this test', function () { });
+    it('should never run this test', function () {
+      throw new Error('never run this test');
+    });
 
     after(function () {
       throw new Error('first level after should not run');
@@ -23,7 +27,9 @@ describe('skip in before with nested describes', function () {
         throw new Error('second level before should not run');
       });
 
-      it('should never run this test', function () { });
+      it('should never run this test', function () {
+        throw new Error('never run this test');
+      });
 
       after(function () {
         throw new Error('second level after should not run');

test/integration/fixtures/pending/skip-sync-before.fixture.js

@@ -1,18 +1,20 @@
 'use strict';
 
 describe('outer describe', function () {
-
   it('should run this test', function () {});
 
   describe('skip in before', function () {
     before(function () {
       this.skip();
     });
 
-    it('should never run this test', function () {});
-    it('should never run this test', function () {});
+    it('should never run this test', function () {
+      throw new Error('never run this test');
+    });
+    it('should never run this test', function () {
+      throw new Error('never run this test');
+    });
   });
 
   it('should run this test', function () {});
-
 });

test/integration/fixtures/pending/skip-sync-beforeEach.fixture.js

@@ -6,10 +6,10 @@ describe('skip in beforeEach', function () {
   });
 
   it('should never run this test', function () {
-    throw new Error('never thrown');
+    throw new Error('never run this test');
   });
 
   it('should never run this test', function () {
-    throw new Error('never thrown');
+    throw new Error('never run this test');
   });
 });

test/integration/fixtures/pending/skip-sync-spec.fixture.js

@@ -3,10 +3,8 @@
 describe('skip in test', function () {
   it('should skip immediately', function () {
     this.skip();
-    throw new Error('never thrown');
+    throw new Error('never run this test');
   });
 
-  it('should run other tests in the suite', function () {
-    // Do nothing
-  });
+  it('should run other tests in the suite', function () {});
 });