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

Add E2E test for deleting a table #1123

Closed
Tracked by #1122
seancolsen opened this issue Mar 2, 2022 · 2 comments · Fixed by #1159
Closed
Tracked by #1122

Add E2E test for deleting a table #1123

seancolsen opened this issue Mar 2, 2022 · 2 comments · Fixed by #1159
Assignees
@A1O
Labels
good first issue Everything in "Help wanted", PLUS being relatively easy and straightforward to implement. help wanted Community contributors can implement this type: enhancement New feature or request work: frontend Related to frontend code in the mathesar_ui directory
Milestone
[06.1] 2022-03 im...

Comments

@seancolsen
Copy link
Contributor

Context

See #1122 for more info and instructions on writing E2E tests.

This test

Write an E2E test which asserts that "deleting a table" works as-expected. Here's a visual demo of what the test should do:

Peek 2022-03-02 16-41

Tips

  • Assert that the modal closes, that the tab is no longer visible, and that the entry in the sidebar is no longer visible
@seancolsen seancolsen added ready Ready for implementation type: enhancement New feature or request work: frontend Related to frontend code in the mathesar_ui directory good first issue Everything in "Help wanted", PLUS being relatively easy and straightforward to implement. help wanted Community contributors can implement this labels Mar 2, 2022
@seancolsen seancolsen added this to the [12.1] 2022-05 improvements milestone Mar 2, 2022
@seancolsen seancolsen mentioned this issue Mar 2, 2022
Closed
9 tasks
@A1O
Copy link
Contributor

A1O commented Mar 10, 2022

I’d like to work on this issue.

@silentninja silentninja added status: started and removed ready Ready for implementation labels Mar 10, 2022
@silentninja
Copy link
Contributor

Thanks, @A1O! I have assigned you the issue. Let us know if you need any help. Feel free to ask questions here or in Matrix (our chat system).

@A1O A1O mentioned this issue Mar 11, 2022
Merged
7 tasks
@kgodey kgodey modified the milestones: [12.1] 2022-05 improvements, [06.1] 2022-03 improvements Mar 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@A1O A1O

Labels
good first issue Everything in "Help wanted", PLUS being relatively easy and straightforward to implement. help wanted Community contributors can implement this type: enhancement New feature or request work: frontend Related to frontend code in the mathesar_ui directory
Projects
No open projects
[NO LONGER USED] Mathesar work tracker
Archived in project
Milestone
[06.1] 2022-03 improvements
Development

Successfully merging a pull request may close this issue.

E2E test for deleting a table
4 participants
@seancolsen @kgodey @silentninja @A1O