Skip to content

Commit

Permalink
Set correct uri for WebTests
Browse files Browse the repository at this point in the history
  • Loading branch information
MKodde committed Dec 19, 2023
1 parent 73c549f commit 4590b1b
Show file tree
Hide file tree
Showing 4 changed files with 30 additions and 3 deletions.
5 changes: 4 additions & 1 deletion .github/workflows/test-acceptance.yml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
name: Run acceptance tests (Behat)
name: Run acceptance tests (WebTests & Behat)
on: [pull_request]

jobs:
Expand All @@ -19,6 +19,9 @@ jobs:
- name: Copy certificate material
run: mkdir /config && cp -r ./ci/config/* /config/

- name: Grab the right parameters.yaml
run: cp ./config/openconext/parameters.yaml.test.dist ./config/openconext/parameters.yaml

- name: Composer install
run: composer install

Expand Down
24 changes: 24 additions & 0 deletions config/openconext/parameters.yaml.test.dist
Original file line number Diff line number Diff line change
@@ -0,0 +1,24 @@
parameters:
app_env: test
app_debug: false
app_secret: [email protected]

support_url:
en_GB: "https://support.example.org/faq-strong-authentication"
nl_NL: "https://support.example.org/faq-sterke-authenticatie"
support_email: [email protected]
locales:
- nl_NL
- en_GB
default_locale: en_GB


saml_idp_publickey: '/config/azuremfa/azuremfa_idp.crt'
saml_idp_privatekey: '/config/azuremfa/azuremfa_idp.key'
saml_metadata_publickey: '/config/azuremfa/azuremfa_idp.crt'
saml_metadata_privatekey: '/config/axuremfa/azuremfa_idp.key'
saml_remote_sp_entity_id: 'https://gateway.dev.openconext.local/gssp/azuremfa/metadata'
saml_remote_sp_certificate: '/config/gateway/gateway_gssp_sp.crt'
saml_remote_sp_acs: 'https://gateway.dev.openconext.local/gssp/azuremfa/consume-assertion'
# Authentication issuers matching this regex will send ForceAuthn to the Azure MFA. The at sign is used as delimiter, be sure to escape the first sign as Symfony would see it as a reference to a service.
ra_issuer_entity_id_regex: '@@^https://(?:[a-z0-9](?:[a-z0-9-]{0,61}[a-z0-9])?\.)+[a-z0-9][a-z0-9-]{0,61}[a-z0-9]/vetting-procedure/gssf/azuremfa/metadata$@'
2 changes: 1 addition & 1 deletion config/packages/framework.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ framework:
trusted_proxies: '192.0.0.1,10.0.0.0/8'
trusted_headers: [ 'x-forwarded-for', 'x-forwarded-host', 'x-forwarded-proto', 'x-forwarded-port', 'x-forwarded-prefix' ]

secret: '%app_env%'
secret: '%app_secret%'
csrf_protection: true
http_method_override: true
error_controller: Surfnet\AzureMfa\Infrastructure\Controller\ExceptionController::show
Expand Down
2 changes: 1 addition & 1 deletion tests/Functional/WebTests/DefaultControllerTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ class DefaultControllerTest extends WebTestCase
public function testIndex()
{
$client = static::createClient();
$crawler = $client->request('GET', '/');
$crawler = $client->request('GET', 'https://azuremfa.dev.openconext.local');
$this->assertEquals(200, $client->getInternalResponse()->getStatusCode());
$this->assertStringContainsString('Welcome to the Azure MFA', $crawler->filter('h2')->text());
}
Expand Down

0 comments on commit 4590b1b

Please sign in to comment.