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

Create tests for main table export button #4117

Open
jpaten opened this issue Aug 21, 2024 · 3 comments · May be fixed by #4217
Open

Create tests for main table export button #4117

jpaten opened this issue Aug 21, 2024 · 3 comments · May be fixed by #4217
Assignees
@jpaten
Labels
canary Done by the Clever Canary team

Comments

@jpaten
Copy link
Contributor

jpaten commented Aug 21, 2024 •
edited
Loading

Create a test to do the following on at least two tabs each:

  • Check that the sidebar values on the export menu match the values on the main table
  • Attempt to download the smallest possible file from the export menu and check that:
    • Checkboxes and loading screens show properly
    • The download url downloads a file
    • The "Download Manifest" button downloads a file
@github-actions github-actions bot added the canary Done by the Clever Canary team label Aug 21, 2024
@jpaten jpaten self-assigned this Aug 21, 2024
@NoopDog
Copy link
Collaborator

NoopDog commented Oct 24, 2024 •
edited
Loading

@jpaten, if you have any WIP associated with this, can you please push it to a draft PR? Thanks!

jpaten added a commit that referenced this issue Oct 30, 2024
@jpaten
test: first draft index export tests (#4117)
770ec45
jpaten added a commit that referenced this issue Oct 30, 2024
@jpaten
test: reduced duplication between index and details export workflows (#…
f75785c 
…4117)
jpaten added a commit that referenced this issue Oct 30, 2024
@jpaten
test: added readme, cleanup on test descriptions (#4117)
afdeea6
@jpaten jpaten linked a pull request Oct 30, 2024 that will close this issue
Open
@jpaten jpaten linked a pull request Oct 30, 2024 that will close this issue
test: create tests for index export button (#4117) #4217
Open
jpaten added a commit that referenced this issue Oct 30, 2024
@jpaten
test: renamed 'testIndexExportDetails' to 'testIndexExportSummary' (#…
a99ca5d 
…4117)
@jpaten
Copy link
Contributor Author

jpaten commented Nov 1, 2024

Note - Current draft PR had a failure when expecting loading text to be visible, potentially because the text had disappeared before the locator evaluated? Check if there's a way to workaround this, otherwise remove checks that loading screen text is correct.

@jpaten
Copy link
Contributor Author

jpaten commented Nov 1, 2024

@NoopDog I've created a draft PR with my WIP! Still has a couple of issues before it's ready to be merged though.

jpaten added a commit that referenced this issue Nov 4, 2024
@jpaten
test: addressed anvil index flakiness issues (#4117)
7cc3e57
jpaten added a commit that referenced this issue Nov 4, 2024
@jpaten
test: first draft index export tests (#4117)
b603f17
jpaten added a commit that referenced this issue Nov 4, 2024
@jpaten
test: reduced duplication between index and details export workflows (#…
97e9c4b 
…4117)
jpaten added a commit that referenced this issue Nov 4, 2024
@jpaten
test: added readme, cleanup on test descriptions (#4117)
3d74697
jpaten added a commit that referenced this issue Nov 4, 2024
@jpaten
test: renamed 'testIndexExportDetails' to 'testIndexExportSummary' (#…
f9ebce2 
…4117)
jpaten added a commit that referenced this issue Nov 4, 2024
@jpaten
test: addressed anvil index flakiness issues (#4117)
f05f64d
jpaten added a commit that referenced this issue Nov 6, 2024
@jpaten
test: cleaned up test code (#4117)
0c81d37
jpaten added a commit that referenced this issue Nov 6, 2024
@jpaten
test: first draft index export tests (#4117)
20efb58
jpaten added a commit that referenced this issue Nov 6, 2024
@jpaten
test: reduced duplication between index and details export workflows (#…
79dc78e 
…4117)
jpaten added a commit that referenced this issue Nov 6, 2024
@jpaten
test: added readme, cleanup on test descriptions (#4117)
afc61ae
jpaten added a commit that referenced this issue Nov 6, 2024
@jpaten
test: renamed 'testIndexExportDetails' to 'testIndexExportSummary' (#…
fe9b3ed 
…4117)
jpaten added a commit that referenced this issue Nov 6, 2024
@jpaten
test: addressed anvil index flakiness issues (#4117)
0e4889b
jpaten added a commit that referenced this issue Nov 6, 2024
@jpaten
test: cleaned up test code (#4117)
85d46fd
jpaten added a commit that referenced this issue Nov 6, 2024
@jpaten
test: cleaned up test code (#4117)
30b8182
jpaten added a commit that referenced this issue Nov 8, 2024
@jpaten
test: first draft index export tests (#4117)
858a800
jpaten added a commit that referenced this issue Nov 8, 2024
@jpaten
test: reduced duplication between index and details export workflows (#…
74e1772 
…4117)
jpaten added a commit that referenced this issue Nov 8, 2024
@jpaten
test: added readme, cleanup on test descriptions (#4117)
ea3ac5e
jpaten added a commit that referenced this issue Nov 8, 2024
@jpaten
test: renamed 'testIndexExportDetails' to 'testIndexExportSummary' (#…
9aed779 
…4117)
jpaten added a commit that referenced this issue Nov 8, 2024
@jpaten
test: addressed anvil index flakiness issues (#4117)
8e6da72
jpaten added a commit that referenced this issue Nov 8, 2024
@jpaten
test: cleaned up test code (#4117)
2629723
jpaten added a commit that referenced this issue Nov 8, 2024
@jpaten
test: increased timeouts for export action (#4117)
fc99db6
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jpaten jpaten

Labels
canary Done by the Clever Canary team
Projects
None yet
Milestone
No milestone
2 participants
@NoopDog @jpaten