Skip to content

Commit 41bab4e

Browse files
committed
[pom] Move surefire configuration to parent and make properties more readable
1 parent 105914b commit 41bab4e

File tree

2 files changed

+2
-14
lines changed

2 files changed

+2
-14
lines changed

.github/workflows/ci.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -48,10 +48,10 @@ jobs:
4848
run: echo 'ARG_LINE=-D"excludedGroups=TestcontainersTests,RequireIllegalAccess"' >> $GITHUB_ENV
4949
- name: Set env command line option
5050
if: ${{ matrix.os == 'ubuntu-latest' }}
51-
run: echo 'ENV_GITHUB="-Denv.GITHUB"' >> $GITHUB_ENV
51+
run: echo 'CI_ONLY_TEST_GROUPS="-Denv.CI_ONLY_TEST_GROUPS"' >> $GITHUB_ENV
5252
- name: Test with Maven
5353
if: ${{ matrix.os != 'windows-latest' }}
54-
run: ./mvnw test -B -D"license.skip=true" -D"jacoco.skip=true" $ENV_GITHUB $ARG_LINE
54+
run: ./mvnw test -B -D"license.skip=true" -D"jacoco.skip=true" $CI_ONLY_TEST_GROUPS $ARG_LINE
5555
- name: Test with Maven
5656
if: ${{ matrix.os == 'windows-latest' && matrix.java == '11' }}
5757
run: ./mvnw test -B -D"license.skip=true" -D"jacoco.skip=true" -D"argLine=--illegal-access=permit"

pom.xml

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -418,18 +418,6 @@
418418
</build>
419419

420420
<profiles>
421-
<profile>
422-
<!-- Run slow tests only on github ci, to force run otherwise use -D"env.GITHUB" -->
423-
<id>github-ci</id>
424-
<activation>
425-
<property>
426-
<name>env.GITHUB</name>
427-
</property>
428-
</activation>
429-
<properties>
430-
<excludedGroups />
431-
</properties>
432-
</profile>
433421
<profile>
434422
<id>pre16</id>
435423
<activation>

0 commit comments

Comments
 (0)