X-Git-Url: https://git.piment-noir.org/?a=blobdiff_plain;f=tests%2Fpools%2Fselection-strategies%2Fselection-strategies.test.mjs;h=0e7f19fbdf0abe23685f31059328caa975ced2a3;hb=e41b02718bb9355f7934f0198568cc0ac6f70f9a;hp=bca3c82b1647e22146b6094321d7c9ac60485403;hpb=e44639e9af74427b71f1556ff7ec9f7606373e0d;p=poolifier.git diff --git a/tests/pools/selection-strategies/selection-strategies.test.mjs b/tests/pools/selection-strategies/selection-strategies.test.mjs index bca3c82b..0e7f19fb 100644 --- a/tests/pools/selection-strategies/selection-strategies.test.mjs +++ b/tests/pools/selection-strategies/selection-strategies.test.mjs @@ -1,4 +1,8 @@ +import { randomInt } from 'node:crypto' + import { expect } from 'expect' + +import { CircularArray } from '../../../lib/circular-array.cjs' import { DynamicClusterPool, DynamicThreadPool, @@ -6,7 +10,6 @@ import { FixedThreadPool, WorkerChoiceStrategies } from '../../../lib/index.cjs' -import { CircularArray } from '../../../lib/circular-array.cjs' describe('Selection strategies test suite', () => { const min = 0 @@ -35,6 +38,9 @@ describe('Selection strategies test suite', () => { expect(pool.opts.workerChoiceStrategy).toBe( WorkerChoiceStrategies.ROUND_ROBIN ) + expect(pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy).toBe( + WorkerChoiceStrategies.ROUND_ROBIN + ) // We need to clean up the resources after our test await pool.destroy() }) @@ -47,9 +53,9 @@ describe('Selection strategies test suite', () => { { workerChoiceStrategy } ) expect(pool.opts.workerChoiceStrategy).toBe(workerChoiceStrategy) - expect(pool.workerChoiceStrategyContext.workerChoiceStrategy).toBe( - workerChoiceStrategy - ) + expect( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy + ).toBe(workerChoiceStrategy) await pool.destroy() } }) @@ -63,9 +69,9 @@ describe('Selection strategies test suite', () => { ) pool.setWorkerChoiceStrategy(workerChoiceStrategy) expect(pool.opts.workerChoiceStrategy).toBe(workerChoiceStrategy) - expect(pool.workerChoiceStrategyContext.workerChoiceStrategy).toBe( - workerChoiceStrategy - ) + expect( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy + ).toBe(workerChoiceStrategy) await pool.destroy() } for (const workerChoiceStrategy of Object.values(WorkerChoiceStrategies)) { @@ -76,26 +82,27 @@ describe('Selection strategies test suite', () => { ) pool.setWorkerChoiceStrategy(workerChoiceStrategy) expect(pool.opts.workerChoiceStrategy).toBe(workerChoiceStrategy) - expect(pool.workerChoiceStrategyContext.workerChoiceStrategy).toBe( - workerChoiceStrategy - ) + expect( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy + ).toBe(workerChoiceStrategy) await pool.destroy() } }) it('Verify available strategies default internals at pool creation', async () => { - const pool = new FixedThreadPool( - max, - './tests/worker-files/thread/testWorker.mjs' - ) for (const workerChoiceStrategy of Object.values(WorkerChoiceStrategies)) { + const pool = new FixedThreadPool( + max, + './tests/worker-files/thread/testWorker.mjs', + { workerChoiceStrategy } + ) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( workerChoiceStrategy ).nextWorkerNodeKey ).toBe(0) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( workerChoiceStrategy ).previousWorkerNodeKey ).toBe(0) @@ -103,7 +110,7 @@ describe('Selection strategies test suite', () => { workerChoiceStrategy === WorkerChoiceStrategies.WEIGHTED_ROUND_ROBIN ) { expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( workerChoiceStrategy ).workerNodeVirtualTaskRunTime ).toBe(0) @@ -112,35 +119,35 @@ describe('Selection strategies test suite', () => { WorkerChoiceStrategies.INTERLEAVED_WEIGHTED_ROUND_ROBIN ) { expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( workerChoiceStrategy ).workerNodeVirtualTaskRunTime ).toBe(0) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( workerChoiceStrategy ).roundId ).toBe(0) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( workerChoiceStrategy ).workerNodeId ).toBe(0) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( workerChoiceStrategy ).roundWeights.length ).toBe(1) expect( Number.isSafeInteger( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( workerChoiceStrategy ).roundWeights[0] ) ).toBe(true) } + await pool.destroy() } - await pool.destroy() }) it('Verify ROUND_ROBIN strategy default policy', async () => { @@ -150,7 +157,7 @@ describe('Selection strategies test suite', () => { './tests/worker-files/thread/testWorker.mjs', { workerChoiceStrategy } ) - expect(pool.workerChoiceStrategyContext.getStrategyPolicy()).toStrictEqual({ + expect(pool.workerChoiceStrategiesContext.getPolicy()).toStrictEqual({ dynamicWorkerUsage: false, dynamicWorkerReady: true }) @@ -161,7 +168,7 @@ describe('Selection strategies test suite', () => { './tests/worker-files/thread/testWorker.mjs', { workerChoiceStrategy } ) - expect(pool.workerChoiceStrategyContext.getStrategyPolicy()).toStrictEqual({ + expect(pool.workerChoiceStrategiesContext.getPolicy()).toStrictEqual({ dynamicWorkerUsage: false, dynamicWorkerReady: true }) @@ -177,7 +184,7 @@ describe('Selection strategies test suite', () => { { workerChoiceStrategy } ) expect( - pool.workerChoiceStrategyContext.getTaskStatisticsRequirements() + pool.workerChoiceStrategiesContext.getTaskStatisticsRequirements() ).toStrictEqual({ runTime: { aggregate: false, @@ -203,7 +210,7 @@ describe('Selection strategies test suite', () => { { workerChoiceStrategy } ) expect( - pool.workerChoiceStrategyContext.getTaskStatisticsRequirements() + pool.workerChoiceStrategiesContext.getTaskStatisticsRequirements() ).toStrictEqual({ runTime: { aggregate: false, @@ -267,13 +274,13 @@ describe('Selection strategies test suite', () => { }) } expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).nextWorkerNodeKey ).toBe(0) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).previousWorkerNodeKey ).toBe(pool.workerNodes.length - 1) // We need to clean up the resources after our test @@ -327,13 +334,13 @@ describe('Selection strategies test suite', () => { ) } expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).nextWorkerNodeKey ).toBe(0) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).previousWorkerNodeKey ).toBe(pool.workerNodes.length - 1) // We need to clean up the resources after our test @@ -366,62 +373,54 @@ describe('Selection strategies test suite', () => { await pool.destroy() }) - it('Verify ROUND_ROBIN strategy internals are resets after setting it', async () => { + it("Verify ROUND_ROBIN strategy internals aren't reset after setting it", async () => { const workerChoiceStrategy = WorkerChoiceStrategies.ROUND_ROBIN let pool = new FixedThreadPool( max, './tests/worker-files/thread/testWorker.mjs', - { workerChoiceStrategy: WorkerChoiceStrategies.WEIGHTED_ROUND_ROBIN } + { workerChoiceStrategy } ) - expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy - ).nextWorkerNodeKey - ).toBeDefined() - expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy - ).previousWorkerNodeKey - ).toBeDefined() + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy + ).nextWorkerNodeKey = randomInt(1, max - 1) + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy + ).previousWorkerNodeKey = randomInt(1, max - 1) pool.setWorkerChoiceStrategy(workerChoiceStrategy) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).nextWorkerNodeKey - ).toBe(0) + ).toBeGreaterThan(0) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).previousWorkerNodeKey - ).toBe(0) + ).toBeGreaterThan(0) await pool.destroy() pool = new DynamicThreadPool( min, max, './tests/worker-files/thread/testWorker.mjs', - { workerChoiceStrategy: WorkerChoiceStrategies.WEIGHTED_ROUND_ROBIN } + { workerChoiceStrategy } ) - expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy - ).nextWorkerNodeKey - ).toBeDefined() - expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy - ).previousWorkerNodeKey - ).toBeDefined() + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy + ).nextWorkerNodeKey = randomInt(1, max - 1) + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy + ).previousWorkerNodeKey = randomInt(1, max - 1) pool.setWorkerChoiceStrategy(workerChoiceStrategy) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).nextWorkerNodeKey - ).toBe(0) + ).toBeGreaterThan(0) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).previousWorkerNodeKey - ).toBe(0) + ).toBeGreaterThan(0) // We need to clean up the resources after our test await pool.destroy() }) @@ -433,7 +432,7 @@ describe('Selection strategies test suite', () => { './tests/worker-files/thread/testWorker.mjs', { workerChoiceStrategy } ) - expect(pool.workerChoiceStrategyContext.getStrategyPolicy()).toStrictEqual({ + expect(pool.workerChoiceStrategiesContext.getPolicy()).toStrictEqual({ dynamicWorkerUsage: false, dynamicWorkerReady: true }) @@ -444,7 +443,7 @@ describe('Selection strategies test suite', () => { './tests/worker-files/thread/testWorker.mjs', { workerChoiceStrategy } ) - expect(pool.workerChoiceStrategyContext.getStrategyPolicy()).toStrictEqual({ + expect(pool.workerChoiceStrategiesContext.getPolicy()).toStrictEqual({ dynamicWorkerUsage: false, dynamicWorkerReady: true }) @@ -460,7 +459,7 @@ describe('Selection strategies test suite', () => { { workerChoiceStrategy } ) expect( - pool.workerChoiceStrategyContext.getTaskStatisticsRequirements() + pool.workerChoiceStrategiesContext.getTaskStatisticsRequirements() ).toStrictEqual({ runTime: { aggregate: false, @@ -486,7 +485,7 @@ describe('Selection strategies test suite', () => { { workerChoiceStrategy } ) expect( - pool.workerChoiceStrategyContext.getTaskStatisticsRequirements() + pool.workerChoiceStrategiesContext.getTaskStatisticsRequirements() ).toStrictEqual({ runTime: { aggregate: false, @@ -553,13 +552,13 @@ describe('Selection strategies test suite', () => { ) } expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).nextWorkerNodeKey ).toEqual(expect.any(Number)) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).previousWorkerNodeKey ).toEqual(expect.any(Number)) // We need to clean up the resources after our test @@ -612,13 +611,13 @@ describe('Selection strategies test suite', () => { ) } expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).nextWorkerNodeKey ).toEqual(expect.any(Number)) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).previousWorkerNodeKey ).toEqual(expect.any(Number)) // We need to clean up the resources after our test @@ -632,7 +631,7 @@ describe('Selection strategies test suite', () => { './tests/worker-files/thread/testWorker.mjs', { workerChoiceStrategy } ) - expect(pool.workerChoiceStrategyContext.getStrategyPolicy()).toStrictEqual({ + expect(pool.workerChoiceStrategiesContext.getPolicy()).toStrictEqual({ dynamicWorkerUsage: false, dynamicWorkerReady: true }) @@ -643,7 +642,7 @@ describe('Selection strategies test suite', () => { './tests/worker-files/thread/testWorker.mjs', { workerChoiceStrategy } ) - expect(pool.workerChoiceStrategyContext.getStrategyPolicy()).toStrictEqual({ + expect(pool.workerChoiceStrategiesContext.getPolicy()).toStrictEqual({ dynamicWorkerUsage: false, dynamicWorkerReady: true }) @@ -659,7 +658,7 @@ describe('Selection strategies test suite', () => { { workerChoiceStrategy } ) expect( - pool.workerChoiceStrategyContext.getTaskStatisticsRequirements() + pool.workerChoiceStrategiesContext.getTaskStatisticsRequirements() ).toStrictEqual({ runTime: { aggregate: true, @@ -685,7 +684,7 @@ describe('Selection strategies test suite', () => { { workerChoiceStrategy } ) expect( - pool.workerChoiceStrategyContext.getTaskStatisticsRequirements() + pool.workerChoiceStrategiesContext.getTaskStatisticsRequirements() ).toStrictEqual({ runTime: { aggregate: true, @@ -762,13 +761,13 @@ describe('Selection strategies test suite', () => { } } expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).nextWorkerNodeKey ).toEqual(expect.any(Number)) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).previousWorkerNodeKey ).toEqual(expect.any(Number)) // We need to clean up the resources after our test @@ -831,13 +830,13 @@ describe('Selection strategies test suite', () => { } } expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).nextWorkerNodeKey ).toEqual(expect.any(Number)) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).previousWorkerNodeKey ).toEqual(expect.any(Number)) // We need to clean up the resources after our test @@ -851,7 +850,7 @@ describe('Selection strategies test suite', () => { './tests/worker-files/thread/testWorker.mjs', { workerChoiceStrategy } ) - expect(pool.workerChoiceStrategyContext.getStrategyPolicy()).toStrictEqual({ + expect(pool.workerChoiceStrategiesContext.getPolicy()).toStrictEqual({ dynamicWorkerUsage: false, dynamicWorkerReady: true }) @@ -862,7 +861,7 @@ describe('Selection strategies test suite', () => { './tests/worker-files/thread/testWorker.mjs', { workerChoiceStrategy } ) - expect(pool.workerChoiceStrategyContext.getStrategyPolicy()).toStrictEqual({ + expect(pool.workerChoiceStrategiesContext.getPolicy()).toStrictEqual({ dynamicWorkerUsage: false, dynamicWorkerReady: true }) @@ -878,7 +877,7 @@ describe('Selection strategies test suite', () => { { workerChoiceStrategy } ) expect( - pool.workerChoiceStrategyContext.getTaskStatisticsRequirements() + pool.workerChoiceStrategiesContext.getTaskStatisticsRequirements() ).toStrictEqual({ runTime: { aggregate: false, @@ -904,7 +903,7 @@ describe('Selection strategies test suite', () => { { workerChoiceStrategy } ) expect( - pool.workerChoiceStrategyContext.getTaskStatisticsRequirements() + pool.workerChoiceStrategiesContext.getTaskStatisticsRequirements() ).toStrictEqual({ runTime: { aggregate: false, @@ -987,13 +986,13 @@ describe('Selection strategies test suite', () => { } } expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).nextWorkerNodeKey ).toEqual(expect.any(Number)) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).previousWorkerNodeKey ).toEqual(expect.any(Number)) // We need to clean up the resources after our test @@ -1062,13 +1061,13 @@ describe('Selection strategies test suite', () => { } } expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).nextWorkerNodeKey ).toEqual(expect.any(Number)) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).previousWorkerNodeKey ).toEqual(expect.any(Number)) // We need to clean up the resources after our test @@ -1082,7 +1081,7 @@ describe('Selection strategies test suite', () => { './tests/worker-files/thread/testWorker.mjs', { workerChoiceStrategy } ) - expect(pool.workerChoiceStrategyContext.getStrategyPolicy()).toStrictEqual({ + expect(pool.workerChoiceStrategiesContext.getPolicy()).toStrictEqual({ dynamicWorkerUsage: false, dynamicWorkerReady: true }) @@ -1093,7 +1092,7 @@ describe('Selection strategies test suite', () => { './tests/worker-files/thread/testWorker.mjs', { workerChoiceStrategy } ) - expect(pool.workerChoiceStrategyContext.getStrategyPolicy()).toStrictEqual({ + expect(pool.workerChoiceStrategiesContext.getPolicy()).toStrictEqual({ dynamicWorkerUsage: false, dynamicWorkerReady: true }) @@ -1109,7 +1108,7 @@ describe('Selection strategies test suite', () => { { workerChoiceStrategy } ) expect( - pool.workerChoiceStrategyContext.getTaskStatisticsRequirements() + pool.workerChoiceStrategiesContext.getTaskStatisticsRequirements() ).toStrictEqual({ runTime: { aggregate: true, @@ -1135,7 +1134,7 @@ describe('Selection strategies test suite', () => { { workerChoiceStrategy } ) expect( - pool.workerChoiceStrategyContext.getTaskStatisticsRequirements() + pool.workerChoiceStrategiesContext.getTaskStatisticsRequirements() ).toStrictEqual({ runTime: { aggregate: true, @@ -1229,13 +1228,13 @@ describe('Selection strategies test suite', () => { expect(workerNode.strategyData.virtualTaskEndTimestamp).toBeGreaterThan(0) } expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).nextWorkerNodeKey ).toEqual(expect.any(Number)) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).previousWorkerNodeKey ).toEqual(expect.any(Number)) // We need to clean up the resources after our test @@ -1315,13 +1314,13 @@ describe('Selection strategies test suite', () => { expect(workerNode.strategyData.virtualTaskEndTimestamp).toBeGreaterThan(0) } expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).nextWorkerNodeKey ).toEqual(expect.any(Number)) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).previousWorkerNodeKey ).toEqual(expect.any(Number)) // We need to clean up the resources after our test @@ -1406,20 +1405,20 @@ describe('Selection strategies test suite', () => { expect(workerNode.strategyData.virtualTaskEndTimestamp).toBeGreaterThan(0) } expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).nextWorkerNodeKey ).toEqual(expect.any(Number)) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).previousWorkerNodeKey ).toEqual(expect.any(Number)) // We need to clean up the resources after our test await pool.destroy() }) - it('Verify FAIR_SHARE strategy internals are resets after setting it', async () => { + it("Verify FAIR_SHARE strategy internals aren't reset after setting it", async () => { const workerChoiceStrategy = WorkerChoiceStrategies.FAIR_SHARE let pool = new FixedThreadPool( max, @@ -1432,7 +1431,7 @@ describe('Selection strategies test suite', () => { } pool.setWorkerChoiceStrategy(workerChoiceStrategy) for (const workerNode of pool.workerNodes) { - expect(workerNode.strategyData.virtualTaskEndTimestamp).toBeUndefined() + expect(workerNode.strategyData.virtualTaskEndTimestamp).toBeGreaterThan(0) } await pool.destroy() pool = new DynamicThreadPool( @@ -1447,7 +1446,7 @@ describe('Selection strategies test suite', () => { } pool.setWorkerChoiceStrategy(workerChoiceStrategy) for (const workerNode of pool.workerNodes) { - expect(workerNode.strategyData.virtualTaskEndTimestamp).toBeUndefined() + expect(workerNode.strategyData.virtualTaskEndTimestamp).toBeGreaterThan(0) } // We need to clean up the resources after our test await pool.destroy() @@ -1460,7 +1459,7 @@ describe('Selection strategies test suite', () => { './tests/worker-files/thread/testWorker.mjs', { workerChoiceStrategy } ) - expect(pool.workerChoiceStrategyContext.getStrategyPolicy()).toStrictEqual({ + expect(pool.workerChoiceStrategiesContext.getPolicy()).toStrictEqual({ dynamicWorkerUsage: false, dynamicWorkerReady: true }) @@ -1471,7 +1470,7 @@ describe('Selection strategies test suite', () => { './tests/worker-files/thread/testWorker.mjs', { workerChoiceStrategy } ) - expect(pool.workerChoiceStrategyContext.getStrategyPolicy()).toStrictEqual({ + expect(pool.workerChoiceStrategiesContext.getPolicy()).toStrictEqual({ dynamicWorkerUsage: false, dynamicWorkerReady: true }) @@ -1487,7 +1486,7 @@ describe('Selection strategies test suite', () => { { workerChoiceStrategy } ) expect( - pool.workerChoiceStrategyContext.getTaskStatisticsRequirements() + pool.workerChoiceStrategiesContext.getTaskStatisticsRequirements() ).toStrictEqual({ runTime: { aggregate: true, @@ -1513,7 +1512,7 @@ describe('Selection strategies test suite', () => { { workerChoiceStrategy } ) expect( - pool.workerChoiceStrategyContext.getTaskStatisticsRequirements() + pool.workerChoiceStrategiesContext.getTaskStatisticsRequirements() ).toStrictEqual({ runTime: { aggregate: true, @@ -1590,18 +1589,18 @@ describe('Selection strategies test suite', () => { } } expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).nextWorkerNodeKey ).toBe(0) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).previousWorkerNodeKey ).toEqual(0) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).workerNodeVirtualTaskRunTime ).toBeGreaterThanOrEqual(0) // We need to clean up the resources after our test @@ -1664,18 +1663,18 @@ describe('Selection strategies test suite', () => { } } expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).nextWorkerNodeKey ).toEqual(0) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).previousWorkerNodeKey ).toEqual(0) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).workerNodeVirtualTaskRunTime ).toBeGreaterThanOrEqual(0) // We need to clean up the resources after our test @@ -1743,98 +1742,88 @@ describe('Selection strategies test suite', () => { } } expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).nextWorkerNodeKey ).toEqual(0) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).previousWorkerNodeKey ).toEqual(0) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).workerNodeVirtualTaskRunTime ).toBeGreaterThanOrEqual(0) // We need to clean up the resources after our test await pool.destroy() }) - it('Verify WEIGHTED_ROUND_ROBIN strategy internals are resets after setting it', async () => { + it("Verify WEIGHTED_ROUND_ROBIN strategy internals aren't reset after setting it", async () => { const workerChoiceStrategy = WorkerChoiceStrategies.WEIGHTED_ROUND_ROBIN let pool = new FixedThreadPool( max, - './tests/worker-files/thread/testWorker.mjs' + './tests/worker-files/thread/testWorker.mjs', + { workerChoiceStrategy } ) - expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - workerChoiceStrategy - ).nextWorkerNodeKey - ).toBeDefined() - expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - workerChoiceStrategy - ).previousWorkerNodeKey - ).toBeDefined() - expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - workerChoiceStrategy - ).workerNodeVirtualTaskRunTime - ).toBeDefined() + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + workerChoiceStrategy + ).nextWorkerNodeKey = randomInt(1, max - 1) + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + workerChoiceStrategy + ).previousWorkerNodeKey = randomInt(1, max - 1) + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + workerChoiceStrategy + ).workerNodeVirtualTaskRunTime = randomInt(100, 1000) pool.setWorkerChoiceStrategy(workerChoiceStrategy) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).nextWorkerNodeKey - ).toBe(0) + ).toBeGreaterThan(0) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).previousWorkerNodeKey - ).toBe(0) + ).toBeGreaterThan(0) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).workerNodeVirtualTaskRunTime - ).toBe(0) + ).toBeGreaterThan(99) await pool.destroy() pool = new DynamicThreadPool( min, max, - './tests/worker-files/thread/testWorker.mjs' + './tests/worker-files/thread/testWorker.mjs', + { workerChoiceStrategy } ) - expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - workerChoiceStrategy - ).nextWorkerNodeKey - ).toBeDefined() - expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - workerChoiceStrategy - ).previousWorkerNodeKey - ).toBeDefined() - expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - workerChoiceStrategy - ).workerNodeVirtualTaskRunTime - ).toBeDefined() + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + workerChoiceStrategy + ).nextWorkerNodeKey = randomInt(1, max - 1) + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + workerChoiceStrategy + ).previousWorkerNodeKey = randomInt(1, max - 1) + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + workerChoiceStrategy + ).workerNodeVirtualTaskRunTime = randomInt(100, 1000) pool.setWorkerChoiceStrategy(workerChoiceStrategy) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).nextWorkerNodeKey - ).toBe(0) + ).toBeGreaterThan(0) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).previousWorkerNodeKey - ).toBe(0) + ).toBeGreaterThan(0) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).workerNodeVirtualTaskRunTime - ).toBe(0) + ).toBeGreaterThan(99) // We need to clean up the resources after our test await pool.destroy() }) @@ -1847,7 +1836,7 @@ describe('Selection strategies test suite', () => { './tests/worker-files/thread/testWorker.mjs', { workerChoiceStrategy } ) - expect(pool.workerChoiceStrategyContext.getStrategyPolicy()).toStrictEqual({ + expect(pool.workerChoiceStrategiesContext.getPolicy()).toStrictEqual({ dynamicWorkerUsage: false, dynamicWorkerReady: true }) @@ -1858,7 +1847,7 @@ describe('Selection strategies test suite', () => { './tests/worker-files/thread/testWorker.mjs', { workerChoiceStrategy } ) - expect(pool.workerChoiceStrategyContext.getStrategyPolicy()).toStrictEqual({ + expect(pool.workerChoiceStrategiesContext.getPolicy()).toStrictEqual({ dynamicWorkerUsage: false, dynamicWorkerReady: true }) @@ -1875,7 +1864,7 @@ describe('Selection strategies test suite', () => { { workerChoiceStrategy } ) expect( - pool.workerChoiceStrategyContext.getTaskStatisticsRequirements() + pool.workerChoiceStrategiesContext.getTaskStatisticsRequirements() ).toStrictEqual({ runTime: { aggregate: true, @@ -1901,7 +1890,7 @@ describe('Selection strategies test suite', () => { { workerChoiceStrategy } ) expect( - pool.workerChoiceStrategyContext.getTaskStatisticsRequirements() + pool.workerChoiceStrategiesContext.getTaskStatisticsRequirements() ).toStrictEqual({ runTime: { aggregate: true, @@ -1971,34 +1960,34 @@ describe('Selection strategies test suite', () => { ) } expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).roundId ).toBe(0) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).workerNodeId ).toBe(0) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).nextWorkerNodeKey ).toBe(0) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).previousWorkerNodeKey ).toEqual(0) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).roundWeights.length ).toBe(1) expect( Number.isSafeInteger( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).roundWeights[0] ) ).toBe(true) @@ -2055,34 +2044,34 @@ describe('Selection strategies test suite', () => { ) } expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).roundId ).toBe(0) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).workerNodeId ).toBe(0) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).nextWorkerNodeKey ).toBe(0) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).previousWorkerNodeKey ).toEqual(0) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).roundWeights.length ).toBe(1) expect( Number.isSafeInteger( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).roundWeights[0] ) ).toBe(true) @@ -2090,68 +2079,59 @@ describe('Selection strategies test suite', () => { await pool.destroy() }) - it('Verify INTERLEAVED_WEIGHTED_ROUND_ROBIN strategy internals are resets after setting it', async () => { + it("Verify INTERLEAVED_WEIGHTED_ROUND_ROBIN strategy internals aren't resets after setting it", async () => { const workerChoiceStrategy = WorkerChoiceStrategies.INTERLEAVED_WEIGHTED_ROUND_ROBIN let pool = new FixedThreadPool( max, - './tests/worker-files/thread/testWorker.mjs' + './tests/worker-files/thread/testWorker.mjs', + { workerChoiceStrategy } ) - expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - workerChoiceStrategy - ).roundId - ).toBeDefined() - expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - workerChoiceStrategy - ).workerNodeId - ).toBeDefined() - expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - workerChoiceStrategy - ).nextWorkerNodeKey - ).toBeDefined() - expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - workerChoiceStrategy - ).previousWorkerNodeKey - ).toBeDefined() - expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - workerChoiceStrategy - ).roundWeights - ).toBeDefined() + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + workerChoiceStrategy + ).roundId = randomInt(1, max - 1) + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + workerChoiceStrategy + ).workerNodeId = randomInt(1, max - 1) + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + workerChoiceStrategy + ).nextWorkerNodeKey = randomInt(1, max - 1) + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + workerChoiceStrategy + ).previousWorkerNodeKey = randomInt(1, max - 1) + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + workerChoiceStrategy + ).roundWeights = [randomInt(1, max - 1), randomInt(1, max - 1)] pool.setWorkerChoiceStrategy(workerChoiceStrategy) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).roundId - ).toBe(0) + ).toBeGreaterThan(0) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).workerNodeId - ).toBe(0) + ).toBeGreaterThan(0) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).nextWorkerNodeKey - ).toBe(0) + ).toBeGreaterThan(0) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).previousWorkerNodeKey - ).toBe(0) + ).toBeGreaterThan(0) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).roundWeights.length - ).toBe(1) + ).toBeGreaterThan(1) expect( Number.isSafeInteger( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).roundWeights[0] ) ).toBe(true) @@ -2159,63 +2139,54 @@ describe('Selection strategies test suite', () => { pool = new DynamicThreadPool( min, max, - './tests/worker-files/thread/testWorker.mjs' + './tests/worker-files/thread/testWorker.mjs', + { workerChoiceStrategy } ) - expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - workerChoiceStrategy - ).roundId - ).toBeDefined() - expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - workerChoiceStrategy - ).workerNodeId - ).toBeDefined() - expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - workerChoiceStrategy - ).nextWorkerNodeKey - ).toBeDefined() - expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - workerChoiceStrategy - ).previousWorkerNodeKey - ).toBeDefined() - expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - workerChoiceStrategy - ).roundWeights - ).toBeDefined() + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + workerChoiceStrategy + ).roundId = randomInt(1, max - 1) + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + workerChoiceStrategy + ).workerNodeId = randomInt(1, max - 1) + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + workerChoiceStrategy + ).nextWorkerNodeKey = randomInt(1, max - 1) + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + workerChoiceStrategy + ).previousWorkerNodeKey = randomInt(1, max - 1) + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + workerChoiceStrategy + ).roundWeights = [randomInt(1, max - 1), randomInt(1, max - 1)] pool.setWorkerChoiceStrategy(workerChoiceStrategy) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).roundId - ).toBe(0) + ).toBeGreaterThan(0) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).workerNodeId - ).toBe(0) + ).toBeGreaterThan(0) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).nextWorkerNodeKey - ).toBe(0) + ).toBeGreaterThan(0) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).previousWorkerNodeKey - ).toBe(0) + ).toBeGreaterThan(0) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).roundWeights.length - ).toBe(1) + ).toBeGreaterThan(1) expect( Number.isSafeInteger( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).roundWeights[0] ) ).toBe(true)