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 "danger zone" UI tests #1151

Merged
merged 2 commits into from
Apr 3, 2024
Merged

Conversation

owi92
Copy link
Member

@owi92 owi92 commented Mar 22, 2024

This adds adjusted tests for user realm and regular realm "danger zone" functions, i.e. renaming the realm path (only applicable to regular realms) and deleting realms.

@owi92 owi92 added the changelog:dev Changes primarily for developers label Mar 22, 2024
@owi92 owi92 changed the title Add Danger zone tests Add "danger zone" UI tests Mar 22, 2024
@github-actions github-actions bot temporarily deployed to test-deployment-pr1151 March 22, 2024 17:15 Destroyed
@owi92 owi92 force-pushed the danger-zone-tests branch from f92f504 to 9be64e9 Compare March 22, 2024 17:33
@github-actions github-actions bot temporarily deployed to test-deployment-pr1151 March 22, 2024 17:35 Destroyed
@owi92 owi92 marked this pull request as draft March 23, 2024 16:20
@owi92 owi92 force-pushed the danger-zone-tests branch from 9be64e9 to 79dca90 Compare March 23, 2024 16:46
@github-actions github-actions bot temporarily deployed to test-deployment-pr1151 March 23, 2024 16:51 Destroyed
@owi92 owi92 marked this pull request as ready for review March 23, 2024 16:59
Copy link
Member

@LukasKalbertodt LukasKalbertodt left a comment

Choose a reason for hiding this comment

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

Just a few things.

frontend/tests/fixtures/standard.sql Show resolved Hide resolved
frontend/tests/dangerZone.spec.ts Outdated Show resolved Hide resolved
frontend/tests/dangerZone.spec.ts Outdated Show resolved Hide resolved
frontend/tests/dangerZone.spec.ts Outdated Show resolved Hide resolved
owi92 added 2 commits April 3, 2024 11:34
This adds a user realm and another empty realm as well
as a subrealm to both of these. They can be used in
tests that require such realms but don't specifically
test realm creation.
@owi92 owi92 force-pushed the danger-zone-tests branch from 79dca90 to d562944 Compare April 3, 2024 10:21
@github-actions github-actions bot temporarily deployed to test-deployment-pr1151 April 3, 2024 10:26 Destroyed
@LukasKalbertodt LukasKalbertodt merged commit 47c8f65 into elan-ev:master Apr 3, 2024
3 checks passed
@owi92 owi92 deleted the danger-zone-tests branch December 19, 2024 15:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog:dev Changes primarily for developers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants