From d02bcb30e3dc12dddd539155ed6fae36cf0f4a34 Mon Sep 17 00:00:00 2001 From: Ilya Goncharov Date: Fri, 29 Nov 2019 11:06:16 +0300 Subject: [PATCH] [Gradle, JS] Move classNameOnlyRegExp to class level --- .../tools/kotlin-test-js-runner/src/KotlinTestsFilter.ts | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/libraries/tools/kotlin-test-js-runner/src/KotlinTestsFilter.ts b/libraries/tools/kotlin-test-js-runner/src/KotlinTestsFilter.ts index 4cec6913b42..e492f135603 100644 --- a/libraries/tools/kotlin-test-js-runner/src/KotlinTestsFilter.ts +++ b/libraries/tools/kotlin-test-js-runner/src/KotlinTestsFilter.ts @@ -124,7 +124,11 @@ export class StartsWithFilter implements KotlinTestsFilter { } export class ExactFilter implements KotlinTestsFilter { + private readonly classNameOnlyRegExp: RegExp; + constructor(public fqn: string) { + // Exact filter by class name only + this.classNameOnlyRegExp = RegExp(`^${escapeRegExp(this.fqn + ".")}[^\.]+$`); } mayContainTestsFromSuite(fqn: string): boolean { @@ -136,9 +140,7 @@ export class ExactFilter implements KotlinTestsFilter { return true } - // Exact filter by class name only - const regexp = RegExp(`^${escapeRegExp(this.fqn + ".")}[^\.]+$`); - return regexp.test(fqn) + return this.classNameOnlyRegExp.test(fqn) } }