Changed how to determine if mockjax should be loaded

[begedin]

Description

This PR resolves the issue described in #124

If the described issue is in fact not an issue, feel free to ignore and close it.

[danielspaniel]

You are right about that one @begedin .. was good catch .. thanks for the PR.

[danielspaniel]

v1.3.9 has this change

[cherewaty]

Just bumped my Ember CLI app’s ember-data-factory-guy version to bring in this change. Now mockjax won’t import into my qunit-driven acceptance tests in the browser if I simply run ember server. The mockjax import happens if I run ember server --environment test, but then I can't simultaneously run my app on localhost:4200 and my tests at localhost:4200/tests.

I think the original implementation is also how ember-cli-qunit determines whether to import its own assets from the bower directory (ditto ember-cli-simple-auth-testing):

if (app.tests) {
  app.import(app.bowerDirectory + '/jquery-mockjax/dist/jquery.mockjax.js');
}

[danielspaniel]

yikes .. your even more right .. @cherewaty. What was I thinking before I changed this?? Gulp. That was dumb. I will comment that to make sure I don't forget again. Fixing.

[danielspaniel]

fixed in v2.0.3

[danielspaniel]

@begedin, until that raven issue is fixed .. you can stay with v2.0.2 I reckon