Browse Source

Add nullability annotations to tests in module/spring-boot-jooq

See gh-47263
pull/47415/head
Moritz Halbritter 2 months ago
parent
commit
1f293cc653
  1. 4
      module/spring-boot-jooq/build.gradle
  2. 3
      module/spring-boot-jooq/src/test/java/org/springframework/boot/jooq/autoconfigure/DefaultExceptionTranslatorExecuteListenerTests.java
  3. 2
      module/spring-boot-jooq/src/test/java/org/springframework/boot/jooq/autoconfigure/JooqAutoConfigurationTests.java
  4. 3
      module/spring-boot-jooq/src/test/java/org/springframework/boot/jooq/autoconfigure/JooqPropertiesTests.java

4
module/spring-boot-jooq/build.gradle

@ -43,3 +43,7 @@ dependencies { @@ -43,3 +43,7 @@ dependencies {
testRuntimeOnly("org.glassfish.jaxb:jaxb-runtime")
testRuntimeOnly("org.hsqldb:hsqldb")
}
tasks.named("compileTestJava") {
options.nullability.checking = "tests"
}

3
module/spring-boot-jooq/src/test/java/org/springframework/boot/jooq/autoconfigure/DefaultExceptionTranslatorExecuteListenerTests.java

@ -50,6 +50,7 @@ class DefaultExceptionTranslatorExecuteListenerTests { @@ -50,6 +50,7 @@ class DefaultExceptionTranslatorExecuteListenerTests {
private final ExceptionTranslatorExecuteListener listener = new DefaultExceptionTranslatorExecuteListener();
@Test
@SuppressWarnings("NullAway") // Test null check
void createWhenTranslatorFactoryIsNullThrowsException() {
assertThatIllegalArgumentException()
.isThrownBy(() -> new DefaultExceptionTranslatorExecuteListener(
@ -74,7 +75,7 @@ class DefaultExceptionTranslatorExecuteListenerTests { @@ -74,7 +75,7 @@ class DefaultExceptionTranslatorExecuteListenerTests {
@Test
void exceptionWhenHasCustomTranslatorFactory() {
SQLExceptionTranslator translator = BadSqlGrammarException::new;
SQLExceptionTranslator translator = (task, sql, ex) -> new BadSqlGrammarException(task, "sql", ex);
ExceptionTranslatorExecuteListener listener = new DefaultExceptionTranslatorExecuteListener(
(context) -> translator);
SQLException sqlException = sqlException(123);

2
module/spring-boot-jooq/src/test/java/org/springframework/boot/jooq/autoconfigure/JooqAutoConfigurationTests.java

@ -361,7 +361,7 @@ class JooqAutoConfigurationTests { @@ -361,7 +361,7 @@ class JooqAutoConfigurationTests {
@Override
public ExecuteListener provide() {
return null;
return mock(ExecuteListener.class);
}
}

3
module/spring-boot-jooq/src/test/java/org/springframework/boot/jooq/autoconfigure/JooqPropertiesTests.java

@ -23,6 +23,7 @@ import java.sql.SQLException; @@ -23,6 +23,7 @@ import java.sql.SQLException;
import javax.sql.DataSource;
import org.jooq.SQLDialect;
import org.jspecify.annotations.Nullable;
import org.junit.jupiter.api.AfterEach;
import org.junit.jupiter.api.Test;
@ -44,7 +45,7 @@ import static org.mockito.Mockito.never; @@ -44,7 +45,7 @@ import static org.mockito.Mockito.never;
*/
class JooqPropertiesTests {
private AnnotationConfigApplicationContext context;
private @Nullable AnnotationConfigApplicationContext context;
@AfterEach
void close() {

Loading…
Cancel
Save