Skip to content

Commit e625ace

Browse files
committed
[actions] Drop usage of ci only tests group as unnecessary / directly set excludedGroups
profile is not needed. All we are doing is setting a property.
1 parent 1a4edef commit e625ace

File tree

1 file changed

+1
-4
lines changed

1 file changed

+1
-4
lines changed

.github/workflows/ci.yaml

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -46,14 +46,11 @@ jobs:
4646
- name: Skip tests that require illegal reflective access
4747
if: ${{ matrix.os != 'ubuntu-latest' && matrix.java != '11' }}
4848
run: echo 'ARG_LINE=-D"excludedGroups=TestcontainersTests,RequireIllegalAccess"' >> $GITHUB_ENV
49-
- name: Set env command line option
50-
if: ${{ matrix.os == 'ubuntu-latest' }}
51-
run: echo 'CI_ONLY_TEST_GROUPS=-D"env.CI_ONLY_TEST_GROUPS"' >> $GITHUB_ENV
5249
- name: Active Profiles
5350
run: ./mvnw help:active-profiles
5451
- name: Test with Maven
5552
if: ${{ matrix.os != 'windows-latest' }}
56-
run: ./mvnw test -B -D"license.skip=true" -D"jacoco.skip=true" $CI_ONLY_TEST_GROUPS $ARG_LINE
53+
run: ./mvnw test -B -D"license.skip=true" -D"jacoco.skip=true" -D"excludedGroups=" $ARG_LINE
5754
- name: Test with Maven
5855
if: ${{ matrix.os == 'windows-latest' && matrix.java == '11' }}
5956
run: ./mvnw test -B -D"license.skip=true" -D"jacoco.skip=true" -D"argLine=--illegal-access=permit"

0 commit comments

Comments
 (0)