Browse Source

Use Integer.parseInt instead of Integer.valueOf for primitive int

Closes gh-25456
pull/25461/head
XenoAmess 6 years ago committed by GitHub
parent
commit
cd12583e13
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
  1. 6
      spring-aspects/src/test/java/org/springframework/cache/aspectj/AbstractCacheAnnotationTests.java
  2. 2
      spring-context/src/test/java/org/springframework/cache/config/EnableCachingIntegrationTests.java
  3. 6
      spring-context/src/testFixtures/java/org/springframework/context/testfixture/cache/AbstractCacheAnnotationTests.java
  4. 2
      spring-core/src/test/java/org/springframework/util/xml/TransformerUtilsTests.java
  5. 2
      spring-websocket/src/test/java/org/springframework/web/socket/sockjs/client/SockJsUrlInfoTests.java

6
spring-aspects/src/test/java/org/springframework/cache/aspectj/AbstractCacheAnnotationTests.java vendored

@ -416,11 +416,11 @@ public abstract class AbstractCacheAnnotationTests { @@ -416,11 +416,11 @@ public abstract class AbstractCacheAnnotationTests {
Integer three = 3;
Cache cache = this.cm.getCache("testCache");
assertThat((int) Integer.valueOf(service.conditionalUpdate(one).toString())).isEqualTo((int) one);
assertThat(Integer.parseInt(service.conditionalUpdate(one).toString())).isEqualTo((int) one);
assertThat(cache.get(one)).isNull();
assertThat((int) Integer.valueOf(service.conditionalUpdate(three).toString())).isEqualTo((int) three);
assertThat((int) Integer.valueOf(cache.get(three).get().toString())).isEqualTo((int) three);
assertThat(Integer.parseInt(service.conditionalUpdate(three).toString())).isEqualTo((int) three);
assertThat(Integer.parseInt(cache.get(three).get().toString())).isEqualTo((int) three);
}
protected void testMultiCache(CacheableService<?> service) {

2
spring-context/src/test/java/org/springframework/cache/config/EnableCachingIntegrationTests.java vendored

@ -195,7 +195,7 @@ public class EnableCachingIntegrationTests { @@ -195,7 +195,7 @@ public class EnableCachingIntegrationTests {
@Bean
public Bar bar() {
return new Bar(Boolean.valueOf(env.getProperty("bar.enabled")));
return new Bar(Boolean.parseBoolean(env.getProperty("bar.enabled")));
}

6
spring-context/src/testFixtures/java/org/springframework/context/testfixture/cache/AbstractCacheAnnotationTests.java vendored

@ -411,11 +411,11 @@ public abstract class AbstractCacheAnnotationTests { @@ -411,11 +411,11 @@ public abstract class AbstractCacheAnnotationTests {
Integer three = 3;
Cache cache = this.cm.getCache("testCache");
assertThat((int) Integer.valueOf(service.conditionalUpdate(one).toString())).isEqualTo((int) one);
assertThat(Integer.parseInt(service.conditionalUpdate(one).toString())).isEqualTo((int) one);
assertThat(cache.get(one)).isNull();
assertThat((int) Integer.valueOf(service.conditionalUpdate(three).toString())).isEqualTo((int) three);
assertThat((int) Integer.valueOf(cache.get(three).get().toString())).isEqualTo((int) three);
assertThat(Integer.parseInt(service.conditionalUpdate(three).toString())).isEqualTo((int) three);
assertThat(Integer.parseInt(cache.get(three).get().toString())).isEqualTo((int) three);
}
protected void testMultiCache(CacheableService<?> service) {

2
spring-core/src/test/java/org/springframework/util/xml/TransformerUtilsTests.java

@ -55,7 +55,7 @@ class TransformerUtilsTests { @@ -55,7 +55,7 @@ class TransformerUtilsTests {
void enableIndentingSunnyDayWithCustomKosherIndentAmount() throws Exception {
final String indentAmountProperty = "10";
Transformer transformer = new StubTransformer();
TransformerUtils.enableIndenting(transformer, Integer.valueOf(indentAmountProperty));
TransformerUtils.enableIndenting(transformer, Integer.parseInt(indentAmountProperty));
String indent = transformer.getOutputProperty(OutputKeys.INDENT);
assertThat(indent).isNotNull();
assertThat(indent).isEqualTo("yes");

2
spring-websocket/src/test/java/org/springframework/web/socket/sockjs/client/SockJsUrlInfoTests.java

@ -34,7 +34,7 @@ public class SockJsUrlInfoTests { @@ -34,7 +34,7 @@ public class SockJsUrlInfoTests {
@Test
public void serverId() throws Exception {
SockJsUrlInfo info = new SockJsUrlInfo(new URI("https://example.com"));
int serverId = Integer.valueOf(info.getServerId());
int serverId = Integer.parseInt(info.getServerId());
assertThat(serverId >= 0 && serverId < 1000).as("Invalid serverId: " + serverId).isTrue();
}

Loading…
Cancel
Save