Tests: remove unused variables
authorJérôme Benoit <jerome.benoit@sap.com>
Wed, 12 Oct 2022 21:55:21 +0000 (23:55 +0200)
committerJérôme Benoit <jerome.benoit@sap.com>
Wed, 12 Oct 2022 21:55:21 +0000 (23:55 +0200)
Signed-off-by: Jérôme Benoit <jerome.benoit@sap.com>
tests/pools/cluster/dynamic.test.js
tests/pools/cluster/fixed.test.js
tests/pools/thread/dynamic.test.js
tests/pools/thread/fixed.test.js

index 1e7fb8703f9a026e0abde5048e25468abef14a43..b1fd166e87b6be473d9e697a57af7fbc017e9ea9 100644 (file)
@@ -27,11 +27,10 @@ describe('Dynamic cluster pool test suite', () => {
   })
 
   it('Verify that new workers are created when required, max size is not exceeded and that after a while new workers will die', async () => {
-    const promises = []
     let poolBusy = 0
     pool.emitter.on('busy', () => poolBusy++)
     for (let i = 0; i < max * 2; i++) {
-      promises.push(pool.execute())
+      pool.execute()
     }
     expect(pool.workers.length).toBeLessThanOrEqual(max)
     expect(pool.workers.length).toBeGreaterThan(min)
index 4c36d8eb8e55f9666d59ea10122a2a77061eb7c5..a05ccc0f60e73865df231d046cf6c3a4b2d9b8d5 100644 (file)
@@ -74,11 +74,10 @@ describe('Fixed cluster pool test suite', () => {
   })
 
   it('Verify that busy event is emitted', async () => {
-    const promises = []
     let poolBusy = 0
     pool.emitter.on('busy', () => poolBusy++)
     for (let i = 0; i < numberOfWorkers * 2; i++) {
-      promises.push(pool.execute())
+      pool.execute()
     }
     // The `busy` event is triggered when the number of submitted tasks at once reach the number of fixed pool workers.
     // So in total numberOfWorkers + 1 times for a loop submitting up to numberOfWorkers * 2 tasks to the fixed pool.
index 9cd4e72c57b1098256b9c2f356fcfc0ded6da730..fad9dab8cbe3440e26a1f8e97b07b300ed662d04 100644 (file)
@@ -27,11 +27,10 @@ describe('Dynamic thread pool test suite', () => {
   })
 
   it('Verify that new workers are created when required, max size is not exceeded and that after a while new workers will die', async () => {
-    const promises = []
     let poolBusy = 0
     pool.emitter.on('busy', () => poolBusy++)
     for (let i = 0; i < max * 2; i++) {
-      promises.push(pool.execute())
+      pool.execute()
     }
     expect(pool.workers.length).toBeLessThanOrEqual(max)
     expect(pool.workers.length).toBeGreaterThan(min)
index 01622f1622c87237e1dc501313ff14ef348cc2dd..a0fd08c8038d9455dabe67b0891d7907b93d98d5 100644 (file)
@@ -74,11 +74,10 @@ describe('Fixed thread pool test suite', () => {
   })
 
   it('Verify that busy event is emitted', async () => {
-    const promises = []
     let poolBusy = 0
     pool.emitter.on('busy', () => poolBusy++)
     for (let i = 0; i < numberOfThreads * 2; i++) {
-      promises.push(pool.execute())
+      pool.execute()
     }
     // The `busy` event is triggered when the number of submitted tasks at once reach the number of fixed pool workers.
     // So in total numberOfThreads + 1 times for a loop submitting up to numberOfThreads * 2 tasks to the fixed pool.