Failed to instantiate module function fakeRouteProvider about bardjs

[wardbell]

Put a breakpoint at the top and check to see if sinon exists. If it doesn't, you're doing something wrong.

Expecting karma framework config to be something like this:

frameworks: ['mocha', 'chai', 'sinon', 'chai-sinon'],

Shouldn't be in karma config files section. Maybe that is the problem.

Look again at the bardjs readme. If it isn't clear enough, please make a suggested improvement in a PR.

[bbohling]

Yep, I have the frameworks setup identically to what is in the readme. Though I had to add karma-sinon (and karma-chai-sinon) via npm...otherwise it would give me an error.

Sinon is undefined though so definitely not being injected somehow. Yours and John's Play by Play was great, but I'm ready for the entire Testing (deep dive) training series John keeps threatening. ;-)

[bbohling]

Actually I was wrong, sinon is an object when I set a breakpoint at the top of the spec.

[bbohling]

Thanks to this article I found the issue. I needed to change my sinon bower dependency to point to the release file. Seems a bit odd that one would need to do that, but it seems to have fixed the issue. Now onto the next issue. :)

[wardbell]

How can we give the appropriate guidance in the bardjs readme? Propose something. I take PRs :-)

[bbohling]

Since I'm a newbie in this space I would first like to verify that my implementation is required, then I can absolutely do a PR. Can you verify that your reference to sinon in your bower.json file looks like:

"sinon": "http://sinonjs.org/releases/sinon-1.9.0.js"

rather than:

"sinon": "^1.14.1",

I'm guessing the other variable at play here is how we put together our tests (i.e., karma.config.js) via Grunt, Gulp or similar.

[wardbell]

Actually I just updated (v.0.1.4) the sinon to the most recent version by request (see issue #8 ).

sinon": "http://sinonjs.org/releases/sinon-1.15.0.js"

And, yes, I have to do it this way because there is no official bower package for sinon

Someone is trying to maintain a sinon bower package independently (sinonjs) ... I believe that is the package to which you refer. But his is not "official" and it's several releases behind right now. I'm tempted but am not quite comfortable ... yet.

[wardbell]

Closing because I have not heard from you and have no reason to believe there is a problem for me to fix. Happy to reopen if you think I should