Skip to content

Commit afc6d4b

Browse files
committed
jenkins: fixup to ensure all nock scopes are asserted
Will be squased into 28d8a86 before merged to master.
1 parent 9917909 commit afc6d4b

File tree

1 file changed

+8
-2
lines changed

1 file changed

+8
-2
lines changed

test/integration/push-jenkins-update.test.js

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,10 @@ tap.test('Allows repository name to be provided with URL parameter when pushing
6767
.reply(201)
6868

6969
t.plan(1)
70-
t.tearDown(() => prCommitsScope.done() && scope.done())
70+
t.tearDown(() => {
71+
prCommitsScope.done()
72+
scope.done()
73+
})
7174

7275
supertest(app)
7376
.post('/citgm/jenkins/end')
@@ -93,7 +96,10 @@ tap.test('Forwards payload provided in incoming POST to GitHub status API', (t)
9396
.reply(201)
9497

9598
t.plan(1)
96-
t.tearDown(() => prCommitsScope.done() && scope.done())
99+
t.tearDown(() => {
100+
prCommitsScope.done()
101+
scope.done()
102+
})
97103

98104
supertest(app)
99105
.post('/node/jenkins/start')

0 commit comments

Comments
 (0)