test(comma-spacing): make tests more strict (#2812)

ST-DDT · web-flow · commit 62d70c69c5b4 · 2025-07-22T10:40:29.000+02:00
diff --git a/tests/lib/rules/comma-spacing.js b/tests/lib/rules/comma-spacing.js
@@ -91,12 +91,16 @@ tester.run('comma-spacing', rule, {
         {
           message: "There should be no space before ','.",
           line: 4,
-          column: 25
+          column: 25,
+          endLine: 4,
+          endColumn: 26
         },
         {
           message: "A space is required after ','.",
           line: 4,
-          column: 25
+          column: 25,
+          endLine: 4,
+          endColumn: 26
         }
       ]
     },
@@ -112,19 +116,31 @@ tester.run('comma-spacing', rule, {
       errors: [
         {
           message: "There should be no space before ','.",
-          line: 3
+          line: 3,
+          column: 36,
+          endLine: 3,
+          endColumn: 37
         },
         {
           message: "A space is required after ','.",
-          line: 3
+          line: 3,
+          column: 36,
+          endLine: 3,
+          endColumn: 37
         },
         {
           message: "There should be no space before ','.",
-          line: 3
+          line: 3,
+          column: 40,
+          endLine: 3,
+          endColumn: 41
         },
         {
           message: "A space is required after ','.",
-          line: 3
+          line: 3,
+          column: 40,
+          endLine: 3,
+          endColumn: 41
         }
       ]
     },
@@ -140,19 +156,31 @@ tester.run('comma-spacing', rule, {
       errors: [
         {
           message: "There should be no space before ','.",
-          line: 3
+          line: 3,
+          column: 39,
+          endLine: 3,
+          endColumn: 40
         },
         {
           message: "A space is required after ','.",
-          line: 3
+          line: 3,
+          column: 39,
+          endLine: 3,
+          endColumn: 40
         },
         {
           message: "There should be no space before ','.",
-          line: 3
+          line: 3,
+          column: 46,
+          endLine: 3,
+          endColumn: 47
         },
         {
           message: "A space is required after ','.",
-          line: 3
+          line: 3,
+          column: 46,
+          endLine: 3,
+          endColumn: 47
         }
       ]
     },
@@ -168,11 +196,17 @@ tester.run('comma-spacing', rule, {
       errors: [
         {
           message: "There should be no space before ','.",
-          line: 3
+          line: 3,
+          column: 33,
+          endLine: 3,
+          endColumn: 34
         },
         {
           message: "A space is required after ','.",
-          line: 3
+          line: 3,
+          column: 33,
+          endLine: 3,
+          endColumn: 34
         }
       ]
     },
@@ -188,11 +222,17 @@ tester.run('comma-spacing', rule, {
       errors: [
         {
           message: "There should be no space before ','.",
-          line: 3
+          line: 3,
+          column: 30,
+          endLine: 3,
+          endColumn: 31
         },
         {
           message: "A space is required after ','.",
-          line: 3
+          line: 3,
+          column: 30,
+          endLine: 3,
+          endColumn: 31
         }
       ]
     },
@@ -208,11 +248,17 @@ tester.run('comma-spacing', rule, {
       errors: [
         {
           message: "There should be no space before ','.",
-          line: 3
+          line: 3,
+          column: 39,
+          endLine: 3,
+          endColumn: 40
         },
         {
           message: "A space is required after ','.",
-          line: 3
+          line: 3,
+          column: 39,
+          endLine: 3,
+          endColumn: 40
         }
       ]
     },
@@ -232,11 +278,17 @@ tester.run('comma-spacing', rule, {
       errors: [
         {
           message: "There should be no space before ','.",
-          line: 4
+          line: 4,
+          column: 31,
+          endLine: 4,
+          endColumn: 32
         },
         {
           message: "A space is required after ','.",
-          line: 4
+          line: 4,
+          column: 31,
+          endLine: 4,
+          endColumn: 32
         }
       ]
     },
@@ -252,11 +304,17 @@ tester.run('comma-spacing', rule, {
       errors: [
         {
           message: "There should be no space before ','.",
-          line: 3
+          line: 3,
+          column: 29,
+          endLine: 3,
+          endColumn: 30
         },
         {
           message: "A space is required after ','.",
-          line: 3
+          line: 3,
+          column: 29,
+          endLine: 3,
+          endColumn: 30
         }
       ]
     },
@@ -277,11 +335,17 @@ tester.run('comma-spacing', rule, {
       errors: [
         {
           message: "A space is required before ','.",
-          line: 4
+          line: 4,
+          column: 17,
+          endLine: 4,
+          endColumn: 18
         },
         {
           message: "There should be no space after ','.",
-          line: 4
+          line: 4,
+          column: 17,
+          endLine: 4,
+          endColumn: 18
         }
       ]
     },
@@ -301,7 +365,10 @@ tester.run('comma-spacing', rule, {
       errors: [
         {
           message: "A space is required after ','.",
-          line: 4
+          line: 4,
+          column: 29,
+          endLine: 4,
+          endColumn: 30
         }
       ]
     }

Original file line number	Diff line number	Diff line change
`@@ -91,12 +91,16 @@ tester.run('comma-spacing', rule, {`
`91`	`91`	`{`
`92`	`92`	`message: "There should be no space before ','.",`
`93`	`93`	`line: 4,`
`94`		`- column: 25`
	`94`	`+ column: 25,`
	`95`	`+ endLine: 4,`
	`96`	`+ endColumn: 26`
`95`	`97`	`},`
`96`	`98`	`{`
`97`	`99`	`message: "A space is required after ','.",`
`98`	`100`	`line: 4,`
`99`		`- column: 25`
	`101`	`+ column: 25,`
	`102`	`+ endLine: 4,`
	`103`	`+ endColumn: 26`
`100`	`104`	`}`
`101`	`105`	`]`
`102`	`106`	`},`
`@@ -112,19 +116,31 @@ tester.run('comma-spacing', rule, {`
`112`	`116`	`errors: [`
`113`	`117`	`{`
`114`	`118`	`message: "There should be no space before ','.",`
`115`		`- line: 3`
	`119`	`+ line: 3,`
	`120`	`+ column: 36,`
	`121`	`+ endLine: 3,`
	`122`	`+ endColumn: 37`
`116`	`123`	`},`
`117`	`124`	`{`
`118`	`125`	`message: "A space is required after ','.",`
`119`		`- line: 3`
	`126`	`+ line: 3,`
	`127`	`+ column: 36,`
	`128`	`+ endLine: 3,`
	`129`	`+ endColumn: 37`
`120`	`130`	`},`
`121`	`131`	`{`
`122`	`132`	`message: "There should be no space before ','.",`
`123`		`- line: 3`
	`133`	`+ line: 3,`
	`134`	`+ column: 40,`
	`135`	`+ endLine: 3,`
	`136`	`+ endColumn: 41`
`124`	`137`	`},`
`125`	`138`	`{`
`126`	`139`	`message: "A space is required after ','.",`
`127`		`- line: 3`
	`140`	`+ line: 3,`
	`141`	`+ column: 40,`
	`142`	`+ endLine: 3,`
	`143`	`+ endColumn: 41`
`128`	`144`	`}`
`129`	`145`	`]`
`130`	`146`	`},`
`@@ -140,19 +156,31 @@ tester.run('comma-spacing', rule, {`
`140`	`156`	`errors: [`
`141`	`157`	`{`
`142`	`158`	`message: "There should be no space before ','.",`
`143`		`- line: 3`
	`159`	`+ line: 3,`
	`160`	`+ column: 39,`
	`161`	`+ endLine: 3,`
	`162`	`+ endColumn: 40`
`144`	`163`	`},`
`145`	`164`	`{`
`146`	`165`	`message: "A space is required after ','.",`
`147`		`- line: 3`
	`166`	`+ line: 3,`
	`167`	`+ column: 39,`
	`168`	`+ endLine: 3,`
	`169`	`+ endColumn: 40`
`148`	`170`	`},`
`149`	`171`	`{`
`150`	`172`	`message: "There should be no space before ','.",`
`151`		`- line: 3`
	`173`	`+ line: 3,`
	`174`	`+ column: 46,`
	`175`	`+ endLine: 3,`
	`176`	`+ endColumn: 47`
`152`	`177`	`},`
`153`	`178`	`{`
`154`	`179`	`message: "A space is required after ','.",`
`155`		`- line: 3`
	`180`	`+ line: 3,`
	`181`	`+ column: 46,`
	`182`	`+ endLine: 3,`
	`183`	`+ endColumn: 47`
`156`	`184`	`}`
`157`	`185`	`]`
`158`	`186`	`},`
`@@ -168,11 +196,17 @@ tester.run('comma-spacing', rule, {`
`168`	`196`	`errors: [`
`169`	`197`	`{`
`170`	`198`	`message: "There should be no space before ','.",`
`171`		`- line: 3`
	`199`	`+ line: 3,`
	`200`	`+ column: 33,`
	`201`	`+ endLine: 3,`
	`202`	`+ endColumn: 34`
`172`	`203`	`},`
`173`	`204`	`{`
`174`	`205`	`message: "A space is required after ','.",`
`175`		`- line: 3`
	`206`	`+ line: 3,`
	`207`	`+ column: 33,`
	`208`	`+ endLine: 3,`
	`209`	`+ endColumn: 34`
`176`	`210`	`}`
`177`	`211`	`]`
`178`	`212`	`},`
`@@ -188,11 +222,17 @@ tester.run('comma-spacing', rule, {`
`188`	`222`	`errors: [`
`189`	`223`	`{`
`190`	`224`	`message: "There should be no space before ','.",`
`191`		`- line: 3`
	`225`	`+ line: 3,`
	`226`	`+ column: 30,`
	`227`	`+ endLine: 3,`
	`228`	`+ endColumn: 31`
`192`	`229`	`},`
`193`	`230`	`{`
`194`	`231`	`message: "A space is required after ','.",`
`195`		`- line: 3`
	`232`	`+ line: 3,`
	`233`	`+ column: 30,`
	`234`	`+ endLine: 3,`
	`235`	`+ endColumn: 31`
`196`	`236`	`}`
`197`	`237`	`]`
`198`	`238`	`},`
`@@ -208,11 +248,17 @@ tester.run('comma-spacing', rule, {`
`208`	`248`	`errors: [`
`209`	`249`	`{`
`210`	`250`	`message: "There should be no space before ','.",`
`211`		`- line: 3`
	`251`	`+ line: 3,`
	`252`	`+ column: 39,`
	`253`	`+ endLine: 3,`
	`254`	`+ endColumn: 40`
`212`	`255`	`},`
`213`	`256`	`{`
`214`	`257`	`message: "A space is required after ','.",`
`215`		`- line: 3`
	`258`	`+ line: 3,`
	`259`	`+ column: 39,`
	`260`	`+ endLine: 3,`
	`261`	`+ endColumn: 40`
`216`	`262`	`}`
`217`	`263`	`]`
`218`	`264`	`},`
`@@ -232,11 +278,17 @@ tester.run('comma-spacing', rule, {`
`232`	`278`	`errors: [`
`233`	`279`	`{`
`234`	`280`	`message: "There should be no space before ','.",`
`235`		`- line: 4`
	`281`	`+ line: 4,`
	`282`	`+ column: 31,`
	`283`	`+ endLine: 4,`
	`284`	`+ endColumn: 32`
`236`	`285`	`},`
`237`	`286`	`{`
`238`	`287`	`message: "A space is required after ','.",`
`239`		`- line: 4`
	`288`	`+ line: 4,`
	`289`	`+ column: 31,`
	`290`	`+ endLine: 4,`
	`291`	`+ endColumn: 32`
`240`	`292`	`}`
`241`	`293`	`]`
`242`	`294`	`},`
`@@ -252,11 +304,17 @@ tester.run('comma-spacing', rule, {`
`252`	`304`	`errors: [`
`253`	`305`	`{`
`254`	`306`	`message: "There should be no space before ','.",`
`255`		`- line: 3`
	`307`	`+ line: 3,`
	`308`	`+ column: 29,`
	`309`	`+ endLine: 3,`
	`310`	`+ endColumn: 30`
`256`	`311`	`},`
`257`	`312`	`{`
`258`	`313`	`message: "A space is required after ','.",`
`259`		`- line: 3`
	`314`	`+ line: 3,`
	`315`	`+ column: 29,`
	`316`	`+ endLine: 3,`
	`317`	`+ endColumn: 30`
`260`	`318`	`}`
`261`	`319`	`]`
`262`	`320`	`},`
`@@ -277,11 +335,17 @@ tester.run('comma-spacing', rule, {`
`277`	`335`	`errors: [`
`278`	`336`	`{`
`279`	`337`	`message: "A space is required before ','.",`
`280`		`- line: 4`
	`338`	`+ line: 4,`
	`339`	`+ column: 17,`
	`340`	`+ endLine: 4,`
	`341`	`+ endColumn: 18`
`281`	`342`	`},`
`282`	`343`	`{`
`283`	`344`	`message: "There should be no space after ','.",`
`284`		`- line: 4`
	`345`	`+ line: 4,`
	`346`	`+ column: 17,`
	`347`	`+ endLine: 4,`
	`348`	`+ endColumn: 18`
`285`	`349`	`}`
`286`	`350`	`]`
`287`	`351`	`},`
`@@ -301,7 +365,10 @@ tester.run('comma-spacing', rule, {`
`301`	`365`	`errors: [`
`302`	`366`	`{`
`303`	`367`	`message: "A space is required after ','.",`
`304`		`- line: 4`
	`368`	`+ line: 4,`
	`369`	`+ column: 29,`
	`370`	`+ endLine: 4,`
	`371`	`+ endColumn: 30`
`305`	`372`	`}`
`306`	`373`	`]`
`307`	`374`	`}`