From d204083b1202559d959170f241adce0f7b7ce3ee Mon Sep 17 00:00:00 2001 From: Mario Vitale Date: Thu, 14 Nov 2024 22:38:29 +0100 Subject: [PATCH] Demote unstable tests (#29250) * [skip ci] Demote unstable tests * [skip ci] Further demotions, fix artifact jumbling --- .github/workflows/e2e-tests-ci-template.yml | 1 + .../channels/bot_accounts/display_name_spec.ts | 2 +- .../bot_accounts/managing_bot_accounts_not_cloud_spec.ts | 2 +- .../custom_status_expiry/custom_status_expiry_3_spec.ts | 4 ++-- .../custom_status_expiry/custom_status_expiry_4_spec.ts | 8 ++++---- .../accessibility/accessibility_modals_dialogs_spec.ts | 4 ++-- .../channels/enterprise/ldap/ldap_login_spec.ts | 1 - .../custom_slash_commands/custom_slash_commands_spec.js | 1 - .../incoming_webhook/attachment_does_not_collapse_spec.ts | 1 - .../integration/channels/messaging/pinned_posts_2_spec.js | 1 - .../notifications/ignore_channel_mentions_spec.js | 1 - .../channels/system_console/unsaved_changes_spec.js | 1 - .../playbooks/channels/rhs/start_run_rhs_spec.js | 1 - .../playbooks/channels/slash_command/test_spec.js | 1 - 14 files changed, 11 insertions(+), 18 deletions(-) diff --git a/.github/workflows/e2e-tests-ci-template.yml b/.github/workflows/e2e-tests-ci-template.yml index 9ab0ee2eb0..d8989cf167 100644 --- a/.github/workflows/e2e-tests-ci-template.yml +++ b/.github/workflows/e2e-tests-ci-template.yml @@ -390,6 +390,7 @@ jobs: - name: ci/download-artifacts uses: actions/download-artifact@fa0a91b85d4f404e444e00e005971372dc801d16 # v4.1.8 with: + pattern: e2e-test-results-${{ inputs.TEST }}-* path: e2e-tests/${{ inputs.TEST }}/ merge-multiple: true - name: ci/upload-report-global diff --git a/e2e-tests/cypress/tests/integration/channels/bot_accounts/display_name_spec.ts b/e2e-tests/cypress/tests/integration/channels/bot_accounts/display_name_spec.ts index cb7fe9b2f2..38f81411db 100644 --- a/e2e-tests/cypress/tests/integration/channels/bot_accounts/display_name_spec.ts +++ b/e2e-tests/cypress/tests/integration/channels/bot_accounts/display_name_spec.ts @@ -46,7 +46,7 @@ describe('Bot display name', () => { }); }); - it('MM-T1813 Display name for bots stays current', () => { + it.skip('MM-T1813 Display name for bots stays current', () => { cy.makeClient({user: otherSysadmin}).then((client) => { // # Create a bot and get bot user id cy.apiCreateBot().then(({bot}) => { diff --git a/e2e-tests/cypress/tests/integration/channels/bot_accounts/managing_bot_accounts_not_cloud_spec.ts b/e2e-tests/cypress/tests/integration/channels/bot_accounts/managing_bot_accounts_not_cloud_spec.ts index 0d8a30bce7..b88f3fe461 100644 --- a/e2e-tests/cypress/tests/integration/channels/bot_accounts/managing_bot_accounts_not_cloud_spec.ts +++ b/e2e-tests/cypress/tests/integration/channels/bot_accounts/managing_bot_accounts_not_cloud_spec.ts @@ -80,7 +80,7 @@ describe('Managing bot accounts', () => { }); }); - it('MM-T1853 Bots managed plugins can be created when Enable Bot Account Creation is set to false', () => { + it.skip('MM-T1853 Bots managed plugins can be created when Enable Bot Account Creation is set to false', () => { // # Upload and enable "matterpoll" plugin cy.apiUploadAndEnablePlugin(matterpollPlugin); diff --git a/e2e-tests/cypress/tests/integration/channels/custom_status/custom_status_expiry/custom_status_expiry_3_spec.ts b/e2e-tests/cypress/tests/integration/channels/custom_status/custom_status_expiry/custom_status_expiry_3_spec.ts index fc57ee0390..67eaebd3ae 100644 --- a/e2e-tests/cypress/tests/integration/channels/custom_status/custom_status_expiry/custom_status_expiry_3_spec.ts +++ b/e2e-tests/cypress/tests/integration/channels/custom_status/custom_status_expiry/custom_status_expiry_3_spec.ts @@ -99,7 +99,7 @@ describe('MM-T4065 Setting manual status clear time less than 7 days away', () = cy.get('#custom_status_modal .dateTime').should('exist'); }); - it('MM-T4065_6 should show selected date in the date input field', () => { + it.skip('MM-T4065_6 should show selected date in the date input field', () => { // # Click on DayPicker input field cy.get('.dateTime__calendar-icon').click(); @@ -144,7 +144,7 @@ describe('MM-T4065 Setting manual status clear time less than 7 days away', () = should('have.attr', 'data-emoticon', customStatus.emoji); }); - it('MM-T4065_9 should show the set custom status with expiry when status dropdown is opened', () => { + it.skip('MM-T4065_9 should show the set custom status with expiry when status dropdown is opened', () => { // # Click on the sidebar header to open status dropdown cy.get('.MenuWrapper .status-wrapper').click(); diff --git a/e2e-tests/cypress/tests/integration/channels/custom_status/custom_status_expiry/custom_status_expiry_4_spec.ts b/e2e-tests/cypress/tests/integration/channels/custom_status/custom_status_expiry/custom_status_expiry_4_spec.ts index 04bfaa618a..d75de18c17 100644 --- a/e2e-tests/cypress/tests/integration/channels/custom_status/custom_status_expiry/custom_status_expiry_4_spec.ts +++ b/e2e-tests/cypress/tests/integration/channels/custom_status/custom_status_expiry/custom_status_expiry_4_spec.ts @@ -87,7 +87,7 @@ describe('MM-T4066 Setting manual status clear time more than 7 days away', () = defaultDurations.map((duration, index) => cy.get(`#custom_status_modal #statusExpiryMenu li#expiry_menu_item_${index}`).should('have.text', duration)); }); - it('MM-T4066_5 should show date/time input on selecting Choose date and time', () => { + it.skip('MM-T4066_5 should show date/time input on selecting Choose date and time', () => { // * Check that the date and time input are not present cy.get('#custom_status_modal .dateTime').should('not.exist'); @@ -98,7 +98,7 @@ describe('MM-T4066 Setting manual status clear time more than 7 days away', () = cy.get('#custom_status_modal .dateTime').should('exist'); }); - it('MM-T4066_6 should show selected date in the date input field', () => { + it.skip('MM-T4066_6 should show selected date in the date input field', () => { // # Click on DayPicker input field cy.get('.dateTime__calendar-icon').click(); @@ -128,7 +128,7 @@ describe('MM-T4066 Setting manual status clear time more than 7 days away', () = should('have.attr', 'data-emoticon', customStatus.emoji); }); - it('MM-T4066_8 should show the set custom status with expiry when status dropdown is opened', () => { + it.skip('MM-T4066_8 should show the set custom status with expiry when status dropdown is opened', () => { // # Click on the sidebar header to open status dropdown cy.get('.MenuWrapper .status-wrapper').click(); @@ -143,7 +143,7 @@ describe('MM-T4066 Setting manual status clear time more than 7 days away', () = cy.get('.status-dropdown-menu .custom_status__expiry time').should('have.text', dateToBeSelected.format('MMM DD')); }); - it('MM-52881 should show the selected date when reopening the date picker', () => { + it.skip('MM-52881 should show the selected date when reopening the date picker', () => { // # clear the status cy.get('.input-clear-x').click(); diff --git a/e2e-tests/cypress/tests/integration/channels/enterprise/accessibility/accessibility_modals_dialogs_spec.ts b/e2e-tests/cypress/tests/integration/channels/enterprise/accessibility/accessibility_modals_dialogs_spec.ts index a3ec143980..5486671a20 100644 --- a/e2e-tests/cypress/tests/integration/channels/enterprise/accessibility/accessibility_modals_dialogs_spec.ts +++ b/e2e-tests/cypress/tests/integration/channels/enterprise/accessibility/accessibility_modals_dialogs_spec.ts @@ -44,7 +44,7 @@ describe('Verify Accessibility Support in Modals & Dialogs', () => { cy.visit(`/${testTeam.name}/channels/town-square`); }); - it('MM-T1466 Accessibility Support in Direct Messages Dialog screen', () => { + it.skip('MM-T1466 Accessibility Support in Direct Messages Dialog screen', () => { // * Verify the aria-label in create direct message button cy.uiAddDirectMessage().click(); @@ -137,7 +137,7 @@ describe('Verify Accessibility Support in Modals & Dialogs', () => { }); }); - it('MM-T1468 Accessibility Support in Add people to Channel Dialog screen', () => { + it.skip('MM-T1468 Accessibility Support in Add people to Channel Dialog screen', () => { // # Add atleast 5 users for (let i = 0; i < 5; i++) { cy.apiCreateUser().then(({user}) => { // eslint-disable-line diff --git a/e2e-tests/cypress/tests/integration/channels/enterprise/ldap/ldap_login_spec.ts b/e2e-tests/cypress/tests/integration/channels/enterprise/ldap/ldap_login_spec.ts index c86b605ea7..9b9b75762c 100644 --- a/e2e-tests/cypress/tests/integration/channels/enterprise/ldap/ldap_login_spec.ts +++ b/e2e-tests/cypress/tests/integration/channels/enterprise/ldap/ldap_login_spec.ts @@ -7,7 +7,6 @@ // - Use element ID when selecting an element. Create one if none. // *************************************************************** -// Stage: @prod // Group: @channels @enterprise @ldap import {UserProfile} from '@mattermost/types/users'; diff --git a/e2e-tests/cypress/tests/integration/channels/integrations/custom_slash_commands/custom_slash_commands_spec.js b/e2e-tests/cypress/tests/integration/channels/integrations/custom_slash_commands/custom_slash_commands_spec.js index 159cc73a36..808c75de79 100644 --- a/e2e-tests/cypress/tests/integration/channels/integrations/custom_slash_commands/custom_slash_commands_spec.js +++ b/e2e-tests/cypress/tests/integration/channels/integrations/custom_slash_commands/custom_slash_commands_spec.js @@ -7,7 +7,6 @@ // - Use element ID when selecting an element. Create one if none. // *************************************************************** -// Stage: @prod // Group: @channels @integrations /** diff --git a/e2e-tests/cypress/tests/integration/channels/integrations/incoming_webhook/attachment_does_not_collapse_spec.ts b/e2e-tests/cypress/tests/integration/channels/integrations/incoming_webhook/attachment_does_not_collapse_spec.ts index 72b427e147..fc2a6e96bc 100644 --- a/e2e-tests/cypress/tests/integration/channels/integrations/incoming_webhook/attachment_does_not_collapse_spec.ts +++ b/e2e-tests/cypress/tests/integration/channels/integrations/incoming_webhook/attachment_does_not_collapse_spec.ts @@ -7,7 +7,6 @@ // - Use element ID when selecting an element. Create one if none. // *************************************************************** -// Stage: @prod // Group: @channels @incoming_webhook describe('Integrations/Incoming Webhook', () => { diff --git a/e2e-tests/cypress/tests/integration/channels/messaging/pinned_posts_2_spec.js b/e2e-tests/cypress/tests/integration/channels/messaging/pinned_posts_2_spec.js index fa157e76ed..4068858b2b 100644 --- a/e2e-tests/cypress/tests/integration/channels/messaging/pinned_posts_2_spec.js +++ b/e2e-tests/cypress/tests/integration/channels/messaging/pinned_posts_2_spec.js @@ -7,7 +7,6 @@ // - Use element ID when selecting an element. Create one if none. // *************************************************************** -// Stage: @prod // Group: @channels @messaging import {getAdminAccount} from '../../../support/env'; diff --git a/e2e-tests/cypress/tests/integration/channels/notifications/ignore_channel_mentions_spec.js b/e2e-tests/cypress/tests/integration/channels/notifications/ignore_channel_mentions_spec.js index 11395b4528..162a8138a1 100644 --- a/e2e-tests/cypress/tests/integration/channels/notifications/ignore_channel_mentions_spec.js +++ b/e2e-tests/cypress/tests/integration/channels/notifications/ignore_channel_mentions_spec.js @@ -7,7 +7,6 @@ // - Use element ID when selecting an element. Create one if none. // *************************************************************** -// Stage: @prod // Group: @channels @notifications import * as TIMEOUTS from '../../../fixtures/timeouts'; diff --git a/e2e-tests/cypress/tests/integration/channels/system_console/unsaved_changes_spec.js b/e2e-tests/cypress/tests/integration/channels/system_console/unsaved_changes_spec.js index 84e16c3971..c4892db493 100644 --- a/e2e-tests/cypress/tests/integration/channels/system_console/unsaved_changes_spec.js +++ b/e2e-tests/cypress/tests/integration/channels/system_console/unsaved_changes_spec.js @@ -7,7 +7,6 @@ // - Use element ID when selecting an element. Create one if none. // *************************************************************** -// Stage: @prod // Group: @channels @not_cloud @system_console import {fileSizeToString} from '../../../utils'; diff --git a/e2e-tests/cypress/tests/integration/playbooks/channels/rhs/start_run_rhs_spec.js b/e2e-tests/cypress/tests/integration/playbooks/channels/rhs/start_run_rhs_spec.js index 35e8dab785..29bcbbcb8f 100644 --- a/e2e-tests/cypress/tests/integration/playbooks/channels/rhs/start_run_rhs_spec.js +++ b/e2e-tests/cypress/tests/integration/playbooks/channels/rhs/start_run_rhs_spec.js @@ -6,7 +6,6 @@ // - [*] indicates an assertion (e.g. * Check the title) // *************************************************************** -// Stage: @prod // Group: @playbooks describe('channels rhs > start a run', {testIsolation: true}, () => { diff --git a/e2e-tests/cypress/tests/integration/playbooks/channels/slash_command/test_spec.js b/e2e-tests/cypress/tests/integration/playbooks/channels/slash_command/test_spec.js index 51411c3f30..45aa54dcec 100644 --- a/e2e-tests/cypress/tests/integration/playbooks/channels/slash_command/test_spec.js +++ b/e2e-tests/cypress/tests/integration/playbooks/channels/slash_command/test_spec.js @@ -6,7 +6,6 @@ // - [*] indicates an assertion (e.g. * Check the title) // *************************************************************** -// Stage: @prod // Group: @playbooks describe('channels > slash command > test', {testIsolation: true}, () => {