JS: Bugfix in tsconfig file inclusion handling

asgerf · asgerf · commit acc28df78519 · 2021-04-01T16:33:05.000+01:00
diff --git a/javascript/extractor/src/com/semmle/js/parser/ParsedProject.java b/javascript/extractor/src/com/semmle/js/parser/ParsedProject.java
@@ -21,7 +21,7 @@ public File getTsConfigFile() {
 
   /** Absolute paths to the files included in this project. */
   public Set<File> getOwnFiles() {
-    return allFiles;
+    return ownFiles;
   }
 
   /** Absolute paths to the files included in or referenced by this project. */
diff --git a/javascript/ql/test/library-tests/TypeScript/BaseUrl/test.expected b/javascript/ql/test/library-tests/TypeScript/BaseUrl/test.expected
@@ -1,2 +1,3 @@
 | A/index.ts:1:1:1:35 | import  ... c/foo"; | B/src/foo.ts:1:1:4:0 | <toplevel> |
+| B/src/foo.ts:1:1:1:30 | import  ... b/bar"; | B/src/lib/bar.ts:1:1:4:0 | <toplevel> |
 | B/src/lib/bar.ts:1:1:1:26 | import  ...  "foo"; | B/src/foo.ts:1:1:4:0 | <toplevel> |

Original file line number	Diff line number	Diff line change
`@@ -21,7 +21,7 @@ public File getTsConfigFile() {`
`21`	`21`
`22`	`22`	`/** Absolute paths to the files included in this project. */`
`23`	`23`	`public Set<File> getOwnFiles() {`
`24`		`- return allFiles;`
	`24`	`+ return ownFiles;`
`25`	`25`	`}`
`26`	`26`
`27`	`27`	`/** Absolute paths to the files included in or referenced by this project. */`
Original file line number	Diff line number	Diff line change
`@@ -1,2 +1,3 @@`
`1`	`1`	`\| A/index.ts:1:1:1:35 \| import ... c/foo"; \| B/src/foo.ts:1:1:4:0 \| <toplevel> \|`
	`2`	`+\| B/src/foo.ts:1:1:1:30 \| import ... b/bar"; \| B/src/lib/bar.ts:1:1:4:0 \| <toplevel> \|`
`2`	`3`	`\| B/src/lib/bar.ts:1:1:1:26 \| import ... "foo"; \| B/src/foo.ts:1:1:4:0 \| <toplevel> \|`