Skip to content

Commit 1fc207c

Browse files
committed
feat: support option skipModuleLoader
1 parent 767c3fb commit 1fc207c

2 files changed

Lines changed: 44 additions & 4 deletions

File tree

spec/bundler.spec.js

Lines changed: 36 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -117,6 +117,42 @@ test('Bundler traces files', t => {
117117
.then(t.end);
118118
});
119119

120+
test('Bundler can optionally skip dumber-module-loader', t => {
121+
const fakeFs = {
122+
'local/setup.js': 'setup',
123+
'local/after.js': 'after',
124+
125+
};
126+
const bundler = createBundler(fakeFs, {
127+
skipModuleLoader: true,
128+
prepends: ['dev-dumber-module-loader']
129+
});
130+
131+
Promise.resolve()
132+
.then(() => bundler.capture({path: 'src/app.js', contents: '', moduleId: 'app'}))
133+
.then(() => bundler.resolve())
134+
.then(() => bundler.bundle())
135+
.then(
136+
bundleMap => {
137+
t.deepEqual(bundleMap, {
138+
'entry-bundle': {
139+
files: [
140+
{contents: 'dev-dumber-module-loader;'},
141+
{contents: 'define.switchToUserSpace();'},
142+
{path: 'src/app.js', contents: "define('app',[],1);", sourceMap: undefined}
143+
],
144+
config: {
145+
baseUrl: 'dist',
146+
bundles: {}
147+
}
148+
}
149+
})
150+
},
151+
err => t.fail(err.stack)
152+
)
153+
.then(t.end);
154+
});
155+
120156
test('Bundler traces files, split bundles', t => {
121157
const fakeFs = {
122158
'local/setup.js': 'setup',

src/index.js

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -51,10 +51,14 @@ export default class Bundler {
5151
this._onRequire = opts.onRequire || opts.onrequire || opts.onRequiringModule;
5252

5353
this._prepends = (opts.prepends || opts.prepend || []).filter(t => t);
54-
this._prepends.push(
55-
// load dumber-module-loader after prepends
56-
path.join(_resolve('dumber-module-loader'), 'dist/index.js')
57-
);
54+
55+
if (!opts.skipModuleLoader) {
56+
this._prepends.push(
57+
// load dumber-module-loader after prepends
58+
path.join(_resolve('dumber-module-loader'), 'dist/index.js')
59+
);
60+
}
61+
5862
this._appends = (opts.appends || opts.append || []).filter(t => t);
5963

6064
this._dependencies = (opts.dependencies || opts.deps || []).filter(t => t).map(d => new Package(d));

0 commit comments

Comments
 (0)