Fix issue with getScriptFileNames (#71)

Author: jbrantly

index.ts

@@ -271,7 +271,7 @@ function ensureTypeScriptInstance(options: Options, loader: any): { instance?: T
 
     // Create the TypeScript language service
     var servicesHost = {
-        getScriptFileNames: () => Object.keys(files),
+        getScriptFileNames: () => Object.keys(files).filter(filePath => !!filePath.match(/(\.d)?\.tsx?$/)),
         getScriptVersion: fileName => {
             fileName = path.normalize(fileName);
             return files[fileName] && files[fileName].version.toString();

test/issue71/app.ts

@@ -0,0 +1,5 @@
+import a = require('a');
+import b = require('./b');
+
+console.log(a);
+console.log(b);

test/issue71/b.ts

@@ -0,0 +1 @@
+export = 'b';

test/issue71/expectedOutput/bundle.js

@@ -0,0 +1,67 @@
+/******/ (function(modules) { // webpackBootstrap
+/******/ 	// The module cache
+/******/ 	var installedModules = {};
+
+/******/ 	// The require function
+/******/ 	function __webpack_require__(moduleId) {
+
+/******/ 		// Check if module is in cache
+/******/ 		if(installedModules[moduleId])
+/******/ 			return installedModules[moduleId].exports;
+
+/******/ 		// Create a new module (and put it into the cache)
+/******/ 		var module = installedModules[moduleId] = {
+/******/ 			exports: {},
+/******/ 			id: moduleId,
+/******/ 			loaded: false
+/******/ 		};
+
+/******/ 		// Execute the module function
+/******/ 		modules[moduleId].call(module.exports, module, module.exports, __webpack_require__);
+
+/******/ 		// Flag the module as loaded
+/******/ 		module.loaded = true;
+
+/******/ 		// Return the exports of the module
+/******/ 		return module.exports;
+/******/ 	}
+
+
+/******/ 	// expose the modules object (__webpack_modules__)
+/******/ 	__webpack_require__.m = modules;
+
+/******/ 	// expose the module cache
+/******/ 	__webpack_require__.c = installedModules;
+
+/******/ 	// __webpack_public_path__
+/******/ 	__webpack_require__.p = "";
+
+/******/ 	// Load entry module and return exports
+/******/ 	return __webpack_require__(0);
+/******/ })
+/************************************************************************/
+/******/ ([
+/* 0 */
+/***/ function(module, exports, __webpack_require__) {
+
+	var a = __webpack_require__(1);
+	var b = __webpack_require__(2);
+	console.log(a);
+	console.log(b);
+
+
+/***/ },
+/* 1 */
+/***/ function(module, exports) {
+
+	module.exports = 'a';
+
+/***/ },
+/* 2 */
+/***/ function(module, exports) {
+
+	module.exports = 'b';
+
+
+/***/ }
+/******/ ]);

test/issue71/expectedOutput/output.txt

@@ -0,0 +1,6 @@
+    Asset     Size  Chunks             Chunk Names
+bundle.js  1.66 kB       0  [emitted]  main
+chunk    {0} bundle.js (main) 121 bytes [rendered]
+    [0] ./.test/issue71/app.ts 78 bytes {0} [built]
+    [1] ./.test/issue71/~/a/index.js 21 bytes {0} [built]
+    [2] ./.test/issue71/b.ts 22 bytes {0} [built]

test/issue71/node_modules/a/a.d.ts

@@ -0,0 +1,7 @@
+{
+	"compilerOptions": {
+		"noEmitOnError": true
+	},
+	"files": [
+	]
+}

test/issue71/node_modules/a/index.js

@@ -0,0 +1,19 @@
+var path = require('path')
+
+module.exports = {
+    entry: './app.ts',
+    output: {
+        filename: 'bundle.js'
+    },
+    resolve: {
+        extensions: ['', '.ts', '.js']
+    },
+    module: {
+        loaders: [
+            { test: /\.ts$/, loader: 'ts-loader' }
+        ]
+    }
+}
+
+// for test harness purposes only, you would not need this in a normal project
+module.exports.resolveLoader = { alias: { 'ts-loader': require('path').join(__dirname, "../../index.js") } }

test/issue71/node_modules/a/package.json

@@ -31,13 +31,15 @@ fs.readdirSync(__dirname).forEach(function(test) {
         if (test == 'jsxPreserve' && semver.lt(typescript.version, '1.6.0-0')) return;
         if (test == 'es6codeSplitting' && semver.lt(typescript.version, '1.6.0-0')) return;
         if (test == 'nodeResolution' && semver.lt(typescript.version, '1.6.0-0')) return;
+        if (test == 'issue71' && semver.lt(typescript.version, '1.6.0-0')) return;
 
         describe(test, function() {
             it('should have the correct output', createTest(test, testPath, {}));
 
             if (test == 'declarationOutput') { return; }
             if (test == 'declarationWatch') { return; }
             if (test == 'sourceMaps' && semver.lt(typescript.version, '1.6.0-0')) return;
+            if (test == 'issue71') { return; }
             it('should work with transpile', createTest(test, testPath, {transpile: true}));
         });
     }