Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add integration test for OIDC authentication #1995

Merged
merged 2 commits into from
Sep 30, 2024

Conversation

replaceafill
Copy link
Member

Inspired on the integration test of the Storage Service this introduces a Compose environment to set up the Dashboard connected to a Keycloak container to test OIDC authentication.

A new Docker stage has been added to the Dockerfile to set up Firefox and its OS dependencies using playwright for testing the authentication workflows in the web user interface.

Tests have been added to guarantee that:

  • successful OIDC authentication creates a local user in Archivematica's database
  • authentication of existing local users works with OIDC authentication enabled
  • authentication of local users can be disabled by adjusting the authentication backends through settings
  • adjusting the LOGIN_URL setting redirects users to the OIDC authentication page

And finally, a very basic GitHub workflow has also been added which can be triggered manually.

replaceafill and others added 2 commits September 26, 2024 20:05
`playwright` depends on `pytest-base-url` which introduces a new
`base_url` fixture that clashes with the parametrization of these
tests.
Copy link
Contributor

@Dhwaniartefact Dhwaniartefact left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@replaceafill replaceafill merged commit 937f41b into qa/1.x Sep 30, 2024
26 checks passed
@replaceafill replaceafill deleted the dev/oidc-integration-test branch September 30, 2024 22:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants