Some checks failed
Server CI / Postgres (shard 0) (push) Has been skipped
Server CI / Postgres (shard 3) (push) Has been skipped
BuildEnv Docker Image / build-image (push) Failing after 4s
BuildEnv Docker Image / build-image-fips (push) Failing after 4s
Migration Review & Release Notes / Generate migration review & release notes (push) Failing after 3s
Server CI / Build Environment (push) Failing after 3s
Server CI / Postgres (shard 1) (push) Has been skipped
Tools CI / check-style (mattermost-govet) (push) Failing after 4s
Server CI / Postgres (shard 2) (push) Has been skipped
Tools CI / check-style (mmgotool) (push) Failing after 3s
Server CI / Elasticsearch v8 Compatibility (push) Has been skipped
Server CI / OpenSearch v2 Compatibility (push) Has been skipped
Server CI / Postgres FIPS (shard 0) (push) Has been skipped
Tools CI / Test (mattermost-govet) (push) Failing after 3s
Server CI / Postgres FIPS (shard 1) (push) Has been skipped
Server CI / Postgres FIPS (shard 2) (push) Has been skipped
Tools CI / Test (mmgotool) (push) Failing after 3s
Server CI / Postgres FIPS (shard 3) (push) Has been skipped
Server CI / Run mmctl tests (push) Has been skipped
Server CI / Run mmctl tests (FIPS) (push) Has been skipped
Server CI / Merge Postgres Test Results (push) Has been skipped
Server CI / Merge Postgres FIPS Test Results (push) Has been skipped
Server CI / Check backport migrations (push) Has been skipped
Server CI / Check generated files (push) Has been skipped
Server CI / check-style (push) Has been skipped
Server CI / Vet API (push) Has been skipped
Server CI / Build mattermost server app (push) Has been skipped
Server CI / Server CI Complete (push) Failing after 2s
Server CI / Server CI Artifacts (push) Has been skipped
Server CI / Sentry Upload (push) Has been skipped
Server CI / Server CI Report (push) Failing after 5m1s
Server CI Nightly Race / Race Detector (push) Has been skipped
CodeQL / Analyze (go) (push) Has been cancelled
CodeQL / Analyze (javascript) (push) Has been cancelled
Webapp CI Cache Warm / Warm Webapp node_modules Cache (push) Has been cancelled
E2E CI Cache Warm / Warm E2E npm Registry Cache (push) Has been cancelled
API / build (push) Has been cancelled
Web App CI / check-types (push) Has been cancelled
Web App CI / check-lint (push) Has been cancelled
Web App CI / check-i18n (push) Has been cancelled
Web App CI / check-external-links (push) Has been cancelled
Web App CI / test (platform) (push) Has been cancelled
Web App CI / test (mattermost-redux) (push) Has been cancelled
Web App CI / test (channels shard 1/4) (push) Has been cancelled
Web App CI / test (channels shard 2/4) (push) Has been cancelled
Web App CI / test (channels shard 3/4) (push) Has been cancelled
Web App CI / test (channels shard 4/4) (push) Has been cancelled
Web App CI / upload-coverage (push) Has been cancelled
Web App CI / build (push) Has been cancelled
YAML Lint / yamllint (push) Has been cancelled
Push mirrored docker images / cd/Push mirrored docker images (push) Has been cancelled
docs-cd / Build and deploy docs (push) Has been cancelled
Docs CI / build (push) Has been cancelled
E2E Cache Warm: Cypress deps / Warm Cypress deps cache (push) Has been cancelled
Scorecards supply-chain security / Scorecard analysis (push) Has been cancelled
Server CI Weekly / Postgres FIPS (push) Has been skipped
Server CI Weekly / Run mmctl tests (FIPS) (push) Has been skipped
Server CI Weekly / Postgres with binary parameters (push) Has been skipped
Server CI Weekly / Build Environment (push) Failing after 1s
Server CI Cache Warm / Warm Go Module and Build Cache (push) Failing after 2s
Server CI Nightly Race / Build Environment (push) Failing after 1s
196 lines
9.0 KiB
TypeScript
196 lines
9.0 KiB
TypeScript
// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved.
|
|
// See LICENSE.txt for license information.
|
|
|
|
import {expect, test} from '@mattermost/playwright-lib';
|
|
|
|
test.describe('draft channel switch', () => {
|
|
/**
|
|
* @objective Verify a typed draft on one channel persists, restores after
|
|
* switching away and back, and posts only to the origin channel.
|
|
*/
|
|
test('typed draft stays on the origin channel after switching away and back', {tag: '@messaging'}, async ({pw}) => {
|
|
const {team, user} = await pw.initSetup();
|
|
const {channelsPage} = await pw.testBrowser.login(user);
|
|
await channelsPage.goto(team.name, 'off-topic');
|
|
await channelsPage.toBeVisible();
|
|
|
|
const originDraft = `origin-draft-${pw.random.id()}`;
|
|
const destinationMessage = `town-square-${pw.random.id()}`;
|
|
|
|
// # Type a draft in Off-Topic and do not send it
|
|
await channelsPage.centerView.postCreate.writeMessage(originDraft);
|
|
|
|
// # Switch to Town Square via the sidebar
|
|
await channelsPage.sidebarLeft.goToItem('town-square');
|
|
await channelsPage.centerView.header.toHaveTitle('Town Square');
|
|
|
|
// * Destination composer must not inherit the origin draft
|
|
expect(await channelsPage.centerView.postCreate.getInputValue()).toBe('');
|
|
|
|
// * Origin draft was persisted: the channel pencil is in the DOM
|
|
// (often CSS-hidden until hover) and the Drafts sidebar link appears
|
|
await expect(channelsPage.sidebarLeft.item('off-topic').getByTestId('draftIcon')).toHaveCount(1);
|
|
await channelsPage.sidebarLeft.draftsVisible();
|
|
|
|
// # Send a different message from Town Square
|
|
await channelsPage.centerView.postCreate.writeMessage(destinationMessage);
|
|
await channelsPage.centerView.postCreate.sendMessage();
|
|
|
|
// * Town Square shows the destination message
|
|
await channelsPage.centerView.waitUntilLastPostContains(destinationMessage);
|
|
|
|
// # Return to Off-Topic
|
|
await channelsPage.sidebarLeft.goToItem('off-topic');
|
|
await channelsPage.centerView.header.toHaveTitle('Off-Topic');
|
|
|
|
// * Origin draft is still in the composer
|
|
expect(await channelsPage.centerView.postCreate.getInputValue()).toBe(originDraft);
|
|
|
|
// # Send the restored draft
|
|
await channelsPage.centerView.postCreate.sendMessage();
|
|
|
|
// * Off-Topic shows the origin draft message
|
|
await channelsPage.centerView.waitUntilLastPostContains(originDraft);
|
|
|
|
// # Return to Town Square
|
|
await channelsPage.sidebarLeft.goToItem('town-square');
|
|
await channelsPage.centerView.header.toHaveTitle('Town Square');
|
|
|
|
// * Origin draft did not post to Town Square
|
|
await expect(channelsPage.centerView.container).not.toContainText(originDraft);
|
|
});
|
|
|
|
/**
|
|
* @objective Verify Ctrl/Cmd+K restores the destination draft and routes
|
|
* messages to the selected channel.
|
|
*/
|
|
test('quick switcher keeps drafts and messages scoped to their channels', {tag: '@messaging'}, async ({pw}) => {
|
|
const {team, user} = await pw.initSetup();
|
|
const {channelsPage, page} = await pw.testBrowser.login(user);
|
|
|
|
await channelsPage.goto(team.name, 'off-topic');
|
|
await channelsPage.toBeVisible();
|
|
|
|
const originDraft = `quick-switch-origin-${pw.random.id()}`;
|
|
const destinationMessage = `quick-switch-destination-${pw.random.id()}`;
|
|
|
|
// # Leave a draft in Off-Topic
|
|
await channelsPage.centerView.postCreate.writeMessage(originDraft);
|
|
|
|
// # Switch to Town Square using Ctrl/Cmd+K
|
|
await page.keyboard.press('ControlOrMeta+K');
|
|
await expect(channelsPage.findChannelsModal.input).toBeVisible();
|
|
await channelsPage.findChannelsModal.input.fill('town');
|
|
await channelsPage.findChannelsModal.selectChannel('town-square');
|
|
await channelsPage.centerView.header.toHaveTitle('Town Square');
|
|
|
|
// * Town Square did not inherit the Off-Topic draft
|
|
expect(await channelsPage.centerView.postCreate.getInputValue()).toBe('');
|
|
|
|
// # Send a destination-owned message
|
|
await channelsPage.centerView.postCreate.writeMessage(destinationMessage);
|
|
await channelsPage.centerView.postCreate.sendMessage();
|
|
await channelsPage.centerView.waitUntilLastPostContains(destinationMessage);
|
|
|
|
// # Return to Off-Topic using Ctrl/Cmd+K
|
|
await page.keyboard.press('ControlOrMeta+K');
|
|
await expect(channelsPage.findChannelsModal.input).toBeVisible();
|
|
await channelsPage.findChannelsModal.input.fill('off');
|
|
await channelsPage.findChannelsModal.selectChannel('off-topic');
|
|
await channelsPage.centerView.header.toHaveTitle('Off-Topic');
|
|
|
|
// * The origin draft was restored and the destination message was not misrouted
|
|
expect(await channelsPage.centerView.postCreate.getInputValue()).toBe(originDraft);
|
|
await expect(channelsPage.centerView.container).not.toContainText(destinationMessage);
|
|
});
|
|
|
|
/**
|
|
* @objective Verify sending /msg to an existing DM clears the origin
|
|
* channel draft instead of leaving it behind for later restoration.
|
|
*
|
|
* @precondition
|
|
* The DM channel already exists so the redirect uses the fast path with no
|
|
* createDirectChannel round trip.
|
|
*/
|
|
test(
|
|
'sending /msg to an existing DM clears the origin draft instead of restoring it',
|
|
{tag: '@slash_commands'},
|
|
async ({pw}) => {
|
|
const {adminClient, userClient, team, user} = await pw.initSetup();
|
|
const [target] = await adminClient.createUsers(team.id, 1, 'draft-msg');
|
|
|
|
await userClient.createDirectChannel([user.id, target.id]);
|
|
|
|
const {channelsPage, page} = await pw.testBrowser.login(user);
|
|
await channelsPage.goto(team.name, 'off-topic');
|
|
await channelsPage.toBeVisible();
|
|
|
|
// Trailing space dismisses the @mention autocomplete.
|
|
await channelsPage.centerView.postCreate.writeMessage(`/msg @${target.username} `);
|
|
await channelsPage.centerView.postCreate.sendMessage();
|
|
|
|
await channelsPage.centerView.header.toHaveTitle(target.username);
|
|
await expect(page).toHaveURL(new RegExp(`/${team.name}/messages/@${target.username}`));
|
|
|
|
// * Destination composer is empty — it did not adopt the /msg text
|
|
expect(await channelsPage.centerView.postCreate.getInputValue()).toBe('');
|
|
|
|
// # Return to Off-Topic
|
|
await channelsPage.sidebarLeft.goToItem('off-topic');
|
|
await channelsPage.centerView.header.toHaveTitle('Off-Topic');
|
|
|
|
// * Origin draft was cleared by the submit, not left behind as /msg
|
|
expect(await channelsPage.centerView.postCreate.getInputValue()).toBe('');
|
|
await expect(channelsPage.sidebarLeft.item('off-topic').getByTestId('draftIcon')).toHaveCount(0);
|
|
},
|
|
);
|
|
|
|
/**
|
|
* @objective Verify a message typed after a settled /msg redirect to an
|
|
* existing DM posts to the DM, not the origin channel.
|
|
*
|
|
* @precondition
|
|
* The DM channel already exists so the redirect uses the fast path with no
|
|
* createDirectChannel round trip.
|
|
*/
|
|
test(
|
|
'posts a later message to the DM rather than the origin channel after /msg',
|
|
{tag: '@slash_commands'},
|
|
async ({pw}) => {
|
|
const {adminClient, userClient, team, user} = await pw.initSetup();
|
|
const [target] = await adminClient.createUsers(team.id, 1, 'stale-dm');
|
|
|
|
const dmChannel = await userClient.createDirectChannel([user.id, target.id]);
|
|
await userClient.createPost({
|
|
channel_id: dmChannel.id,
|
|
message: 'seeding the existing DM',
|
|
} as Parameters<typeof userClient.createPost>[0]);
|
|
|
|
const {channelsPage, page} = await pw.testBrowser.login(user);
|
|
await channelsPage.goto(team.name, 'off-topic');
|
|
await channelsPage.toBeVisible();
|
|
|
|
// Trailing space dismisses the @mention autocomplete.
|
|
await channelsPage.centerView.postCreate.writeMessage(`/msg @${target.username} `);
|
|
await channelsPage.centerView.postCreate.sendMessage();
|
|
|
|
await channelsPage.centerView.header.toHaveTitle(target.username);
|
|
await expect(page).toHaveURL(new RegExp(`/${team.name}/messages/@${target.username}`));
|
|
|
|
const message = `stale-draft-${pw.random.id()}`;
|
|
await channelsPage.centerView.postCreate.writeMessage(message);
|
|
await channelsPage.centerView.postCreate.sendMessage();
|
|
|
|
// * Follow-up message appears in the DM
|
|
await channelsPage.centerView.waitUntilLastPostContains(message);
|
|
|
|
// # Return to Off-Topic
|
|
await channelsPage.sidebarLeft.goToItem('off-topic');
|
|
await channelsPage.centerView.header.toHaveTitle('Off-Topic');
|
|
|
|
// * Follow-up message did not post to the origin channel
|
|
await expect(channelsPage.centerView.container).not.toContainText(message);
|
|
},
|
|
);
|
|
});
|