From 3d1e1659c294ebf1bc50418dfcc156124090a94d Mon Sep 17 00:00:00 2001 From: slhale Date: Tue, 18 Aug 2015 12:37:47 -0700 Subject: [PATCH] [Search] Update tests for 100% coverage. --- .../test/controllers/SearchControllerSpec.js | 4 ++++ .../test/services/GenericSearchProviderSpec.js | 17 +++++++++++++++++ 2 files changed, 21 insertions(+) diff --git a/platform/search/test/controllers/SearchControllerSpec.js b/platform/search/test/controllers/SearchControllerSpec.js index 39416fab62..720d9bd64a 100644 --- a/platform/search/test/controllers/SearchControllerSpec.js +++ b/platform/search/test/controllers/SearchControllerSpec.js @@ -199,6 +199,10 @@ define( mockPromise.then.mostRecentCall.args[0]({hits: [], total: 0}); expect(mockScope.ngModel.search).toEqual(false); }); + + it("has a default results list to filter from", function () { + expect(mockScope.ngModel.filter()).toBeDefined(); + }); }); } ); \ No newline at end of file diff --git a/platform/search/test/services/GenericSearchProviderSpec.js b/platform/search/test/services/GenericSearchProviderSpec.js index b9dcc34f81..2da7cd343b 100644 --- a/platform/search/test/services/GenericSearchProviderSpec.js +++ b/platform/search/test/services/GenericSearchProviderSpec.js @@ -119,8 +119,12 @@ define( // Call through the children-getting part mockTimeout.mostRecentCall.args[0](); + // Array argument indicates multiple children mockCapabilityPromise.then.mostRecentCall.args[0]([]); mockTimeout.mostRecentCall.args[0](); + // Call again, but for single child + mockCapabilityPromise.then.mostRecentCall.args[0]({}); + mockTimeout.mostRecentCall.args[0](); expect(mockWorker.postMessage).toHaveBeenCalled(); }); @@ -150,6 +154,19 @@ define( }); }); + it("gives an empty result for an empty query", function () { + var timestamp = Date.now(), + queryOutput; + + queryOutput = provider.query('', timestamp, 1, 2); + expect(queryOutput.hits).toEqual([]); + expect(queryOutput.total).toEqual(0); + + queryOutput = provider.query(); + expect(queryOutput.hits).toEqual([]); + expect(queryOutput.total).toEqual(0); + }); + it("handles responses from the worker", function () { var timestamp = Date.now(), event = {