Skip to content

Commit d1e5b08

Browse files
committed
Fixed maintainer requests
1 parent 2f754d0 commit d1e5b08

2 files changed

Lines changed: 3 additions & 5 deletions

File tree

source/memory-store.ts

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -129,9 +129,6 @@ export default class MemoryStore implements Store {
129129
* @public
130130
*/
131131
shutdown(): void {
132-
if (this.interval !== undefined) {
133-
clearInterval(this.interval)
134-
this.interval = undefined
135-
}
132+
clearInterval(this.interval)
136133
}
137134
}

test/library/memory-store-test.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@ import MemoryStore from '../../source/memory-store.js'
99
describe('memory store test', () => {
1010
beforeEach(() => {
1111
jest.useFakeTimers()
12+
jest.spyOn(global, 'clearInterval')
1213
})
1314
afterEach(() => {
1415
jest.useRealTimers()
@@ -81,7 +82,7 @@ describe('memory store test', () => {
8182
store.init({ windowMs: -1 } as Options)
8283
expect(store.interval).toBeDefined()
8384
store.shutdown()
84-
expect(store.interval).toBeUndefined()
85+
expect(clearInterval).toHaveBeenCalledWith(store.interval)
8586
})
8687

8788
describe('reset time', () => {

0 commit comments

Comments
 (0)