Skip to content

Revert Jest upgrade#3202

Closed
JPeer264 wants to merge 2 commits intomasterfrom
jp/revert-jest
Closed

Revert Jest upgrade#3202
JPeer264 wants to merge 2 commits intomasterfrom
jp/revert-jest

Conversation

@JPeer264
Copy link
Member

revert #3201 and #3191 because releasing won't work with this update.

@JPeer264 JPeer264 self-assigned this Mar 10, 2026
@JPeer264 JPeer264 requested review from a team and szokeasaurusrex as code owners March 10, 2026 09:47
@JPeer264 JPeer264 requested review from Lms24 and nicohrubec March 10, 2026 09:47
@github-actions
Copy link
Contributor

Fails
🚫 Please consider adding a changelog entry for the next release.

Instructions and example for changelog

Please add an entry to CHANGELOG.md to the "Unreleased" section. Make sure the entry includes this PR's number.

Example:

## Unreleased

### Features

- Revert Jest upgrade ([#3202](https://github.com/getsentry/sentry-cli/pull/3202))

If none of the above apply, you can opt out of this check by adding #skip-changelog to the PR description or adding a skip-changelog label.

Generated by 🚫 dangerJS against afb79da

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

"testPathIgnorePatterns": [
"<rootDir>/src"
]
},
Copy link

Choose a reason for hiding this comment

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

Jest config in package.json silently ignored by jest.config.js

High Severity

The "jest" configuration block added to package.json (collectCoverage, testEnvironment, testPathIgnorePatterns) is silently ignored because jest.config.js exists. Jest uses jest.config.js as the sole configuration source when present. This means collectCoverage won't be enabled, testEnvironment will default to "jsdom" (Jest 27's default) instead of "node", and the testPathIgnorePatterns from package.json won't apply. The testEnvironment mismatch is especially likely to cause test failures since Node-specific APIs won't be available under jsdom.

Additional Locations (1)

Fix in Cursor Fix in Web

setupFiles: [require.resolve('./setupTests.js')],
testPathIgnorePatterns: ['<rootDir>/src/', '<rootDir>/tests/integration/'],
setupFiles: ['<rootDir>/setupTests.js'],
testPathIgnorePatterns: ['<rootDir>/src/'],
Copy link

Choose a reason for hiding this comment

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

Integration test ignore pattern silently dropped from config

Medium Severity

The testPathIgnorePatterns entry for '<rootDir>/tests/integration/' was removed from jest.config.js and not restored anywhere in the effective configuration. Integration tests that were previously excluded from the default test run will now execute, potentially causing unexpected failures or slowdowns in the regular test suite.

Fix in Cursor Fix in Web

@JPeer264
Copy link
Member Author

closed in favor in favor of #3194

@JPeer264 JPeer264 closed this Mar 10, 2026
@JPeer264 JPeer264 deleted the jp/revert-jest branch March 10, 2026 09:56
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.

1 participant