Skip to content

add unit hook tests#543

Merged
MiloszFilimowski merged 7 commits into
mainfrom
FCE-3030/react-client-tests
Jul 17, 2026
Merged

add unit hook tests#543
MiloszFilimowski merged 7 commits into
mainfrom
FCE-3030/react-client-tests

Conversation

@czerwiukk

@czerwiukk czerwiukk commented Jun 8, 2026

Copy link
Copy Markdown
Member

Description

Adds unit tests for the react client level public hooks.

Motivation and Context

Introducing hook tests that capture the current state of features will make moving logic to the typescript client easier. The goal is to move the features to the typescript client level without changing the behavior of the hooks.

Documentation impact

  • Documentation update required
  • Documentation updated in another PR
  • No documentation update required

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to
    not work as expected)

@czerwiukk czerwiukk self-assigned this Jun 8, 2026
@linear

linear Bot commented Jun 8, 2026

Copy link
Copy Markdown

FCE-3030

@czerwiukk
czerwiukk force-pushed the FCE-3030/react-client-tests branch from 66d271a to 3c2cbc4 Compare July 8, 2026 08:23
Comment thread packages/react-client/src/tests/support/fakeFishjamClient.ts Outdated
Comment thread packages/react-client/src/tests/support/fakeMediaStream.ts
Comment thread packages/react-client/src/tests/connection.spec.ts Outdated
…ctConfig

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Comment thread packages/react-client/src/tests/screenShare.spec.ts Outdated
@MiloszFilimowski
MiloszFilimowski enabled auto-merge (squash) July 17, 2026 10:06
@MiloszFilimowski
MiloszFilimowski merged commit ed82706 into main Jul 17, 2026
2 checks passed
@MiloszFilimowski
MiloszFilimowski deleted the FCE-3030/react-client-tests branch July 17, 2026 10:13
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