ExternalVendorCode/openmct
1
0
Fork 0
mirror of https://github.com/nasa/openmct.git synced 2025-02-20 09:26:45 +00:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

[Search] Style

Browse Source 
Removed use of the keyword 'catch' when dealing
with promises, and instead used the second
parameter of then.
This commit is contained in:
slhale 2015-08-03 10:46:34 -07:00
parent eb0bdba666
commit d0bad46627
2 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
platform/search/src/providers/ElasticsearchSearchProvider.js
View File

@ -166,7 +166,7 @@ define(
}).then(function (rawResults) {
// ...then process the data
return processResults(rawResults, timestamp);
})['catch'](function (err) {
}, function (err) {
// In case of error, return nothing. (To prevent
// infinite loading time.)
return {hits: [], total: 0};

6
platform/search/test/providers/ElasticsearchSearchProviderSpec.js
View File

@ -47,7 +47,7 @@ define(
mockHttp = jasmine.createSpy("$http");
mockHttpPromise = jasmine.createSpyObj(
"promise",
[ "then", "catch" ]
[ "then" ]
);
mockHttp.andReturn(mockHttpPromise);
// allow chaining of promise.then().catch();
@ -59,7 +59,7 @@ define(
);
mockObjectPromise = jasmine.createSpyObj(
"promise",
[ "then", "catch" ]
[ "then" ]
);
mockObjectService.getObjects.andReturn(mockObjectPromise);
@ -107,7 +107,7 @@ define(
});
it("returns something when there is an ElasticSearch error", function () {
mockProviderResults = mockHttpPromise['catch'].mostRecentCall.args[0]();
mockProviderResults = mockHttpPromise.then.mostRecentCall.args[1]();
expect(mockProviderResults).toBeDefined();
});
});