linkedin / linkedin/parseq

Make BaseEngineTest behave as BaseEngineParTest by default

Open
#127 0 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
Java
Stars
1.2k
Forks
268
PR merge metrics
No merged PRs in 30d

Description

When BaseEngineTest is used in multithreaded unit test it will throw exception explaining that perhaps BaseEngineParTest shoul dbe used instead.
It would be better however to make it default by simply renaming:
BaseEngineTest -> BaseIsolatedEngineTest
BaseEngineParTest -> BaseEngineTest
BaseEngineParTest should be just alias to BaseEngineTest and left for backwards compatibility.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by locating BaseEngineTest and BaseEngineParTest and reviewing their usages in the multithreaded unit tests. Confirm that the default test base behaves like the parallel version, that the isolated name remains available, and that the old parallel name still works as an alias.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
testing
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.