mirror of
https://github.com/hanzoai/team.git
synced 2026-09-30 01:57:35 +00:00
* ci: backport test-system-io v2 E2E dispatch onto release-10.11 Manually port dual v1/v2 routing from release-11.7/11.9 after #36376 cherry-pick conflicted heavily on missing Playwright specs. Adds template-v2 workflows pinned to test-system-io@1631d8fcea24, runner-prep- openldap, and E2E_USE_TEST_IO_DISPATCH gating while keeping legacy v1 shard/S3 path intact. Adapted for 10.11 webapp-setup (no read-only cache). Prerequisite for cherry-picking #37570. Co-authored-by: sabril <[email protected]> * E2E/Playwright: Add testcontainers to playwright-lib (#37570) * add testcontainers to playwright-lib * include retry mechanism for transient failures * - Introduced a new command `testcontainers:up` in package.json to run Playwright tests with Testcontainers. - Created a standalone Playwright configuration file `playwright.testcontainers-up.config.ts` for managing Testcontainers. - Added a no-op test `testcontainers_up.spec.ts` to ensure the Testcontainers stack is up during the test run. - Implemented a global setup script `testcontainers_up_global_setup.ts` to start and stop the Testcontainers stack. - Updated dependencies in package.json, including adding `chalk` for logging. * fix package-lock * fix tsc and restore waiting for all migrations to complete --------- Co-authored-by: Mattermost Build <[email protected]> * fix(e2e): remove obsolete CRT intro modal dismiss from uiClickSidebarItem The CRT tutorial modal was removed in MM-66470; dismissing #genericModalLabel races and flakes MM-T4261_3. Co-authored-by: sabril <[email protected]> * fix(e2e): sync Playwright navigations to live testcontainers baseURL After restartMattermostContainer remaps the host port, relative page.goto must follow testConfig.baseURL rather than the worker-start config value. Co-authored-by: sabril <[email protected]> * fix(e2e): ignore webhook assets in eslint; prettier README Co-authored-by: sabril <[email protected]> * ci(e2e): fully adopt test-system-io and lighten Playwright services Port #37413: drop the dual v1/v2 dispatch path, promote the test-system-io templates to the canonical names, and remove legacy calculate-results / AWS artifact upload plumbing. For Playwright testcontainers on this release, leave optional sidecars (openldap, keycloak, elasticsearch, opensearch, minio, azurite, webhook) off by default and remove the related service-specific specs that are not part of the older suite. Co-authored-by: sabril <[email protected]> * style(e2e): prettier stack.ts after optional webhook change Co-authored-by: sabril <[email protected]> * ci(e2e): grant contents:read to test-system-io workflow callers Reusable Playwright/Cypress templates require contents:read. Without it on the caller job, GitHub Actions fails the workflow at startup. Co-authored-by: sabril <[email protected]> * ci: retrigger enterprise build after prior failure Co-authored-by: sabril <[email protected]> * test(e2e): align testcontainers feature flags with release docker-compose Mirror MM_FEATUREFLAGS_* from e2e-tests/.ci/server.generate.sh for each release, dropping flags that do not exist in that release's FeatureFlags struct (or were never enabled in the compose stack). Co-authored-by: sabril <[email protected]> * ci(e2e): use Node 24 for Playwright testcontainers on release-10.11 testcontainers 12 pulls undici 8, which requires Node ≥22.5 (webidl.util.markAsUncloneable). release-10.11 .nvmrc is still Node 20, so prep-deps and dispatch-run switch to Node 24.11 after webapp-setup. Co-authored-by: sabril <[email protected]> * e2e(playwright): omit unset Testcontainers env keys; align release docs When optional webhook/sidecars are off, .env.testcontainers no longer writes PW_*=undefined. Docs match empty DEFAULT_TESTCONTAINERS_SERVICES. Co-authored-by: sabril <[email protected]> * e2e(playwright): prettier-format lib/README.md table Fixes CI npm run check (prettier) failures from the services table edit. Co-authored-by: sabril <[email protected]> * e2e(playwright): use getClientConfigOld on release-10.11 for mmctl version check getClientConfig was renamed after 10.11; the cherry-picked mmctl_remote spec failed with TypeError: adminClient.getClientConfig is not a function. Co-authored-by: sabril <[email protected]> --------- Co-authored-by: sabril <[email protected]> Co-authored-by: Mattermost Build <[email protected]> Co-authored-by: Hanzo Dev <[email protected]>
89 lines
3.6 KiB
JavaScript
89 lines
3.6 KiB
JavaScript
#!/usr/bin/env node
|
|
// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved.
|
|
// See LICENSE.txt for license information.
|
|
|
|
// Reused containers are deliberately left running after a Playwright process exits, so there's no
|
|
// in-process teardown to call from a fresh invocation of this script — it finds and removes them
|
|
// directly via Docker, by the label every container this project starts carries.
|
|
//
|
|
// Also doubles as CI's final teardown step: with PW_TESTCONTAINERS_REUSE, the stack stays up
|
|
// across every per-spec test invocation in a job, so none of them ever run the real (non-adopted)
|
|
// teardown path that collects container logs or archives the boot-env drift history. This script
|
|
// does both before removing anything, so `ci/upload-debug-artifacts` still picks them up.
|
|
|
|
import {execSync} from 'node:child_process';
|
|
import fs from 'node:fs';
|
|
import path from 'node:path';
|
|
|
|
const LABEL_FILTER = 'label=mm-playwright-testcontainers=true';
|
|
const LOG_DIR = path.resolve(process.cwd(), 'logs');
|
|
const ENV_FILE_PATH = path.resolve(process.cwd(), '.env.testcontainers');
|
|
|
|
function run(command) {
|
|
try {
|
|
return execSync(command, {encoding: 'utf-8'}).trim();
|
|
} catch {
|
|
return '';
|
|
}
|
|
}
|
|
|
|
// Reads the last-written PW_TESTCONTAINERS_NETWORK_NAME= line from .env.testcontainers before
|
|
// it's archived/removed, so this run's network can be removed by name instead of via a host-wide
|
|
// `docker network prune`. Entries are appended on each write, so the last match is current.
|
|
function readTestcontainersNetworkName() {
|
|
if (!fs.existsSync(ENV_FILE_PATH)) {
|
|
return undefined;
|
|
}
|
|
const matches = [...fs.readFileSync(ENV_FILE_PATH, 'utf-8').matchAll(/^PW_TESTCONTAINERS_NETWORK_NAME=(.*)$/gm)];
|
|
return matches.at(-1)?.[1]?.trim() || undefined;
|
|
}
|
|
|
|
// Archives, then removes, .env.testcontainers — a stale copy left behind would seed the next
|
|
// fresh boot's config overrides from a container that no longer exists, wrongly convincing a
|
|
// later pw.ensure*() call that some old setting is already active.
|
|
function archiveEnvFile() {
|
|
if (!fs.existsSync(ENV_FILE_PATH)) {
|
|
return;
|
|
}
|
|
fs.mkdirSync(LOG_DIR, {recursive: true});
|
|
fs.copyFileSync(ENV_FILE_PATH, path.join(LOG_DIR, 'testcontainers_env_history.log'));
|
|
fs.rmSync(ENV_FILE_PATH);
|
|
}
|
|
|
|
function collectLogs(containerIds) {
|
|
fs.mkdirSync(LOG_DIR, {recursive: true});
|
|
for (const id of containerIds) {
|
|
const image = run(`docker inspect -f "{{.Config.Image}}" ${id}`) || id;
|
|
const safeName = image.replace(/[^a-zA-Z0-9_.-]/g, '_');
|
|
const outPath = path.join(LOG_DIR, `${safeName}-${id.slice(0, 12)}.log`);
|
|
try {
|
|
execSync(`docker logs "${id}" > "${outPath}" 2>&1`);
|
|
} catch {
|
|
// Best-effort — a container that's already gone or never logged anything shouldn't
|
|
// block teardown.
|
|
}
|
|
}
|
|
}
|
|
|
|
const containerIds = run(`docker ps -aq --filter "${LABEL_FILTER}"`).split('\n').filter(Boolean);
|
|
|
|
const networkName = readTestcontainersNetworkName();
|
|
archiveEnvFile();
|
|
|
|
if (containerIds.length === 0) {
|
|
// eslint-disable-next-line no-console
|
|
console.log('No Testcontainers-managed containers found (nothing to remove).');
|
|
} else {
|
|
collectLogs(containerIds);
|
|
|
|
// eslint-disable-next-line no-console
|
|
console.log(`Removing ${containerIds.length} Testcontainers-managed container(s): ${containerIds.join(', ')}`);
|
|
execSync(`docker rm -f ${containerIds.join(' ')}`, {stdio: 'inherit'});
|
|
|
|
if (networkName) {
|
|
// eslint-disable-next-line no-console
|
|
console.log(`Removing Testcontainers-managed network: ${networkName}`);
|
|
run(`docker network rm ${networkName}`);
|
|
}
|
|
}
|