Skip to content

Move test skipping from Jest config to a filter file#1525

Closed
robhogan wants to merge 1 commit into
mainfrom
export-D77656335
Closed

Move test skipping from Jest config to a filter file#1525
robhogan wants to merge 1 commit into
mainfrom
export-D77656335

Conversation

@robhogan

@robhogan robhogan commented Jul 2, 2025

Copy link
Copy Markdown
Contributor

Summary:
Currently we use testPathIgnorePatterns to skip known-broken tests on some configurations (mainly tests that weren't written for Windows).

Using testPathIgnorePatterns ignores the tests on the file-crawling level. This has the side-effect of changing the file map cache key, which means Metro tests on Windows don't benefit from sharing a cache with posix (i.e., Saved State).

This uses
Changelog: Internal

Differential Revision: D77656335

Summary:
Currently we use `testPathIgnorePatterns` to skip known-broken tests on some configurations (mainly tests that weren't written for Windows).

Using `testPathIgnorePatterns` ignores the tests on the file-crawling level. This has the side-effect of changing the file map cache key, which means Metro tests on Windows don't benefit from sharing a cache with posix (i.e., Saved State).

This uses 
Changelog: Internal

Differential Revision: D77656335
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jul 2, 2025
@facebook-github-bot

Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D77656335

@facebook-github-bot

Copy link
Copy Markdown
Contributor

This pull request has been merged in f7a7e0b.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants