JS-526 Support ESLint configuration comment syntax (#5075)

SonarSource · Jan 31, 2025 · 66f2fb6 · 66f2fb6
1 parent 872478d
commit 66f2fb6
Show file tree

Hide file tree

Showing 149 changed files with 5,638 additions and 19,693 deletions.
diff --git a/its/plugin/tests/src/test/java/com/sonar/javascript/it/plugin/TypeScriptAnalysisTest.java b/its/plugin/tests/src/test/java/com/sonar/javascript/it/plugin/TypeScriptAnalysisTest.java
@@ -55,8 +55,9 @@ void test() throws Exception {
 
     String sampleFileKey = projectKey + ":sample.lint.ts";
     List<Issue> issuesList = getIssues(sampleFileKey);
-    assertThat(issuesList).hasSize(1);
-    assertThat(issuesList.get(0).getLine()).isEqualTo(4);
+    assertThat(issuesList).hasSize(2);
+    assertThat(issuesList.get(0).getLine()).isEqualTo(3);
+    assertThat(issuesList.get(1).getLine()).isEqualTo(4);
 
     assertThat(OrchestratorStarter.getMeasureAsInt(sampleFileKey, "ncloc")).isEqualTo(7);
     assertThat(OrchestratorStarter.getMeasureAsInt(sampleFileKey, "classes")).isEqualTo(0);

diff --git a/its/ruling/src/test/expected/jsts/Ghost/javascript-S3827.json b/its/ruling/src/test/expected/jsts/Ghost/javascript-S3827.json
@@ -1,56 +1,19 @@
 {
-"Ghost:core/client/app/components/gh-cm-editor.js": [
-24
-],
 "Ghost:core/client/app/components/gh-navitem-url-input.js": [
 18
 ],
 "Ghost:core/client/app/components/gh-profile-image.js": [
 51
 ],
-"Ghost:core/client/app/components/gh-search-input.js": [
-116
-],
-"Ghost:core/client/app/components/gh-tag-settings-form.js": [
-102
-],
-"Ghost:core/client/app/components/gh-trim-focus-input.js": [
-13
-],
-"Ghost:core/client/app/components/modals/base.js": [
-16
-],
 "Ghost:core/client/app/controllers/setup/three.js": [
 45
 ],
-"Ghost:core/client/app/helpers/gh-format-html.js": [
-22
-],
-"Ghost:core/client/app/helpers/gh-format-markdown.js": [
-7,
-28
-],
-"Ghost:core/client/app/mixins/ed-editor-shortcuts.js": [
-106
-],
-"Ghost:core/client/app/mixins/shortcuts.js": [
-9
-],
 "Ghost:core/client/app/models/user.js": [
 67
 ],
-"Ghost:core/client/app/routes/settings/tags.js": [
-103
-],
-"Ghost:core/client/app/utils/random-password.js": [
-4
-],
 "Ghost:core/client/app/utils/validator-extensions.js": [
 8
 ],
-"Ghost:core/client/app/utils/word-count.js": [
-5
-],
 "Ghost:core/client/app/validators/invite-user.js": [
 9
 ],
@@ -98,51 +61,5 @@
 ],
 "Ghost:core/client/tests/acceptance/team-test.js": [
 37
-],
-"Ghost:core/client/tests/integration/components/gh-profile-image-test.js": [
-59
-],
-"Ghost:core/test/functional/client/about_test.js": [
-6
-],
-"Ghost:core/test/functional/client/app_test.js": [
-6,
-142
-],
-"Ghost:core/test/functional/client/content_test.js": [
-6,
-26
-],
-"Ghost:core/test/functional/client/editor_test.js": [
-5,
-42
-],
-"Ghost:core/test/functional/client/psm_test.js": [
-6,
-66
-],
-"Ghost:core/test/functional/client/settings_test.js": [
-10
-],
-"Ghost:core/test/functional/client/signin_test.js": [
-6,
-27,
-43,
-73
-],
-"Ghost:core/test/functional/client/signout_test.js": [
-5
-],
-"Ghost:core/test/functional/client/signup_test.js": [
-6
-],
-"Ghost:core/test/functional/client/team_test.js": [
-10
-],
-"Ghost:core/test/functional/setup/setup_test.js": [
-5,
-11,
-20,
-79
 ]
 }
diff --git a/its/ruling/src/test/expected/jsts/ace/javascript-S103.json b/its/ruling/src/test/expected/jsts/ace/javascript-S103.json
@@ -80,8 +80,7 @@
 ],
 "ace:lib/ace/mode/yaml/yaml-lint.js": [
 1,
-79,
-2879
+79
 ],
 "ace:src/autocomplete_test.js": [
 29,

diff --git a/its/ruling/src/test/expected/jsts/ace/javascript-S122.json b/its/ruling/src/test/expected/jsts/ace/javascript-S122.json
@@ -403,8 +403,7 @@
 696
 ],
 "ace:src/snippets_test.js": [
-196,
-228
+196
 ],
 "ace:src/test/all_browser.js": [
 117

diff --git a/its/ruling/src/test/expected/jsts/ace/javascript-S1442.json b/its/ruling/src/test/expected/jsts/ace/javascript-S1442.json
@@ -5,9 +5,6 @@
 "ace:demo/kitchen-sink/docs/javascript.js": [
 3
 ],
-"ace:src/ext/emmet.js": [
-211
-],
 "ace:src/keyboard/vim.js": [
 5475,
 5500

diff --git a/its/ruling/src/test/expected/jsts/ace/javascript-S2137.json b/its/ruling/src/test/expected/jsts/ace/javascript-S2137.json
@@ -18,9 +18,5 @@
 4191,
 4192,
 4214
-],
-"ace:src/snippets_test.js": [
-228,
-244
 ]
 }
diff --git a/its/ruling/src/test/expected/jsts/ace/javascript-S2424.json b/its/ruling/src/test/expected/jsts/ace/javascript-S2424.json
@@ -54,10 +54,6 @@
 "ace:src/snippets.js": [
 109
 ],
-"ace:src/snippets_test.js": [
-228,
-244
-],
 "ace:src/test/asyncjs/async.js": [
 272,
 277

diff --git a/its/ruling/src/test/expected/jsts/ace/javascript-S3523.json b/its/ruling/src/test/expected/jsts/ace/javascript-S3523.json
@@ -9,10 +9,5 @@
 248,
 2408,
 3316
-],
-"ace:lib/ace/mode/yaml/yaml-lint.js": [
-3528,
-3533,
-4037
 ]
 }
diff --git a/its/ruling/src/test/expected/jsts/ace/javascript-S3827.json b/its/ruling/src/test/expected/jsts/ace/javascript-S3827.json
@@ -1,8 +1,4 @@
 {
-"ace:Makefile.dryice.js": [
-725,
-725
-],
 "ace:build_support/editor.html": [
 33
 ],

diff --git a/its/ruling/src/test/expected/jsts/angular.js/javascript-S103.json b/its/ruling/src/test/expected/jsts/angular.js/javascript-S103.json
@@ -5,13 +5,6 @@
 "angular.js:src/ng/animate.js": [
 307
 ],
-"angular.js:src/ng/directive/input.js": [
-27
-],
-"angular.js:src/ng/directive/ngOptions.js": [
-237,
-238
-],
 "angular.js:src/ng/sce.js": [
 622,
 625,

diff --git a/its/ruling/src/test/expected/jsts/angular.js/javascript-S1143.json b/its/ruling/src/test/expected/jsts/angular.js/javascript-S1143.json
diff --git a/its/ruling/src/test/expected/jsts/angular.js/javascript-S1440.json b/its/ruling/src/test/expected/jsts/angular.js/javascript-S1440.json
diff --git a/its/ruling/src/test/expected/jsts/angular.js/javascript-S2189.json b/its/ruling/src/test/expected/jsts/angular.js/javascript-S2189.json
diff --git a/its/ruling/src/test/expected/jsts/angular.js/javascript-S2430.json b/its/ruling/src/test/expected/jsts/angular.js/javascript-S2430.json
diff --git a/its/ruling/src/test/expected/jsts/angular.js/javascript-S2703.json b/its/ruling/src/test/expected/jsts/angular.js/javascript-S2703.json
diff --git a/its/ruling/src/test/expected/jsts/angular.js/javascript-S3523.json b/its/ruling/src/test/expected/jsts/angular.js/javascript-S3523.json
diff --git a/its/ruling/src/test/expected/jsts/angular.js/javascript-S3696.json b/its/ruling/src/test/expected/jsts/angular.js/javascript-S3696.json
diff --git a/its/ruling/src/test/expected/jsts/angular.js/javascript-S3798.json b/its/ruling/src/test/expected/jsts/angular.js/javascript-S3798.json
@@ -328,10 +328,7 @@
 534,
 541,
 554,
-578,
-601,
 605,
-609,
 618,
 626,
 809,
@@ -371,7 +368,6 @@
 34
 ],
 "angular.js:src/ng/browser.js": [
-4,
 9,
 34,
 342
@@ -502,12 +498,6 @@
 93
 ],
 "angular.js:src/ng/directive/ngModel.js": [
-16,
-17,
-18,
-19,
-20,
-21,
 22,
 23,
 25,
@@ -678,7 +668,6 @@
 59,
 81,
 92,
-98,
 103,
 108,
 122,
@@ -943,10 +932,6 @@
 "angular.js:src/ngMessageFormat/messageFormatService.js": [
 175,
 202,
-212,
-213,
-214,
-215,
 216,
 218
 ],
@@ -1022,15 +1007,8 @@
 84
 ],
 "angular.js:src/ngTouch/touch.js": [
-25,
 29
 ],
-"angular.js:src/routeToRegExp.js": [
-17
-],
-"angular.js:src/shallowCopy.js": [
-10
-],
 "angular.js:src/stringify.js": [
 5,
 28
-Original file line number
+Diff line change
@@ Expand Up / @@ -80,8 +80,7 @@ @@
     ],
     "ace:lib/ace/mode/yaml/yaml-lint.js": [
 ,
-,
     ],
     "ace:src/autocomplete_test.js": [
 ,
@@ Expand Down @@