Skip to content

Commit c620e5f

Browse files
committed
test: also tests properly for round robin strategy
Signed-off-by: Jérôme Benoit <[email protected]>
1 parent 7e160a5 commit c620e5f

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

tests/pools/selection-strategies/worker-choice-strategies-context.test.mjs

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@ import { InterleavedWeightedRoundRobinWorkerChoiceStrategy } from '../../../src/
1111
import { LeastBusyWorkerChoiceStrategy } from '../../../src/pools/selection-strategies/least-busy-worker-choice-strategy.ts'
1212
// import { LeastEluWorkerChoiceStrategy } from '../../../src/pools/selection-strategies/least-elu-worker-choice-strategy.ts'
1313
import { LeastUsedWorkerChoiceStrategy } from '../../../src/pools/selection-strategies/least-used-worker-choice-strategy.ts'
14+
import { RoundRobinWorkerChoiceStrategy } from '../../../src/pools/selection-strategies/round-robin-worker-choice-strategy.ts'
1415
import { WeightedRoundRobinWorkerChoiceStrategy } from '../../../src/pools/selection-strategies/weighted-round-robin-worker-choice-strategy.ts'
1516
import { WorkerChoiceStrategiesContext } from '../../../src/pools/selection-strategies/worker-choice-strategies-context.ts'
1617

@@ -206,13 +207,13 @@ describe('Worker choice strategies context test suite', () => {
206207
),
207208
).toBeInstanceOf(LeastUsedWorkerChoiceStrategy)
208209
workerChoiceStrategiesContext.setDefaultWorkerChoiceStrategy(
209-
WorkerChoiceStrategies.LEAST_USED,
210+
WorkerChoiceStrategies.ROUND_ROBIN,
210211
)
211212
expect(
212213
workerChoiceStrategiesContext.workerChoiceStrategies.get(
213214
workerChoiceStrategiesContext.defaultWorkerChoiceStrategy,
214215
),
215-
).toBeInstanceOf(LeastUsedWorkerChoiceStrategy)
216+
).toBeInstanceOf(RoundRobinWorkerChoiceStrategy)
216217
})
217218

218219
it('Verify that setDefaultWorkerChoiceStrategy() works with ROUND_ROBIN and dynamic pool', () => {
@@ -225,13 +226,13 @@ describe('Worker choice strategies context test suite', () => {
225226
),
226227
).toBeInstanceOf(LeastUsedWorkerChoiceStrategy)
227228
workerChoiceStrategiesContext.setDefaultWorkerChoiceStrategy(
228-
WorkerChoiceStrategies.LEAST_USED,
229+
WorkerChoiceStrategies.ROUND_ROBIN,
229230
)
230231
expect(
231232
workerChoiceStrategiesContext.workerChoiceStrategies.get(
232233
workerChoiceStrategiesContext.defaultWorkerChoiceStrategy,
233234
),
234-
).toBeInstanceOf(LeastUsedWorkerChoiceStrategy)
235+
).toBeInstanceOf(RoundRobinWorkerChoiceStrategy)
235236
})
236237

237238
it('Verify that setDefaultWorkerChoiceStrategy() works with LEAST_USED and fixed pool', () => {

0 commit comments

Comments
 (0)