Skip to content

Commit c7c6fe1

Browse files
committed
Use block eslint-disable for naming-convention in tests
Replace single-line eslint-disable-next-line comments with explicit block disables/enables around the assertions in src/test/unittest/common/environment.unit.test.ts. This silences @typescript-eslint/naming-convention for the expected object literals (e.g. VAR1/VAR2) while keeping the rule enabled elsewhere.
1 parent a7baca7 commit c7c6fe1

1 file changed

Lines changed: 4 additions & 2 deletions

File tree

src/test/unittest/common/environment.unit.test.ts

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,11 +11,12 @@ suite('Environment File Parsing Tests', () => {
1111
const content = 'VAR1=value1\nVAR2=value2';
1212
const result = parseEnvFile(content);
1313

14-
// eslint-disable-next-line @typescript-eslint/naming-convention
14+
/* eslint-disable @typescript-eslint/naming-convention */
1515
expect(result).to.deep.equal({
1616
VAR1: 'value1',
1717
VAR2: 'value2',
1818
});
19+
/* eslint-enable @typescript-eslint/naming-convention */
1920
});
2021

2122
test('Should parse single-quoted multiline values', () => {
@@ -72,11 +73,12 @@ suite('Environment File Parsing Tests', () => {
7273
const content = 'VAR1=value1\nInvalid line\nVAR2=value2';
7374
const result = parseEnvFile(content);
7475

75-
// eslint-disable-next-line @typescript-eslint/naming-convention
76+
/* eslint-disable @typescript-eslint/naming-convention */
7677
expect(result).to.deep.equal({
7778
VAR1: 'value1',
7879
VAR2: 'value2',
7980
});
81+
/* eslint-enable @typescript-eslint/naming-convention */
8082
});
8183

8284
test('Should handle multiline value with multiple newlines', () => {

0 commit comments

Comments
 (0)