-
Notifications
You must be signed in to change notification settings - Fork 34
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Failed to instantiate module function fakeRouteProvider #7
Comments
Put a breakpoint at the top and check to see if Expecting karma framework config to be something like this:
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. |
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. ;-) |
Actually I was wrong, sinon is an object when I set a breakpoint at the top of the spec. |
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. :) |
How can we give the appropriate guidance in the bardjs readme? Propose something. I take PRs :-) |
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:
rather than:
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. |
Actually I just updated (v.0.1.4) the sinon to the most recent version by request (see issue #8 ).
And, yes, I have to do it this way because there is no official bower package for sinon
|
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 |
Sounds similar to some other closed issues, but no solution jumped out at me. Using 0.1.3 of bardjs I am getting the following error:
My spec is a complete rip off of something John Papa did in one of his Pluralsight trainings:
And I verified that sinon is indeed included in the karma files array. Any ideas what I am doing wrong? I am assuming it's my fault. :)
The text was updated successfully, but these errors were encountered: