JS: Fix a bug in a unit test

The 'extractTypeScriptFiles' override did not incorporate the file type and one of our unit tests was expecting this. The test was previously passing for the wrong reasons.

Author: asgerf

javascript/extractor/test/com/semmle/js/extractor/test/AutoBuildTests.java

@@ -111,12 +111,17 @@ private void runTest() throws IOException {
     try {
       Set<String> actual = new LinkedHashSet<>();
       new AutoBuild() {
-        @Override
-        protected CompletableFuture<?> extract(FileExtractor extractor, Path file, boolean concurrent) {
+        private void markExtracted(Path file, FileExtractor extractor) {
           String extracted = file.toString();
-          if (extractor.getConfig().hasFileType())
+          if (extractor.getConfig().hasFileType()) {
             extracted += ":" + extractor.getFileType(file.toFile());
+          }
           actual.add(extracted);
+        }
+
+        @Override
+        protected CompletableFuture<?> extract(FileExtractor extractor, Path file, boolean concurrent) {
+          markExtracted(file, extractor);
           return CompletableFuture.completedFuture(null);
         }
 
@@ -134,7 +139,7 @@ public void extractTypeScriptFiles(
             java.util.Set<Path> extractedFiles,
             FileExtractors extractors) {
           for (Path f : files) {
-            actual.add(f.toString());
+            markExtracted(f, extractors.forFile(f));
             extractedFiles.add(f);
           }
         }