Skip to content

Commit 0af670f

Browse files
committed
Merge branch 'master' of github.com:krausest/js-framework-benchmark
2 parents 787f660 + 09aa57f commit 0af670f

30 files changed

+554
-781
lines changed

webdriver-ts/check.dot

Lines changed: 0 additions & 87 deletions
This file was deleted.

webdriver-ts/index.ejs

Lines changed: 0 additions & 35 deletions
This file was deleted.

webdriver-ts/package.json

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,6 @@
1313
"resultsChrome": "node dist/createResultJS.js --browser chrome && cd .. && cd webdriver-ts-results && npm run build-prod",
1414
"resultsFirefox": "node dist/createResultJS.js --browser firefox && cd .. && cd webdriver-ts-results && npm run build-prod",
1515
"resultsSafari": "node dist/createResultJS.js --browser safari && cd .. && cd webdriver-ts-results && npm run build-prod",
16-
"index": "node dist/createIndex.js",
1716
"isKeyed": "cross-env LANG=\"en_US.UTF-8\" node dist/isKeyed.js",
1817
"test": "vitest --run",
1918
"test:watch": "vitest",

webdriver-ts/src/benchmarkRunner.ts

Lines changed: 21 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
import yargs from "yargs";
22
import {
33
BenchmarkOptions,
4-
BENCHMARK_RUNNER,
4+
BenchmarkRunner,
55
config,
66
ErrorAndWarning,
77
FrameworkData,
@@ -21,7 +21,7 @@ import {
2121
} from "./benchmarksCommon.js";
2222
import { StartupBenchmarkResult } from "./benchmarksLighthouse.js";
2323
import { writeResults } from "./writeResults.js";
24-
import { PlausibilityCheck, parseCPUTrace } from "./timeline.js";
24+
import { PlausibilityCheck } from "./timeline.js";
2525

2626
function forkAndCallBenchmark(
2727
framework: FrameworkData,
@@ -32,13 +32,13 @@ function forkAndCallBenchmark(
3232
let forkedRunner = null;
3333
if (benchmarkInfo.type === BenchmarkType.STARTUP_MAIN) {
3434
forkedRunner = "dist/forkedBenchmarkRunnerLighthouse.js";
35-
} else if (config.BENCHMARK_RUNNER == BENCHMARK_RUNNER.WEBDRIVER_CDP) {
35+
} else if (config.BENCHMARK_RUNNER == BenchmarkRunner.WEBDRIVER_CDP) {
3636
forkedRunner = "dist/forkedBenchmarkRunnerWebdriverCDP.js";
37-
} else if (config.BENCHMARK_RUNNER == BENCHMARK_RUNNER.PLAYWRIGHT) {
37+
} else if (config.BENCHMARK_RUNNER == BenchmarkRunner.PLAYWRIGHT) {
3838
forkedRunner = "dist/forkedBenchmarkRunnerPlaywright.js";
39-
} else if (config.BENCHMARK_RUNNER == BENCHMARK_RUNNER.WEBDRIVER) {
39+
} else if (config.BENCHMARK_RUNNER == BenchmarkRunner.WEBDRIVER) {
4040
forkedRunner = "dist/forkedBenchmarkRunnerWebdriver.js";
41-
} else if (config.BENCHMARK_RUNNER == BENCHMARK_RUNNER.WEBDRIVER_AFTERFRAME) {
41+
} else if (config.BENCHMARK_RUNNER == BenchmarkRunner.WEBDRIVER_AFTERFRAME) {
4242
forkedRunner = "dist/forkedBenchmarkRunnerWebdriverAfterframe.js";
4343
} else {
4444
forkedRunner = "dist/forkedBenchmarkRunnerPuppeteer.js";
@@ -126,6 +126,7 @@ async function runBenchmakLoop(
126126
framework: FrameworkData,
127127
benchmarkInfo: CPUBenchmarkInfo | MemBenchmarkInfo,
128128
benchmarkOptions: BenchmarkOptions,
129+
// eslint-disable-next-line @typescript-eslint/no-unused-vars
129130
plausibilityCheck: PlausibilityCheck
130131
): Promise<{ errors: string[]; warnings: string[] }> {
131132
let warnings: string[] = [];
@@ -240,7 +241,7 @@ async function runBench(
240241
warnings = warnings.concat(result.warnings);
241242
} catch (e) {
242243
console.log("UNHANDELED ERROR", e);
243-
errors.push(e);
244+
errors.push(e as string);
244245
}
245246
}
246247
}
@@ -300,21 +301,21 @@ console.log("args", args);
300301
let runner = args.runner;
301302
if (
302303
[
303-
BENCHMARK_RUNNER.WEBDRIVER_CDP,
304-
BENCHMARK_RUNNER.WEBDRIVER,
305-
BENCHMARK_RUNNER.WEBDRIVER_AFTERFRAME,
306-
BENCHMARK_RUNNER.PLAYWRIGHT,
307-
BENCHMARK_RUNNER.PUPPETEER,
304+
BenchmarkRunner.WEBDRIVER_CDP,
305+
BenchmarkRunner.WEBDRIVER,
306+
BenchmarkRunner.WEBDRIVER_AFTERFRAME,
307+
BenchmarkRunner.PLAYWRIGHT,
308+
BenchmarkRunner.PUPPETEER,
308309
].includes(runner)
309310
) {
310311
console.log(`INFO: Using ${runner} benchmark runner`);
311312
config.BENCHMARK_RUNNER = runner;
312313
} else {
313314
console.log("ERROR: argument driver has illegal value " + runner, [
314-
BENCHMARK_RUNNER.WEBDRIVER_CDP,
315-
BENCHMARK_RUNNER.WEBDRIVER,
316-
BENCHMARK_RUNNER.PLAYWRIGHT,
317-
BENCHMARK_RUNNER.PUPPETEER,
315+
BenchmarkRunner.WEBDRIVER_CDP,
316+
BenchmarkRunner.WEBDRIVER,
317+
BenchmarkRunner.PLAYWRIGHT,
318+
BenchmarkRunner.PUPPETEER,
318319
]);
319320
process.exit(1);
320321
}
@@ -360,8 +361,8 @@ console.log("benchmarkOptions", benchmarkOptions);
360361
args.benchmark && args.benchmark.length > 0 ? args.benchmark : [""];
361362
let runBenchmarks: Array<BenchmarkInfo> = benchmarkInfos.filter(
362363
(b) =>
363-
// afterframe currently only targets CPU benchmarks
364-
(config.BENCHMARK_RUNNER !== BENCHMARK_RUNNER.WEBDRIVER_AFTERFRAME ||
364+
// afterframe currently only targets CPU benchmarks
365+
(config.BENCHMARK_RUNNER !== BenchmarkRunner.WEBDRIVER_AFTERFRAME ||
365366
b.type == BenchmarkType.CPU) &&
366367
runBenchmarksArgs.some((name) => b.id.toLowerCase().indexOf(name) > -1)
367368
);
@@ -370,7 +371,7 @@ console.log("benchmarkOptions", benchmarkOptions);
370371
let matchesDirectoryArg = (directoryName: string) =>
371372
frameworkArgument.length == 0 || frameworkArgument.some((arg: string) => arg == directoryName);
372373
runFrameworks = (await initializeFrameworks(benchmarkOptions, matchesDirectoryArg)).filter(
373-
(f) => f.keyed || config.BENCHMARK_RUNNER !== BENCHMARK_RUNNER.WEBDRIVER_AFTERFRAME
374+
(f) => f.keyed || config.BENCHMARK_RUNNER !== BenchmarkRunner.WEBDRIVER_AFTERFRAME
374375
);
375376

376377
console.log("ARGS.smotest", args.smoketest);
@@ -386,7 +387,7 @@ console.log("benchmarkOptions", benchmarkOptions);
386387
});
387388
console.log("Using smoketest config ", JSON.stringify(config));
388389
}
389-
if (config.BENCHMARK_RUNNER == BENCHMARK_RUNNER.WEBDRIVER_AFTERFRAME) {
390+
if (config.BENCHMARK_RUNNER == BenchmarkRunner.WEBDRIVER_AFTERFRAME) {
390391
benchmarkOptions.resultsDirectory = "results_client_" + benchmarkOptions.browser;
391392
}
392393
if (!fs.existsSync(benchmarkOptions.resultsDirectory))

0 commit comments

Comments
 (0)