Skip to content
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

Update testing dependencies #3437

Merged
merged 1 commit into from
Mar 24, 2023
Merged

Update testing dependencies #3437

merged 1 commit into from
Mar 24, 2023

Conversation

MikeMcC399
Copy link
Contributor

@MikeMcC399 MikeMcC399 commented Mar 24, 2023

This PR fixes the warning message:

"Opening Cypress...
Couldn't find tsconfig.json. tsconfig-paths will be skipped"

which appears when using Cypress, for instance in the log files of https://github.com/corona-warn-app/cwa-website/actions/workflows/cypress-test-prod.yml

The error no longer occurs when browser-sync is updated to 2.29.1 (latest), since this version removes the dependency on "typescript": "^4.6.2" (see BrowserSync/browser-sync@v2.29.0...v2.29.1 where it persists in devDependencies only).

At the same time:

as part of general maintenance, since these components work together for testing.

Verification

Local test

Execute

npm ci
npm test

and confirm successful completion.

There should be no message "Couldn't find tsconfig.json. tsconfig-paths will be skipped" in the logs.

GitHub test

Run .github/workflows/build-and-test.yml

Confirm successful completion.

There should be no message "Couldn't find tsconfig.json. tsconfig-paths will be skipped" in the logs after the log message
"Opening Cypress..."

Note


Internal Tracking ID: EXPOSUREAPP-14986

browser-sync to 2.29.1
cypress to 12.8.1
start-server-and-test to 2.0.0

resolves issue with cypress warning
"Couldn't find tsconfig.json. tsconfig-paths will be skipped"
@MikeMcC399 MikeMcC399 marked this pull request as ready for review March 24, 2023 08:57
@MikeMcC399 MikeMcC399 requested a review from a team March 24, 2023 08:57
@larswmh larswmh self-assigned this Mar 24, 2023
Copy link
Member

@larswmh larswmh left a comment

Choose a reason for hiding this comment

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

Thanks for updating the dependencies @MikeMcC399!

@larswmh larswmh merged commit a4cdf4a into corona-warn-app:master Mar 24, 2023
@MikeMcC399 MikeMcC399 deleted the fix/cypress-tsconfig-warning branch March 24, 2023 09:39
@MikeMcC399
Copy link
Contributor Author

@larswmh

Thanks for updating the dependencies

You're welcome! ... and thank you for merging!

Contributors should fetch the updated master branch and run
npm install

They can also run
npx cypress cache prune
to remove unused older Cypress versions.

@larswmh larswmh mentioned this pull request Mar 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cypress Issues
2 participants