JS: Fix copy-pasta bug

asger-semmle · asger-semmle · commit c115451b9d0c · 2019-02-15T16:48:42.000Z
diff --git a/javascript/ql/src/semmle/javascript/StringOps.qll b/javascript/ql/src/semmle/javascript/StringOps.qll
@@ -408,7 +408,7 @@ module StringOps {
     /**
      * A call of form `_.endsWith(A, B)` or `ramda.endsWith(A, B)`.
      */
-    private class EndsWith_Library extends StartsWith, DataFlow::CallNode {
+    private class EndsWith_Library extends Range, DataFlow::CallNode {
       EndsWith_Library() {
         getNumArgument() = 2 and
         exists(DataFlow::SourceNode callee | this = callee.getACall() |
diff --git a/javascript/ql/test/library-tests/StringOps/EndsWith/EndsWith.expected b/javascript/ql/test/library-tests/StringOps/EndsWith/EndsWith.expected
@@ -1 +1,3 @@
 | tst.js:5:7:5:19 | A.endsWith(B) | tst.js:5:7:5:7 | A | tst.js:5:18:5:18 | B | true |
+| tst.js:6:7:6:22 | _.endsWith(A, B) | tst.js:6:18:6:18 | A | tst.js:6:21:6:21 | B | true |
+| tst.js:7:7:7:22 | R.endsWith(A, B) | tst.js:7:18:7:18 | A | tst.js:7:21:7:21 | B | true |

Original file line number	Diff line number	Diff line change
`@@ -1 +1,3 @@`
`1`	`1`	`\| tst.js:5:7:5:19 \| A.endsWith(B) \| tst.js:5:7:5:7 \| A \| tst.js:5:18:5:18 \| B \| true \|`
	`2`	`+\| tst.js:6:7:6:22 \| _.endsWith(A, B) \| tst.js:6:18:6:18 \| A \| tst.js:6:21:6:21 \| B \| true \|`
	`3`	`+\| tst.js:7:7:7:22 \| R.endsWith(A, B) \| tst.js:7:18:7:18 \| A \| tst.js:7:21:7:21 \| B \| true \|`