diff --git a/Jenkinsfile b/Jenkinsfile
index d7b972c25..ca8038866 100644
--- a/Jenkinsfile
+++ b/Jenkinsfile
@@ -46,22 +46,6 @@ pipeline {
}
}
}
- stage('Publish JDK 11 + MongoDB 4.2') {
- when {
- changeset "ci/openjdk11-mongodb-4.2/**"
- }
- agent { label 'data' }
- options { timeout(time: 30, unit: 'MINUTES') }
-
- steps {
- script {
- def image = docker.build("springci/spring-data-openjdk11-with-mongodb-4.2.0", "ci/openjdk11-mongodb-4.2/")
- docker.withRegistry('', 'hub.docker.com-springbuildmaster') {
- image.push()
- }
- }
- }
- }
stage('Publish JDK 13 + MongoDB 4.2') {
when {
changeset "ci/openjdk13-mongodb-4.2/**"
@@ -155,26 +139,6 @@ pipeline {
}
}
- stage("test: baseline (jdk11)") {
- agent {
- docker {
- image 'springci/spring-data-openjdk11-with-mongodb-4.2.0:latest'
- label 'data'
- args '-v $HOME:/tmp/jenkins-home'
- }
- }
- options { timeout(time: 30, unit: 'MINUTES') }
- steps {
- sh 'rm -rf ?'
- sh 'mkdir -p /tmp/mongodb/db /tmp/mongodb/log'
- sh 'mongod --setParameter transactionLifetimeLimitSeconds=90 --setParameter maxTransactionLockRequestTimeoutMillis=10000 --dbpath /tmp/mongodb/db --replSet rs0 --fork --logpath /tmp/mongodb/log/mongod.log &'
- sh 'sleep 10'
- sh 'mongo --eval "rs.initiate({_id: \'rs0\', members:[{_id: 0, host: \'127.0.0.1:27017\'}]});"'
- sh 'sleep 15'
- sh 'MAVEN_OPTS="-Duser.name=jenkins -Duser.home=/tmp/jenkins-home" ./mvnw -Pjava11 clean dependency:list test -Duser.name=jenkins -Dsort -U -B'
- }
- }
-
stage("test: baseline (jdk13)") {
agent {
docker {
diff --git a/spring-data-mongodb/pom.xml b/spring-data-mongodb/pom.xml
index 76df2c09e..872db5d42 100644
--- a/spring-data-mongodb/pom.xml
+++ b/spring-data-mongodb/pom.xml
@@ -344,12 +344,6 @@
src/test/resources/logging.properties
true
-
-
- listener
- org.springframework.data.mongodb.test.util.CleanMongoDBJunitRunListener
-
-
diff --git a/spring-data-mongodb/src/test/java/ConfigClassInDefaultPackageUnitTests.java b/spring-data-mongodb/src/test/java/ConfigClassInDefaultPackageUnitTests.java
index 48450f141..2ba51d37c 100644
--- a/spring-data-mongodb/src/test/java/ConfigClassInDefaultPackageUnitTests.java
+++ b/spring-data-mongodb/src/test/java/ConfigClassInDefaultPackageUnitTests.java
@@ -13,7 +13,9 @@
* See the License for the specific language governing permissions and
* limitations under the License.
*/
-import org.junit.Test;
+
+import org.junit.jupiter.api.Test;
+
import org.springframework.context.annotation.AnnotationConfigApplicationContext;
/**
diff --git a/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/DependencyTests.java b/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/DependencyTests.java
index d1d92aaca..166ae9029 100644
--- a/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/DependencyTests.java
+++ b/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/DependencyTests.java
@@ -16,11 +16,11 @@
package org.springframework.data.mongodb;
import static de.schauderhaft.degraph.check.JCheck.*;
-import static org.junit.Assert.*;
+import static org.hamcrest.MatcherAssert.*;
import de.schauderhaft.degraph.configuration.NamedPattern;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
/**
* Tests package dependency constraints.
@@ -28,10 +28,10 @@ import org.junit.Test;
* @author Jens Schauder
* @author Oliver Gierke
*/
-public class DependencyTests {
+class DependencyTests {
@Test
- public void noInternalPackageCycles() {
+ void noInternalPackageCycles() {
assertThat(classpath() //
.noJars() //
@@ -43,7 +43,7 @@ public class DependencyTests {
}
@Test
- public void onlyConfigMayUseRepository() {
+ void onlyConfigMayUseRepository() {
assertThat(classpath() //
.including("org.springframework.data.**") //
@@ -60,7 +60,7 @@ public class DependencyTests {
}
@Test
- public void commonsInternaly() {
+ void commonsInternaly() {
assertThat(classpath() //
.noJars() //
diff --git a/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/MongoDatabaseUtilsUnitTests.java b/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/MongoDatabaseUtilsUnitTests.java
index 700b6f6a6..b51468e3f 100644
--- a/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/MongoDatabaseUtilsUnitTests.java
+++ b/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/MongoDatabaseUtilsUnitTests.java
@@ -22,12 +22,11 @@ import static org.mockito.Mockito.*;
import javax.transaction.Status;
import javax.transaction.UserTransaction;
-import org.junit.After;
-import org.junit.Before;
-import org.junit.Test;
-import org.junit.runner.RunWith;
+import org.junit.jupiter.api.AfterEach;
+import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.extension.ExtendWith;
import org.mockito.Mock;
-import org.mockito.junit.MockitoJUnitRunner;
+import org.mockito.junit.jupiter.MockitoExtension;
import org.springframework.transaction.TransactionStatus;
import org.springframework.transaction.jta.JtaTransactionManager;
@@ -42,8 +41,8 @@ import com.mongodb.session.ServerSession;
/**
* @author Christoph Strobl
*/
-@RunWith(MockitoJUnitRunner.class)
-public class MongoDatabaseUtilsUnitTests {
+@ExtendWith(MockitoExtension.class)
+class MongoDatabaseUtilsUnitTests {
@Mock ClientSession session;
@Mock ServerSession serverSession;
@@ -52,23 +51,8 @@ public class MongoDatabaseUtilsUnitTests {
@Mock UserTransaction userTransaction;
- @Before
- public void setUp() {
-
- when(dbFactory.getSession(any())).thenReturn(session);
-
- when(dbFactory.withSession(session)).thenReturn(dbFactory);
-
- when(dbFactory.getMongoDatabase()).thenReturn(db);
-
- when(session.getServerSession()).thenReturn(serverSession);
- when(session.hasActiveTransaction()).thenReturn(true);
-
- when(serverSession.isClosed()).thenReturn(false);
- }
-
- @After
- public void verifyTransactionSynchronizationManagerState() {
+ @AfterEach
+ void verifyTransactionSynchronizationManagerState() {
assertThat(TransactionSynchronizationManager.getResourceMap().isEmpty()).isTrue();
assertThat(TransactionSynchronizationManager.isSynchronizationActive()).isFalse();
@@ -79,7 +63,7 @@ public class MongoDatabaseUtilsUnitTests {
}
@Test // DATAMONGO-2130
- public void isTransactionActiveShouldDetectTxViaFactory() {
+ void isTransactionActiveShouldDetectTxViaFactory() {
when(dbFactory.isTransactionActive()).thenReturn(true);
@@ -87,7 +71,7 @@ public class MongoDatabaseUtilsUnitTests {
}
@Test // DATAMONGO-2130
- public void isTransactionActiveShouldReturnFalseIfNoTxActive() {
+ void isTransactionActiveShouldReturnFalseIfNoTxActive() {
when(dbFactory.isTransactionActive()).thenReturn(false);
@@ -95,7 +79,12 @@ public class MongoDatabaseUtilsUnitTests {
}
@Test // DATAMONGO-2130
- public void isTransactionActiveShouldLookupTxForActiveTransactionSynchronizationViaTxManager() {
+ void isTransactionActiveShouldLookupTxForActiveTransactionSynchronizationViaTxManager() {
+
+ when(dbFactory.getSession(any())).thenReturn(session);
+ when(session.getServerSession()).thenReturn(serverSession);
+ when(session.hasActiveTransaction()).thenReturn(true);
+ when(serverSession.isClosed()).thenReturn(false);
when(dbFactory.isTransactionActive()).thenReturn(false);
@@ -112,7 +101,7 @@ public class MongoDatabaseUtilsUnitTests {
}
@Test // DATAMONGO-1920
- public void shouldNotStartSessionWhenNoTransactionOngoing() {
+ void shouldNotStartSessionWhenNoTransactionOngoing() {
MongoDatabaseUtils.getDatabase(dbFactory, SessionSynchronization.ON_ACTUAL_TRANSACTION);
@@ -121,7 +110,14 @@ public class MongoDatabaseUtilsUnitTests {
}
@Test // DATAMONGO-1920
- public void shouldParticipateInOngoingJtaTransactionWithCommitWhenSessionSychronizationIsAny() throws Exception {
+ void shouldParticipateInOngoingJtaTransactionWithCommitWhenSessionSychronizationIsAny() throws Exception {
+
+ when(dbFactory.getSession(any())).thenReturn(session);
+ when(dbFactory.withSession(session)).thenReturn(dbFactory);
+ when(dbFactory.getMongoDatabase()).thenReturn(db);
+ when(session.getServerSession()).thenReturn(serverSession);
+ when(session.hasActiveTransaction()).thenReturn(true);
+ when(serverSession.isClosed()).thenReturn(false);
when(userTransaction.getStatus()).thenReturn(Status.STATUS_NO_TRANSACTION, Status.STATUS_ACTIVE,
Status.STATUS_ACTIVE);
@@ -152,7 +148,14 @@ public class MongoDatabaseUtilsUnitTests {
}
@Test // DATAMONGO-1920
- public void shouldParticipateInOngoingJtaTransactionWithRollbackWhenSessionSychronizationIsAny() throws Exception {
+ void shouldParticipateInOngoingJtaTransactionWithRollbackWhenSessionSychronizationIsAny() throws Exception {
+
+ when(dbFactory.getSession(any())).thenReturn(session);
+ when(dbFactory.withSession(session)).thenReturn(dbFactory);
+ when(dbFactory.getMongoDatabase()).thenReturn(db);
+ when(session.getServerSession()).thenReturn(serverSession);
+ when(session.hasActiveTransaction()).thenReturn(true);
+ when(serverSession.isClosed()).thenReturn(false);
when(userTransaction.getStatus()).thenReturn(Status.STATUS_NO_TRANSACTION, Status.STATUS_ACTIVE,
Status.STATUS_ACTIVE);
@@ -185,8 +188,7 @@ public class MongoDatabaseUtilsUnitTests {
}
@Test // DATAMONGO-1920
- public void shouldNotParticipateInOngoingJtaTransactionWithRollbackWhenSessionSychronizationIsNative()
- throws Exception {
+ void shouldNotParticipateInOngoingJtaTransactionWithRollbackWhenSessionSychronizationIsNative() throws Exception {
when(userTransaction.getStatus()).thenReturn(Status.STATUS_NO_TRANSACTION, Status.STATUS_ACTIVE,
Status.STATUS_ACTIVE);
@@ -219,7 +221,13 @@ public class MongoDatabaseUtilsUnitTests {
}
@Test // DATAMONGO-1920
- public void shouldParticipateInOngoingMongoTransactionWhenSessionSychronizationIsNative() {
+ void shouldParticipateInOngoingMongoTransactionWhenSessionSychronizationIsNative() {
+
+ when(dbFactory.getSession(any())).thenReturn(session);
+ when(dbFactory.withSession(session)).thenReturn(dbFactory);
+ when(dbFactory.getMongoDatabase()).thenReturn(db);
+ when(session.getServerSession()).thenReturn(serverSession);
+ when(serverSession.isClosed()).thenReturn(false);
MongoTransactionManager txManager = new MongoTransactionManager(dbFactory);
TransactionTemplate txTemplate = new TransactionTemplate(txManager);
@@ -245,7 +253,13 @@ public class MongoDatabaseUtilsUnitTests {
}
@Test // DATAMONGO-1920
- public void shouldParticipateInOngoingMongoTransactionWhenSessionSychronizationIsAny() {
+ void shouldParticipateInOngoingMongoTransactionWhenSessionSynchronizationIsAny() {
+
+ when(dbFactory.getSession(any())).thenReturn(session);
+ when(dbFactory.withSession(session)).thenReturn(dbFactory);
+ when(dbFactory.getMongoDatabase()).thenReturn(db);
+ when(session.getServerSession()).thenReturn(serverSession);
+ when(serverSession.isClosed()).thenReturn(false);
MongoTransactionManager txManager = new MongoTransactionManager(dbFactory);
TransactionTemplate txTemplate = new TransactionTemplate(txManager);
diff --git a/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/MongoTransactionManagerUnitTests.java b/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/MongoTransactionManagerUnitTests.java
index e391866ae..338d35ada 100644
--- a/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/MongoTransactionManagerUnitTests.java
+++ b/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/MongoTransactionManagerUnitTests.java
@@ -18,12 +18,12 @@ package org.springframework.data.mongodb;
import static org.assertj.core.api.Assertions.*;
import static org.mockito.Mockito.*;
-import org.junit.After;
-import org.junit.Before;
-import org.junit.Test;
-import org.junit.runner.RunWith;
+import org.junit.jupiter.api.AfterEach;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.extension.ExtendWith;
import org.mockito.Mock;
-import org.mockito.junit.MockitoJUnitRunner;
+import org.mockito.junit.jupiter.MockitoExtension;
import org.springframework.data.mongodb.core.MongoTemplate;
import org.springframework.transaction.TransactionDefinition;
@@ -41,7 +41,7 @@ import com.mongodb.session.ServerSession;
/**
* @author Christoph Strobl
*/
-@RunWith(MockitoJUnitRunner.class)
+@ExtendWith(MockitoExtension.class)
public class MongoTransactionManagerUnitTests {
@Mock ClientSession session;
@@ -52,24 +52,16 @@ public class MongoTransactionManagerUnitTests {
@Mock MongoDatabase db;
@Mock MongoDatabase db2;
- @Before
+ @BeforeEach
public void setUp() {
when(dbFactory.getSession(any())).thenReturn(session, session2);
-
when(dbFactory.withSession(session)).thenReturn(dbFactory);
- when(dbFactory.withSession(session2)).thenReturn(dbFactory2);
-
when(dbFactory.getMongoDatabase()).thenReturn(db);
- when(dbFactory2.getMongoDatabase()).thenReturn(db2);
-
when(session.getServerSession()).thenReturn(serverSession);
- when(session2.getServerSession()).thenReturn(serverSession);
-
- when(serverSession.isClosed()).thenReturn(false);
}
- @After
+ @AfterEach
public void verifyTransactionSynchronizationManager() {
assertThat(TransactionSynchronizationManager.getResourceMap().isEmpty()).isTrue();
@@ -238,6 +230,11 @@ public class MongoTransactionManagerUnitTests {
@Test // DATAMONGO-1920
public void suspendTransactionWhilePropagationRequiresNew() {
+ when(dbFactory.withSession(session2)).thenReturn(dbFactory2);
+ when(dbFactory2.getMongoDatabase()).thenReturn(db2);
+ when(session2.getServerSession()).thenReturn(serverSession);
+ when(serverSession.isClosed()).thenReturn(false);
+
MongoTransactionManager txManager = new MongoTransactionManager(dbFactory);
TransactionStatus txStatus = txManager.getTransaction(new DefaultTransactionDefinition());
diff --git a/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/ReactiveMongoDatabaseUtilsUnitTests.java b/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/ReactiveMongoDatabaseUtilsUnitTests.java
index 9a46ff82a..f05e1c676 100644
--- a/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/ReactiveMongoDatabaseUtilsUnitTests.java
+++ b/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/ReactiveMongoDatabaseUtilsUnitTests.java
@@ -22,11 +22,10 @@ import static org.mockito.Mockito.*;
import reactor.core.publisher.Mono;
import reactor.test.StepVerifier;
-import org.junit.Before;
-import org.junit.Test;
-import org.junit.runner.RunWith;
+import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.extension.ExtendWith;
import org.mockito.Mock;
-import org.mockito.junit.MockitoJUnitRunner;
+import org.mockito.junit.jupiter.MockitoExtension;
import org.springframework.transaction.reactive.TransactionSynchronizationManager;
import org.springframework.transaction.reactive.TransactionalOperator;
@@ -42,26 +41,16 @@ import com.mongodb.session.ServerSession;
* @author Mark Paluch
* @author Christoph Strobl
*/
-@RunWith(MockitoJUnitRunner.class)
-public class ReactiveMongoDatabaseUtilsUnitTests {
+@ExtendWith(MockitoExtension.class)
+class ReactiveMongoDatabaseUtilsUnitTests {
@Mock ClientSession session;
@Mock ServerSession serverSession;
@Mock ReactiveMongoDatabaseFactory databaseFactory;
@Mock MongoDatabase db;
- @Before
- public void setUp() {
-
- when(databaseFactory.getSession(any())).thenReturn(Mono.just(session));
- when(databaseFactory.getMongoDatabase()).thenReturn(db);
-
- when(session.getServerSession()).thenReturn(serverSession);
- when(session.hasActiveTransaction()).thenReturn(true);
- }
-
@Test // DATAMONGO-2265
- public void isTransactionActiveShouldDetectTxViaFactory() {
+ void isTransactionActiveShouldDetectTxViaFactory() {
when(databaseFactory.isTransactionActive()).thenReturn(true);
@@ -71,7 +60,7 @@ public class ReactiveMongoDatabaseUtilsUnitTests {
}
@Test // DATAMONGO-2265
- public void isTransactionActiveShouldReturnFalseIfNoTxActive() {
+ void isTransactionActiveShouldReturnFalseIfNoTxActive() {
when(databaseFactory.isTransactionActive()).thenReturn(false);
@@ -81,8 +70,11 @@ public class ReactiveMongoDatabaseUtilsUnitTests {
}
@Test // DATAMONGO-2265
- public void isTransactionActiveShouldLookupTxForActiveTransactionSynchronizationViaTxManager() {
+ void isTransactionActiveShouldLookupTxForActiveTransactionSynchronizationViaTxManager() {
+ when(session.getServerSession()).thenReturn(serverSession);
+ when(session.hasActiveTransaction()).thenReturn(true);
+ when(databaseFactory.getSession(any())).thenReturn(Mono.just(session));
when(databaseFactory.isTransactionActive()).thenReturn(false);
when(session.commitTransaction()).thenReturn(Mono.empty());
@@ -96,7 +88,9 @@ public class ReactiveMongoDatabaseUtilsUnitTests {
}
@Test // DATAMONGO-2265
- public void shouldNotStartSessionWhenNoTransactionOngoing() {
+ void shouldNotStartSessionWhenNoTransactionOngoing() {
+
+ when(databaseFactory.getMongoDatabase()).thenReturn(db);
ReactiveMongoDatabaseUtils.getDatabase(databaseFactory, SessionSynchronization.ON_ACTUAL_TRANSACTION) //
.as(StepVerifier::create) //
@@ -108,7 +102,10 @@ public class ReactiveMongoDatabaseUtilsUnitTests {
}
@Test // DATAMONGO-2265
- public void shouldParticipateInOngoingMongoTransactionWhenSessionSychronizationIsNative() {
+ void shouldParticipateInOngoingMongoTransactionWhenSessionSychronizationIsNative() {
+
+ when(session.getServerSession()).thenReturn(serverSession);
+ when(databaseFactory.getSession(any())).thenReturn(Mono.just(session));
ReactiveMongoTransactionManager txManager = new ReactiveMongoTransactionManager(databaseFactory);
when(session.abortTransaction()).thenReturn(Mono.empty());
diff --git a/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/ReactiveMongoTransactionManagerUnitTests.java b/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/ReactiveMongoTransactionManagerUnitTests.java
index a0feccb79..d04db2529 100644
--- a/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/ReactiveMongoTransactionManagerUnitTests.java
+++ b/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/ReactiveMongoTransactionManagerUnitTests.java
@@ -15,24 +15,21 @@
*/
package org.springframework.data.mongodb;
-import static org.assertj.core.api.Assertions.*;
import static org.mockito.Mockito.*;
import reactor.core.publisher.Mono;
import reactor.test.StepVerifier;
-import org.junit.After;
-import org.junit.Before;
-import org.junit.Test;
-import org.junit.runner.RunWith;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.extension.ExtendWith;
import org.mockito.Mock;
-import org.mockito.junit.MockitoJUnitRunner;
+import org.mockito.junit.jupiter.MockitoExtension;
import org.springframework.data.mongodb.core.ReactiveMongoTemplate;
import org.springframework.transaction.TransactionDefinition;
import org.springframework.transaction.reactive.TransactionalOperator;
import org.springframework.transaction.support.DefaultTransactionDefinition;
-import org.springframework.transaction.support.TransactionSynchronizationManager;
import com.mongodb.reactivestreams.client.ClientSession;
import com.mongodb.reactivestreams.client.MongoDatabase;
@@ -44,8 +41,8 @@ import com.mongodb.session.ServerSession;
* @author Mark Paluch
* @author Christoph Strobl
*/
-@RunWith(MockitoJUnitRunner.class)
-public class ReactiveMongoTransactionManagerUnitTests {
+@ExtendWith(MockitoExtension.class)
+class ReactiveMongoTransactionManagerUnitTests {
@Mock ClientSession session;
@Mock ClientSession session2;
@@ -55,30 +52,16 @@ public class ReactiveMongoTransactionManagerUnitTests {
@Mock MongoDatabase db;
@Mock MongoDatabase db2;
- @Before
- public void setUp() {
-
+ @BeforeEach
+ void setUp() {
when(databaseFactory.getSession(any())).thenReturn(Mono.just(session), Mono.just(session2));
-
when(databaseFactory.withSession(session)).thenReturn(databaseFactory);
- when(databaseFactory.withSession(session2)).thenReturn(databaseFactory2);
-
when(databaseFactory.getMongoDatabase()).thenReturn(db);
- when(databaseFactory2.getMongoDatabase()).thenReturn(db2);
-
when(session.getServerSession()).thenReturn(serverSession);
- when(session2.getServerSession()).thenReturn(serverSession);
- }
-
- @After
- public void verifyTransactionSynchronizationManager() {
-
- assertThat(TransactionSynchronizationManager.getResourceMap().isEmpty()).isTrue();
- assertThat(TransactionSynchronizationManager.isSynchronizationActive()).isFalse();
}
@Test // DATAMONGO-2265
- public void triggerCommitCorrectly() {
+ void triggerCommitCorrectly() {
ReactiveMongoTransactionManager txManager = new ReactiveMongoTransactionManager(databaseFactory);
ReactiveMongoTemplate template = new ReactiveMongoTemplate(databaseFactory);
@@ -103,7 +86,7 @@ public class ReactiveMongoTransactionManagerUnitTests {
}
@Test // DATAMONGO-2265
- public void participateInOnGoingTransactionWithCommit() {
+ void participateInOnGoingTransactionWithCommit() {
ReactiveMongoTransactionManager txManager = new ReactiveMongoTransactionManager(databaseFactory);
ReactiveMongoTemplate template = new ReactiveMongoTemplate(databaseFactory);
@@ -131,7 +114,7 @@ public class ReactiveMongoTransactionManagerUnitTests {
}
@Test // DATAMONGO-2265
- public void participateInOnGoingTransactionWithRollbackOnly() {
+ void participateInOnGoingTransactionWithRollbackOnly() {
ReactiveMongoTransactionManager txManager = new ReactiveMongoTransactionManager(databaseFactory);
ReactiveMongoTemplate template = new ReactiveMongoTemplate(databaseFactory);
@@ -156,7 +139,7 @@ public class ReactiveMongoTransactionManagerUnitTests {
}
@Test // DATAMONGO-2265
- public void suspendTransactionWhilePropagationNotSupported() {
+ void suspendTransactionWhilePropagationNotSupported() {
ReactiveMongoTransactionManager txManager = new ReactiveMongoTransactionManager(databaseFactory);
ReactiveMongoTemplate template = new ReactiveMongoTemplate(databaseFactory);
@@ -195,7 +178,11 @@ public class ReactiveMongoTransactionManagerUnitTests {
}
@Test // DATAMONGO-2265
- public void suspendTransactionWhilePropagationRequiresNew() {
+ void suspendTransactionWhilePropagationRequiresNew() {
+
+ when(databaseFactory.withSession(session2)).thenReturn(databaseFactory2);
+ when(databaseFactory2.getMongoDatabase()).thenReturn(db2);
+ when(session2.getServerSession()).thenReturn(serverSession);
ReactiveMongoTransactionManager txManager = new ReactiveMongoTransactionManager(databaseFactory);
ReactiveMongoTemplate template = new ReactiveMongoTemplate(databaseFactory);
@@ -237,7 +224,7 @@ public class ReactiveMongoTransactionManagerUnitTests {
}
@Test // DATAMONGO-2265
- public void readonlyShouldInitiateASessionStartAndCommitTransaction() {
+ void readonlyShouldInitiateASessionStartAndCommitTransaction() {
ReactiveMongoTransactionManager txManager = new ReactiveMongoTransactionManager(databaseFactory);
ReactiveMongoTemplate template = new ReactiveMongoTemplate(databaseFactory);
diff --git a/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/SessionAwareMethodInterceptorUnitTests.java b/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/SessionAwareMethodInterceptorUnitTests.java
index 85711d834..e27eb7de5 100644
--- a/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/SessionAwareMethodInterceptorUnitTests.java
+++ b/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/SessionAwareMethodInterceptorUnitTests.java
@@ -24,11 +24,12 @@ import java.lang.reflect.Method;
import java.lang.reflect.Proxy;
import org.bson.Document;
-import org.junit.Before;
-import org.junit.Test;
-import org.junit.runner.RunWith;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.extension.ExtendWith;
import org.mockito.Mock;
-import org.mockito.junit.MockitoJUnitRunner;
+import org.mockito.junit.jupiter.MockitoExtension;
+
import org.springframework.aop.framework.ProxyFactory;
import org.springframework.data.mongodb.SessionAwareMethodInterceptor.MethodCache;
import org.springframework.test.util.ReflectionTestUtils;
@@ -44,7 +45,7 @@ import com.mongodb.client.MongoDatabase;
*
* @author Christoph Strobl
*/
-@RunWith(MockitoJUnitRunner.class)
+@ExtendWith(MockitoExtension.class)
public class SessionAwareMethodInterceptorUnitTests {
@Mock ClientSession session;
@@ -54,7 +55,7 @@ public class SessionAwareMethodInterceptorUnitTests {
MongoCollection collection;
MongoDatabase database;
- @Before
+ @BeforeEach
public void setUp() {
collection = createProxyInstance(session, targetCollection, MongoCollection.class);
diff --git a/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/config/AbstractMongoConfigurationUnitTests.java b/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/config/AbstractMongoConfigurationUnitTests.java
index 50debf50b..d51b6d234 100644
--- a/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/config/AbstractMongoConfigurationUnitTests.java
+++ b/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/config/AbstractMongoConfigurationUnitTests.java
@@ -25,7 +25,8 @@ import java.util.Collection;
import java.util.Collections;
import java.util.Set;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
+
import org.springframework.beans.factory.NoSuchBeanDefinitionException;
import org.springframework.context.annotation.AnnotationConfigApplicationContext;
import org.springframework.context.annotation.Bean;
diff --git a/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/config/AbstractReactiveMongoConfigurationUnitTests.java b/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/config/AbstractReactiveMongoConfigurationUnitTests.java
index 6cef594d6..a89bdd299 100644
--- a/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/config/AbstractReactiveMongoConfigurationUnitTests.java
+++ b/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/config/AbstractReactiveMongoConfigurationUnitTests.java
@@ -25,7 +25,7 @@ import java.util.Collection;
import java.util.Collections;
import java.util.Set;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
import org.mockito.Mockito;
import org.springframework.beans.factory.NoSuchBeanDefinitionException;
import org.springframework.context.annotation.AnnotationConfigApplicationContext;
diff --git a/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/config/AuditingIntegrationTests.java b/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/config/AuditingIntegrationTests.java
index 14712ebcf..d97d97dae 100644
--- a/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/config/AuditingIntegrationTests.java
+++ b/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/config/AuditingIntegrationTests.java
@@ -18,7 +18,7 @@ package org.springframework.data.mongodb.config;
import static org.assertj.core.api.Assertions.*;
import org.joda.time.DateTime;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
import org.springframework.context.support.AbstractApplicationContext;
import org.springframework.context.support.ClassPathXmlApplicationContext;
diff --git a/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/config/MappingMongoConverterParserIntegrationTests.java b/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/config/MappingMongoConverterParserIntegrationTests.java
index 551819050..df7080bed 100644
--- a/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/config/MappingMongoConverterParserIntegrationTests.java
+++ b/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/config/MappingMongoConverterParserIntegrationTests.java
@@ -21,7 +21,7 @@ import java.util.Collections;
import java.util.Set;
import org.bson.Document;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
import org.springframework.beans.factory.config.BeanDefinition;
import org.springframework.beans.factory.config.BeanReference;
diff --git a/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/config/MongoAuditingRegistrarUnitTests.java b/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/config/MongoAuditingRegistrarUnitTests.java
index b4cdc51f9..202264895 100644
--- a/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/config/MongoAuditingRegistrarUnitTests.java
+++ b/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/config/MongoAuditingRegistrarUnitTests.java
@@ -17,10 +17,10 @@ package org.springframework.data.mongodb.config;
import static org.assertj.core.api.Assertions.*;
-import org.junit.Test;
-import org.junit.runner.RunWith;
+import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.extension.ExtendWith;
import org.mockito.Mock;
-import org.mockito.junit.MockitoJUnitRunner;
+import org.mockito.junit.jupiter.MockitoExtension;
import org.springframework.beans.factory.support.BeanDefinitionRegistry;
import org.springframework.core.type.AnnotationMetadata;
@@ -30,21 +30,21 @@ import org.springframework.core.type.AnnotationMetadata;
*
* @author Oliver Gierke
*/
-@RunWith(MockitoJUnitRunner.class)
-public class MongoAuditingRegistrarUnitTests {
+@ExtendWith(MockitoExtension.class)
+class MongoAuditingRegistrarUnitTests {
- MongoAuditingRegistrar registrar = new MongoAuditingRegistrar();
+ private MongoAuditingRegistrar registrar = new MongoAuditingRegistrar();
@Mock AnnotationMetadata metadata;
@Mock BeanDefinitionRegistry registry;
@Test // DATAMONGO-792
- public void rejectsNullAnnotationMetadata() {
+ void rejectsNullAnnotationMetadata() {
assertThatIllegalArgumentException().isThrownBy(() -> registrar.registerBeanDefinitions(null, registry));
}
@Test // DATAMONGO-792
- public void rejectsNullBeanDefinitionRegistry() {
+ void rejectsNullBeanDefinitionRegistry() {
assertThatIllegalArgumentException().isThrownBy(() -> registrar.registerBeanDefinitions(metadata, null));
}
}
diff --git a/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/config/MongoClientParserIntegrationTests.java b/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/config/MongoClientParserIntegrationTests.java
index 62a89ef08..e76740e23 100644
--- a/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/config/MongoClientParserIntegrationTests.java
+++ b/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/config/MongoClientParserIntegrationTests.java
@@ -19,8 +19,8 @@ import static org.assertj.core.api.Assertions.*;
import java.util.concurrent.TimeUnit;
-import org.junit.Before;
-import org.junit.Test;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
import org.springframework.beans.factory.support.BeanDefinitionReader;
import org.springframework.beans.factory.support.DefaultListableBeanFactory;
import org.springframework.beans.factory.xml.XmlBeanDefinitionReader;
@@ -47,7 +47,7 @@ public class MongoClientParserIntegrationTests {
DefaultListableBeanFactory factory;
BeanDefinitionReader reader;
- @Before
+ @BeforeEach
public void setUp() {
this.factory = new DefaultListableBeanFactory();
diff --git a/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/config/MongoDbFactoryParserIntegrationTests.java b/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/config/MongoDbFactoryParserIntegrationTests.java
index 51c8aefd6..129eafc92 100644
--- a/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/config/MongoDbFactoryParserIntegrationTests.java
+++ b/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/config/MongoDbFactoryParserIntegrationTests.java
@@ -17,8 +17,8 @@ package org.springframework.data.mongodb.config;
import static org.assertj.core.api.Assertions.*;
-import org.junit.Before;
-import org.junit.Test;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
import org.springframework.beans.factory.config.BeanDefinition;
import org.springframework.beans.factory.config.ConstructorArgumentValues;
@@ -51,7 +51,7 @@ public class MongoDbFactoryParserIntegrationTests {
DefaultListableBeanFactory factory;
BeanDefinitionReader reader;
- @Before
+ @BeforeEach
public void setUp() {
factory = new DefaultListableBeanFactory();
reader = new XmlBeanDefinitionReader(factory);
diff --git a/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/config/MongoParserIntegrationTests.java b/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/config/MongoParserIntegrationTests.java
index f37eb7ebd..b030622a5 100644
--- a/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/config/MongoParserIntegrationTests.java
+++ b/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/config/MongoParserIntegrationTests.java
@@ -19,9 +19,9 @@ import static org.assertj.core.api.Assertions.*;
import java.util.List;
-import org.junit.Before;
-import org.junit.Ignore;
-import org.junit.Test;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Disabled;
+import org.junit.jupiter.api.Test;
import org.springframework.beans.PropertyValue;
import org.springframework.beans.factory.config.BeanDefinition;
@@ -44,7 +44,7 @@ public class MongoParserIntegrationTests {
DefaultListableBeanFactory factory;
BeanDefinitionReader reader;
- @Before
+ @BeforeEach
public void setUp() {
this.factory = new DefaultListableBeanFactory();
@@ -52,7 +52,7 @@ public class MongoParserIntegrationTests {
}
@Test
- @Ignore
+ @Disabled
public void readsMongoAttributesCorrectly() {
reader.loadBeanDefinitions(new ClassPathResource("namespace/mongo-bean.xml"));
diff --git a/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/config/ReadPreferencePropertyEditorUnitTests.java b/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/config/ReadPreferencePropertyEditorUnitTests.java
index 3053e955c..3aed67e9e 100644
--- a/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/config/ReadPreferencePropertyEditorUnitTests.java
+++ b/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/config/ReadPreferencePropertyEditorUnitTests.java
@@ -17,8 +17,8 @@ package org.springframework.data.mongodb.config;
import static org.assertj.core.api.Assertions.*;
-import org.junit.Before;
-import org.junit.Test;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
import com.mongodb.ReadPreference;
@@ -31,7 +31,7 @@ public class ReadPreferencePropertyEditorUnitTests {
ReadPreferencePropertyEditor editor;
- @Before
+ @BeforeEach
public void setUp() {
editor = new ReadPreferencePropertyEditor();
}
diff --git a/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/config/StringToWriteConcernConverterUnitTests.java b/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/config/StringToWriteConcernConverterUnitTests.java
index ade0f678a..5842fd147 100644
--- a/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/config/StringToWriteConcernConverterUnitTests.java
+++ b/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/config/StringToWriteConcernConverterUnitTests.java
@@ -17,7 +17,7 @@ package org.springframework.data.mongodb.config;
import static org.assertj.core.api.Assertions.*;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
import com.mongodb.WriteConcern;
diff --git a/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/config/WriteConcernPropertyEditorUnitTests.java b/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/config/WriteConcernPropertyEditorUnitTests.java
index ad937d3d3..482339804 100644
--- a/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/config/WriteConcernPropertyEditorUnitTests.java
+++ b/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/config/WriteConcernPropertyEditorUnitTests.java
@@ -17,8 +17,8 @@ package org.springframework.data.mongodb.config;
import static org.assertj.core.api.Assertions.*;
-import org.junit.Before;
-import org.junit.Test;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
import com.mongodb.WriteConcern;
@@ -32,7 +32,7 @@ public class WriteConcernPropertyEditorUnitTests {
WriteConcernPropertyEditor editor;
- @Before
+ @BeforeEach
public void setUp() {
editor = new WriteConcernPropertyEditor();
}
diff --git a/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/core/ChangeStreamOptionsUnitTests.java b/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/core/ChangeStreamOptionsUnitTests.java
index 2f03056ce..635951448 100644
--- a/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/core/ChangeStreamOptionsUnitTests.java
+++ b/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/core/ChangeStreamOptionsUnitTests.java
@@ -18,7 +18,7 @@ package org.springframework.data.mongodb.core;
import static org.assertj.core.api.Assertions.*;
import org.bson.BsonDocument;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
/**
* Unit tests for {@link ChangeStreamOptions}.
diff --git a/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/core/CloseableIterableCursorAdapterUnitTests.java b/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/core/CloseableIterableCursorAdapterUnitTests.java
index 4db0adf31..37417f1c5 100644
--- a/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/core/CloseableIterableCursorAdapterUnitTests.java
+++ b/spring-data-mongodb/src/test/java/org/springframework/data/mongodb/core/CloseableIterableCursorAdapterUnitTests.java
@@ -15,14 +15,16 @@
*/
package org.springframework.data.mongodb.core;
+import static org.assertj.core.api.Assertions.*;
import static org.mockito.Mockito.*;
import org.bson.Document;
-import org.junit.Before;
-import org.junit.Test;
-import org.junit.runner.RunWith;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.extension.ExtendWith;
import org.mockito.Mock;
-import org.mockito.junit.MockitoJUnitRunner;
+import org.mockito.junit.jupiter.MockitoExtension;
+
import org.springframework.dao.support.PersistenceExceptionTranslator;
import org.springframework.data.mongodb.core.MongoTemplate.CloseableIterableCursorAdapter;
import org.springframework.data.mongodb.core.MongoTemplate.DocumentCallback;
@@ -35,40 +37,39 @@ import com.mongodb.client.MongoCursor;
*
* @author Oliver Gierke
*/
-@RunWith(MockitoJUnitRunner.class)
-public class CloseableIterableCursorAdapterUnitTests {
+@ExtendWith(MockitoExtension.class)
+class CloseableIterableCursorAdapterUnitTests {
@Mock PersistenceExceptionTranslator exceptionTranslator;
@Mock DocumentCallback