Only show deprecation for catch exceptions if you tell Jasmine not to catch

- Fixes #1497
2018-02-07 14:47:10 -08:00
parent 4530f9431f
commit fbec3cc230
4 changed files with 8 additions and 4 deletions
--- a/src/core/Env.js
+++ b/src/core/Env.js
@@ -198,8 +198,10 @@ getJasmineRequireObj().Env = function(j$) {

    // TODO: fix this naming, and here's where the value comes in
    this.catchExceptions = function(value) {
-      this.deprecated('The catchExceptions option is deprecated and will be replaced with stopOnSpecFailure in Jasmine 3.0');
      catchExceptions = !!value;
+      if (!catchExceptions) {
+        this.deprecated('The catchExceptions option is deprecated and will be replaced with stopOnSpecFailure in Jasmine 3.0');
+      }
      return catchExceptions;
    };