fixes 3 failing find-in-files tests (#12973)

zaggino · ficristo · commit 1936493c77b6 · 2016-12-13T20:39:49.000+01:00
* fixes 3 failing find-in-files tests

* implement proposed change
diff --git a/src/search/SearchModel.js b/src/search/SearchModel.js
@@ -111,6 +111,7 @@ define(function (require, exports, module) {
      * Clears out the model to an empty state.
      */
     SearchModel.prototype.clear = function () {
+        var numMatchesBefore = this.numMatches;
         this.results = {};
         this.queryInfo = null;
         this.queryExpr = null;
@@ -120,7 +121,9 @@ define(function (require, exports, module) {
         this.numMatches = 0;
         this.foundMaximum = false;
         this.exceedsMaximum = false;
-        this.fireChanged();
+        if (numMatchesBefore !== 0) {
+            this.fireChanged();
+        }
     };
 
     /**
diff --git a/test/spec/FindInFiles-test.js b/test/spec/FindInFiles-test.js
@@ -785,6 +785,8 @@ define(function (require, exports, module) {
                 gotChange = false;
                 oldResults = null;
                 wasQuickChange = false;
+
+                FindInFiles.clearSearch(); // calls FindInFiles.searchModel.clear internally
                 FindInFiles.searchModel.on("change.FindInFilesTest", function (event, quickChange) {
                     gotChange = true;
                     wasQuickChange = quickChange;
