Option for replacing modules with a stub in the bundle, "bundlerOptio…

…ns.ignore", #53, #57

README.md

@@ -120,6 +120,8 @@ Valid options are the same as for the `compilerOptions` section in `tsconfig.jso
 
 * `karmaTypescriptConfig.bundlerOptions.exclude` - An array of npm module names that will be excluded from the bundle.
 
+* `karmaTypescriptConfig.bundlerOptions.ignore` - An array of npm module names that will be bundled as stubs, `module.exports = {};`.
+
 * `karmaTypescriptConfig.bundlerOptions.nodeGlobals` - Boolean indicating whether the global variables `process` and `Buffer` should be included in the bundle. Defaults to `true`.
 
 * `karmaTypescriptConfig.bundlerOptions.noParse` - An array of module names that will be bunded as-is, without being parsed for dependencies.

integration-tests@latest/karma.conf.js

@@ -12,7 +12,6 @@ module.exports = function(config) {
             { pattern: "src/misc/no-module-tester.spec.ts" },
             { pattern: "src/misc/!(emptyfile)/**/*.ts" },
             { pattern: "src/node/**/*.ts" },
-            { pattern: "src/socket.io/*.ts" },
             { pattern: "src/typescript/**/*.ts" },
             { pattern: "src/x-performance/**/*.ts" }
         ],
@@ -36,9 +35,10 @@ module.exports = function(config) {
             include: ["**/*.ts", "**/*.tsx"],
             exclude: ["broken"],
             bundlerOptions: {
+                exclude: ["react/addons"],
+                ignore: ["ws"],
                 nodeGlobals: true,
-                noParse: ["jquery"],
-                exclude: ["ws"]
+                noParse: ["jquery"]
             },
             disableCodeCoverageInstrumentation: false,
             excludeFromCoverage: /\.(d|spec|test)\.ts/,

integration-tests@latest/src/bundling/ignore/ignore-tester.spec.ts

@@ -0,0 +1,11 @@
+import { IgnoreTester } from "./ignore-tester";
+
+describe("IgnoreTester", () => {
+
+    let tester = new IgnoreTester();
+
+    it("should be required without ignored dependencies", () => {
+
+        expect(tester.testRequire()).not.toBeUndefined();
+    });
+});

integration-tests@latest/src/bundling/ignore/ignore-tester.ts

@@ -0,0 +1,10 @@
+import * as io from "socket.io";
+
+export class IgnoreTester {
+
+    public testRequire(): boolean {
+
+        // socket.io requires ws which has dependencies that are incompatible
+        return io !== undefined;
+    }
+}

lib/bundlers/builtin/bundler.js

@@ -17,6 +17,7 @@ var acorn = require("acorn"),
     entrypointFilenames = [],
     exclude,
     filenameCache = [],
+    ignore,
     log,
     lookupNameCache = {},
     nodeGlobals,
@@ -37,6 +38,7 @@ function Bundler() {
 
         bundlerOptions = bundlerOptions || {};
         exclude = bundlerOptions.exclude || [];
+        ignore = bundlerOptions.ignore || [];
         nodeGlobals = bundlerOptions.nodeGlobals !== undefined ? bundlerOptions.nodeGlobals : true;
         noParse = bundlerOptions.noParse || [];
 
@@ -227,8 +229,7 @@ function Bundler() {
     function resolveModules(requiringModule, requiredModule) {
 
         if(exclude.indexOf(requiredModule.moduleName) !== -1) {
-
-            log.error("Excluding module %s", requiredModule.moduleName);
+            log.debug("Excluding module %s", requiredModule.moduleName);
             return requiredModule.moduleName;
         }
 
@@ -271,8 +272,13 @@ function Bundler() {
 
     function readSource(requiredModule) {
 
-        requiredModule.source = fs.readFileSync(requiredModule.filename).toString();
-        requiredModule.source = removeSourceMapUrl(requiredModule.source);
+        if(ignore.indexOf(requiredModule.moduleName) !== -1) {
+            requiredModule.source = "module.exports={};";
+        }
+        else {
+            requiredModule.source = fs.readFileSync(requiredModule.filename).toString();
+            requiredModule.source = removeSourceMapUrl(requiredModule.source);
+        }
     }
 
     function addDependencies(requiredModule) {