Skip to content

Conversation

@deepjyoti30-st
Copy link
Contributor

Summary of change

This PR updates all tests to use the containerized core to run the tests.

Related issues

Test Plan

All tests should pass with the containerized core running

Documentation changes

(If relevant, please create a PR in our docs repo, or create a checklist here highlighting the necessary changes)

Checklist for important updates

  • Changelog has been updated
  • frontendDriverInterfaceSupported.json file has been updated (if needed)
  • Changes to the version if needed
    • In package.json
    • In package-lock.json
    • In lib/ts/version.ts
  • Had run npm run build-pretty
  • Had installed and ran the pre-commit hook
  • Issue this PR against the latest non released version branch.
    • To know which one it is, run find the latest released tag (git tag) in the format vX.Y.Z, and then find the latest branch (git branch --all) whose X.Y is greater than the latest released tag.
    • If no such branch exists, then create one from the latest released branch.
  • If added a new recipe interface, then make sure that the implementation of it uses NON arrow functions only (like someFunc: function () {..}).
  • If I added a new recipe, I also added the recipe entry point into the size-limit section of package.json with the size limit set to the current size rounded up.
  • If I added a new recipe, I also added the recipe entry point into the rollup.config.mjs
  • If I added a new login method, I modified the list in lib/ts/types.ts
  • If I added a factor id, I modified the list in lib/ts/recipe/multifactorauth/types.ts

namsnath added 30 commits April 8, 2025 12:15
- Adds screenshots to report
- Adds defaults to `.mocharc.yml`
- Each test runs independently
- Adds some additional steps to ensure tests are independent
- Cleans up test hooks
- Fixes verification failure test that relied on core being stopped
    - Adds intercept to fail verification query
- Each test runs independently
- Adds some additional steps to ensure tests are independent
- Cleans up test hooks
- Same appId can be re-used and will not create new apps
- Each test runs independently
- Adds some additional steps to ensure tests are independent
- Cleans up test hooks
- Each test runs independently
- Cleans up test hooks
@promptless
Copy link

promptless bot commented Apr 9, 2025

✅ No documentation updates required.

1 similar comment
@promptless
Copy link

promptless bot commented Apr 9, 2025

✅ No documentation updates required.

@sattvikc sattvikc merged commit 69437a1 into 0.41 Apr 18, 2025
1 of 26 checks passed
@sattvikc sattvikc deleted the ci/github-actions/auth-react/v0.41.0 branch April 18, 2025 07:09
@deepjyoti30-st deepjyoti30-st restored the ci/github-actions/auth-react/v0.41.0 branch April 21, 2025 10:19
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.

5 participants