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

Initial Roborazzi support #359

Merged
merged 6 commits into from
Sep 28, 2023
Merged

Initial Roborazzi support #359

merged 6 commits into from
Sep 28, 2023

Conversation

timothyfroehlich
Copy link
Member

@timothyfroehlich timothyfroehlich commented Sep 11, 2023

This adds a basic Roborazzi test to HelloWorld and includes documentation on it. Subsequent PRs will add roborazzi tests for Validation and the DesignSwitcher.

Please review the implementation and let me know if there's any questions. Once approved I'll update the Validation draft with the changes and finish the DesignSwitcher Roborazzi tests.

See the Readme and new docs for instructions.

@timothyfroehlich timothyfroehlich self-assigned this Sep 11, 2023
@timothyfroehlich timothyfroehlich changed the base branch from main to wb/froeht/test-all-validation September 11, 2023 20:29
@timothyfroehlich timothyfroehlich linked an issue Sep 11, 2023 that may be closed by this pull request
@timothyfroehlich timothyfroehlich added Type: New Feature New feature or request engProd Engineering productivity - build system, testing, etc labels Sep 12, 2023
@timothyfroehlich timothyfroehlich added this to the 0.24 milestone Sep 12, 2023
@timothyfroehlich timothyfroehlich force-pushed the wb/froeht/roborazzi branch 2 times, most recently from cfa7dae to 82cceca Compare September 12, 2023 21:42
@timothyfroehlich timothyfroehlich changed the base branch from wb/froeht/test-all-validation to wb/froeht/designcompose-test-fixture September 12, 2023 21:42
Base automatically changed from wb/froeht/designcompose-test-fixture to main September 14, 2023 20:55
@timothyfroehlich
Copy link
Member Author

This is stalled for a bit, I'm running into a crash while trying to do screenshot tests of the DesignSwitcher. I've asked for help here: takahirom/roborazzi#164

This adds a basic Roborazzi test to HelloWorld and includes documentation on it.
.github/workflows/main.yml Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@rylin8 rylin8 left a comment

Choose a reason for hiding this comment

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

Did a test where I changed the rendering code slightly and it worked!

@timothyfroehlich timothyfroehlich added this pull request to the merge queue Sep 28, 2023
Merged via the queue into main with commit f5d01d2 Sep 28, 2023
@timothyfroehlich timothyfroehlich deleted the wb/froeht/roborazzi branch September 28, 2023 17:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
engProd Engineering productivity - build system, testing, etc Type: New Feature New feature or request
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Figma Integration and Screenshot testing omnibus
3 participants