Skip to content

Commit a85bdd5

Browse files
committed
Revert "Container debugging profile" and update pr-*-functional-tests.yml files
1 parent 1f94019 commit a85bdd5

3 files changed

Lines changed: 41 additions & 20 deletions

File tree

.github/workflows/pr-functional-tests.yml

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -30,8 +30,4 @@ jobs:
3030
java-version: ${{ matrix.java }}
3131

3232
- name: Build with Maven
33-
run: mvn -B verify \
34-
-DskipUnitTests=true \
35-
-Drun-functional-tests=true \
36-
-DdockerfileName=Dockerfile \
37-
--file extra/pom.xml
33+
run: mvn -B verify -DskipUnitTests=true -Drun-functional-tests=true -DdockerfileName=Dockerfile --file extra/pom.xml

.github/workflows/pr-module-functional-tests.yml

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -30,13 +30,7 @@ jobs:
3030
java-version: ${{ matrix.java }}
3131

3232
- name: Build with Maven
33-
run: mvn package \
34-
-DskipUnitTests=true \
35-
--file extra/pom.xml
33+
run: mvn package -DskipUnitTests=true --file extra/pom.xml
3634

3735
- name: Run module tests
38-
run: mvn -B verify \
39-
-DskipUnitTests=true \
40-
-Drun-module-functional-tests=true
41-
-DdockerfileName=Dockerfile-modules \
42-
--file extra/pom.xml
36+
run: mvn -B verify -DskipUnitTests=true -Drun-module-functional-tests=true -DdockerfileName=Dockerfile-modules --file extra/pom.xml

pom.xml

Lines changed: 38 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
<?xml version="1.0" encoding="UTF-8"?>
2-
<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
2+
<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
3+
xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
34
<modelVersion>4.0.0</modelVersion>
45

56
<parent>
@@ -372,7 +373,7 @@
372373
<version>${maven-surefire-plugin.version}</version>
373374
<configuration>
374375
<systemPropertyVariables>
375-
<launchContainers>false</launchContainers>
376+
<launchContainers>true</launchContainers>
376377
</systemPropertyVariables>
377378
<skipTests>${skipUnitTests}</skipTests>
378379
<excludeJUnit5Engines>spock</excludeJUnit5Engines>
@@ -385,11 +386,6 @@
385386
<configuration>
386387
<systemPropertyVariables>
387388
<launchContainers>true</launchContainers>
388-
<mockserver.version>${mockserver.version}</mockserver.version>
389-
<pbs.version>${project.version}</pbs.version>
390-
<tests.max-container-count>5</tests.max-container-count>
391-
<!-- Run only one application container instance on fixed ports for debugging purposes -->
392-
<tests.fixed-container-ports>false</tests.fixed-container-ports>
393389
</systemPropertyVariables>
394390
</configuration>
395391
</plugin>
@@ -589,6 +585,23 @@
589585
</build>
590586

591587
<profiles>
588+
<profile>
589+
<!-- Run only one application container instance on fixed ports for debugging purposes -->
590+
<id>Container debugging</id>
591+
<build>
592+
<plugins>
593+
<plugin>
594+
<groupId>org.apache.maven.plugins</groupId>
595+
<artifactId>maven-failsafe-plugin</artifactId>
596+
<configuration>
597+
<systemPropertyVariables>
598+
<tests.fixed-container-ports>true</tests.fixed-container-ports>
599+
</systemPropertyVariables>
600+
</configuration>
601+
</plugin>
602+
</plugins>
603+
</build>
604+
</profile>
592605
<profile>
593606
<id>Run general functional tests without modules</id>
594607
<activation>
@@ -624,6 +637,15 @@
624637
<plugin>
625638
<groupId>org.apache.maven.plugins</groupId>
626639
<artifactId>maven-failsafe-plugin</artifactId>
640+
<configuration>
641+
<systemPropertyVariables>
642+
<mockserver.version>${mockserver.version}</mockserver.version>
643+
<pbs.version>${project.version}</pbs.version>
644+
<tests.max-container-count>5</tests.max-container-count>
645+
<!-- Run only one application container instance on fixed ports for debugging purposes -->
646+
<tests.fixed-container-ports>false</tests.fixed-container-ports>
647+
</systemPropertyVariables>
648+
</configuration>
627649
<executions>
628650
<execution>
629651
<id>general-functional-tests</id>
@@ -680,6 +702,15 @@
680702
<plugin>
681703
<groupId>org.apache.maven.plugins</groupId>
682704
<artifactId>maven-failsafe-plugin</artifactId>
705+
<configuration>
706+
<systemPropertyVariables>
707+
<mockserver.version>${mockserver.version}</mockserver.version>
708+
<pbs.version>${project.version}</pbs.version>
709+
<tests.max-container-count>5</tests.max-container-count>
710+
<!-- Run only one application container instance on fixed ports for debugging purposes -->
711+
<tests.fixed-container-ports>false</tests.fixed-container-ports>
712+
</systemPropertyVariables>
713+
</configuration>
683714
<executions>
684715
<execution>
685716
<id>module-functional-tests</id>

0 commit comments

Comments
 (0)