Skip to content
This repository was archived by the owner on May 12, 2021. It is now read-only.

Commit 346a852

Browse files
committed
Annotations are not inherited in overridden methods in JUnit 5, so @beforeeach needed in all subclasses
1 parent 6ae1713 commit 346a852

34 files changed

Lines changed: 114 additions & 34 deletions

File tree

core/runtime/src/test/java/org/apache/polygene/runtime/entity/EntityCompositeEqualityTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,8 +44,8 @@ public class EntityCompositeEqualityTest
4444
private UnitOfWork unitOfWork;
4545
private EntityBuilder<MyComposite> myCompositeBuilder;
4646

47-
@BeforeEach
4847
@Override
48+
@BeforeEach
4949
public void setUp()
5050
throws Exception
5151
{

core/runtime/src/test/java/org/apache/polygene/runtime/query/QueryBuilderFactoryImplTest.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -55,6 +55,7 @@ public void assemble( ModuleAssembly module )
5555
module.transients( TestComposite.class );
5656
}
5757

58+
@Override
5859
@BeforeEach
5960
public void setUp()
6061
throws Exception

core/runtime/src/test/java/org/apache/polygene/runtime/value/AssociationToValueTest.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -39,6 +39,7 @@
3939
import org.apache.polygene.bootstrap.ModuleAssembly;
4040
import org.apache.polygene.test.AbstractPolygeneTest;
4141
import org.apache.polygene.test.EntityTestAssembler;
42+
import org.junit.jupiter.api.BeforeEach;
4243
import org.junit.jupiter.api.Test;
4344

4445
import static org.hamcrest.MatcherAssert.assertThat;
@@ -110,6 +111,7 @@ public void assemble( ModuleAssembly module )
110111
}
111112

112113
@Override
114+
@BeforeEach
113115
public void setUp()
114116
throws Exception
115117
{

core/testsupport/src/main/java/org/apache/polygene/test/AbstractPolygeneTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -64,8 +64,8 @@ public abstract class AbstractPolygeneTest extends AbstractPolygeneBaseTest
6464
@Structure
6565
protected ModuleDescriptor module;
6666

67-
@BeforeEach
6867
@Override
68+
@BeforeEach
6969
public void setUp()
7070
throws Exception
7171
{
@@ -87,8 +87,8 @@ protected void defineApplication( ApplicationAssembly applicationAssembly )
8787
assemble( module );
8888
}
8989

90-
@AfterEach
9190
@Override
91+
@AfterEach
9292
public void tearDown()
9393
{
9494
if( unitOfWorkFactory != null && unitOfWorkFactory.isUnitOfWorkActive() )

core/testsupport/src/main/java/org/apache/polygene/test/cache/AbstractCachePoolTest.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,7 @@
2626
import org.apache.polygene.spi.cache.Cache;
2727
import org.apache.polygene.spi.cache.CachePool;
2828
import org.apache.polygene.test.AbstractPolygeneTest;
29+
import org.junit.jupiter.api.BeforeEach;
2930
import org.junit.jupiter.api.Test;
3031

3132
import static org.hamcrest.MatcherAssert.assertThat;
@@ -44,6 +45,7 @@ public abstract class AbstractCachePoolTest
4445
protected Cache<String> cache;
4546

4647
@Override
48+
@BeforeEach
4749
public void setUp()
4850
throws Exception
4951
{

core/testsupport/src/main/java/org/apache/polygene/test/indexing/AbstractAnyQueryTest.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,7 @@
3737
import org.apache.polygene.test.model.Protocol;
3838
import org.apache.polygene.test.model.QueryParam;
3939
import org.apache.polygene.test.model.URL;
40+
import org.junit.jupiter.api.BeforeEach;
4041

4142
/**
4243
* Abstract satisfiedBy with tests for any queries against Index/Query engines.
@@ -76,6 +77,7 @@ protected void assembleValues( ModuleAssembly module, Visibility visibility )
7677
}
7778

7879
@Override
80+
@BeforeEach
7981
public void setUp()
8082
throws Exception
8183
{

core/testsupport/src/main/java/org/apache/polygene/test/indexing/AbstractEntityFinderTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -90,8 +90,8 @@ public abstract class AbstractEntityFinderTest
9090

9191
private EntityFinder entityFinder;
9292

93-
@BeforeEach
9493
@Override
94+
@BeforeEach
9595
public void setUp()
9696
throws Exception
9797
{

extensions/cache-memcache/build.gradle

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,7 @@ dependencies {
3232
runtimeOnly polygene.core.runtime
3333

3434
testImplementation polygene.internals.testsupport
35+
testImplementation 'com.github.faustxvi:junit5-docker:1.0.0-RC5'
3536

3637
testRuntimeOnly libraries.logback
3738
}

extensions/cache-memcache/src/test/java/org/apache/polygene/cache/memcache/MemcacheCachePoolTest.java

Lines changed: 21 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,9 @@
1919
*/
2020
package org.apache.polygene.cache.memcache;
2121

22+
import com.github.junit5docker.Docker;
23+
import com.github.junit5docker.Port;
24+
import com.github.junit5docker.WaitFor;
2225
import org.apache.polygene.api.common.Visibility;
2326
import org.apache.polygene.bootstrap.AssemblyException;
2427
import org.apache.polygene.bootstrap.ModuleAssembly;
@@ -31,18 +34,29 @@
3134
/**
3235
* Memcache CachePool Test.
3336
*/
37+
@Docker( image = "memcached",
38+
ports = @Port( exposed = 11211, inner = 11211 ),
39+
newForEachCase = false )
3440
public class MemcacheCachePoolTest
3541
extends AbstractCachePoolTest
3642
{
37-
@ClassRule
38-
public static final DockerRule DOCKER = new DockerRule( "memcached", 11211 );
43+
// @ClassRule
44+
// public static final DockerRule DOCKER = new DockerRule( "memcached", 11211 );
3945

4046
@Override
4147
// START SNIPPET: assembly
4248
public void assemble( ModuleAssembly module )
4349
throws AssemblyException
4450
{
4551
// END SNIPPET: assembly
52+
try
53+
{
54+
Thread.sleep(10000);
55+
}
56+
catch( InterruptedException e )
57+
{
58+
e.printStackTrace();
59+
}
4660
ModuleAssembly confModule = module.layer().module( "confModule" );
4761
new EntityTestAssembler().visibleIn( Visibility.layer ).assemble( confModule );
4862
// START SNIPPET: assembly
@@ -52,8 +66,11 @@ public void assemble( ModuleAssembly module )
5266
assemble( module );
5367
// END SNIPPET: assembly
5468
MemcacheConfiguration memcacheConf = confModule.forMixin( MemcacheConfiguration.class ).declareDefaults();
55-
String dockerHost = DOCKER.getDockerHost();
56-
int dockerPort = DOCKER.getExposedContainerPort( "11211/tcp" );
69+
// String dockerHost = DOCKER.getDockerHost();
70+
// int dockerPort = DOCKER.getExposedContainerPort( "11211/tcp" );
71+
String dockerHost = "localhost";
72+
int dockerPort = 11211;
73+
5774
memcacheConf.addresses().set( dockerHost + ':' + dockerPort );
5875
memcacheConf.protocol().set( "binary" );
5976
// START SNIPPET: assembly

extensions/entitystore-cassandra/src/test/java/org/apache/polygene/entitystore/cassandra/CassandraEntityStoreTest.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@
2828
import org.apache.polygene.test.entity.AbstractEntityStoreTest;
2929
import org.apache.polygene.test.entity.CanRemoveAll;
3030
import org.junit.ClassRule;
31+
import org.junit.jupiter.api.BeforeEach;
3132

3233
/**
3334
* Test the CassandraEntityStoreService.
@@ -68,6 +69,7 @@ public void assemble( ModuleAssembly module )
6869
// END SNIPPET: assembly
6970

7071
@Override
72+
@BeforeEach
7173
public void setUp()
7274
throws Exception
7375
{

0 commit comments

Comments
 (0)