perf(document): Improve preset test performance

acot-a11y · Dec 13, 2020 · 51aa3d2 · 51aa3d2
1 parent d48a4d3
commit 51aa3d2
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 4 deletions.
diff --git a/packages/cli/src/commands/preset-test.ts b/packages/cli/src/commands/preset-test.ts
@@ -59,7 +59,7 @@ export default createCommand({
     port,
   });
 
-  const tester = new DocTester(server);
+  const tester = new DocTester(server, {});
   const reporter = new DocResultReporter({
     origin: `http://localhost:${port}`,
   });

diff --git a/packages/document/src/doc-tester.ts b/packages/document/src/doc-tester.ts
@@ -1,3 +1,4 @@
+import os from 'os';
 import type { TestcaseResult } from '@acot/types';
 import { Acot } from '@acot/core';
 import plur from 'plur';
@@ -8,11 +9,20 @@ import type { DocProject } from './doc-project';
 import type { DocResult } from './doc-result';
 import { debug } from './logging';
 
+export type DocTesterConfig = {
+  parallel: number;
+};
+
 export class DocTester {
   private _server: DocServer;
+  private _config: DocTesterConfig;
 
-  public constructor(server: DocServer) {
+  public constructor(server: DocServer, config: Partial<DocTesterConfig> = {}) {
     this._server = server;
+    this._config = {
+      parallel: os.cpus().length,
+      ...config,
+    };
   }
 
   public async test(project: DocProject): Promise<DocResult> {
@@ -30,7 +40,7 @@ export class DocTester {
       launchOptions: {
         executablePath: chromium?.executablePath,
       },
-      parallel: 1, // TODO Handle parameters
+      parallel: this._config.parallel,
       presets: [project.preset],
       origin: `http://localhost:${port}`,
     });
@@ -106,7 +116,6 @@ export class DocTester {
         }
       });
     } catch (e) {
-      // TODO logging
       debug('Unexpected doc-test error:', e);
     }