Raw result update (removes test failures)

Author: asgerf

javascript/ql/test/query-tests/JSDoc/BadParamTag/BadParamTag.expected

@@ -1,10 +1,4 @@
-#select
 | tst.js:3:4:3:9 | @param | @param tag is missing description. |
 | tst.js:3:4:3:9 | @param | @param tag is missing name. |
 | tst.js:7:4:7:9 | @param | @param tag is missing description. |
 | tst.js:11:4:11:9 | @param | @param tag is missing description. |
-testFailures
-| tst.js:3:4:3:9 | @param tag is missing description. | Unexpected result: Alert |
-| tst.js:3:4:3:9 | @param tag is missing name. | Unexpected result: Alert |
-| tst.js:7:4:7:9 | @param tag is missing description. | Unexpected result: Alert |
-| tst.js:11:4:11:9 | @param tag is missing description. | Unexpected result: Alert |

javascript/ql/test/query-tests/LanguageFeatures/SyntaxError/SyntaxError.expected

@@ -1,7 +1,3 @@
-#select
 | arrows.js:1:5:1:5 | Error: Argument name clash | Error: Argument name clash |
 | destructingPrivate.js:4:6:4:6 | Error: Unexpected token | Error: Unexpected token |
 | tst.js:2:12:2:12 | Error: Unterminated string constant | Error: Unterminated string constant |
-testFailures
-| destructingPrivate.js:4:6:4:6 | Error: Unexpected token | Unexpected result: Alert |
-| tst.js:2:12:2:12 | Error: Unterminated string constant | Unexpected result: Alert |

javascript/ql/test/query-tests/React/UnusedOrUndefinedStateProperty/UnusedOrUndefinedStateProperty.expected

@@ -1,15 +1,7 @@
-#select
-| issue7506.js:12:1:12:34 | class C ... e\\n  }\\n} | Component state property 'p1' is $@, but it is never read. | issue7506.js:15:5:15:10 | p1: '' | written |
-| undefined.js:1:1:1:34 | class C ...     }\\n} | Component state property 'notWritten' is $@, but it is never written. | undefined.js:8:9:8:29 | this.st ... Written | read |
-| undefined.js:1:1:1:34 | class C ...     }\\n} | Component state property 'notWrittenButReadInChain' is $@, but it is never written. | undefined.js:9:9:9:43 | this.st ... InChain | read |
-| undefined.js:32:1:32:34 | class C ...     }\\n} | Component state property 'notWrittenThrougExternalPropertyAccess' is $@, but it is never written. | undefined.js:35:9:35:57 | this.st ... yAccess | read |
+| issue7506.js:12:1:12:45 | class C ... e\\n  }\\n} | Component state property 'p1' is $@, but it is never read. | issue7506.js:15:5:15:10 | p1: '' | written |
+| undefined.js:1:1:1:45 | class C ...     }\\n} | Component state property 'notWritten' is $@, but it is never written. | undefined.js:8:9:8:29 | this.st ... Written | read |
+| undefined.js:1:1:1:45 | class C ...     }\\n} | Component state property 'notWrittenButReadInChain' is $@, but it is never written. | undefined.js:9:9:9:43 | this.st ... InChain | read |
+| undefined.js:32:1:32:45 | class C ...     }\\n} | Component state property 'notWrittenThrougExternalPropertyAccess' is $@, but it is never written. | undefined.js:35:9:35:57 | this.st ... yAccess | read |
 | undefined.js:61:19:61:30 | { // $  ... ;\\n  }\\n} | Component state property 'notWrittenInKnownInitializerObject' is $@, but it is never written. | undefined.js:64:9:64:53 | this.st ... rObject | read |
-| unused.js:1:1:1:34 | class C ...     }\\n} | Component state property 'notRead' is $@, but it is never read. | unused.js:6:9:6:26 | this.state.notRead | written |
-| unused.js:27:1:27:34 | class C ...     }\\n} | Component state property 'notReadThrougExternaPropertyAccess' is $@, but it is never read. | unused.js:30:9:30:53 | this.st ... yAccess | written |
-testFailures
-| issue7506.js:12:1:12:34 | Component state property 'p1' is $@, but it is never read. | Unexpected result: Alert |
-| undefined.js:1:1:1:34 | Component state property 'notWritten' is $@, but it is never written. | Unexpected result: Alert |
-| undefined.js:1:1:1:34 | Component state property 'notWrittenButReadInChain' is $@, but it is never written. | Unexpected result: Alert |
-| undefined.js:32:1:32:34 | Component state property 'notWrittenThrougExternalPropertyAccess' is $@, but it is never written. | Unexpected result: Alert |
-| unused.js:1:1:1:34 | Component state property 'notRead' is $@, but it is never read. | Unexpected result: Alert |
-| unused.js:27:1:27:34 | Component state property 'notReadThrougExternaPropertyAccess' is $@, but it is never read. | Unexpected result: Alert |
+| unused.js:1:1:1:45 | class C ...     }\\n} | Component state property 'notRead' is $@, but it is never read. | unused.js:6:9:6:26 | this.state.notRead | written |
+| unused.js:27:1:27:45 | class C ...     }\\n} | Component state property 'notReadThrougExternaPropertyAccess' is $@, but it is never read. | unused.js:30:9:30:53 | this.st ... yAccess | written |

javascript/ql/test/query-tests/Security/CWE-020/UselessCharacterEscape/UselessRegExpCharacterEscape.expected

@@ -1193,12 +1193,6 @@ testFailures
 | tainted-access-paths.js:39:24:39:30 | req.url | Unexpected result: Source |
 | tainted-access-paths.js:48:24:48:30 | req.url | Unexpected result: Source |
 | tainted-promise-steps.js:6:24:6:30 | req.url | Unexpected result: Source |
-| tainted-sendFile.js:27:16:27:33 | req.param("gimme") | Unexpected result: Alert |
-| tainted-sendFile.js:30:16:30:48 | homeDir ... arams.x | Unexpected result: Alert |
-| tainted-sendFile.js:30:37:30:48 | req.params.x | Unexpected result: Alert |
-| tainted-sendFile.js:32:16:32:46 | path.jo ... rams.x) | Unexpected result: Alert |
-| tainted-sendFile.js:32:34:32:45 | req.params.x | Unexpected result: Alert |
-| tainted-sendFile.js:35:43:35:58 | req.param("dir") | Unexpected result: Alert |
 | tainted-string-steps.js:6:24:6:30 | req.url | Unexpected result: Source |
 | torrents.js:5:13:5:38 | parseTo ... t).name | Unexpected result: Source |
 | typescript.ts:9:24:9:30 | req.url | Unexpected result: Source |

javascript/ql/test/query-tests/Security/CWE-022/TaintedPath/TaintedPath.expected

@@ -119,7 +119,6 @@ subpaths
 testFailures
 | FileAccessToHttp.js:4:15:4:47 | fs.read ... "utf8") | Unexpected result: Source |
 | bufferRead.js:12:22:12:43 | new Buf ... s.size) | Unexpected result: Source |
-| googlecompiler.js:14:32:14:91 | // $ Al ... on body | Missing result: Alert |
 | googlecompiler.js:43:54:43:57 | data | Unexpected result: Source |
 | readFileSync.js:5:12:5:39 | fs.read ... t.txt") | Unexpected result: Source |
 | readStreamRead.js:13:21:13:35 | readable.read() | Unexpected result: Source |

javascript/ql/test/query-tests/Security/CWE-200/FileAccessToHttp.expected

@@ -1,4 +1,3 @@
-#select
 | lib/tst.js:7:1:7:45 | app.use ... rname)) | Serves the folder lib, which can contain private information. |
 | lib/tst.js:9:1:9:43 | app.use ... otDir)) | Serves the folder lib, which can contain private information. |
 | lib/tst.js:11:1:11:52 | app.use ... + '/')) | Serves the folder lib, which can contain private information. |
@@ -22,6 +21,3 @@
 | private-file-exposure.js:51:5:51:88 | app.use ... les'))) | Serves the folder "../node_modules", which can contain private information. |
 | private-file-exposure.js:70:5:70:71 | serveHa ... ular"}) | Serves the folder "./node_modules/angular", which can contain private information. |
 | subfolder/private-file-exposure-2.js:6:1:6:34 | app.use ... rname)) | Serves the folder subfolder, which can contain private information. |
-testFailures
-| googlecompiler.js:14:32:14:91 | // $ Al ... on body | Missing result: Alert |
-| googlecompiler.js:37:30:37:81 | // $ Al ... st body | Missing result: Alert |

javascript/ql/test/query-tests/Security/CWE-200/PrivateFileExposure.expected

@@ -34,5 +34,4 @@ nodes
 subpaths
 testFailures
 | CleartextStorage.js:5:12:5:40 | req.par ... sword") | Unexpected result: Source |
-| CleartextStorage.js:6:26:6:27 | pw | Fixed missing result: Alert |
 | CleartextStorage2.js:5:12:5:58 | url.par ... assword | Unexpected result: Source |

javascript/ql/test/query-tests/Security/CWE-312/CleartextStorage.expected

@@ -1,4 +1,3 @@
-#select
 | highlight.js:2:26:2:979 | ((traffic-flow\|traffic-generator\|firewall\|scheduler\|aaa\|accounting\|address-list\|address\|align\|area\|bandwidth-server\|bfd\|bgp\|bridge\|client\|clock\|community\|config\|connection\|console\|customer\|default\|dhcp-client\|dhcp-server\|discovery\|dns\|e-mail\|ethernet\|filter\|firewall\|firmware\|gps\|graphing\|group\|hardware\|health\|hotspot\|identity\|igmp-proxy\|incoming\|instance\|interface\|ip\|ipsec\|ipv6\|irq\|l2tp-server\|lcd\|ldp\|logging\|mac-server\|mac-winbox\|mangle\|manual\|mirror\|mme\|mpls\|nat\|nd\|neighbor\|network\|note\|ntp\|ospf\|ospf-v3\|ovpn-server\|page\|peer\|pim\|ping\|policy\|pool\|port\|ppp\|pppoe-client\|pptp-server\|prefix\|profile\|proposal\|proxy\|queue\|radius\|resource\|rip\|ripng\|route\|routing\|screen\|script\|security-profiles\|server\|service\|service-port\|settings\|shares\|smb\|sms\|sniffer\|snmp\|snooper\|socks\|sstp-server\|system\|tool\|tracking\|type\|upgrade\|upnp\|user-manager\|users\|user\|vlan\|secret\|vrrp\|watchdog\|web-access\|wireless\|pptp\|pppoe\|lan\|wan\|layer7-protocol\|lease\|simple\|raw);?\\s)+ | This part of the regular expression may cause exponential backtracking on strings starting with '/' and containing many repetitions of 'firewall\\t'. |
 | highlight.js:6:12:6:695 | (Add\|Clear\|Close\|Copy\|Enter\|Exit\|Find\|Format\|Get\|Hide\|Join\|Lock\|Move\|New\|Open\|Optimize\|Pop\|Push\|Redo\|Remove\|Rename\|Reset\|Resize\|Search\|Select\|Set\|Show\|Skip\|Split\|Step\|Switch\|Undo\|Unlock\|Watch\|Backup\|Checkpoint\|Compare\|Compress\|Convert\|ConvertFrom\|ConvertTo\|Dismount\|Edit\|Expand\|Export\|Group\|Import\|Initialize\|Limit\|Merge\|New\|Out\|Publish\|Restore\|Save\|Sync\|Unpublish\|Update\|Approve\|Assert\|Complete\|Confirm\|Deny\|Disable\|Enable\|Install\|Invoke\|Register\|Request\|Restart\|Resume\|Start\|Stop\|Submit\|Suspend\|Uninstall\|Unregister\|Wait\|Debug\|Measure\|Ping\|Repair\|Resolve\|Test\|Trace\|Connect\|Disconnect\|Read\|Receive\|Send\|Write\|Block\|Grant\|Protect\|Revoke\|Unblock\|Unprotect\|Use\|ForEach\|Sort\|Tee\|Where)+ | This part of the regular expression may cause exponential backtracking on strings containing many repetitions of 'New'. |
 | highlight.js:10:22:10:32 | (\\\\.\|[^/])* | This part of the regular expression may cause exponential backtracking on strings starting with 's/' and containing many repetitions of '\\\\.'. |
@@ -203,5 +202,3 @@
 | tst.js:341:25:341:35 | (\\u0000\|.)+ | This part of the regular expression may cause exponential backtracking on strings starting with '\\n\\u0000' and containing many repetitions of '\\u0000'. |
 | tst.js:343:44:343:57 | (\ud83d\ude80\|.)+ | This part of the regular expression may cause exponential backtracking on strings starting with '\\n\\u{1f680}' and containing many repetitions of '\\u{1f680}'. |
 | tst.js:345:22:345:23 | a? | This part of the regular expression may cause exponential backtracking on strings starting with 'a' and containing many repetitions of 'aa'. |
-testFailures
-| tst.js:345:22:345:23 | This part of the regular expression may cause exponential backtracking on strings starting with 'a' and containing many repetitions of 'aa'. | Unexpected result: Alert |

javascript/ql/test/query-tests/Security/CWE-400/ReDoS/ReDoS.expected

@@ -437,12 +437,6 @@ testFailures
 | tst15.js:2:15:2:31 | document.location | Unexpected result: Source |
 | tst15.js:7:16:7:32 | document.location | Unexpected result: Source |
 | tst15.js:12:18:12:41 | documen ... .search | Unexpected result: Source |
-| tst16.js:5:21:5:59 | querySt ... h).data | Unexpected result: Alert |
-| tst16.js:5:39:5:53 | location.search | Unexpected result: Alert |
-| tst16.js:6:21:6:56 | querySt ... search) | Unexpected result: Alert |
-| tst16.js:6:41:6:55 | location.search | Unexpected result: Alert |
-| tst16.js:7:21:7:62 | queryst ... h).data | Unexpected result: Alert |
-| tst16.js:7:42:7:56 | location.search | Unexpected result: Alert |
 | tst2.js:2:14:2:33 | window.location.href | Unexpected result: Source |
 | tst6.js:2:18:2:45 | $locati ... irect') | Unexpected result: Source |
 | typed.ts:4:22:4:36 | location.search | Unexpected result: Source |