Merge pull request #520 from Cognifide/bugfix/tiles-are-not-visable

Bugfix/tiles are not visable
wttech · Aug 14, 2019 · a6b5a81 · a6b5a81
2 parents 03cd733 + 2940a2a
commit a6b5a81
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 5 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -18,6 +18,7 @@ All notable changes to AET will be documented in this file.
 - [PR-459](https://github.com/Cognifide/aet/pull/459) Print more meaningful error messages when suite.xml is malformed ([#436](https://github.com/Cognifide/aet/issues/436)) 
 - [PR-517](https://github.com/Cognifide/aet/pull/517) Single URL rerun fixed for named URLs ([#487](https://github.com/Cognifide/aet/issues/487))
 - [PR-516](https://github.com/Cognifide/aet/pull/516) Added option to change headers added by Chrome instance. ([#515](https://github.com/Cognifide/aet/issues/515))
+- [PR-520](https://github.com/Cognifide/aet/pull/520) Fix displaying test tiles
 
 ## Version 3.2.2
 

diff --git a/report/src/main/webapp/app/components/testSearch.filter.js b/report/src/main/webapp/app/components/testSearch.filter.js
@@ -31,15 +31,15 @@ define(['angularAMD'], function (angularAMD) {
         return tests;
       }
 
-      return tests.filter(function (test) {
-        return _.some(test.urls, function (url) {
+      return tests.filter(function (testWrapper) {
+        return _.some(testWrapper.test.urls, function (url) {
           return matches(searchPhrase, url.name);
         });
       });
     }
 
     function matches(searchPhrase, name) {
-      return name && name.indexOf(searchPhrase) > 0;
+      return name && name.includes(searchPhrase);
     }
   }
 });
diff --git a/report/src/main/webapp/app/components/testStatus.filter.js b/report/src/main/webapp/app/components/testStatus.filter.js
@@ -30,8 +30,8 @@ define(['angularAMD'], function (angularAMD) {
     function filter(tests, statuses) {
       var filteredTests = tests;
       if (statuses && statuses.length > 0) {
-        filteredTests = _.filter(tests, function (test) {
-          return _.some(test.urls, function (url) {
+        filteredTests = _.filter(tests, function (testWrapper) {
+          return _.some(testWrapper.test.urls, function (url) {
             return statuses.indexOf(url.getStatus()) > -1;
           });
         });