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
chore(jest): add typeahead for watch mode #12467
chore(jest): add typeahead for watch mode #12467
Conversation
Perf AnalysisNo significant results to display. All results
Perf Analysis (Fluent)Perf comparison
Perf tests with no regressions
|
Asset size changesSize Auditor did not detect a change in bundle size for any component! Baseline commit: 16345e00d8e3ff4071793da22ee52b5a8e49ccb3 (build) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe please if you could add some usage example? :) (how to use it, what everything is possible to do with this plugin)
Or if the whole functionality is auto suggest then of course this comment is not relevant :)
@@ -12,4 +12,5 @@ module.exports = { | |||
'test/(.*)$': `<rootDir>/test/$1`, | |||
}, | |||
setupFilesAfterEnv: [...commonConfig.setupFilesAfterEnv, './test/setup.ts'], | |||
watchPlugins: ['jest-watch-typeahead/filename', 'jest-watch-typeahead/testname'], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please move this to shared config
@@ -38,6 +38,7 @@ | |||
"@types/simulant": "^0.2.0", | |||
"@uifabric/build": "^7.0.0", | |||
"faker": "^4.1.0", | |||
"jest-watch-typeahead": "^0.4.2", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kenotron or @ecraig12345 should this just go in the scripts/package.json file?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think so
It'd be great if we could add to I'd love for there to be one standard config really, but that might be outside the scope here. |
Yes, standardizing on one config is definitely out of scope for this PR. |
@ecraig12345 @dzearing I pushed latest change, moved the devDependency to the scripts/package.json. |
* add package and configure jest * add watch plugin to main jest config * move devDepend to scripts package.json * add to watchPlugins to jest-resources as well
Pull request checklist
$ yarn change
Description of changes
Adds https://github.com/jest-community/jest-watch-typeahead to our jest config.
You now have autocomplete/typeahead functionality when you use filtering by file name or by test name in your
yarn test:watch
.Focus areas to test
(optional)
Microsoft Reviewers: Open in CodeFlow