Merge branch 'no-children' of https://github.com/johnjbarton/jasmine into 4.0

* Merges #1742 from @johnjbarton

lib/jasmine-core/jasmine.js

@@ -2099,11 +2099,7 @@ getJasmineRequireObj().Env = function(j$) {
       }
       addSpecsToSuite(suite, specDefinitions);
       if (suite.parentSuite && !suite.children.length) {
-        this.deprecated(
-          'describe with no children (describe() or it()) is ' +
-            'deprecated and will be removed in a future version of Jasmine. ' +
-            'Please either remove the describe or add children to it.'
-        );
+        throw new Error('describe with no children (describe() or it())');
       }
       return suite;
     };

spec/core/EnvSpec.js

@@ -149,15 +149,10 @@ describe('Env', function() {
       );
     });
 
-    it('logs a deprecation when it has no children', function() {
-      spyOn(env, 'deprecated');
-      env.describe('no children', function() {});
-      expect(env.deprecated).toHaveBeenCalledWith(
-        'describe with no children' +
-          ' (describe() or it()) is deprecated and will be removed in a future ' +
-          'version of Jasmine. Please either remove the describe or add ' +
-          'children to it.'
-      );
+    it('throws an error when it has no children', function() {
+      expect(function() {
+        env.describe('done method', function() {});
+      }).toThrowError('describe with no children (describe() or it())');
     });
   });
 

spec/core/integration/EnvSpec.js

@@ -2036,6 +2036,7 @@ describe('Env integration', function() {
       } catch (e) {
         exception = e;
       }
+      env.it('has a test', function() {});
     });
 
     env.execute(null, function() {

src/core/Env.js

@@ -1097,11 +1097,7 @@ getJasmineRequireObj().Env = function(j$) {
       }
       addSpecsToSuite(suite, specDefinitions);
       if (suite.parentSuite && !suite.children.length) {
-        this.deprecated(
-          'describe with no children (describe() or it()) is ' +
-            'deprecated and will be removed in a future version of Jasmine. ' +
-            'Please either remove the describe or add children to it.'
-        );
+        throw new Error('describe with no children (describe() or it())');
       }
       return suite;
     };