-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
[No QA][TS migration] migrate workflow tests #38491
Changes from 8 commits
5d56fb0
94702fb
d6aefab
9c18645
fe4547d
c6f9af1
d886ff2
e80c43d
7d176db
f0fe698
fd2b2b5
56aca75
ad69176
c45f367
6df0fe8
95c70e8
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
This file was deleted.
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,13 @@ | ||
/* eslint-disable @typescript-eslint/naming-convention */ | ||
import type {Config} from 'jest'; | ||
|
||
const config: Config = { | ||
verbose: true, | ||
transform: { | ||
'^.+\\.(js|jsx|ts|tsx)$': 'ts-jest', | ||
}, | ||
clearMocks: true, | ||
resetMocks: true, | ||
}; | ||
|
||
export default config; |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -11,7 +11,7 @@ const VALIDATEGITHUBACTIONS__VERIFY__STEP_MOCKS = [ | |
VALIDATEGITHUBACTIONS__VERIFY__SETUP_NODE__STEP_MOCK, | ||
VALIDATEGITHUBACTIONS__VERIFY__VERIFY_JAVASCRIPT_ACTION_BUILDS__STEP_MOCK, | ||
VALIDATEGITHUBACTIONS__VERIFY__VALIDATE_ACTIONS_AND_WORKFLOWS__STEP_MOCK, | ||
] as const; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. same |
||
]; | ||
|
||
export { | ||
// eslint-disable-next-line import/prefer-default-export | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -5,11 +5,7 @@ import {createMockStep} from '../utils/utils'; | |
const VERIFYPODFILE__VERIFY__CHECKOUT__STEP_MOCK = createMockStep('Checkout', 'Checkout', 'VERIFY'); | ||
const VERIFYPODFILE__VERIFY__SETUP_NODE__STEP_MOCK = createMockStep('Setup Node', 'Setup Node', 'VERIFY', [], []); | ||
const VERIFYPODFILE__VERIFY__VERIFY_PODFILE__STEP_MOCK = createMockStep('Verify podfile', 'Verify podfile', 'VERIFY', [], []); | ||
const VERIFYPODFILE__VERIFY__STEP_MOCKS = [ | ||
VERIFYPODFILE__VERIFY__CHECKOUT__STEP_MOCK, | ||
VERIFYPODFILE__VERIFY__SETUP_NODE__STEP_MOCK, | ||
VERIFYPODFILE__VERIFY__VERIFY_PODFILE__STEP_MOCK, | ||
] as const; | ||
const VERIFYPODFILE__VERIFY__STEP_MOCKS = [VERIFYPODFILE__VERIFY__CHECKOUT__STEP_MOCK, VERIFYPODFILE__VERIFY__SETUP_NODE__STEP_MOCK, VERIFYPODFILE__VERIFY__VERIFY_PODFILE__STEP_MOCK]; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. same |
||
|
||
export { | ||
// eslint-disable-next-line import/prefer-default-export | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -9,7 +9,7 @@ const VERIFYSIGNEDCOMMITS__VERIFYSIGNEDCOMMITS__VERIFY_SIGNED_COMMITS__STEP_MOCK | |
['GITHUB_TOKEN'], | ||
[], | ||
); | ||
const VERIFYSIGNEDCOMMITS__VERIFYSIGNEDCOMMITS__STEP_MOCKS = [VERIFYSIGNEDCOMMITS__VERIFYSIGNEDCOMMITS__VERIFY_SIGNED_COMMITS__STEP_MOCK] as const; | ||
const VERIFYSIGNEDCOMMITS__VERIFYSIGNEDCOMMITS__STEP_MOCKS = [VERIFYSIGNEDCOMMITS__VERIFYSIGNEDCOMMITS__VERIFY_SIGNED_COMMITS__STEP_MOCK]; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. same |
||
|
||
export { | ||
// eslint-disable-next-line import/prefer-default-export | ||
|
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.
why remove as const? Those arrays aren't supposed to be modified, so that's fine in this file.
UPDATE: it's been causing erorrs, because it lacked
satisfies StepIdentifier[]