Fixed maintainer requests

express-rate-limit · Sep 2, 2022 · d1e5b08 · d1e5b08
1 parent 2f754d0
commit d1e5b08
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/source/memory-store.ts b/source/memory-store.ts
@@ -129,9 +129,6 @@ export default class MemoryStore implements Store {
 	 * @public
 	 */
 	shutdown(): void {
-		if (this.interval !== undefined) {
-			clearInterval(this.interval)
-			this.interval = undefined
-		}
+		clearInterval(this.interval)
 	}
 }
diff --git a/test/library/memory-store-test.ts b/test/library/memory-store-test.ts
@@ -9,6 +9,7 @@ import MemoryStore from '../../source/memory-store.js'
 describe('memory store test', () => {
 	beforeEach(() => {
 		jest.useFakeTimers()
+		jest.spyOn(global, 'clearInterval')
 	})
 	afterEach(() => {
 		jest.useRealTimers()
@@ -81,7 +82,7 @@ describe('memory store test', () => {
 		store.init({ windowMs: -1 } as Options)
 		expect(store.interval).toBeDefined()
 		store.shutdown()
-		expect(store.interval).toBeUndefined()
+		expect(clearInterval).toHaveBeenCalledWith(store.interval)
 	})
 
 	describe('reset time', () => {