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

Dev Deps: Bump Jest to 29.4.1 #952

Merged
merged 1 commit into from
Feb 7, 2023
Merged

Conversation

lucasgarfield
Copy link
Collaborator

This commit bumps Jest to 29.4.1. This requires adding two additional development dependencies.

As of Jest 28 jest-environment-jsdom is no longer shipped with Jest by default, and has therefore been added as a dependency.

uuid was also added as a dev dependency. Jest 29 introduced some compatibility issues with uuid that were fixed in uuid@9: uuidjs/uuid#451 (comment)

We import and use uuid in our tests, but as it was a dependencies dependency we did not control which version was installed. Version 8 (what was installed) causes Jest tests to fail like so:

    SyntaxError: Unexpected token 'export'

    > 29 | import { v4 as uuidv4 } from 'uuid';

This commit bumps Jest to 29.4.1. This requires adding two additional
development dependencies.

As of Jest 28 `jest-environment-jsdom` is no longer shipped with Jest by
default, and has therefore been added as a dependency.

`uuid` was also added as a dev dependency. Jest 29 introduced some
compatibility issues with `uuid` that were fixed in `uuid@9`:
uuidjs/uuid#451 (comment)

We import and use `uuid` in our tests, but as it was a dependencies
dependency we did not control which version was installed. Version 8
(what was installed) causes Jest tests to fail like so:
```
    SyntaxError: Unexpected token 'export'

    > 29 | import { v4 as uuidv4 } from 'uuid';
```
@codecov
Copy link

codecov bot commented Feb 7, 2023

Codecov Report

Merging #952 (9cf7030) into main (d4eb5f7) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #952   +/-   ##
=======================================
  Coverage   76.00%   76.00%           
=======================================
  Files          61       61           
  Lines        1542     1542           
  Branches      384      384           
=======================================
  Hits         1172     1172           
  Misses        338      338           
  Partials       32       32           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d4eb5f7...9cf7030. Read the comment docs.

Copy link
Member

@croissanne croissanne left a comment

Choose a reason for hiding this comment

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

thanks!

@croissanne croissanne merged commit 0e4d12b into osbuild:main Feb 7, 2023
@lucasgarfield lucasgarfield deleted the bump-jest branch February 7, 2023 17:15
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