ci(react): Add basic catalog integration tests.#1625
Conversation
d845bba to
d34360e
Compare
|
/gemini review |
There was a problem hiding this comment.
Code Review
This pull request introduces deep cloning of messages using structuredClone in App.tsx to prevent unintended mutations, refactors the test utility whenSettled to avoid nested act blocks, and adds a comprehensive suite of tests for various UI examples under tests/v0_9/. The review feedback highlights several opportunities to improve code safety in the new test files by replacing the non-null assertion operator (!) with optional chaining (?.) on lookup results that may fail.
|
OK, this should be ready for @josemontespg to take a look! |
1a8aa0a to
749bb77
Compare
|
Amazing, thank you for adding integration tests for the react renderer! |
|
I'll wait for the actions to finish, it's taking them a while to get scheduled again :/ Update: The new react tests have run and passed here: https://github.com/a2ui-project/a2ui/actions/runs/27647541272/job/81763115586?pr=1625#step:5:705 |
Description
This PR adds the remaining integration tests to the React adapter.
There are some minor changes to the react integration testing harness to improve the interaction with elements that require some more than a "click".
Pre-launch Checklist
If you need help, consider asking for advice on the discussion board.