resizeService not injected in test environment about ember-resize HOT 7 CLOSED

In this case, we're talking about integration tests, rather than unit tests.

For mine, the expected behaviour for component integration tests is that service injection mirrors development/prod. If a service is injected in development/prod, it's injected in test. This matches the behaviour of other service-injecting Ember add-ons.

In my case, I had to change my production code to enable test-specific behaviour. I'm already being explicit about the fact I need the resizeService, as per the Ember testing docs.

The bug is that, even after I register the service with

this.register('service:resize', resizeServiceStub);
this.inject.service('resize', { as: 'resizeService' });

...it's still not available in my tests.

from ember-resize.

I could be wrong, but afaik you should always inject services even in your production code. That is, resizeService: Ember.inject.service('resize') is exactly the way you should do it.
I'd even go so far as to suggest that this addon should not automatically inject itself via an initializer.

from ember-resize.

It's been a while since the last update, just wanted to throw out a +1 that this is an issue. Especially in terms of time it took me fiddling around with tests and finally finding this issue on github.

For now I'm going to (redundantly) inject the service into production code.
My intuition is that this should be the default behaviour, ember-resize should not automatically inject itself into every component. I'm fairly new to Ember, but there is something off here; changing production code to enable test specific behaviour.

from ember-resize.

I think the solution here is to clarify in the README what needs to be done. I'm wary about injecting services into unit tests without the developer being explicit about it

from ember-resize.

My expectation (and I could have the wrong expectation) is that if I don't need to inject the service in my component in order for it to work, and integration test should not then require me to inject it. You end up in a state where your code works, but the tests fail. While updating the documentation could make this easier to debug when a dev comes across it, personally, I think it would be better to either

1. work without the service being injected by the developer just to make the test work
2. require the service to be injected by the developer in order to use the mixin (thereby also working in the test)

from ember-resize.

I've started seeing this with acceptance tests after upgrading to Ember 2.13.

from ember-resize.

🎉 This issue has been resolved in version 0.2.1 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀

from ember-resize.