Skip to content

[MHV-55186] Update all V1 components to V3 web components on SM MHV #41811

[MHV-55186] Update all V1 components to V3 web components on SM MHV

[MHV-55186] Update all V1 components to V3 web components on SM MHV #41811

Triggered via pull request March 18, 2024 21:52
Status Failure
Total duration 6m 53s
Artifacts 1

pull-request.yml

on: pull_request
Fetch Unit Test Stability Allow List
1m 1s
Fetch Unit Test Stability Allow List
ESLint Disable Check
1m 1s
ESLint Disable Check
Icon Check
57s
Icon Check
Sentry Check
1m 2s
Sentry Check
Linting (Files Changed)
1m 8s
Linting (Files Changed)
Check Cross App Imports
5m 36s
Check Cross App Imports
Fit to window
Zoom out
Zoom in

Annotations

10 errors and 25 warnings
Linting (Files Changed): src/applications/mhv/secure-messaging/tests/e2e/pages/FolderManagementPage.js#L133
src/applications/mhv/secure-messaging/tests/e2e/pages/FolderManagementPage.js:133:38:Replace `mockParentMessageDetails.data.attributes.messageId` with `⏎········mockParentMessageDetails.data.attributes.messageId⏎······`
Linting (Files Changed): src/applications/mhv/secure-messaging/tests/e2e/pages/FolderManagementPage.js#L202
src/applications/mhv/secure-messaging/tests/e2e/pages/FolderManagementPage.js:202:42:Replace `mockCustomResponse.data.attributes.threadId` with `⏎········mockCustomResponse.data.attributes.threadId⏎······`
Linting (Files Changed): src/applications/mhv/secure-messaging/tests/e2e/pages/FolderManagementPage.js#L223
src/applications/mhv/secure-messaging/tests/e2e/pages/FolderManagementPage.js:223:41:Replace `mockCustomResponse.data.attributes.threadId` with `⏎········mockCustomResponse.data.attributes.threadId⏎······`
Linting (Files Changed): src/applications/mhv/secure-messaging/tests/e2e/pages/FolderManagementPage.js#L239
src/applications/mhv/secure-messaging/tests/e2e/pages/FolderManagementPage.js:239:41:Replace `mockCustomResponse.data.attributes.threadId` with `⏎········mockCustomResponse.data.attributes.threadId⏎······`
Linting (Files Changed): src/applications/mhv/secure-messaging/tests/e2e/pages/PatientComposePage.js#L205
src/applications/mhv/secure-messaging/tests/e2e/pages/PatientComposePage.js:205:49:Replace `draftMessage.data.attributes.messageId` with `⏎········draftMessage.data.attributes.messageId⏎······`
Linting (Files Changed): src/applications/mhv/secure-messaging/tests/e2e/pages/PatientComposePage.js#L362
src/applications/mhv/secure-messaging/tests/e2e/pages/PatientComposePage.js:362:38:Replace `mockMessageResponse.data.attributes.messageId` with `⏎········mockMessageResponse.data.attributes.messageId⏎······`
Linting (Files Changed): src/applications/mhv/secure-messaging/tests/e2e/pages/PatientInboxPage.js#L145
src/applications/mhv/secure-messaging/tests/e2e/pages/PatientInboxPage.js:145:54:Replace `inputMockMessage.attributes.messageId` with `⏎········inputMockMessage.attributes.messageId⏎······`
Linting (Files Changed): src/applications/mhv/secure-messaging/tests/e2e/pages/PatientInboxPage.js#L167
src/applications/mhv/secure-messaging/tests/e2e/pages/PatientInboxPage.js:167:35:Replace `mockMessages.data[0].attributes.messageId` with `⏎········mockMessages.data[0].attributes.messageId⏎······`
Linting (Files Changed): src/applications/mhv/secure-messaging/tests/e2e/pages/PatientInboxPage.js#L172
src/applications/mhv/secure-messaging/tests/e2e/pages/PatientInboxPage.js:172:35:Replace `this.singleThread.data[0].attributes.messageId` with `⏎········this.singleThread.data[0].attributes.messageId⏎······`
Linting (Files Changed): src/applications/mhv/secure-messaging/tests/e2e/pages/PatientInboxPage.js#L179
src/applications/mhv/secure-messaging/tests/e2e/pages/PatientInboxPage.js:179:37:Replace `this.singleThread.data[1].attributes.messageId` with `⏎··········this.singleThread.data[1].attributes.messageId⏎········`
Icon Check
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: department-of-veterans-affairs/action-inject-ssm-secrets@latest. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
ESLint Disable Check
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: department-of-veterans-affairs/action-inject-ssm-secrets@latest. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
Fetch Unit Test Stability Allow List
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: department-of-veterans-affairs/action-inject-ssm-secrets@latest, actions/checkout@v3, actions/setup-node@v3, actions/upload-artifact@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
Sentry Check
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: department-of-veterans-affairs/action-inject-ssm-secrets@latest. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
Linting (Files Changed)
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: trilom/file-changes-action@v1.2.4. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
Linting (Files Changed)
The following actions uses node12 which is deprecated and will be forced to run on node16: trilom/file-changes-action@v1.2.4. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
Linting (Files Changed): src/applications/mhv/secure-messaging/components/AttachmentsList.jsx#L3
src/applications/mhv/secure-messaging/components/AttachmentsList.jsx:3:1:Importing from platform via platform/monitoring is deprecated. Import from @department-of-veterans-affairs/platform-monitoring instead or check babel.config.json for an alias to the import.
Linting (Files Changed): src/applications/mhv/secure-messaging/components/AttachmentsList.jsx#L69
src/applications/mhv/secure-messaging/components/AttachmentsList.jsx:69:5:React Hook useEffect has a missing dependency: 'setAttachFileSuccess'. Either include it or remove the dependency array.
Linting (Files Changed): src/applications/mhv/secure-messaging/components/AttachmentsList.jsx#L175
src/applications/mhv/secure-messaging/components/AttachmentsList.jsx:175:19:The <va-button> Web Component should be used instead of the button HTML element.
Linting (Files Changed): src/applications/mhv/secure-messaging/components/ComposeForm/ComposeForm.jsx#L165
src/applications/mhv/secure-messaging/components/ComposeForm/ComposeForm.jsx:165:5:React Hook useEffect has a missing dependency: 'defaultRecipientsList'. Either include it or remove the dependency array.
Linting (Files Changed): src/applications/mhv/secure-messaging/components/ComposeForm/ComposeForm.jsx#L200
src/applications/mhv/secure-messaging/components/ComposeForm/ComposeForm.jsx:200:6:React Hook useEffect has missing dependencies: 'draft' and 'recipients'. Either include them or remove the dependency array.
Linting (Files Changed): src/applications/mhv/secure-messaging/components/ComposeForm/ComposeForm.jsx#L231
src/applications/mhv/secure-messaging/components/ComposeForm/ComposeForm.jsx:231:5:React Hook useEffect has missing dependencies: 'attachments', 'category', 'currentFolder?.folderId', 'dispatch', 'draft?.messageId', 'history', 'messageBody', 'selectedRecipient', and 'subject'. Either include them or remove the dependency array.
Linting (Files Changed): src/applications/mhv/secure-messaging/components/ComposeForm/ComposeForm.jsx#L249
src/applications/mhv/secure-messaging/components/ComposeForm/ComposeForm.jsx:249:5:React Hook useEffect has a missing dependency: 'lastFocusableElement'. Either include it or remove the dependency array.
Linting (Files Changed): src/applications/mhv/secure-messaging/components/ComposeForm/ComposeForm.jsx#L445
src/applications/mhv/secure-messaging/components/ComposeForm/ComposeForm.jsx:445:5:React Hook useEffect has missing dependencies: 'draft?.body' and 'savedDraft'. Either include them or remove the dependency array.
Linting (Files Changed): src/applications/mhv/secure-messaging/components/ComposeForm/ComposeForm.jsx#L524
src/applications/mhv/secure-messaging/components/ComposeForm/ComposeForm.jsx:524:5:React Hook useCallback has missing dependencies: 'noTimeout' and 'signOutMessage'. Either include them or remove the dependency array.
Linting (Files Changed): src/applications/mhv/secure-messaging/components/ComposeForm/ComposeForm.jsx#L544
src/applications/mhv/secure-messaging/components/ComposeForm/ComposeForm.jsx:544:5:React Hook useEffect has a missing dependency: 'noTimeout'. Either include it or remove the dependency array.
Linting (Files Changed)
Received event from pull_request, but also received a before(fdbb3a6db966777a1bd6273a1faf3c27afa8f268) or after(7320089267d38a09262ead627c7e0f3ae6f9d559) value. I am assuming you want to use a Push event but forgot something, so I'm giving you a message.
Linting (Files Changed)
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Linting (Files Changed)
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Linting (Files Changed)
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Linting (Files Changed)
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Linting (Files Changed)
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Linting (Files Changed)
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Linting (Files Changed)
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Check Cross App Imports
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/download-artifact@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.

Artifacts

Produced during runtime
Name Size
unit-test-allow-list
552 KB