Skip to content

Commit 0dcbfdf

Browse files
committed
Use modern Java features
1 parent e890575 commit 0dcbfdf

5 files changed

Lines changed: 5 additions & 7 deletions

File tree

src/main/java/com/maxmind/minfraud/request/CustomInputs.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@
33
import com.fasterxml.jackson.annotation.JsonAnyGetter;
44
import com.maxmind.minfraud.AbstractModel;
55

6-
import java.util.Collections;
76
import java.util.HashMap;
87
import java.util.Map;
98
import java.util.regex.Pattern;
@@ -17,7 +16,7 @@ public final class CustomInputs extends AbstractModel {
1716
private final Map<String, Object> inputs;
1817

1918
private CustomInputs(Builder builder) {
20-
inputs = Collections.unmodifiableMap(new HashMap<>(builder.inputs));
19+
inputs = Map.copyOf(builder.inputs);
2120
}
2221

2322
/**

src/main/java/com/maxmind/minfraud/request/Email.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ public final class Email extends AbstractModel {
2222
private static final Map<String, String> typoDomains;
2323

2424
static {
25-
HashMap<String, String> m = new HashMap<String, String>() {{
25+
HashMap<String, String> m = new HashMap<>() {{
2626
// gmail.com
2727
put("35gmai.com", "gmail.com");
2828
put("636gmail.com", "gmail.com");

src/test/java/com/maxmind/minfraud/request/EmailTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ public void testMultipleAtAddress() {
2929

3030
@Test
3131
public void testAddressWithNoValidation() {
32-
Map<String, String> addresses = new HashMap<String, String>() {{
32+
Map<String, String> addresses = new HashMap<>() {{
3333
put("test", null);
3434
put("@test", "test");
3535
put("test@", null);

src/test/java/com/maxmind/minfraud/request/RequestTestHelper.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@
77
import java.net.URI;
88
import java.net.URISyntaxException;
99
import java.net.URL;
10-
import java.nio.charset.StandardCharsets;
1110
import java.nio.file.Files;
1211
import java.nio.file.Paths;
1312
import java.time.ZonedDateTime;
@@ -153,7 +152,7 @@ private static Transaction makeTransaction(Email e) throws Exception {
153152
public static String readJsonFile(String name) throws IOException, URISyntaxException {
154153
URL resource = RequestTestHelper.class
155154
.getResource("/test-data/" + name + ".json");
156-
return new String(Files.readAllBytes(Paths.get(resource.toURI())), StandardCharsets.UTF_8);
155+
return Files.readString(Paths.get(resource.toURI()));
157156
}
158157

159158
public static void verifyRequestFor(String service, String jsonFile) throws IOException, URISyntaxException {

src/test/java/com/maxmind/minfraud/response/IpAddressTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ public void testIpAddress() throws Exception {
4444
.finish()
4545
);
4646

47-
assertEquals("IP risk", new Double(99), address.getRisk());
47+
assertEquals("IP risk", Double.valueOf(99), address.getRisk());
4848
assertEquals("correct local time", time, address.getLocation().getLocalTime());
4949
assertEquals("1.2.0.0/16", address.getTraits().getNetwork().toString());
5050
assertTrue("isHighRisk", address.getCountry().isHighRisk());

0 commit comments

Comments
 (0)