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

feat: cypress integration tests #42

Merged
merged 13 commits into from
Apr 11, 2024
Merged

feat: cypress integration tests #42

merged 13 commits into from
Apr 11, 2024

Conversation

UnicornChance
Copy link
Contributor

Description

Adding cypress integration testing for UDS Core + UDS Identity Config. This tests user authentication and registration processes that can be affected by changing Identity config images.

At this time it's utilizing a test realm.json that disables email verification, MFA, and OCSP checking. Obviously this creates a disparity between realm files, so this issue has been created for hopefully building out more robust integration tests that can also test the email verification and MFA.

Has been re-based against main, so merge conflicts should be resolved.

Tested this in private CI and its work as well as locally. Docs have been included to explain the uds task as well as the different currently implemented tests and how to update the certs that are used by the tests.

To run the tests:
uds run uds-core-integration-tests

Related Issue

Relates to #27

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Other (security config, docs update, etc)

Checklist before merging

@UnicornChance UnicornChance self-assigned this Apr 4, 2024
@UnicornChance UnicornChance requested a review from a team as a code owner April 4, 2024 16:36
@UnicornChance UnicornChance changed the title cypress-integration-tests cypress integration tests Apr 4, 2024
@UnicornChance UnicornChance changed the title cypress integration tests feat: cypress integration tests Apr 4, 2024
@rjferguson21 rjferguson21 mentioned this pull request Apr 10, 2024
5 tasks
rjferguson21
rjferguson21 previously approved these changes Apr 11, 2024
Copy link
Contributor

@mjnagel mjnagel left a comment

Choose a reason for hiding this comment

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

Overall this looks great - really appreciate the work that went into it!

docs/INTEGRATION.md Outdated Show resolved Hide resolved
zarf.yaml Show resolved Hide resolved
src/test/cypress/support/commands.ts Show resolved Hide resolved
@UnicornChance UnicornChance merged commit 12ea0cc into main Apr 11, 2024
6 checks passed
@UnicornChance UnicornChance deleted the integration-tests branch April 12, 2024 14:22
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.

4 participants