Skip to content

Commit b63f4fa

Browse files
novucstim-schilling
authored andcommitted
Actually run the Selenium tests in CI
The matrix entries carry env: followed by a plain scalar, so it parses as the string "DJANGO_SELENIUM_TESTS=true" rather than a mapping, and matrix.env.DJANGO_SELENIUM_TESTS resolves to nothing. The suite has been skipping since the job was added. Make it a mapping. The runner image already provides Firefox and geckodriver, and the tests run headless when CI is set, so no extra setup is needed.
1 parent 263eb79 commit b63f4fa

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

.github/workflows/test.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -133,13 +133,13 @@ jobs:
133133
- python-version: '3.13'
134134
database: psycopg3
135135
env:
136-
DJANGO_SELENIUM_TESTS=true
136+
DJANGO_SELENIUM_TESTS: "true"
137137
- python-version: '3.14'
138138
database: postgis3
139139
- python-version: '3.14'
140140
database: psycopg3
141141
env:
142-
DJANGO_SELENIUM_TESTS=true
142+
DJANGO_SELENIUM_TESTS: "true"
143143

144144
services:
145145
postgres:

0 commit comments

Comments
 (0)