Skip to content

Conversation

@hsubox76
Copy link
Contributor

@hsubox76 hsubox76 commented Nov 10, 2025

TOTP tests currently require a pre-created test user to exist in the JS SDK project, and for that account to be configured properly. Every so often, at unpredictable intervals, the test user account ends up stuck in a state that causes tests to fail, and this user needs to be manually deleted and recreated by an SDK team member.

Disabling these tests until an alternative method can be found to test TOTP that avoids this issue. Using describe.skip allows contributors to easily re-enable the tests locally in order to run them manually.

@changeset-bot
Copy link

changeset-bot bot commented Nov 10, 2025

⚠️ No Changeset found

Latest commit: aceaf75

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@google-oss-bot
Copy link
Contributor

@google-oss-bot
Copy link
Contributor

@hsubox76 hsubox76 marked this pull request as ready for review November 11, 2025 18:17
@hsubox76 hsubox76 merged commit d1d898f into main Nov 14, 2025
49 checks passed
@hsubox76 hsubox76 deleted the ch-totp-disable branch November 14, 2025 19:07
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.

4 participants