Problem with loading Karma test runner

[bolshchikov]

The following code works fine in the browser with AMD but doesn't work in Karma test runner:

define(['pubsub'], function (pubsub) {
  pubsub.publish('test');
});

In Karma, pubsub is undefined, because it takes the route of common js thinking that module is defined

// CommonJS
if (typeof exports === 'object' && typeof module !== 'undefined'){
    module.exports = factory();

// AMD
} else if (typeof define === 'function' && define.amd){
    define(factory);
// Browser
} else {
    root.PubSub = factory();
}

[mroderick]

I guess it would be preferable to test for the AMD case first, as is done in https://github.com/umdjs/umd/blob/master/returnExports.js

[bolshchikov]

@mroderick, so I assume this solution will also resolve the issue #49

[mroderick]

How does the code not work in the Karma test runner?
Do they define both exports and module globals?

Did you try modifying PubSubJS to use the UMD pattern linked above?

[mroderick]

This has been fixed with #63, and is included in 1.5.0