Skip to content

Commit 71fd30a

Browse files
committed
SONARJAVA-5122 S5977: extend rule to be applied for all Random subclasses
1 parent 737c864 commit 71fd30a

2 files changed

Lines changed: 11 additions & 2 deletions

File tree

java-checks-test-sources/default/src/test/java/checks/tests/RandomizedTestDataCheckSample.java

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,13 @@
11
package checks.tests;
22

3+
import java.security.SecureRandom;
34
import java.util.Random;
45
import java.util.UUID;
6+
7+
import org.apache.commons.lang.math.JVMRandom;
58
import org.junit.jupiter.api.Test;
69

710
public class RandomizedTestDataCheckSample {
8-
911
@Test
1012
public void randomizedTest() {
1113

@@ -36,6 +38,13 @@ public void randomizedTest() {
3638
int age5 = new Random().nextInt(42);
3739
// ^^^^^^^^^^^^<
3840
MyRandom myRandom = new MyRandom(); // Compliant
41+
42+
Random random = new Random();
43+
// ^^^^^^^^^^^^<
44+
SecureRandom random1 = new SecureRandom();
45+
// ^^^^^^^^^^^^^^^^^^<
46+
int age6 = new JVMRandom().nextInt(34);
47+
// ^^^^^^^^^^^^^^^<
3948
}
4049

4150
@Test

java-checks/src/main/java/org/sonar/java/checks/tests/RandomizedTestDataCheck.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@
3232
public class RandomizedTestDataCheck extends IssuableSubscriptionVisitor {
3333

3434
private static final MethodMatchers RANDOM_CONSTRUCTOR_METHOD_MATCHER = MethodMatchers.create()
35-
.ofTypes("java.util.Random")
35+
.ofSubTypes("java.util.Random")
3636
.constructor()
3737
.withAnyParameters()
3838
.build();

0 commit comments

Comments
 (0)