Enable back S3827 no-reference-error for JS in HTML files (#3690)

SonarSource · Feb 10, 2023 · e112541 · e112541
1 parent 8e057fd
commit e112541
Show file tree

Hide file tree

Showing 11 changed files with 733 additions and 7 deletions.
diff --git a/its/plugin/tests/src/test/resources/html-blacklist-profile.xml b/its/plugin/tests/src/test/resources/html-blacklist-profile.xml
@@ -8,11 +8,6 @@
       <key>S3923</key> <!-- all duplicate branches -->
       <priority>INFO</priority>
     </rule>
-    <rule>
-      <repositoryKey>javascript</repositoryKey>
-      <key>S3827</key> <!-- no-reference-error -->
-      <priority>INFO</priority>
-    </rule>
     <rule>
       <repositoryKey>javascript</repositoryKey>
       <key>S3504</key> <!-- no-var -->

diff --git a/its/ruling/src/test/expected/js/jStorage/javascript-S3827.json b/its/ruling/src/test/expected/js/jStorage/javascript-S3827.json
@@ -0,0 +1,5 @@
+{
+'jStorage:example/index.html':[
+161,
+],
+}
diff --git a/its/ruling/src/test/expected/js/javascript-test-sources/javascript-S3827.json b/its/ruling/src/test/expected/js/javascript-test-sources/javascript-S3827.json
@@ -3,6 +3,15 @@
 725,
 725,
 ],
+'javascript-test-sources:src/ace/build_support/editor.html':[
+33,
+],
+'javascript-test-sources:src/ace/demo/autocompletion.html':[
+31,
+],
+'javascript-test-sources:src/ace/demo/csp.html':[
+23,
+],
 'javascript-test-sources:src/ace/demo/kitchen-sink/demo.js':[
 266,
 ],
@@ -17,6 +26,21 @@
 'javascript-test-sources:src/ace/doc/template/resources/javascripts/prettify-extension.js':[
 23,
 ],
+'javascript-test-sources:src/ace/experiments/animate_folding.html':[
+117,
+],
+'javascript-test-sources:src/ace/experiments/capture.html':[
+29,
+],
+'javascript-test-sources:src/ace/experiments/debug_mem_leak.html':[
+30,
+],
+'javascript-test-sources:src/ace/experiments/triple_click.html':[
+22,
+],
+'javascript-test-sources:src/ace/index.html':[
+955,
+],
 'javascript-test-sources:src/ace/lib/ace/mode/html/saxparser.js':[
 6274,
 6310,
@@ -37,6 +61,9 @@
 'javascript-test-sources:src/ace/lib/ace/worker/worker_client_v2.js':[
 102,
 ],
+'javascript-test-sources:src/ace/tool/perf-test.html':[
+87,
+],
 'javascript-test-sources:src/ecmascript6/Ghost/core/client/app/components/gh-cm-editor.js':[
 24,
 ],

diff --git a/its/ruling/src/test/expected/js/jquery/javascript-S3827.json b/its/ruling/src/test/expected/js/jquery/javascript-S3827.json
@@ -0,0 +1,104 @@
+{
+'jquery:test/data/ajax/onunload.html':[
+21,
+],
+'jquery:test/data/ajax/unreleasedXHR.html':[
+14,
+],
+'jquery:test/data/core/aliased.html':[
+21,
+21,
+],
+'jquery:test/data/core/dynamic_ready.html':[
+17,
+24,
+],
+'jquery:test/data/core/onready.html':[
+20,
+],
+'jquery:test/data/css/cssWidthBeforeDocReady.html':[
+19,
+],
+'jquery:test/data/css/cssWidthBrowserZoom.html':[
+24,
+],
+'jquery:test/data/data/dataAttrs.html':[
+10,
+],
+'jquery:test/data/dimensions/documentLarge.html':[
+17,
+],
+'jquery:test/data/event/focusElem.html':[
+14,
+],
+'jquery:test/data/event/focusinCrossFrame.html':[
+15,
+],
+'jquery:test/data/event/interactiveReady.html':[
+13,
+],
+'jquery:test/data/event/promiseReady.html':[
+11,
+],
+'jquery:test/data/event/syncReady.html':[
+13,
+],
+'jquery:test/data/event/triggerunload.html':[
+16,
+],
+'jquery:test/data/manipulation/iframe-denied.html':[
+29,
+],
+'jquery:test/data/manipulation/scripts-context.html':[
+12,
+],
+'jquery:test/data/offset/absolute.html':[
+28,
+],
+'jquery:test/data/offset/body.html':[
+20,
+],
+'jquery:test/data/offset/boxes.html':[
+74,
+],
+'jquery:test/data/offset/fixed.html':[
+24,
+],
+'jquery:test/data/offset/relative.html':[
+24,
+],
+'jquery:test/data/offset/scroll.html':[
+28,
+],
+'jquery:test/data/offset/static.html':[
+23,
+],
+'jquery:test/data/offset/table.html':[
+21,
+],
+'jquery:test/data/readywait.html':[
+36,
+],
+'jquery:test/data/selector/cache.html':[
+22,
+],
+'jquery:test/data/selector/html5_selector.html':[
+19,
+],
+'jquery:test/data/selector/mixed_sort.html':[
+16,
+],
+'jquery:test/data/support/bodyBackground.html':[
+25,
+26,
+],
+'jquery:test/index.html':[
+41,
+],
+'jquery:test/integration/data/gh-1764-fullscreen-iframe.html':[
+18,
+],
+'jquery:test/integration/gh-1764-fullscreen.html':[
+31,
+],
+}
diff --git a/its/ruling/src/test/expected/js/mootools-core/javascript-S3827.json b/its/ruling/src/test/expected/js/mootools-core/javascript-S3827.json
@@ -36,4 +36,21 @@
 'mootools-core:Source/Utilities/JSON.js':[
 51,
 ],
+'mootools-core:Tests/Element/Element.Delegation.html':[
+124,
+163,
+197,
+235,
+269,
+291,
+304,
+],
+'mootools-core:Tests/Element/Element.Event.change.html':[
+117,
+],
+'mootools-core:Tests/Fx/Fx.html':[
+120,
+145,
+205,
+],
 }
diff --git a/its/ruling/src/test/expected/js/p5.js/javascript-S3827.json b/its/ruling/src/test/expected/js/p5.js/javascript-S3827.json
@@ -22,4 +22,12 @@
 'p5.js:src/image/image.js':[
 210,
 ],
+'p5.js:test/test-reference.html':[
+27,
+34,
+37,
+],
+'p5.js:test/test.html':[
+30,
+],
 }