Skip to content

Commit d3b4d9f

Browse files
reduce flake in log updates
1 parent f8e818e commit d3b4d9f

File tree

1 file changed

+22
-11
lines changed

1 file changed

+22
-11
lines changed

packages/driver/cypress/integration/commands/sessions/sessions.spec.js

Lines changed: 22 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -111,6 +111,7 @@ describe('cy.session', { retries: 0 }, () => {
111111
setupTestContext()
112112
cy.log('create new session to test against')
113113
cy.session('session-1', setup)
114+
.wait(4) // ensure last log clear page has been updated
114115
})
115116

116117
it('successfully creates new session', () => {
@@ -194,6 +195,7 @@ describe('cy.session', { retries: 0 }, () => {
194195
cy.log('create new session with validation to test against')
195196

196197
cy.session('session-1', setup, { validate })
198+
.wait(4) // ensure last log clear page has been updated
197199
})
198200

199201
it('successfully creates new session and validates it', () => {
@@ -273,12 +275,16 @@ describe('cy.session', { retries: 0 }, () => {
273275
setupTestContext()
274276
cy.log('create new session with validation to test against')
275277

276-
cy.on('fail', (err) => {
277-
expect(setup).to.be.calledOnce
278-
expect(validate).to.be.calledOnce
279-
expect(clearPageCount, 'total times session cleared the page').to.eq(2)
280-
expect(err.message).to.contain('Your `cy.session` **validate** callback returned false')
281-
done()
278+
cy.once('fail', (err) => {
279+
new Promise((resolve) => {
280+
setTimeout(() => {
281+
expect(setup).to.be.calledOnce
282+
expect(validate).to.be.calledOnce
283+
expect(clearPageCount, 'total times session cleared the page').to.eq(2)
284+
expect(err.message).to.contain('Your `cy.session` **validate** callback returned false')
285+
done()
286+
}, 4)
287+
})
282288
})
283289

284290
validate.callsFake(() => false)
@@ -350,6 +356,7 @@ describe('cy.session', { retries: 0 }, () => {
350356

351357
cy.log('restore session to test against')
352358
cy.session('session-1', setup)
359+
.wait(4) // ensure last log clear page has been updated
353360
})
354361

355362
it('successfully restores saved session', () => {
@@ -407,6 +414,7 @@ describe('cy.session', { retries: 0 }, () => {
407414

408415
cy.log('restore session to test against')
409416
cy.session('session-1', setup, { validate })
417+
.wait(4) // ensure last log clear page has been updated
410418
})
411419

412420
it('successfully restores saved session', () => {
@@ -481,6 +489,7 @@ describe('cy.session', { retries: 0 }, () => {
481489

482490
cy.log('restore session to test against')
483491
cy.session('session-1', setup, { validate })
492+
.wait(4) // ensure last log clear page has been updated
484493
})
485494

486495
it('successfully recreates session', () => {
@@ -596,11 +605,13 @@ describe('cy.session', { retries: 0 }, () => {
596605
})
597606

598607
cy.once('fail', (err) => {
599-
expect(err.message).to.contain('Your `cy.session` **validate** callback returned false')
600-
expect(setup).to.be.calledOnce
601-
expect(validate).to.be.calledTwice
602-
expect(clearPageCount, 'total times session cleared the page').to.eq(3)
603-
done()
608+
return setTimeout(() => {
609+
expect(err.message).to.contain('Your `cy.session` **validate** callback returned false')
610+
expect(setup).to.be.calledOnce
611+
expect(validate).to.be.calledTwice
612+
expect(clearPageCount, 'total times session cleared the page').to.eq(3)
613+
done()
614+
}, 4)// ensure last log clear page has been updated
604615
})
605616

606617
cy.log('restore session to test against')

0 commit comments

Comments
 (0)