bundle.require doesn't appear to work in prebundle hook about karma-browserify HOT 11 CLOSED

Did a little investigation and this would have worked with karma-bro < 0.7

from karma-browserify.

So the reason this is happening is that the bundle gets reset on rebuilds:

https://github.com/Nikku/karma-browserify/blob/master/lib/bro.js#L208

Immediate fix is to do this in prebundle:

bundle.require('./foo', {expose: 'foo'});
bundle.on('reset', function () {
    bundle.require('./foo', {expose: 'foo'});
});

external among other bundle configuration calls is not reset on reset. Technically running prebundle on every run wouldn't break anything but it would balloon arrays like b._external with long running watchify instances. I think the answer here is to provide an extra hook that runs on every bundle call but I'll wait on @nikku's thoughts before we ship anything.

from karma-browserify.

Thanks for digging @bendrucker.

To pre-empt questions about why on earth I'd want or need to do this: I have a project, "foo", which contains some plugins. These plugins depend on "foo-plugintool", an external dependency which in turn depends on "foo". When testing, I'd like that require('foo') to resolve back to the working copy of the project rather than the released version. Hope that makes sense.

from karma-browserify.

Haha, I already had an update for that behavior implemented but reverted it again because no reported a bug on it yet.

The intention during early karma-bro times was to actually be able to configure the whole bundle thing once. Too bad browserify nowadays resets file configurations and so forth.

My suggestion is: We fire a custom event users can listen to that resembles the configuration before actual bundling for the users:

bundle.on('prebundle', function () {
  bundle.require('./foo', {expose: 'foo'});
});

We could then rename the actual method to something like configure or the like to indicate manual configuration of browserify. Prebundle is definitely the wrong name for what it offers today.

from karma-browserify.

Looks like this is absolutely equivalent:

bundle.on('reset', function () {
  bundle.require('./foo', {expose: 'foo'});
});

The initial configuration is not needed as we always reset the bundle prior to any bundling operations.

from karma-browserify.

You're right. Honestly I'm inclined to just document this. The gain from having a clearer event name isn't worth creating code that isn't portable (i.e. wouldn't work with regular watchify). configure makes sense but would also mean a major version change.

from karma-browserify.

This sounds fine as far as it goes, but it seems that it introduces another related bug. If you do

bundle.on('reset', function () {
  bundle.require('./foo', {expose: 'foo'});
});

then certainly require('foo') works, but oddly require('./foo') no longer does. This is at odds with the usual behaviour of browserify AFAICT. If I have foo.js:

module.exports = 4;

and relative.js:

var foo = require('./foo');
console.log("foo =", foo);

then I can still browserify this using

browserify -r ./foo.js:foo relative.js | node

and the relative require continues to work.

from karma-browserify.

This is related to #52 and a browserify/watchify bug. It looks as if watchify cannot handle files that are required using their relative path names and a global alias at the same time.

As much as I can tell there are two options to fix this, none of them is without pain.

from karma-browserify.

We are going to fix this in karma-browserify by exposing all files via their absolute path.

This is something watchify requires. In order for the above code to work, you should use

bundle.require('abs-path-to-foo', {expose: 'foo'});

from karma-browserify.

Forget what I posted before 😀.

The next version of karma-browserify will remove the prebundle hook in favor of a configure hook.
Users may hook into prebundle actions by listening to the prebundle event.

Example configuration:

{

  files: [ ],
  preprocessors: [ ],

  browserify: {
    configure: function(b) {
      b.on('prebundle', function() {
        b.external('lib/external.js');
      });
    }
  }
}

from karma-browserify.

Thank you! 🍰

from karma-browserify.