From 4690a5353b71c10533a83464b7a3fc4ac9df99dc Mon Sep 17 00:00:00 2001 From: kl Date: Mon, 9 Mar 2026 10:12:48 +0800 Subject: [PATCH] test(e2e): expand archive coverage to tar/tgz/7z/rar --- .github/workflows/nightly-e2e.yml | 4 +- .github/workflows/pr-e2e-mvp.yml | 4 +- tests/e2e/.gitignore | 2 +- tests/e2e/README.md | 4 +- tests/e2e/fixtures/sample.7z | Bin 0 -> 177 bytes tests/e2e/fixtures/sample.rar | 1 + tests/e2e/fixtures/sample.tar | Bin 0 -> 4096 bytes tests/e2e/fixtures/sample.tgz | Bin 0 -> 375 bytes tests/e2e/scripts/generate-fixtures.mjs | 50 +++++++++++++++++------- tests/e2e/specs/preview-smoke.spec.ts | 28 ++++++++++++- 10 files changed, 70 insertions(+), 23 deletions(-) create mode 100644 tests/e2e/fixtures/sample.7z create mode 100644 tests/e2e/fixtures/sample.rar create mode 100644 tests/e2e/fixtures/sample.tar create mode 100644 tests/e2e/fixtures/sample.tgz diff --git a/.github/workflows/nightly-e2e.yml b/.github/workflows/nightly-e2e.yml index 3543706b..431ff4a0 100644 --- a/.github/workflows/nightly-e2e.yml +++ b/.github/workflows/nightly-e2e.yml @@ -36,10 +36,10 @@ jobs: with: python-version: '3.11' - - name: Install LibreOffice + zip + - name: Install LibreOffice + archive tools run: | sudo apt-get update - sudo apt-get install -y libreoffice zip + sudo apt-get install -y libreoffice zip p7zip-full - name: Setup Python deps for office fixtures run: | diff --git a/.github/workflows/pr-e2e-mvp.yml b/.github/workflows/pr-e2e-mvp.yml index 161b525d..62e111d5 100644 --- a/.github/workflows/pr-e2e-mvp.yml +++ b/.github/workflows/pr-e2e-mvp.yml @@ -36,10 +36,10 @@ jobs: with: python-version: '3.11' - - name: Install LibreOffice + zip + - name: Install LibreOffice + archive tools run: | sudo apt-get update - sudo apt-get install -y libreoffice zip + sudo apt-get install -y libreoffice zip p7zip-full - name: Setup Python deps for office fixtures run: | diff --git a/tests/e2e/.gitignore b/tests/e2e/.gitignore index f3b87092..55cbce0a 100644 --- a/tests/e2e/.gitignore +++ b/tests/e2e/.gitignore @@ -3,7 +3,7 @@ playwright-report/ test-results/ __pycache__/ -fixtures/zip-tmp/ +fixtures/archive-tmp/ fixtures/sample.docx fixtures/sample.xlsx fixtures/sample.pptx diff --git a/tests/e2e/README.md b/tests/e2e/README.md index c5336162..ce570a87 100644 --- a/tests/e2e/README.md +++ b/tests/e2e/README.md @@ -6,7 +6,7 @@ This folder contains a first MVP of end-to-end automated tests. - Basic preview smoke checks for common file types (txt/md/json/xml/csv/html/png) - Office Phase-2 smoke checks (docx/xlsx/pptx) -- Archive smoke check (zip) +- Archive smoke checks (zip/tar/tgz/7z/rar) - Basic endpoint reachability - Security regression checks for blocked internal-network hosts (`10.*`) on: - `/onlinePreview` @@ -31,7 +31,7 @@ npx playwright install --with-deps chromium pip3 install -r requirements.txt ``` -> Prerequisite: ensure `zip` command is available in PATH (used for `sample.zip` fixture generation). +> Prerequisite: ensure `zip`, `tar`, and `7z` (or `bsdtar` fallback) are available in PATH for archive fixtures. 3. Generate fixtures and start fixture server: diff --git a/tests/e2e/fixtures/sample.7z b/tests/e2e/fixtures/sample.7z new file mode 100644 index 0000000000000000000000000000000000000000..527017d1c3329b13113960e8bdfc27833de32605 GIT binary patch literal 177 zcmXr7+Ou9=hJm^ElLfOn0|aD2X@MiFo-mj`6uFdRxbA4~64R4)6;J*@-Z-1H@xscr z9}1N&ajdKVB>(?s)dL1b7B&V(PIU%$Zbk-1WoAZ3)>sCH1_mBk1`aO9j}@6a7#LU? z1w|P$8S;QIm7$11kD-L20*DzHM7S6k_E=m}$Xb7gQ3#1G3}mm)z5*5(Wn*MeXjsz7 GzyJV@oHD5Z literal 0 HcmV?d00001 diff --git a/tests/e2e/fixtures/sample.rar b/tests/e2e/fixtures/sample.rar new file mode 100644 index 00000000..088dfdb4 --- /dev/null +++ b/tests/e2e/fixtures/sample.rar @@ -0,0 +1 @@ +Rar!u{1a}u{7}u{0} diff --git a/tests/e2e/fixtures/sample.tar b/tests/e2e/fixtures/sample.tar new file mode 100644 index 0000000000000000000000000000000000000000..89aabcbfdca2b847c570c6399dd7ad685b59fe5a GIT binary patch literal 4096 zcmeHI%}&BV5MD@(5zl(?v`j2BNni5FkM z7jpQCkRs@T#u#^!$2(3-&dg##Su!&2qA)Vj9|5eHC_puv23Y9kD7fiv)1@KOE({sVIwjoNhEsNxBHJ5iRI;X7`>XUHJvDkBK{L*+}~RVb51 zd(Y0p`|I0dqEx5rWS}bJl9E>{+)wWw0BNk&KP3tMb8ZQQ<8=g6 z+HmJGW-?(2lXA`u?0n9$aiv^zYKNtA^T>#06otlDaxFOLW#=TI#hZM!7zeE!&*M52 zRa&cx<}{vL_Ifm)E(7VQe%AMQ`X{E9&_A;%)Bpb&gXu+`&au~3O;6op8M@crP~q_> Wf!hG5_gH8$_bj`XnpZTT@!YE4!@83E?Y~(6#NgBSKc_6c|Nnwc=h*G4rmG%62F|rRP%!)@psmBJ VZZ0JyB_$;>J^`(95R3p2004H?x#<7^ literal 0 HcmV?d00001 diff --git a/tests/e2e/scripts/generate-fixtures.mjs b/tests/e2e/scripts/generate-fixtures.mjs index afd108cb..b617d56d 100644 --- a/tests/e2e/scripts/generate-fixtures.mjs +++ b/tests/e2e/scripts/generate-fixtures.mjs @@ -16,20 +16,42 @@ write('sample.xml', 'kkFileViewtrue'); write('sample.csv', 'name,value\nkkFileView,1\ne2e,1\n'); write('sample.html', '

kkFileView fixture

'); -// zip (contains txt) - only generate if missing to avoid noisy local diffs -const zipPath = path.join(fixturesDir, 'sample.zip'); -if (!fs.existsSync(zipPath)) { - const zipWork = path.join(fixturesDir, 'zip-tmp'); - fs.mkdirSync(zipWork, { recursive: true }); - fs.writeFileSync(path.join(zipWork, 'inner.txt'), 'kkFileView zip inner file'); - try { - execFileSync('zip', ['-X', '-q', '-r', zipPath, 'inner.txt'], { cwd: zipWork }); - } catch (err) { - console.error('Failed to create sample.zip fixture. Ensure "zip" is installed and available in PATH.'); - throw err instanceof Error ? err : new Error(String(err)); - } finally { - fs.rmSync(zipWork, { recursive: true, force: true }); - } +// archive fixtures (contains inner.txt) - generate if missing +const archiveWork = path.join(fixturesDir, 'archive-tmp'); +fs.mkdirSync(archiveWork, { recursive: true }); +fs.writeFileSync(path.join(archiveWork, 'inner.txt'), 'kkFileView archive inner file'); + +const ensureArchive = (name, generator) => { + const out = path.join(fixturesDir, name); + if (fs.existsSync(out)) return; + generator(out); +}; + +try { + ensureArchive('sample.zip', out => { + execFileSync('zip', ['-X', '-q', '-r', out, 'inner.txt'], { cwd: archiveWork }); + }); + + ensureArchive('sample.tar', out => { + execFileSync('tar', ['-cf', out, 'inner.txt'], { cwd: archiveWork }); + }); + + ensureArchive('sample.tgz', out => { + execFileSync('tar', ['-czf', out, 'inner.txt'], { cwd: archiveWork }); + }); + + ensureArchive('sample.7z', out => { + try { + execFileSync('7z', ['a', '-bd', '-y', out, 'inner.txt'], { cwd: archiveWork, stdio: 'ignore' }); + } catch { + execFileSync('bsdtar', ['-a', '-cf', out, 'inner.txt'], { cwd: archiveWork }); + } + }); +} catch (err) { + console.error('Failed to create archive fixtures. Ensure tar/zip/7z (or bsdtar) are available in PATH.'); + throw err instanceof Error ? err : new Error(String(err)); +} finally { + fs.rmSync(archiveWork, { recursive: true, force: true }); } // 1x1 png diff --git a/tests/e2e/specs/preview-smoke.spec.ts b/tests/e2e/specs/preview-smoke.spec.ts index afa769b4..03df155d 100644 --- a/tests/e2e/specs/preview-smoke.spec.ts +++ b/tests/e2e/specs/preview-smoke.spec.ts @@ -25,6 +25,10 @@ test.beforeAll(async () => { 'sample.xlsx', 'sample.pptx', 'sample.zip', + 'sample.tar', + 'sample.tgz', + 'sample.7z', + 'sample.rar', ]; try { @@ -97,13 +101,33 @@ test('12 zip preview', async ({ request }) => { expect(resp.status()).toBe(200); }); -test('13 security: block 10.x host in onlinePreview', async ({ request }) => { +test('13 tar preview', async ({ request }) => { + const resp = await openPreview(request, `${fixtureBase}/sample.tar`); + expect(resp.status()).toBe(200); +}); + +test('14 tgz preview', async ({ request }) => { + const resp = await openPreview(request, `${fixtureBase}/sample.tgz`); + expect(resp.status()).toBe(200); +}); + +test('15 7z preview', async ({ request }) => { + const resp = await openPreview(request, `${fixtureBase}/sample.7z`); + expect(resp.status()).toBe(200); +}); + +test('16 rar preview', async ({ request }) => { + const resp = await openPreview(request, `${fixtureBase}/sample.rar`); + expect(resp.status()).toBe(200); +}); + +test('17 security: block 10.x host in onlinePreview', async ({ request }) => { const resp = await openPreview(request, `http://10.1.2.3/a.pdf`); const body = await resp.text(); expect(body).toContain('不受信任'); }); -test('14 security: block 10.x host in getCorsFile', async ({ request }) => { +test('18 security: block 10.x host in getCorsFile', async ({ request }) => { const encoded = b64('http://10.1.2.3/a.pdf'); const resp = await request.get(`/getCorsFile?urlPath=${encoded}`); const body = await resp.text();