Skip to content

Commit 949accb

Browse files
gustavoliraclaude
andcommitted
fix: comply with camelCase naming convention for coverage fixture
Rename _coverageCollector to coverageCollector to satisfy the @typescript-eslint/naming-convention rule, and suppress the rule for the Istanbul __coverage__ global which cannot be renamed. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent b5c8008 commit 949accb

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

src/playwright/fixtures/test.ts

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ type RHDHDeploymentTestFixtures = {
1010
rhdh: RHDHDeployment;
1111
uiHelper: UIhelper;
1212
loginHelper: LoginHelper;
13-
_coverageCollector: void;
13+
coverageCollector: void;
1414
};
1515

1616
type RHDHDeploymentWorkerFixtures = {
@@ -63,7 +63,7 @@ const baseTest = base.extend<
6363
},
6464
{ scope: "test" },
6565
] as const,
66-
_coverageCollector: [
66+
coverageCollector: [
6767
async ({ page }, use, testInfo) => {
6868
await use();
6969
if (process.env.E2E_COLLECT_COVERAGE !== "1") return;
@@ -72,6 +72,7 @@ const baseTest = base.extend<
7272
() =>
7373
(
7474
globalThis as unknown as {
75+
// eslint-disable-next-line @typescript-eslint/naming-convention
7576
__coverage__?: Record<string, unknown>;
7677
}
7778
).__coverage__,

0 commit comments

Comments
 (0)