Skip to content

Commit

Permalink
Implemented require of browserify in runtime
Browse files Browse the repository at this point in the history
  • Loading branch information
andrewblond committed Aug 31, 2015
1 parent cde8622 commit b6ee487
Show file tree
Hide file tree
Showing 8 changed files with 34 additions and 6 deletions.
4 changes: 2 additions & 2 deletions lib/bundle.js
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
var vowCode = require('./vow-code'),
vow = require('vow'),
promisify = require('vow-node').promisify,
browserify = require('browserify'),
EOL = require('os').EOL;

/**
Expand Down Expand Up @@ -131,7 +130,8 @@ function compileYModule(name, requires, libs) {
* @returns {String}
*/
function compileCommonJSRequire(requires, dirname) {
var browserifyOptions = {
var browserify = require('browserify'),
browserifyOptions = {
basedir: dirname
},
renderer = browserify(browserifyOptions),
Expand Down
4 changes: 2 additions & 2 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -43,10 +43,10 @@
"jscs": "1.13.1",
"jshint": "2.8.0",
"mocha": "2.2.5",
"mock-enb": "0.1.0",
"mocha-phantomjs": "3.5.3",
"mock-enb": "0.1.0",
"mock-fs": "3.0.0",
"mock-require": "1.1.0",
"mock-require": "^1.1.0",
"must": "0.12.0",
"phantomjs": "1.9.18",
"serve-static": "1.9.3",
Expand Down
5 changes: 5 additions & 0 deletions test/lib/mocks.js
Original file line number Diff line number Diff line change
@@ -1 +1,6 @@
var mock = require('mock-require'),
browserifyPath = require.resolve('browserify');

mock(require.resolve(browserifyPath), require(browserifyPath));

require('./job-queue-stub');
3 changes: 3 additions & 0 deletions test/techs/bemhtml/bemhtml--browser.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -101,6 +101,9 @@ function runTest(testContent, options, template, lib) {
bundle: {},
// jscs:disable
node_modules: {
browserify: {
'index.js': ''
},
fake: {
'index.js': 'module.exports = { getText: function () { return "Hello world!"; } };'
},
Expand Down
3 changes: 3 additions & 0 deletions test/techs/bemhtml/bemhtml--node.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -133,6 +133,9 @@ function build(templates, options, lib) {
bundle: {},
// jscs:disable
node_modules: {
browserify: {
'index.js': ''
},
fake: {
'index.js': 'module.exports = { getText: function () { return "^_^"; } };'
},
Expand Down
3 changes: 3 additions & 0 deletions test/techs/bemhtml/bemhtml--ym.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -141,6 +141,9 @@ function runTest(testContent, options, template, lib) {
bundle: {},
// jscs:disable
node_modules: {
browserify: {
'index.js': ''
},
fake: {
'index.js': 'module.exports = { getText: function () { return "^_^"; } };'
},
Expand Down
9 changes: 8 additions & 1 deletion test/techs/bemhtml/bemhtml.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -271,7 +271,14 @@ describe('bemhtml', function () {
function build(templates, options) {
var scheme = {
blocks: {},
bundle: {}
bundle: {},
// jscs:disable
node_modules: {
browserify: {
'index.js': ''
}
}
// jscs:enable
},
bundle, fileList;

Expand Down
9 changes: 8 additions & 1 deletion test/techs/bemtree.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -168,7 +168,14 @@ function build(templates, options) {

var scheme = {
blocks: {},
bundle: {}
bundle: {},
// jscs:disable
node_modules: {
browserify: {
'index.js': ''
}
}
// jscs:enable
},
bundle, fileList;

Expand Down

0 comments on commit b6ee487

Please sign in to comment.