Skip to content

Commit d1133c6

Browse files
committed
ci: align test matrix with redmine_gtt
Adds the postgis 17-3.5 cell and the system-test include (6.1-stable + Ruby 3.4) so both plugins run the same matrix. The system suite is skipped until test/system exists.
1 parent 59f57a3 commit d1133c6

1 file changed

Lines changed: 17 additions & 5 deletions

File tree

.github/workflows/test-postgis.yml

Lines changed: 17 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -27,8 +27,13 @@ jobs:
2727
# 6.1-stable -> 3.3, 3.4
2828
redmine_version: [6.0-stable, 6.1-stable]
2929
ruby_version: ['3.3', '3.4']
30-
# postgis/postgis tags spanning PG 15/17/18 and PostGIS 3.4/3.5/3.6.
31-
db_version: [15-3.4, 18-3.6]
30+
# postgis/postgis tags spanning PG 15/17/18 and PostGIS 3.4/3.5/3.6
31+
# (floor -> ceiling), matching the redmine_gtt matrix.
32+
db_version: [15-3.4, 17-3.5, 18-3.6]
33+
include:
34+
- system_test: true
35+
redmine_version: 6.1-stable
36+
ruby_version: '3.4'
3237
exclude:
3338
# Redmine 6.0 supports Ruby up to 3.3 only.
3439
- redmine_version: 6.0-stable
@@ -117,12 +122,19 @@ jobs:
117122
- name: Run tests
118123
env:
119124
RAILS_ENV: test
125+
# For system tests in the plugin
126+
GOOGLE_CHROME_OPTS_ARGS: "headless,disable-gpu,no-sandbox,disable-dev-shm-usage"
120127
working-directory: redmine
121128
# Only run suites that exist; `rails test` raises LoadError on a
122-
# missing directory (functional/integration get added as Phase 0
123-
# progresses).
129+
# missing directory (functional/integration/system get added as the
130+
# revival progresses). System tests only run on the designated matrix
131+
# cell, like in redmine_gtt.
124132
run: |
125-
for suite in unit functional integration; do
133+
suites="unit functional integration"
134+
if [ "${{ matrix.system_test }}" = "true" ]; then
135+
suites="$suites system"
136+
fi
137+
for suite in $suites; do
126138
dir="plugins/${{ env.PLUGIN_NAME }}/test/$suite"
127139
if [ -d "$dir" ]; then
128140
bundle exec rails test "$dir"

0 commit comments

Comments
 (0)