Fix skipped and todo tests in Github Actions Reporter#14309
Fix skipped and todo tests in Github Actions Reporter#14309SimenB merged 2 commits intojestjs:mainfrom
Conversation
✅ Deploy Preview for jestjs ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site configuration. |
141764a to
c3afed2
Compare
mrazauskas
left a comment
There was a problem hiding this comment.
Thanks! All looks good to my eye.
SimenB
left a comment
There was a problem hiding this comment.
Thanks! Could you resolve the conflict? 🙂
| } from '@jest/test-result'; | ||
| import type {Config} from '@jest/types'; | ||
| import GitHubActionsReporter from '../GitHubActionsReporter'; | ||
| import {normalizeIcons} from '../../../../e2e/Utils'; |
There was a problem hiding this comment.
Can you move this helper to @jest/test-utils?
There was a problem hiding this comment.
I would probably do that in another PR though
There was a problem hiding this comment.
Would be good to this now. @jest/test-utils is an internal package.
|
Conflict solved |
mrazauskas
left a comment
There was a problem hiding this comment.
Thanks! Very good idea to defined it as a function.
|
|
||
| export {normalizeIcons} from '@jest/test-utils'; |
There was a problem hiding this comment.
The reexport is unnecessary. This util is part of the internal testing infrastructure, it isn’t a public API.
| export {normalizeIcons} from '@jest/test-utils'; |
|
This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Summary
Fixes #14278
Test plan
Added relevant unit tests