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

Fix unit tests #305

Merged
merged 1 commit into from
Nov 15, 2023
Merged

Fix unit tests #305

merged 1 commit into from
Nov 15, 2023

Conversation

tvdijen
Copy link
Contributor

@tvdijen tvdijen commented Oct 23, 2023

Tests started failing since yesterday:

yarn install v1.22.19
info No lockfile found.
[1/4] Resolving packages...
warning @symfony/webpack-encore > webpack-dev-server > webpack-dev-middleware > [email protected]: this will be v4
[2/4] Fetching packages...
error @symfony/[email protected]: The engine "node" is incompatible with this module. Expected version ">=16.0.0". Got "14.21.2"

I was able to trace to issue back to a new release of webpack-encore:
https://github.com/symfony/webpack-encore/releases/tag/v4.5.0

This same issue also applies to stepup-selfservice, stepup-gateway, stepup-tiqr and stepup-azure-mfa.
Another approach may be to raise the Node-version, but I wasn't sure how to test/try this.

Tests started failing since yesterday:

```
yarn install v1.22.19
info No lockfile found.
[1/4] Resolving packages...
warning @symfony/webpack-encore > webpack-dev-server > webpack-dev-middleware > [email protected]: this will be v4
[2/4] Fetching packages...
error @symfony/[email protected]: The engine "node" is incompatible with this module. Expected version ">=16.0.0". Got "14.21.2"
```

I was able to trace to issue back to a new release of webpack-encore:
https://github.com/symfony/webpack-encore/releases/tag/v4.5.0
@tvdijen tvdijen requested a review from MKodde October 23, 2023 18:35
@phavekes phavekes merged commit 6a93a99 into develop Nov 15, 2023
1 check passed
@phavekes phavekes deleted the tvdijen-patch-1 branch November 15, 2023 09:22
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.

2 participants