Conversation
This reverts commit ef659de.
Instructions and example for changelogPlease add an entry to 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 |
There was a problem hiding this comment.
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" | ||
| ] | ||
| }, |
There was a problem hiding this comment.
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)
| setupFiles: [require.resolve('./setupTests.js')], | ||
| testPathIgnorePatterns: ['<rootDir>/src/', '<rootDir>/tests/integration/'], | ||
| setupFiles: ['<rootDir>/setupTests.js'], | ||
| testPathIgnorePatterns: ['<rootDir>/src/'], |
There was a problem hiding this comment.
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.
|
closed in favor in favor of #3194 |


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