JS: Update test output and add related TODO in model of 'async'

Author: asgerf

Diff for: javascript/ql/lib/semmle/javascript/frameworks/AsyncPackage.qll

@@ -142,7 +142,7 @@ module AsyncPackage {
     override predicate step(DataFlow::Node pred, DataFlow::Node succ) {
       exists(DataFlow::FunctionNode iteratee, IterationCall call |
         iteratee = call.getIteratorCallback() and // Require a closure to avoid spurious call/return mismatch.
-        pred = call.getCollection() and
+        pred = call.getCollection() and // TODO: needs a flow summary to ensure ArrayElement content is unfolded
         succ = iteratee.getParameter(0)
       )
     }

Diff for: javascript/ql/test/library-tests/frameworks/AsyncPackage/AsyncTaintTracking.expected

@@ -1,10 +1,10 @@
 legacyDataFlowDifference
+| each.js:11:9:11:16 | source() | each.js:13:12:13:15 | item | only flow with OLD data flow library |
+| map.js:10:13:10:20 | source() | map.js:12:14:12:17 | item | only flow with OLD data flow library |
+| map.js:26:13:26:20 | source() | map.js:28:27:28:32 | result | only flow with OLD data flow library |
+| sortBy.js:10:22:10:29 | source() | sortBy.js:12:27:12:32 | result | only flow with OLD data flow library |
 #select
-| each.js:11:9:11:16 | source() | each.js:13:12:13:15 | item |
-| map.js:10:13:10:20 | source() | map.js:12:14:12:17 | item |
 | map.js:20:19:20:26 | source() | map.js:23:27:23:32 | result |
-| map.js:26:13:26:20 | source() | map.js:28:27:28:32 | result |
-| sortBy.js:10:22:10:29 | source() | sortBy.js:12:27:12:32 | result |
 | waterfall.js:8:30:8:37 | source() | waterfall.js:11:12:11:16 | taint |
 | waterfall.js:8:30:8:37 | source() | waterfall.js:20:10:20:14 | taint |
 | waterfall.js:28:18:28:25 | source() | waterfall.js:39:10:39:12 | err |