JS: Really autoformat everything

Author: asgerf

javascript/ql/examples/queries/dataflow/StoredXss/StoredXss.ql

@@ -22,7 +22,8 @@ import DataFlow::PathGraph
  */
 class MysqlSource extends StoredXss::Source {
   MysqlSource() {
-    this = moduleImport("mysql")
+    this =
+      moduleImport("mysql")
           .getAMemberCall("createConnection")
           .getAMethodCall("query")
           .getCallback(1)

javascript/ql/src/semmle/javascript/security/dataflow/TaintedPathCustomizations.qll

@@ -368,7 +368,7 @@ module TaintedPath {
    *   // pathname is safe
    * }
    * ```
-   * 
+   *
    * or
    * ```
    * var relative = path.resolve(pathname); // or path.normalize
@@ -402,9 +402,13 @@ module TaintedPath {
     }
 
     override predicate blocks(boolean outcome, Expr e) {
-      member = "relative" and e = pathCall.getArgument(1).asExpr() and outcome = startsWith.getPolarity().booleanNot()
+      member = "relative" and
+      e = pathCall.getArgument(1).asExpr() and
+      outcome = startsWith.getPolarity().booleanNot()
       or
-      not member = "relative" and e = pathCall.getArgument(0).asExpr() and outcome = startsWith.getPolarity()
+      not member = "relative" and
+      e = pathCall.getArgument(0).asExpr() and
+      outcome = startsWith.getPolarity()
     }
   }
 

javascript/ql/test/library-tests/CallGraphs/AnnotatedTest/Test.ql

@@ -34,9 +34,7 @@ class AnnotatedCall extends InvokeExpr {
 
   AnnotatedFunction getAnExpectedCallee() { result.getCalleeName() = getCallTargetName() }
 
-  int getBoundArgs() {
-    result = getAnnotation(this, "boundArgs").toInt()
-  }
+  int getBoundArgs() { result = getAnnotation(this, "boundArgs").toInt() }
 
   int getBoundArgsOrMinusOne() {
     result = getBoundArgs()

javascript/ql/test/library-tests/Classes/tests.ql

@@ -17,4 +17,4 @@ import ConstructorDefinitions
 import ClassNodeConstructor
 import ClassNodeInstanceMethod
 import PrivateField
-import ClassFlow
+import ClassFlow

javascript/ql/test/library-tests/CustomLoadStoreSteps/test.ql

@@ -13,7 +13,7 @@ class Configuration extends TaintTracking::Configuration {
 
   // When the source code states that "foo" is being read, "bar" is additionally being read.
   override predicate isAdditionalLoadStep(DataFlow::Node pred, DataFlow::Node succ, string prop) {
-    exists(DataFlow::PropRead read | read = succ | 
+    exists(DataFlow::PropRead read | read = succ |
       read.getBase() = pred and
       read.getPropertyName() = "foo"
     ) and

javascript/ql/test/library-tests/DependencyModuleImports/Test.ql

@@ -32,9 +32,8 @@ class SampleVersionSink extends DataFlow::Node {
       functionName = "m1" and
       argNumber = 0
     |
-      this = DataFlow::dependencyModuleImport(dep)
-            .getAMemberCall(functionName)
-            .getArgument(argNumber) and
+      this =
+        DataFlow::dependencyModuleImport(dep).getAMemberCall(functionName).getArgument(argNumber) and
       dep.info(dependencyName, vDep) and
       vDep.maybeBetween(vFrom, vTo)
     )

javascript/ql/test/library-tests/PartialInvokeNode/test.ql

@@ -1,21 +1,21 @@
 import javascript
 
-query
-DataFlow::Node getBoundFunction(DataFlow::PartialInvokeNode invoke, DataFlow::Node callback, int boundArgs) {
+query DataFlow::Node getBoundFunction(
+  DataFlow::PartialInvokeNode invoke, DataFlow::Node callback, int boundArgs
+) {
   result = invoke.getBoundFunction(callback, boundArgs)
 }
 
-query
-predicate isPartialArgument(DataFlow::PartialInvokeNode invoke, DataFlow::Node callback, DataFlow::Node argument, int index) {
+query predicate isPartialArgument(
+  DataFlow::PartialInvokeNode invoke, DataFlow::Node callback, DataFlow::Node argument, int index
+) {
   invoke.isPartialArgument(callback, argument, index)
 }
 
-query
-DataFlow::Node getBoundReceiver(DataFlow::PartialInvokeNode invoke) {
+query DataFlow::Node getBoundReceiver(DataFlow::PartialInvokeNode invoke) {
   result = invoke.getBoundReceiver()
 }
 
-query
-DataFlow::Node clickEvent() {
+query DataFlow::Node clickEvent() {
   result = DataFlow::globalVarRef("addEventListener").getACall().getABoundCallbackParameter(1, 0)
 }

javascript/ql/test/library-tests/Promises/tests.ql

@@ -7,4 +7,4 @@ import PromiseDefinition_getAResolveHandler
 import PromiseDefinition_getRejectParameter
 import PromiseDefinition_getResolveParameter
 import PromiseDefinition_getACatchHandler
-import flow
+import flow

javascript/ql/test/library-tests/RangeAnalysis/DeadBranch.ql

@@ -22,8 +22,8 @@ class AssertionComment extends LineComment {
     isOK = true and
     exists(ConditionGuardNode guard | guard = getAGuardNode() |
       RangeAnalysis::isContradictoryGuardNode(guard) and
-      result = "Error: analysis claims " + getTestExpr() + " is always " +
-          guard.getOutcome().booleanNot()
+      result =
+        "Error: analysis claims " + getTestExpr() + " is always " + guard.getOutcome().booleanNot()
     )
     or
     isOK = false and

javascript/ql/test/library-tests/RegExp/EscapeInString/EscapeInString.ql

@@ -1,6 +1,7 @@
 import javascript
 
 from StringLiteral literal, RegExpDot dot, int pos
-where dot.getParent*() = literal
-  and pos = dot.getLocation().getStartColumn() - literal.getLocation().getStartColumn()
+where
+  dot.getParent*() = literal and
+  pos = dot.getLocation().getStartColumn() - literal.getLocation().getStartColumn()
 select dot, literal.getRawValue().charAt(pos)