Skip to content
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

Using Sinon #12

Closed
glassresistor opened this issue Jun 6, 2014 · 4 comments
Closed

Using Sinon #12

glassresistor opened this issue Jun 6, 2014 · 4 comments

Comments

@glassresistor
Copy link

I've been able to use Sinon with browserify by simple including it as a script before by budled tests. Is there a good way to do this with mochify?

Or do you have another fakeServer solution that works?

@mantoni
Copy link
Owner

mantoni commented Jun 6, 2014

Sinon generally works great with Mochify by just using require('sinon').

However, there is an open issue regarding the fake server implementation here: sinonjs/sinon#319

@glassresistor
Copy link
Author

Ironically thats all I really want sinc IO in an api is unavoidable. Its
why i've been bypassing browserify with it here
https://github.com/motherjones/smokejs/blob/RC1/mocha/test/all.js
This runs in the browser but not from the
browserify | testling | coverify pattern for some reason it seems to run
sinon as if is from node still in that situation.

Does the useFakeXMLHttpRequest(); work? I could probably hack one together
from there.

On Fri, Jun 6, 2014 at 11:37 AM, Maximilian Antoni <notifications@github.com

wrote:

Sinon generally works great with Mochify by just using require('sinon').

However, there is an open issue regarding the fake server implementation
here: sinonjs/sinon#319
sinonjs/sinon#319


Reply to this email directly or view it on GitHub
#12 (comment).

Mikela

@mantoni
Copy link
Owner

mantoni commented Jun 6, 2014

There is an open pull request for that, actually: sinonjs/sinon#439

@glassresistor
Copy link
Author

dances

On Fri, Jun 6, 2014 at 12:03 PM, Maximilian Antoni <notifications@github.com

wrote:

There is an open pull request for that, actually: sinonjs/sinon#439
sinonjs/sinon#439


Reply to this email directly or view it on GitHub
#12 (comment).

Mikela

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants