feat(infra): remove cli-tests DOCSTOOLS-4734

Author: goldserg

.eslintignore

@@ -9,14 +9,14 @@ node_modules
 .env
 .DS_Store
 .eslintrc.js
-vitest.config.mjs
+vitest.units.config.ts
+vitest.integration.config.ts
 /dist
 /cache
 /coverage
 /external
 /scripts
 /tests/mocks
-/tests/bin.mjs
 
 .lintstagedrc.js
 .prettierrc.js

.github/workflows/integration-tests.yml

@@ -35,5 +35,4 @@ jobs:
         run: npm run build
 
       - name: Run e2e tests
-        working-directory: tests
-        run: npm ci && npm test
+        run: npm run e2e

.github/workflows/release.yml

@@ -105,16 +105,6 @@ jobs:
         env:
           NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }}
 
-      - name: Publish stable tests to npm
-        working-directory: tests
-        if: (github.event_name == 'release' && github.event.release.draft == false) || (github.event_name == 'workflow_dispatch' && inputs.release_type == 'stable')
-        run: |
-          npm version ${{steps.package.outputs.version}}
-          npm ci
-          npm publish --provenance --access public
-        env:
-          NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }}
-
       # === PRERELEASE STEPS ===
       - name: Set prerelease version
         if: github.event_name == 'workflow_dispatch' && inputs.release_type == 'prerelease'
@@ -126,20 +116,6 @@ jobs:
         env:
           NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }}
 
-      - name: Set prerelease version for tests
-        working-directory: tests
-        if: github.event_name == 'workflow_dispatch' && inputs.release_type == 'prerelease'
-        run: |
-          npm ci
-          npm version --no-git-tag-version 0.0.0-rc-$(git branch --show-current)-${{github.run_id}}
-
-      - name: Publish prerelease tests to npm
-        working-directory: tests
-        if: github.event_name == 'workflow_dispatch' && inputs.release_type == 'prerelease'
-        run: npm publish --tag $(git branch --show-current) --provenance --access public
-        env:
-          NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }}
-
       # === DEPRECATE STEPS ===
       - name: Validate deprecate inputs
         if: github.event_name == 'workflow_dispatch' && inputs.release_type == 'deprecate'

.prettierignore

@@ -4,7 +4,8 @@
 .env
 .DS_Store
 .eslintrc.js
-vitest.config.mjs
+vitest.units.config.ts
+vitest.integration.config.ts
 node_modules
 /lib
 /dist
@@ -13,4 +14,3 @@ node_modules
 /coverage
 /external
 /tests/mocks
-/tests/bin.mjs

package.json

@@ -34,7 +34,8 @@
     "./manifest": "./build/manifest.json",
     "./mdit-plugins": "./build/mdit-plugins.js",
     "./arcadia-vcs": "./build/arcadia-vcs.js",
-    "./github-vcs": "./build/github-vcs.js"
+    "./github-vcs": "./build/github-vcs.js",
+    "./tests/fixtures": "./tests/fixtures/index.ts"
   },
   "main": "build/index.js",
   "bin": {
@@ -45,18 +46,20 @@
     "lib",
     "build",
     "assets",
-    "schemas"
+    "schemas",
+    "tests",
+    "vitest.integration.config.ts"
   ],
   "scripts": {
     "build": "npm run build:clean && node scripts/build.cli.js",
     "build:types": "tspc -p tsconfig.lib.json || true; tspc -p tsconfig.build.json || true",
     "build:clean": "rm -rf build lib",
-    "e2e": "npm run build && cd tests && npm test",
+    "e2e": "npm run build && vitest run -c vitest.integration.config.ts",
     "prepublishOnly": "npm run lint && npm run build && npm run build:types",
     "start": "node build/index.js",
-    "test": "vitest run",
-    "test:coverage": "vitest run --coverage",
-    "test:watch": "vitest",
+    "test": "vitest run -c vitest.units.config.ts",
+    "test:coverage": "vitest run -c vitest.units.config.ts --coverage",
+    "test:watch": "vitest -c vitest.units.config.ts",
     "typecheck": "tsc --noEmit",
     "regenerate-schemas": "scripts/regenerate-schemas.js",
     "lint": "lint",
@@ -137,6 +140,7 @@
     "esbuild": "^0.27.2",
     "html-escaper": "^3.0.3",
     "husky": "8.0.3",
+    "markdown-it-attrs": "^4.1.0",
     "mime-types": "2.1.35",
     "minimatch": "9.0.3",
     "node-html-parser": "6.1.5",

tests/bin.mjs

@@ -1,4 +1,4 @@
-import type {Report} from '../fixtures/runners/types';
+import type {Report} from '../fixtures/runner';
 
 import {describe, expect, it} from 'vitest';
 

tests/e2e/errors.spec.ts

@@ -1,7 +1,5 @@
-import type {Runner} from './runners';
-
-import {createRunner} from './runners';
-import {cleanupDirectory} from './utils/file';
+import runner from './runner';
+import {cleanupDirectory} from './file';
 
 export interface BuildRunArgs {
     md2md?: boolean;
@@ -18,41 +16,27 @@ export interface TranslateRunArgs {
 }
 
 class Build {
-    private readonly runner: Runner;
-
-    constructor(runner: Runner) {
-        this.runner = runner;
-    }
-
     run(input: string, output: string, args: string[], env?: Record<string, string>) {
-        return this.runner.runYfmDocs(
+        return runner.runYfmDocs(
             ['--input', input, '--output', output, '--quiet', '--allowHtml', ...args],
             env,
         );
     }
 }
 
 class Extract {
-    private readonly runner: Runner;
-
-    constructor(runner: Runner) {
-        this.runner = runner;
-    }
-
     run(input: string, output: string, args: string[], env?: Record<string, string>) {
-        return this.runner.runYfmDocs(
+        return runner.runYfmDocs(
             ['translate', 'extract', '--input', input, '--output', output, '--quiet', ...args],
             env,
         );
     }
 }
 
 export class CliTestAdapter {
-    readonly runner: Runner = createRunner();
-
-    readonly build = new Build(this.runner);
+    readonly build = new Build();
 
-    readonly extract = new Extract(this.runner);
+    readonly extract = new Extract();
 
     async testBuildPass(
         inputPath: string,
@@ -110,7 +94,7 @@ export class CliTestAdapter {
             ...additionalArgs.split(' ').filter(Boolean),
         ];
 
-        await this.runner.runYfmDocs(baseArgs);
+        await runner.runYfmDocs(baseArgs);
     }
 }
 

tests/fixtures/cliAdapter.ts tests/fixtures/cli.tstests/fixtures/cliAdapter.ts renamed to tests/fixtures/cli.ts

@@ -1,13 +1,12 @@
-import type {BuildRunArgs} from '../cliAdapter';
+import type {BuildRunArgs} from './cli';
 
 import {readFileSync} from 'node:fs';
 import {rm} from 'node:fs/promises';
 import {join, resolve} from 'node:path';
 import {glob} from 'glob';
 import {expect, test} from 'vitest';
 
-import {TestAdapter} from '../cliAdapter';
-
+import {TestAdapter} from './cli';
 import {bundleless, hashless, platformless} from './test';
 
 const SYSTEM_DIRS = ['_bundle/', '_search/'];
@@ -124,8 +123,8 @@ type TestPaths = {
 
 export function getTestPaths(testRootPath: string): TestPaths {
     return {
-        inputPath: resolve(__dirname, '../../', join(testRootPath, 'input')),
-        outputPath: resolve(__dirname, '../../', join(testRootPath, 'output')),
+        inputPath: resolve(__dirname, '../', join(testRootPath, 'input')),
+        outputPath: resolve(__dirname, '../', join(testRootPath, 'output')),
     };
 }
 

tests/fixtures/utils/file.ts tests/fixtures/file.tstests/fixtures/utils/file.ts renamed to tests/fixtures/file.ts

@@ -1,3 +1,4 @@
-export * from './cliAdapter';
-export * from './utils/file';
-export * from './utils/test';
+export * from './cli';
+export * from './file';
+export * from './test';
+export * from './runner';