Skip to content

Commit b50a07a

Browse files
committed
Add yaml and Symfony service container linting
Prior to this change, issues in the symfony ci container yaml were not picked up by qa, resulting in runtime errors.
1 parent 41926c2 commit b50a07a

4 files changed

Lines changed: 15 additions & 2 deletions

File tree

.github/workflows/test-integration.yml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -82,6 +82,10 @@ jobs:
8282
if: always()
8383
run: |
8484
cd docker && docker compose exec -T --user www-data engine.dev.openconext.local bash -c '
85+
echo -e "\nContainer lint\n" && \
86+
bin/console lint:container --env=ci && \
87+
echo -e "\nYAML lint\n" && \
88+
bin/console lint:yaml config/ && \
8589
echo -e "\nTwig lint\n" && \
8690
bin/console lint:twig theme/ && \
8791
cd theme && \

ci/qa/lint.sh

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,12 @@ set -e
33

44
cd $(dirname $0)/../../
55

6+
echo -e "\nContainer lint\n"
7+
bin/console lint:container --env=ci
8+
9+
echo -e "\nYAML lint\n"
10+
bin/console lint:yaml config/
11+
612
echo -e "\nTwig lint\n"
713
bin/console lint:twig theme/
814

config/services/ci/controllers.yml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,9 +24,7 @@ services:
2424
engineblock.functional_test.controller.feedback:
2525
class: OpenConext\EngineBlockFunctionalTestingBundle\Controllers\FeedbackController
2626
arguments:
27-
- '@translator'
2827
- '@twig'
29-
- '@engineblock.compat.logger'
3028
- '@OpenConext\EngineBlock\Service\FeedbackStateHelper'
3129

3230
engineblock.functional_test.controller.consent:

src/OpenConext/EngineBlockFunctionalTestingBundle/Features/FeedbackFooters.feature

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -44,6 +44,11 @@ Feature:
4444
And I should see "support@openconext.org"
4545

4646

47+
48+
Scenario: The functional-testing feedback page renders correctly with feedback-info
49+
When I go to Engineblock URL "/functional-testing/feedback?template=session-lost&feedback-info=%7B%22requestId%22%3A%22test-abc%22%2C%22ipAddress%22%3A%221.2.3.4%22%2C%22artCode%22%3A%2231914%22%7D"
50+
Then I should see "your session was lost"
51+
4752
Scenario: When a IdP specific error page is shown and a translation is not configured the support emailaddress of the IdP should be hidden
4853
Given The clock on the IdP "Dummy Idp" is ahead
4954
And I have configured the following translations:

0 commit comments

Comments
 (0)