Merge pull request github#5656 from asgerf/js/files-diagnostics

JS: Add file diagnostics queries

Author: RasmusWL

javascript/ql/src/Diagnostics/ExtractionErrors.ql

@@ -0,0 +1,18 @@
+/**
+ * @name Extraction errors
+ * @description List all extraction errors for files in the source code directory.
+ * @kind diagnostic
+ * @id js/diagnostics/extraction-errors
+ */
+
+import javascript
+
+/** Gets the SARIF severity to associate an error. */
+int getSeverity() { result = 2 }
+
+from Error error
+where
+  exists(error.getFile().getRelativePath()) and
+  error.isFatal()
+select error, "Extraction failed in " + error.getFile() + " with error " + error.getMessage(),
+  getSeverity()

javascript/ql/src/Diagnostics/SuccessfullyExtractedFiles.ql

@@ -0,0 +1,14 @@
+/**
+ * @name Successfully extracted files
+ * @description Lists all files in the source code directory that were extracted without encountering an error in the file.
+ * @kind diagnostic
+ * @id js/diagnostics/successfully-extracted-files
+ */
+
+import javascript
+
+from File f
+where
+  not exists(Error e | e.isFatal() and e.getFile() = f) and
+  exists(f.getRelativePath())
+select f, ""

javascript/ql/src/semmle/javascript/Errors.qll

@@ -10,6 +10,9 @@ abstract class Error extends Locatable {
   abstract string getMessage();
 
   override string toString() { result = getMessage() }
+
+  /** Holds if this error prevented the file from being extracted. */
+  predicate isFatal() { any() }
 }
 
 /** A JavaScript parse error encountered during extraction. */
@@ -21,4 +24,6 @@ class JSParseError extends @js_parse_error, Error {
 
   /** Gets the source text of the line this error occurs on. */
   string getLine() { js_parse_errors(this, _, _, result) }
+
+  override predicate isFatal() { not getTopLevel() instanceof Angular2::TemplateTopLevel }
 }

javascript/ql/src/semmle/javascript/Regexp.qll

@@ -840,6 +840,8 @@ class RegExpParseError extends Error, @regexp_parse_error {
   override string getMessage() { regexp_parse_errors(this, _, result) }
 
   override string toString() { result = getMessage() }
+
+  override predicate isFatal() { none() }
 }
 
 /**

javascript/ql/test/query-tests/Diagnostics/ExtractionErrors.expected

@@ -0,0 +1,4 @@
+| bad1.js:1:7:1:7 | Error: Unexpected token | Extraction failed in bad1.js with error Error: Unexpected token | 2 |
+| bad2.ts:1:11:1:11 | Error: Expression expected. | Extraction failed in bad2.ts with error Error: Expression expected. | 2 |
+| bad2.ts:1:13:1:13 | Error: Expression expected. | Extraction failed in bad2.ts with error Error: Expression expected. | 2 |
+| bad3.html:2:11:2:11 | Error: Unexpected token | Extraction failed in bad3.html with error Error: Unexpected token | 2 |

javascript/ql/test/query-tests/Diagnostics/ExtractionErrors.qlref

@@ -0,0 +1 @@
+Diagnostics/ExtractionErrors.ql

javascript/ql/test/query-tests/Diagnostics/SuccessfullyExtractedFiles.expected

@@ -0,0 +1,4 @@
+| contains-template.js:0:0:0:0 | contains-template.js |  |
+| good1.js:0:0:0:0 | good1.js |  |
+| good2.ts:0:0:0:0 | good2.ts |  |
+| good3.html:0:0:0:0 | good3.html |  |

javascript/ql/test/query-tests/Diagnostics/SuccessfullyExtractedFiles.qlref

@@ -0,0 +1 @@
+Diagnostics/SuccessfullyExtractedFiles.ql

javascript/ql/test/query-tests/Diagnostics/bad1.js

@@ -0,0 +1 @@
+let x x x;

javascript/ql/test/query-tests/Diagnostics/bad2.ts

@@ -0,0 +1 @@
+const z = ??;