Skip to content

Commit

Permalink
Update tests for new provider settings
Browse files Browse the repository at this point in the history
  • Loading branch information
MisterPhilip committed Jul 20, 2018
1 parent 8d1cf0e commit 351d0c5
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
2 changes: 1 addition & 1 deletion test/core/OmnibugSettings.js
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ test("OmnibugSettings should return a list of defaults", t => {
let settings = new OmnibugSettings(chrome);

t.is(typeof settings.defaults.defaultPattern, "string");
t.is(typeof settings.defaults.disabledProviders, "object");
t.is(typeof settings.defaults.providers, "object");
t.is(typeof settings.defaults.highlightKeys, "object");
t.is(typeof settings.defaults.alwaysExpand, "boolean");
t.is(typeof settings.defaults.showQuotes, "boolean");
Expand Down
5 changes: 3 additions & 2 deletions test/providers/OmnibugProvider.js
Original file line number Diff line number Diff line change
Expand Up @@ -33,9 +33,10 @@ test("Patterns should exist", t => {

test("Patterns should only return for enabled providers", t => {
OmnibugProvider.addProvider(new OmnibugTestProvider());
let defaultPattern = OmnibugProvider.getPattern(["OMNIBUG-TEST-PROVIDER"]);
OmnibugProvider.addProvider(new OmnibugTestProvider2());
let testPattern = OmnibugProvider.getPattern({"OMNIBUG-TEST-PROVIDER": {"enabled": false}, "OMNIBUG-TEST-PROVIDER-2": {"enabled": true}});

t.is(defaultPattern.source, /omnibug\-test\-provider\-321789/.source, "Default pattern should be a RegExp object");
t.is(testPattern.source, /omnibug\-test\-provider\-321789/.source, "Pattern should be a RegExp object");
});

test("Providers should be returned", t => {
Expand Down

0 comments on commit 351d0c5

Please sign in to comment.