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.

Sign up for GitHub

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

Jump to bottom

enh: add context e2e tests #1149

Merged
merged 3 commits into from
Jul 3, 2024
Merged

enh: add context e2e tests #1149

merged 3 commits into from
Jul 3, 2024

Conversation

enjeck
Copy link
Contributor

@enjeck enjeck commented Jun 17, 2024 •
edited
Loading

Fixes #994

  • Create a new application
  • Add a table to it
  • Add a view to it
  • Check that both are rendered
  • Share with a user
  • Check that the user receives the context
  • Check that the application menu entry shows up
  • Deleting an application (for both owner and shares)
  • Transfering an application
  • Modifying permissions and creating, editing, deleting rows/columns

@enjeck enjeck self-assigned this Jun 17, 2024
@enjeck enjeck force-pushed the context-cypress branch 2 times, most recently from 2249dab to 16c9830 Compare June 20, 2024 06:57
@enjeck enjeck marked this pull request as ready for review June 28, 2024 06:25
@enjeck enjeck force-pushed the context-cypress branch 2 times, most recently from 68fd75d to 535f205 Compare June 28, 2024 07:17
blizzz
blizzz approved these changes Jun 28, 2024
View reviewed changes
Copy link
Member

@blizzz blizzz left a comment

Choose a reason for hiding this comment

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

just one test

@enjeck enjeck changed the base branch from main to fix-context-update-nodes July 2, 2024 13:54
@enjeck
enh: add context e2e tests
9abe8d0 
Signed-off-by: Cleopatra Enjeck M <patrathewhiz@gmail.com>
@enjeck
enh: add more context e2e tests
3233487 
Signed-off-by: Cleopatra Enjeck M <patrathewhiz@gmail.com>
@enjeck
fix: update context e2e tests selector
7ff4ab7 
Signed-off-by: Cleopatra Enjeck M <patrathewhiz@gmail.com>
@enjeck
Copy link
Contributor Author

enjeck commented Jul 2, 2024

Needed a fix at #1178 to make that test pass

@juliushaertl juliushaertl merged commit 90defc8 into fix-context-update-nodes Jul 3, 2024
42 checks passed
@juliushaertl juliushaertl deleted the context-cypress branch July 3, 2024 20:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@blizzz blizzz blizzz approved these changes

@elzody elzody elzody approved these changes

@juliushaertl juliushaertl Awaiting requested review from juliushaertl

Assignees

@enjeck enjeck

Labels
None yet
Projects
📝 Office team
Status: ☑️ Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Applications: Frontend tests
4 participants
@enjeck @blizzz @elzody @juliushaertl