Format test QL libaries with codeql query format

Author: mario-campos

ql/ql/test/queries/performance/MissingNoInline/Test.qll

@@ -5,7 +5,8 @@ import ql
  *
  * This predicate exists to fix a join order.
  */
-predicate missingNoInline(AddExpr add, Expr e1, Expr e2) { // $ Alert
+predicate missingNoInline(AddExpr add, Expr e1, Expr e2) {
+  // $ Alert
   // BAD
   add.getLeftOperand() = e1 and
   add.getRightOperand() = e2

ql/ql/test/queries/style/AcronymsShouldBeCamelCase/Test.qll

@@ -5,7 +5,8 @@ predicate isXML() { any() } // $ Alert
 predicate isAES() { any() }
 
 // BAD
-newtype TXMLElements = // $ Alert
+newtype TXMLElements =
+  // $ Alert
   TXmlElement() or // GOOD
   TXMLElement() // $ Alert // BAD
 

ql/ql/test/queries/style/DataFlowConfigModuleNaming/Test.qll

@@ -8,14 +8,16 @@ module EmptyConfig implements DataFlow::ConfigSig {
 }
 
 // BAD - does not end with "Config"
-module EmptyConfiguration implements DataFlow::ConfigSig { // $ Alert
+module EmptyConfiguration implements DataFlow::ConfigSig {
+  // $ Alert
   predicate isSource(DataFlow::Node src) { none() }
 
   predicate isSink(DataFlow::Node sink) { none() }
 }
 
 // BAD - does not end with "Config"
-module EmptyFlow implements DataFlow::ConfigSig { // $ Alert
+module EmptyFlow implements DataFlow::ConfigSig {
+  // $ Alert
   predicate isSource(DataFlow::Node src) { none() }
 
   predicate isSink(DataFlow::Node sink) { none() }

ql/ql/test/queries/style/DeadCode/Foo.qll

@@ -43,7 +43,8 @@ private module Input1 implements InputSig {
   predicate foo() { any() }
 }
 
-private module Input2 implements InputSig { // $ Alert
+private module Input2 implements InputSig {
+  // $ Alert
   predicate foo() { any() }
 }
 

ql/ql/test/queries/style/Misspelling/Test.qll

@@ -1,7 +1,8 @@
 /**
  * A string that's deliberately mispelled (and so is that last word).
  */ // $ Alert
-class PublicallyAccessible extends string { // $ Alert
+class PublicallyAccessible extends string {
+  // $ Alert
   int numOccurences; // $ Alert // should be 'occurrences'
 
   PublicallyAccessible() { this = "publically" and numOccurences = 123 }
@@ -16,7 +17,8 @@ class PublicallyAccessible extends string { // $ Alert
  * A class whose name contains a British-English spelling.
  * And here's the word 'colour'.
  */ // $ Alert
-class AnalysedInt extends int { // $ Alert
+class AnalysedInt extends int {
+  // $ Alert
   AnalysedInt() { this = 7 }
 
   // 'analyses' should not be flagged

ql/ql/test/queries/style/NonDocBlock/Foo.qll

@@ -1,13 +1,15 @@
 /*
  * This should be QLDoc.
- */ // $ Alert
+ */
 
+// $ Alert
 /**
  * this is fine
  */
 predicate foo() { any() }
 
-/* Note: this is bad. */ // $ Alert
+/* Note: this is bad. */
+// $ Alert
 class Foo extends string {
   Foo() { this = "FOo" }
 }

ql/ql/test/queries/style/UseInstanceofExtension/Foo.qll

@@ -4,28 +4,32 @@ class Range extends string {
   string getAChild() { result = "test" }
 }
 
-class Inst extends string { // $ Alert
+class Inst extends string {
+  // $ Alert
   Range range;
 
   Inst() { this = range }
 
   string getAChild() { result = range.getAChild() }
 }
 
-class Inst2 extends string { // $ Alert
+class Inst2 extends string {
+  // $ Alert
   Inst2() { this instanceof Range }
 
   string getAChild() { result = this.(Range).getAChild() }
 }
 
-class Inst3 extends string { // $ Alert
+class Inst3 extends string {
+  // $ Alert
   Range range;
 
   Inst3() { this = range }
 
   Range getRange() { result = range }
 }
 
-class Inst4 extends string { // $ Alert
+class Inst4 extends string {
+  // $ Alert
   Inst4() { this instanceof Range }
 }