X-Git-Url: https://git.piment-noir.org/?a=blobdiff_plain;f=tests%2Fpools%2Fselection-strategies%2Fselection-strategies.test.mjs;h=0e7f19fbdf0abe23685f31059328caa975ced2a3;hb=2717674c56c2caae531507e7d4e0068f4d4c8918;hp=0feb5387ca7dcf162541023b3f0db56c472e4239;hpb=146eaa19f0f0ad8b0aecfef0ad3d8552dd064f33;p=poolifier.git diff --git a/tests/pools/selection-strategies/selection-strategies.test.mjs b/tests/pools/selection-strategies/selection-strategies.test.mjs index 0feb5387..0e7f19fb 100644 --- a/tests/pools/selection-strategies/selection-strategies.test.mjs +++ b/tests/pools/selection-strategies/selection-strategies.test.mjs @@ -1,12 +1,15 @@ +import { randomInt } from 'node:crypto' + import { expect } from 'expect' + +import { CircularArray } from '../../../lib/circular-array.cjs' import { DynamicClusterPool, DynamicThreadPool, FixedClusterPool, FixedThreadPool, WorkerChoiceStrategies -} from '../../../lib/index.js' -import { CircularArray } from '../../../lib/circular-array.js' +} from '../../../lib/index.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,65 +69,40 @@ describe('Selection strategies test suite', () => { ) pool.setWorkerChoiceStrategy(workerChoiceStrategy) expect(pool.opts.workerChoiceStrategy).toBe(workerChoiceStrategy) - expect(pool.workerChoiceStrategyContext.workerChoiceStrategy).toBe( - workerChoiceStrategy - ) - expect(pool.opts.workerChoiceStrategyOptions).toBeUndefined() - expect(pool.workerChoiceStrategyContext.opts).toStrictEqual({ - retries: - pool.info.maxSize + - Object.keys(pool.workerChoiceStrategyContext.opts.weights).length, - runTime: { median: false }, - waitTime: { median: false }, - elu: { median: false }, - weights: expect.objectContaining({ - 0: expect.any(Number), - [pool.info.maxSize - 1]: expect.any(Number) - }) - }) + expect( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy + ).toBe(workerChoiceStrategy) await pool.destroy() } for (const workerChoiceStrategy of Object.values(WorkerChoiceStrategies)) { const pool = new DynamicClusterPool( min, max, - './tests/worker-files/cluster/testWorker.js' + './tests/worker-files/cluster/testWorker.cjs' ) pool.setWorkerChoiceStrategy(workerChoiceStrategy) expect(pool.opts.workerChoiceStrategy).toBe(workerChoiceStrategy) - expect(pool.workerChoiceStrategyContext.workerChoiceStrategy).toBe( - workerChoiceStrategy - ) - expect(pool.opts.workerChoiceStrategyOptions).toBeUndefined() - expect(pool.workerChoiceStrategyContext.opts).toStrictEqual({ - retries: - pool.info.maxSize + - Object.keys(pool.workerChoiceStrategyContext.opts.weights).length, - runTime: { median: false }, - waitTime: { median: false }, - elu: { median: false }, - weights: expect.objectContaining({ - 0: expect.any(Number), - [pool.info.maxSize - 1]: expect.any(Number) - }) - }) + 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) @@ -129,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) @@ -138,47 +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.workerChoiceStrategiesContext.workerChoiceStrategies.get( + workerChoiceStrategy + ).roundWeights[0] + ) + ).toBe(true) } + await pool.destroy() } - await pool.destroy() - }) - - it('Verify strategies wait for worker node readiness in dynamic pool', async () => { - const pool = new DynamicThreadPool( - min, - max, - './tests/worker-files/thread/testWorker.mjs' - ) - expect(pool.starting).toBe(false) - expect(pool.workerNodes.length).toBe(min) - const maxMultiplier = 10000 - const promises = new Set() - for (let i = 0; i < max * maxMultiplier; i++) { - promises.add(pool.execute()) - } - await Promise.all(promises) - expect(pool.workerNodes.length).toBe(max) - // We need to clean up the resources after our test - await pool.destroy() }) it('Verify ROUND_ROBIN strategy default policy', async () => { @@ -188,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 }) @@ -199,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 }) @@ -215,7 +184,7 @@ describe('Selection strategies test suite', () => { { workerChoiceStrategy } ) expect( - pool.workerChoiceStrategyContext.getTaskStatisticsRequirements() + pool.workerChoiceStrategiesContext.getTaskStatisticsRequirements() ).toStrictEqual({ runTime: { aggregate: false, @@ -241,7 +210,7 @@ describe('Selection strategies test suite', () => { { workerChoiceStrategy } ) expect( - pool.workerChoiceStrategyContext.getTaskStatisticsRequirements() + pool.workerChoiceStrategiesContext.getTaskStatisticsRequirements() ).toStrictEqual({ runTime: { aggregate: false, @@ -305,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 @@ -365,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 @@ -382,7 +351,7 @@ describe('Selection strategies test suite', () => { const workerChoiceStrategy = WorkerChoiceStrategies.ROUND_ROBIN let pool = new FixedClusterPool( max, - './tests/worker-files/cluster/testWorker.js', + './tests/worker-files/cluster/testWorker.cjs', { workerChoiceStrategy } ) let results = new Set() @@ -404,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() }) @@ -471,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 }) @@ -482,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 }) @@ -498,7 +459,7 @@ describe('Selection strategies test suite', () => { { workerChoiceStrategy } ) expect( - pool.workerChoiceStrategyContext.getTaskStatisticsRequirements() + pool.workerChoiceStrategiesContext.getTaskStatisticsRequirements() ).toStrictEqual({ runTime: { aggregate: false, @@ -524,7 +485,7 @@ describe('Selection strategies test suite', () => { { workerChoiceStrategy } ) expect( - pool.workerChoiceStrategyContext.getTaskStatisticsRequirements() + pool.workerChoiceStrategiesContext.getTaskStatisticsRequirements() ).toStrictEqual({ runTime: { aggregate: false, @@ -591,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 @@ -650,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 @@ -670,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 }) @@ -681,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 }) @@ -697,7 +658,7 @@ describe('Selection strategies test suite', () => { { workerChoiceStrategy } ) expect( - pool.workerChoiceStrategyContext.getTaskStatisticsRequirements() + pool.workerChoiceStrategiesContext.getTaskStatisticsRequirements() ).toStrictEqual({ runTime: { aggregate: true, @@ -723,7 +684,7 @@ describe('Selection strategies test suite', () => { { workerChoiceStrategy } ) expect( - pool.workerChoiceStrategyContext.getTaskStatisticsRequirements() + pool.workerChoiceStrategiesContext.getTaskStatisticsRequirements() ).toStrictEqual({ runTime: { aggregate: true, @@ -800,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 @@ -869,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 @@ -889,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 }) @@ -900,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 }) @@ -916,7 +877,7 @@ describe('Selection strategies test suite', () => { { workerChoiceStrategy } ) expect( - pool.workerChoiceStrategyContext.getTaskStatisticsRequirements() + pool.workerChoiceStrategiesContext.getTaskStatisticsRequirements() ).toStrictEqual({ runTime: { aggregate: false, @@ -942,7 +903,7 @@ describe('Selection strategies test suite', () => { { workerChoiceStrategy } ) expect( - pool.workerChoiceStrategyContext.getTaskStatisticsRequirements() + pool.workerChoiceStrategiesContext.getTaskStatisticsRequirements() ).toStrictEqual({ runTime: { aggregate: false, @@ -1025,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 @@ -1100,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 @@ -1120,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 }) @@ -1131,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 }) @@ -1147,7 +1108,7 @@ describe('Selection strategies test suite', () => { { workerChoiceStrategy } ) expect( - pool.workerChoiceStrategyContext.getTaskStatisticsRequirements() + pool.workerChoiceStrategiesContext.getTaskStatisticsRequirements() ).toStrictEqual({ runTime: { aggregate: true, @@ -1173,7 +1134,7 @@ describe('Selection strategies test suite', () => { { workerChoiceStrategy } ) expect( - pool.workerChoiceStrategyContext.getTaskStatisticsRequirements() + pool.workerChoiceStrategiesContext.getTaskStatisticsRequirements() ).toStrictEqual({ runTime: { aggregate: true, @@ -1267,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 @@ -1353,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 @@ -1444,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, @@ -1470,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( @@ -1485,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() @@ -1498,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 }) @@ -1509,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 }) @@ -1525,7 +1486,7 @@ describe('Selection strategies test suite', () => { { workerChoiceStrategy } ) expect( - pool.workerChoiceStrategyContext.getTaskStatisticsRequirements() + pool.workerChoiceStrategiesContext.getTaskStatisticsRequirements() ).toStrictEqual({ runTime: { aggregate: true, @@ -1551,7 +1512,7 @@ describe('Selection strategies test suite', () => { { workerChoiceStrategy } ) expect( - pool.workerChoiceStrategyContext.getTaskStatisticsRequirements() + pool.workerChoiceStrategiesContext.getTaskStatisticsRequirements() ).toStrictEqual({ runTime: { aggregate: true, @@ -1628,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(expect.any(Number)) + ).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 @@ -1702,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(expect.any(Number)) + ).toEqual(0) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).previousWorkerNodeKey - ).toEqual(expect.any(Number)) + ).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 @@ -1781,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(expect.any(Number)) + ).toEqual(0) expect( - pool.workerChoiceStrategyContext.workerChoiceStrategies.get( - pool.workerChoiceStrategyContext.workerChoiceStrategy + pool.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy ).previousWorkerNodeKey - ).toEqual(expect.any(Number)) + ).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() }) @@ -1885,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 }) @@ -1896,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 }) @@ -1913,7 +1864,7 @@ describe('Selection strategies test suite', () => { { workerChoiceStrategy } ) expect( - pool.workerChoiceStrategyContext.getTaskStatisticsRequirements() + pool.workerChoiceStrategiesContext.getTaskStatisticsRequirements() ).toStrictEqual({ runTime: { aggregate: true, @@ -1939,7 +1890,7 @@ describe('Selection strategies test suite', () => { { workerChoiceStrategy } ) expect( - pool.workerChoiceStrategyContext.getTaskStatisticsRequirements() + pool.workerChoiceStrategiesContext.getTaskStatisticsRequirements() ).toStrictEqual({ runTime: { aggregate: true, @@ -1961,7 +1912,7 @@ describe('Selection strategies test suite', () => { await pool.destroy() }) - it.skip('Verify INTERLEAVED_WEIGHTED_ROUND_ROBIN strategy can be run in a fixed pool', async () => { + it('Verify INTERLEAVED_WEIGHTED_ROUND_ROBIN strategy can be run in a fixed pool', async () => { const pool = new FixedThreadPool( max, './tests/worker-files/thread/testWorker.mjs', @@ -2009,35 +1960,42 @@ 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(expect.any(Number)) + ).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.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy + ).roundWeights[0] + ) + ).toBe(true) // We need to clean up the resources after our test await pool.destroy() }) - it.skip('Verify INTERLEAVED_WEIGHTED_ROUND_ROBIN strategy can be run in a dynamic pool', async () => { + it('Verify INTERLEAVED_WEIGHTED_ROUND_ROBIN strategy can be run in a dynamic pool', async () => { const pool = new DynamicThreadPool( min, max, @@ -2086,149 +2044,152 @@ 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(expect.any(Number)) + ).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.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy + ).roundWeights[0] + ) + ).toBe(true) // We need to clean up the resources after our test 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.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy + ).roundWeights[0] + ) + ).toBe(true) 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 - ).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.workerChoiceStrategiesContext.workerChoiceStrategies.get( + pool.workerChoiceStrategiesContext.defaultWorkerChoiceStrategy + ).roundWeights[0] + ) + ).toBe(true) // We need to clean up the resources after our test await pool.destroy() })