test(comma-style): make tests more strict (#2813)

ST-DDT · web-flow · commit 52f42d310006 · 2025-07-22T10:44:31.000+02:00
diff --git a/tests/lib/rules/comma-style.js b/tests/lib/rules/comma-style.js
@@ -70,7 +70,10 @@ tester.run('comma-style', rule, {
       errors: [
         {
           message: "',' should be placed last.",
-          line: 5
+          line: 5,
+          column: 13,
+          endLine: 5,
+          endColumn: 14
         }
       ]
     },
@@ -89,7 +92,10 @@ tester.run('comma-style', rule, {
       errors: [
         {
           message: "',' should be placed last.",
-          line: 4
+          line: 4,
+          column: 13,
+          endLine: 4,
+          endColumn: 14
         }
       ]
     },
@@ -107,8 +113,11 @@ tester.run('comma-style', rule, {
       options: ['first', { exceptions: { ArrowFunctionExpression: false } }],
       errors: [
         {
-          message: "',' should be placed first."
-          // line: 3 // eslint v7.0
+          message: "',' should be placed first.",
+          line: 3,
+          column: 40,
+          endLine: 3,
+          endColumn: 41
         }
       ]
     },
@@ -130,8 +139,11 @@ tester.run('comma-style', rule, {
       options: ['first', { exceptions: { FunctionExpression: false } }],
       errors: [
         {
-          message: "',' should be placed first."
-          // line: 3 // eslint v7.0
+          message: "',' should be placed first.",
+          line: 3,
+          column: 36,
+          endLine: 3,
+          endColumn: 37
         }
       ]
     },
@@ -151,8 +163,11 @@ tester.run('comma-style', rule, {
       options: ['last', { exceptions: { FunctionExpression: false } }],
       errors: [
         {
-          message: "',' should be placed last."
-          // line: 3 // eslint v7.0
+          message: "',' should be placed last.",
+          line: 5,
+          column: 13,
+          endLine: 5,
+          endColumn: 14
         }
       ]
     }

Original file line number	Diff line number	Diff line change
`@@ -70,7 +70,10 @@ tester.run('comma-style', rule, {`
`70`	`70`	`errors: [`
`71`	`71`	`{`
`72`	`72`	`message: "',' should be placed last.",`
`73`		`- line: 5`
	`73`	`+ line: 5,`
	`74`	`+ column: 13,`
	`75`	`+ endLine: 5,`
	`76`	`+ endColumn: 14`
`74`	`77`	`}`
`75`	`78`	`]`
`76`	`79`	`},`
`@@ -89,7 +92,10 @@ tester.run('comma-style', rule, {`
`89`	`92`	`errors: [`
`90`	`93`	`{`
`91`	`94`	`message: "',' should be placed last.",`
`92`		`- line: 4`
	`95`	`+ line: 4,`
	`96`	`+ column: 13,`
	`97`	`+ endLine: 4,`
	`98`	`+ endColumn: 14`
`93`	`99`	`}`
`94`	`100`	`]`
`95`	`101`	`},`
`@@ -107,8 +113,11 @@ tester.run('comma-style', rule, {`
`107`	`113`	`options: ['first', { exceptions: { ArrowFunctionExpression: false } }],`
`108`	`114`	`errors: [`
`109`	`115`	`{`
`110`		`- message: "',' should be placed first."`
`111`		`- // line: 3 // eslint v7.0`
	`116`	`+ message: "',' should be placed first.",`
	`117`	`+ line: 3,`
	`118`	`+ column: 40,`
	`119`	`+ endLine: 3,`
	`120`	`+ endColumn: 41`
`112`	`121`	`}`
`113`	`122`	`]`
`114`	`123`	`},`
`@@ -130,8 +139,11 @@ tester.run('comma-style', rule, {`
`130`	`139`	`options: ['first', { exceptions: { FunctionExpression: false } }],`
`131`	`140`	`errors: [`
`132`	`141`	`{`
`133`		`- message: "',' should be placed first."`
`134`		`- // line: 3 // eslint v7.0`
	`142`	`+ message: "',' should be placed first.",`
	`143`	`+ line: 3,`
	`144`	`+ column: 36,`
	`145`	`+ endLine: 3,`
	`146`	`+ endColumn: 37`
`135`	`147`	`}`
`136`	`148`	`]`
`137`	`149`	`},`
`@@ -151,8 +163,11 @@ tester.run('comma-style', rule, {`
`151`	`163`	`options: ['last', { exceptions: { FunctionExpression: false } }],`
`152`	`164`	`errors: [`
`153`	`165`	`{`
`154`		`- message: "',' should be placed last."`
`155`		`- // line: 3 // eslint v7.0`
	`166`	`+ message: "',' should be placed last.",`
	`167`	`+ line: 5,`
	`168`	`+ column: 13,`
	`169`	`+ endLine: 5,`
	`170`	`+ endColumn: 14`
`156`	`171`	`}`
`157`	`172`	`]`
`158`	`173`	`}`