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

Submit testimony e2e #1630

Draft
wants to merge 6 commits into
base: main
Choose a base branch
from
Draft

Conversation

simk209
Copy link
Collaborator

@simk209 simk209 commented Oct 1, 2024

Summary

#1514

Known issues

I am hitting timeout errors related to the Create Testimony button. I have included console logs to check if the page URL updates after clicking on a specific bill within the browse bills page, and it does, however the content does not seem to update. The screenshot taken by playright also shows browse-bills content instead of a specific bill page. This helps explain why playwright cannot find the Create Testimony button, but I am not sure how to remedy it. I have tried adding a longer timeout.

I was also able to log some failed requests, but I am unsure if they are related to this problem.

#playright error screenshot
image

Steps to test/reproduce

run npx playwright test submit-testimony.spec.ts

Copy link

vercel bot commented Oct 1, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
maple-dev ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 23, 2024 8:48pm

@simk209
Copy link
Collaborator Author

simk209 commented Oct 2, 2024

image
getting networkidle issues too, but can successfully waitfor domcontentload. unsure if that provides any clues.

Fixed an issue where the tests would time out when clicking the 'Create Testimony' button in the submit testimony test. Now that I am using getByRole to find the button instead of waitForSelector, all browsers except firefox pass this part of the test. Firefox fails waiting for profile icon in the beforeEach block.
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.

1 participant