diff --git a/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/liquibase/LiquibaseAutoConfigurationTests.java b/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/liquibase/LiquibaseAutoConfigurationTests.java index 3015fce73ac..ffd89d60fbe 100644 --- a/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/liquibase/LiquibaseAutoConfigurationTests.java +++ b/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/liquibase/LiquibaseAutoConfigurationTests.java @@ -320,7 +320,7 @@ class LiquibaseAutoConfigurationTests { .withPropertyValues("spring.liquibase.parameters.foo:bar").run(assertLiquibase((liquibase) -> { Map parameters = (Map) ReflectionTestUtils.getField(liquibase, "parameters"); - assertThat(parameters.containsKey("foo")).isTrue(); + assertThat(parameters).containsKey("foo"); assertThat(parameters.get("foo")).isEqualTo("bar"); })); } diff --git a/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/web/servlet/error/BasicErrorControllerIntegrationTests.java b/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/web/servlet/error/BasicErrorControllerIntegrationTests.java index 3535b1e7417..75c988f6154 100755 --- a/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/web/servlet/error/BasicErrorControllerIntegrationTests.java +++ b/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/web/servlet/error/BasicErrorControllerIntegrationTests.java @@ -1,5 +1,5 @@ /* - * Copyright 2012-2020 the original author or authors. + * Copyright 2012-2021 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -84,13 +84,13 @@ class BasicErrorControllerIntegrationTests { } @Test - @SuppressWarnings("rawtypes") + @SuppressWarnings({ "rawtypes", "unchecked" }) void testErrorForMachineClientDefault() { load(); ResponseEntity entity = new TestRestTemplate().getForEntity(createUrl("?trace=true"), Map.class); assertErrorAttributes(entity.getBody(), "500", "Internal Server Error", null, "", "/"); - assertThat(entity.getBody().containsKey("exception")).isFalse(); - assertThat(entity.getBody().containsKey("trace")).isFalse(); + assertThat(entity.getBody()).doesNotContainKey("exception"); + assertThat(entity.getBody()).doesNotContainKey("trace"); } @Test @@ -137,19 +137,19 @@ class BasicErrorControllerIntegrationTests { "No message available", "/noMessage"); } - @SuppressWarnings("rawtypes") + @SuppressWarnings({ "rawtypes", "unchecked" }) private void exceptionWithStackTraceAndMessage(String path) { ResponseEntity entity = new TestRestTemplate().getForEntity(createUrl(path), Map.class); assertErrorAttributes(entity.getBody(), "500", "Internal Server Error", IllegalStateException.class, "Expected!", "/"); - assertThat(entity.getBody().containsKey("trace")).isTrue(); + assertThat(entity.getBody()).containsKey("trace"); } - @SuppressWarnings("rawtypes") + @SuppressWarnings({ "rawtypes", "unchecked" }) private void exceptionWithoutStackTraceAndMessage(String path) { ResponseEntity entity = new TestRestTemplate().getForEntity(createUrl(path), Map.class); assertErrorAttributes(entity.getBody(), "500", "Internal Server Error", IllegalStateException.class, "", "/"); - assertThat(entity.getBody().containsKey("trace")).isFalse(); + assertThat(entity.getBody()).doesNotContainKey("trace"); } @Test @@ -258,37 +258,37 @@ class BasicErrorControllerIntegrationTests { bindingExceptionWithoutMessage("?message=true"); } - @SuppressWarnings({ "rawtypes" }) + @SuppressWarnings({ "rawtypes", "unchecked" }) private void bindingExceptionWithErrors(String param) { ResponseEntity entity = new TestRestTemplate().getForEntity(createUrl("/bind" + param), Map.class); assertErrorAttributes(entity.getBody(), "400", "Bad Request", BindException.class, "", "/bind"); - assertThat(entity.getBody().containsKey("errors")).isTrue(); + assertThat(entity.getBody()).containsKey("errors"); } - @SuppressWarnings({ "rawtypes" }) + @SuppressWarnings({ "rawtypes", "unchecked" }) private void bindingExceptionWithoutErrors(String param) { ResponseEntity entity = new TestRestTemplate().getForEntity(createUrl("/bind" + param), Map.class); assertErrorAttributes(entity.getBody(), "400", "Bad Request", BindException.class, "", "/bind"); - assertThat(entity.getBody().containsKey("errors")).isFalse(); + assertThat(entity.getBody()).doesNotContainKey("errors"); } - @SuppressWarnings({ "rawtypes" }) + @SuppressWarnings({ "rawtypes", "unchecked" }) private void bindingExceptionWithMessage(String param) { ResponseEntity entity = new TestRestTemplate().getForEntity(createUrl("/bind" + param), Map.class); assertErrorAttributes(entity.getBody(), "400", "Bad Request", BindException.class, "Validation failed for object='test'. Error count: 1", "/bind"); - assertThat(entity.getBody().containsKey("errors")).isFalse(); + assertThat(entity.getBody()).doesNotContainKey("errors"); } - @SuppressWarnings({ "rawtypes" }) + @SuppressWarnings({ "rawtypes", "unchecked" }) private void bindingExceptionWithoutMessage(String param) { ResponseEntity entity = new TestRestTemplate().getForEntity(createUrl("/bind" + param), Map.class); assertErrorAttributes(entity.getBody(), "400", "Bad Request", BindException.class, "", "/bind"); - assertThat(entity.getBody().containsKey("errors")).isFalse(); + assertThat(entity.getBody()).doesNotContainKey("errors"); } @Test - @SuppressWarnings("rawtypes") + @SuppressWarnings({ "rawtypes", "unchecked" }) void testRequestBodyValidationForMachineClient() { load("--server.error.include-exception=true"); RequestEntity request = RequestEntity.post(URI.create(createUrl("/bodyValidation"))) @@ -296,19 +296,19 @@ class BasicErrorControllerIntegrationTests { ResponseEntity entity = new TestRestTemplate().exchange(request, Map.class); assertErrorAttributes(entity.getBody(), "400", "Bad Request", MethodArgumentNotValidException.class, "", "/bodyValidation"); - assertThat(entity.getBody().containsKey("errors")).isFalse(); + assertThat(entity.getBody()).doesNotContainKey("errors"); } @Test - @SuppressWarnings("rawtypes") + @SuppressWarnings({ "rawtypes", "unchecked" }) void testBindingExceptionForMachineClientDefault() { load(); RequestEntity request = RequestEntity.get(URI.create(createUrl("/bind?trace=true,message=true"))) .accept(MediaType.APPLICATION_JSON).build(); ResponseEntity entity = new TestRestTemplate().exchange(request, Map.class); - assertThat(entity.getBody().containsKey("exception")).isFalse(); - assertThat(entity.getBody().containsKey("trace")).isFalse(); - assertThat(entity.getBody().containsKey("errors")).isFalse(); + assertThat(entity.getBody()).doesNotContainKey("exception"); + assertThat(entity.getBody()).doesNotContainKey("trace"); + assertThat(entity.getBody()).doesNotContainKey("errors"); } @Test diff --git a/spring-boot-project/spring-boot/src/test/java/org/springframework/boot/web/embedded/undertow/FileSessionPersistenceTests.java b/spring-boot-project/spring-boot/src/test/java/org/springframework/boot/web/embedded/undertow/FileSessionPersistenceTests.java index 80c554651a6..c1952a1ad1f 100644 --- a/spring-boot-project/spring-boot/src/test/java/org/springframework/boot/web/embedded/undertow/FileSessionPersistenceTests.java +++ b/spring-boot-project/spring-boot/src/test/java/org/springframework/boot/web/embedded/undertow/FileSessionPersistenceTests.java @@ -1,5 +1,5 @@ /* - * Copyright 2012-2019 the original author or authors. + * Copyright 2012-2021 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -82,7 +82,7 @@ class FileSessionPersistenceTests { this.persistence.persistSessions("test", sessionData); Map restored = this.persistence.loadSessionAttributes("test", this.classLoader); assertThat(restored).isNotNull(); - assertThat(restored.containsKey("abc")).isFalse(); + assertThat(restored).doesNotContainKey("abc"); } @Test diff --git a/spring-boot-project/spring-boot/src/test/java/org/springframework/boot/web/reactive/error/DefaultErrorAttributesTests.java b/spring-boot-project/spring-boot/src/test/java/org/springframework/boot/web/reactive/error/DefaultErrorAttributesTests.java index 32e22143622..7fbba7b3b5e 100644 --- a/spring-boot-project/spring-boot/src/test/java/org/springframework/boot/web/reactive/error/DefaultErrorAttributesTests.java +++ b/spring-boot-project/spring-boot/src/test/java/org/springframework/boot/web/reactive/error/DefaultErrorAttributesTests.java @@ -256,8 +256,8 @@ class DefaultErrorAttributesTests { MockServerHttpRequest request = MockServerHttpRequest.get("/test").build(); Map attributes = this.errorAttributes.getErrorAttributes(buildServerRequest(request, ex), ErrorAttributeOptions.defaults()); - assertThat(attributes.get("message")).isEqualTo(""); - assertThat(attributes.containsKey("errors")).isFalse(); + assertThat(attributes).containsEntry("message", ""); + assertThat(attributes).doesNotContainKey("errors"); } private ServerRequest buildServerRequest(MockServerHttpRequest request, Throwable error) { diff --git a/spring-boot-project/spring-boot/src/test/java/org/springframework/boot/web/servlet/error/DefaultErrorAttributesTests.java b/spring-boot-project/spring-boot/src/test/java/org/springframework/boot/web/servlet/error/DefaultErrorAttributesTests.java index 6a882e9b0f1..491ade7d78f 100644 --- a/spring-boot-project/spring-boot/src/test/java/org/springframework/boot/web/servlet/error/DefaultErrorAttributesTests.java +++ b/spring-boot-project/spring-boot/src/test/java/org/springframework/boot/web/servlet/error/DefaultErrorAttributesTests.java @@ -90,7 +90,7 @@ class DefaultErrorAttributesTests { ErrorAttributeOptions.of(Include.MESSAGE)); assertThat(this.errorAttributes.getError(this.webRequest)).isSameAs(ex); assertThat(modelAndView).isNull(); - assertThat(attributes.containsKey("exception")).isFalse(); + assertThat(attributes).doesNotContainKey("exception"); assertThat(attributes.get("message")).isEqualTo("Test"); } @@ -101,7 +101,7 @@ class DefaultErrorAttributesTests { Map attributes = this.errorAttributes.getErrorAttributes(this.webRequest, ErrorAttributeOptions.of(Include.MESSAGE)); assertThat(this.errorAttributes.getError(this.webRequest)).isSameAs(ex); - assertThat(attributes.containsKey("exception")).isFalse(); + assertThat(attributes).doesNotContainKey("exception"); assertThat(attributes.get("message")).isEqualTo("Test"); } @@ -112,7 +112,7 @@ class DefaultErrorAttributesTests { Map attributes = this.errorAttributes.getErrorAttributes(this.webRequest, ErrorAttributeOptions.defaults()); assertThat(this.errorAttributes.getError(this.webRequest)).isSameAs(ex); - assertThat(attributes.containsKey("exception")).isFalse(); + assertThat(attributes).doesNotContainKey("exception"); assertThat(attributes.get("message").toString()).contains(""); } @@ -121,7 +121,7 @@ class DefaultErrorAttributesTests { this.request.setAttribute("javax.servlet.error.message", "Test"); Map attributes = this.errorAttributes.getErrorAttributes(this.webRequest, ErrorAttributeOptions.of(Include.MESSAGE)); - assertThat(attributes.containsKey("exception")).isFalse(); + assertThat(attributes).doesNotContainKey("exception"); assertThat(attributes.get("message")).isEqualTo("Test"); } @@ -130,7 +130,7 @@ class DefaultErrorAttributesTests { this.request.setAttribute("javax.servlet.error.message", "Test"); Map attributes = this.errorAttributes.getErrorAttributes(this.webRequest, ErrorAttributeOptions.defaults()); - assertThat(attributes.containsKey("exception")).isFalse(); + assertThat(attributes).doesNotContainKey("exception"); assertThat(attributes.get("message")).asString().contains(""); } @@ -140,7 +140,7 @@ class DefaultErrorAttributesTests { this.request.setAttribute("javax.servlet.error.message", "Test"); Map attributes = this.errorAttributes.getErrorAttributes(this.webRequest, ErrorAttributeOptions.of(Include.MESSAGE)); - assertThat(attributes.containsKey("exception")).isFalse(); + assertThat(attributes).doesNotContainKey("exception"); assertThat(attributes.get("message")).isEqualTo("Test"); } @@ -149,7 +149,7 @@ class DefaultErrorAttributesTests { this.request.setAttribute("javax.servlet.error.exception", new RuntimeException()); Map attributes = this.errorAttributes.getErrorAttributes(this.webRequest, ErrorAttributeOptions.of(Include.MESSAGE)); - assertThat(attributes.containsKey("exception")).isFalse(); + assertThat(attributes).doesNotContainKey("exception"); assertThat(attributes.get("message")).isEqualTo("No message available"); } @@ -161,7 +161,7 @@ class DefaultErrorAttributesTests { Map attributes = this.errorAttributes.getErrorAttributes(this.webRequest, ErrorAttributeOptions.of(Include.MESSAGE)); assertThat(this.errorAttributes.getError(this.webRequest)).isSameAs(wrapped); - assertThat(attributes.containsKey("exception")).isFalse(); + assertThat(attributes).doesNotContainKey("exception"); assertThat(attributes.get("message")).isEqualTo("Test"); } @@ -172,7 +172,7 @@ class DefaultErrorAttributesTests { Map attributes = this.errorAttributes.getErrorAttributes(this.webRequest, ErrorAttributeOptions.of(Include.MESSAGE)); assertThat(this.errorAttributes.getError(this.webRequest)).isSameAs(error); - assertThat(attributes.containsKey("exception")).isFalse(); + assertThat(attributes).doesNotContainKey("exception"); assertThat(attributes.get("message")).isEqualTo("Test error"); } @@ -216,7 +216,7 @@ class DefaultErrorAttributesTests { assertThat(attributes.get("errors")).isEqualTo(bindingResult.getAllErrors()); } else { - assertThat(attributes.containsKey("errors")).isFalse(); + assertThat(attributes).doesNotContainKey("errors"); } } @@ -246,7 +246,7 @@ class DefaultErrorAttributesTests { this.request.setAttribute("javax.servlet.error.exception", ex); Map attributes = this.errorAttributes.getErrorAttributes(this.webRequest, ErrorAttributeOptions.defaults()); - assertThat(attributes.containsKey("trace")).isFalse(); + assertThat(attributes).doesNotContainKey("trace"); } @Test