Approved.

Sample tests are still unfixed.

fixed

Issue: JUnit uses assertEquals(expected, actual) argument order.

Since JDK 17 java.util.Random implements the RandomGenerator interface which provides default implementations of nextInt(lo, hi) for all random classes. Nevertheless ThreadLocalRandom has the nicer prng algorithm.

• Issue: This is meant to be a bug fixing kata, but your initial solution setup has no real bug (ther than missing return).
• Issue: Random testss usually should have more than one test case. Usually it's aorund 100, but for this kata 10 or 20 should be enough.
• Style: Math.random(), really? There are Random::ints(lo, hi), and Random::nextInt(hi), and ThreadLocalRandom::nextInt(lo, hi) today.