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

Refactor Fillet Playwright Test to the new Fixture-based Approach #4909

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

Conversation

max-mrgrsk
Copy link
Collaborator

Description:

Refactor the existing test to use the new fixture-based testing approach. This will ensure compatibility with Electron and browser environments and align our tests with the team’s current standards.

Closes: #4821

@max-mrgrsk max-mrgrsk linked an issue Jan 3, 2025 that may be closed by this pull request
Copy link

qa-wolf bot commented Jan 3, 2025

QA Wolf here! As you write new code it's important that your test coverage is keeping up.
Click here to request test coverage for this PR!

Copy link

vercel bot commented Jan 3, 2025

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

Name Status Preview Updated (UTC)
modeling-app ✅ Ready (Inspect) Visit Preview Jan 3, 2025 7:45pm

@max-mrgrsk max-mrgrsk self-assigned this Jan 3, 2025
@max-mrgrsk max-mrgrsk added refactor tests Pull requests that update or improve our test suite labels Jan 3, 2025
@max-mrgrsk max-mrgrsk added this to the v1 Modeling App Launch milestone Jan 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor tests Pull requests that update or improve our test suite
Projects
None yet
Development

Successfully merging this pull request may close these issues.

refactor fillet playwright test to the new fixture-based approach
1 participant