-
Notifications
You must be signed in to change notification settings - Fork 95
[v0.48.0] feat: e2e tests use containerized core #887
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from 5 commits
Commits
Show all changes
48 commits
Select commit
Hold shift + click to select a range
18163e7
update: makes test utils/server use applications
namsnath 6ec5b6d
feat: setup mochawesome
namsnath a6417a0
update: `emailverification` tests use core apps
namsnath fd624eb
update: `embed` tests use core apps
namsnath 77cdb86
update: `generalerror` tests use core apps
namsnath e36f2b1
update: `getRedirectionURL` tests use core apps
namsnath fe872b1
update: remove assertion on creating new apps
namsnath 7ddf577
update: `mfa.chooserscreen` tests use core apps
namsnath 41a3fd1
feat: adds mocha envs to test config
namsnath a46fb7b
lint: format code
namsnath d65eaf4
update: `accountlinking` tests use core apps
namsnath d55fda8
update: `mfa.default_reqs` tests use core apps
namsnath d71e74a
refactor: split app creation and ST setup utils
namsnath c11dae0
update: `mfa.factorscreen.otp` tests use core apps
namsnath b6e5bb1
refactor: split app creation and ST setup utils
namsnath 2041010
update: `mfa.factorscreen.totp` tests use core apps
namsnath b5a04ba
update: add missing backend hooks
namsnath b5a1b76
update: `mfa.firstFactors` tests use core apps
namsnath 646f0d4
update: `mfa.requirement_handling` tests use core apps
namsnath 87f778c
update: `mfa.signin` tests use core apps
namsnath d9f3daf
update: `multitenancy.tenant_interactions` tests use core apps
namsnath 23dadd6
update: `oauth2provider` tests use core apps
namsnath d7a55f1
update: `resetpasswordusingtoken` tests use core apps
namsnath c37e670
update: `signin-rrdv5/6` tests use core apps
namsnath d1c7f48
update: few `thirdparty` tests use core apps
namsnath 88e5ecf
update: removes `visual` tests
namsnath f54f19e
removes usages of old helper functions
namsnath eef281a
update: `userContext`/`userroles` tests use core apps
namsnath 3da8a6c
fix: broken tests, update configs
namsnath 10ef8d0
update: remove log
namsnath a965aac
fix: correct setup ST url
namsnath 5e586a0
test: only run few tests
namsnath 46d7103
revert: test: only run few tests
namsnath 5bbad1f
Fix package-lock issues
deepjyoti30-st aa04571
Add some fixes for userContext and test server
deepjyoti30-st 67c72c5
update: remove spec config in mocha
namsnath c9e289e
Add fixes for some failing updates
deepjyoti30-st 3c77a8c
Add workflow and docker compose file for runs
deepjyoti30-st d0e0bb4
Modify the workflow to run in a separate workflow for test
deepjyoti30-st b8032dd
Fix FDI versions and branch ref
deepjyoti30-st ab28870
Update .github/workflows/auth-react-test-1.yml
deepjyoti30-st 165daa3
Update workflow to make it ready for merge
deepjyoti30-st 7e5c70f
Add unit test workflow
deepjyoti30-st 2c8d6ec
Get rid of using random UUID for appId
deepjyoti30-st 32c6cef
Rename Core test to Core Functionality
deepjyoti30-st fbbcb06
Get rid of a TODO
deepjyoti30-st ecad24b
Get rid of using timeout in tests
deepjyoti30-st f215d04
Get rid of setting appId from tests directly
deepjyoti30-st File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,25 @@ | ||
| name: Auth-React Unit Tests | ||
|
|
||
| on: | ||
| pull_request: | ||
| types: | ||
| - opened | ||
| - reopened | ||
| - synchronize | ||
| push: | ||
| branches: | ||
| - master | ||
| - "v[0-9]+.[0-9]+" | ||
| tags: | ||
| - "(dev-)?v[0-9]+.[0-9]+.[0-9]+" | ||
|
|
||
| jobs: | ||
| unit-tests: | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - uses: actions/checkout@v4 | ||
| - uses: actions/setup-node@v4 | ||
| with: | ||
| node-version: 18 | ||
| - run: npm install | ||
| - run: npm run test-unit |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -40,7 +40,6 @@ import { | |
|
|
||
| import { TEST_CLIENT_BASE_URL, SOMETHING_WENT_WRONG_ERROR } from "../constants"; | ||
| import { tryEmailPasswordSignUp, tryPasswordlessSignInUp } from "./mfa.helpers"; | ||
| import { randomUUID } from "crypto"; | ||
|
|
||
| const examplePhoneNumber = "+36701231212"; | ||
| const exampleEmail = "[email protected]"; | ||
|
|
@@ -129,9 +128,8 @@ export function getPasswordlessTestCases({ authRecipe, logId, generalErrorRecipe | |
| const contactMethod = "EMAIL_OR_PHONE"; | ||
|
|
||
| before(async function () { | ||
| const appId = randomUUID(); | ||
| const coreUrl = await setupCoreApp({ | ||
| appId, | ||
| appId: "test-app-id", | ||
deepjyoti30-st marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
| coreConfig: { | ||
| passwordless_code_lifetime: 4000, | ||
| passwordless_max_code_input_attempts: 3, | ||
|
|
@@ -338,15 +336,14 @@ export function getPasswordlessTestCases({ authRecipe, logId, generalErrorRecipe | |
|
|
||
| let accountLinkingSupported; | ||
| let coreUrl; | ||
| const appId = randomUUID(); | ||
| const coreConfig = { | ||
| passwordless_code_lifetime: 4000, | ||
| passwordless_max_code_input_attempts: 3, | ||
| }; | ||
|
|
||
| before(async function () { | ||
| coreUrl = await setupCoreApp({ | ||
| appId, | ||
| appId: "test-app-id", | ||
| coreConfig, | ||
| }); | ||
| }); | ||
|
|
||
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.