Skip to content

chore(react): fixing tests. - #230

Merged
filipi87 merged 3 commits into
mainfrom
filipi/fix_client_react_tests
Jun 19, 2026
Merged

filipi87 merged 3 commits into
mainfrom
filipi/fix_client_react_tests

Conversation

@filipi87

Copy link
Copy Markdown
Collaborator

chore(react): fixing tests.

@filipi87
filipi87 force-pushed the filipi/fix_client_react_tests branch from 5b56b2e to 4e8b904 Compare June 18, 2026 17:16

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

These tests started to fail after my last commit when I added support for the RTVI 2.0.0 protocol.

I have fixed the tests, the messages should include text.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

These tests started to fail after the changes in this PR:

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

New workflow to make sure the tests are always executed when we open a PR. 🙂

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

These tests started to fail after the changes in this PR:

@filipi87
filipi87 requested review from Regaddi and mattieruth June 18, 2026 21:04
@filipi87
filipi87 marked this pull request as ready for review June 18, 2026 21:06

@mattieruth mattieruth left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@filipi87
filipi87 merged commit 5fbff17 into main Jun 19, 2026
3 checks passed
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.

3 participants