Skip to content

Commit 72b82cc

Browse files
committed
fixup tests
1 parent 76e7a61 commit 72b82cc

File tree

4 files changed

+10
-10
lines changed

4 files changed

+10
-10
lines changed

test/unit/MetadataService.spec.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,7 @@ describe("MetadataService", function() {
4747
});
4848

4949
it("should use metadata on settings", function(done) {
50+
Log.level = Log.DEBUG;
5051
settings.metadata = "test";
5152

5253
let p = subject.getMetadata();

test/unit/ResponseValidator.spec.js

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -168,9 +168,9 @@ describe("ResponseValidator", function () {
168168

169169
});
170170

171-
describe("validateSignoutResponse", function (done) {
171+
describe("validateSignoutResponse", function () {
172172

173-
it("should validate that the client state matches response state", function () {
173+
it("should validate that the client state matches response state", function (done) {
174174

175175
stubResponse.state = "not_the_id";
176176
subject.validateSignoutResponse(stubState, stubResponse).then(null, err => {
@@ -211,7 +211,7 @@ describe("ResponseValidator", function () {
211211

212212
});
213213

214-
describe("validateSigninResponse", function (done) {
214+
describe("validateSigninResponse", function () {
215215

216216
it("should process signin params", function (done) {
217217

@@ -343,7 +343,7 @@ describe("ResponseValidator", function () {
343343
});
344344
});
345345

346-
it("should validate that the client state matches response state", function () {
346+
it("should validate that the client state matches response state", function (done) {
347347

348348
stubResponse.state = "not_the_id";
349349
subject._processSigninParams(stubState, stubResponse).then(null, err => {

test/unit/UserInfoService.spec.js

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -46,11 +46,10 @@ describe("UserInfoService", function() {
4646
});
4747

4848
it("should require a token", function(done) {
49-
subject.getClaims().then(null,
50-
err => {
51-
err.message.should.contain("token");
52-
done();
53-
});
49+
subject.getClaims().catch(err => {
50+
err.message.should.contain("token");
51+
done();
52+
});
5453
});
5554

5655
it("should call userinfo endpoint and pass token", function(done) {

test/unit/WebStorageStateStore.spec.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ describe("WebStorageStateStore", function() {
3434
})
3535
});
3636

37-
it("should use prefix if specified", function() {
37+
it("should use prefix if specified", function(done) {
3838
prefix = "foo.";
3939
subject = new WebStorageStateStore({ prefix: prefix, store: store });
4040

0 commit comments

Comments
 (0)