4

I'm using ember-cli 0.0.35, and injecting a dependency onto my component via an initializer. It works great in development, but the property isn't present when I run tests. It appears that testing calls loadInitializers, but the dependency is not showing up on this.subject({});

I don't want to manually inject it for the tests. Is there a better way to handle this?

Initializer:

var FooServiceInitializer = {
  name: 'foo',
  initialize: function (container, application) {
   application.inject('component:foo', 'foo', 'service:foo');
  }
};
export default FooServiceInitializer;

Failing Test:

moduleForComponent('bar', 'Component: Bar', {
  setup: function() {
    App = startApp();
    component = this.subject({});
  },
  teardown: function () {
    Ember.run(App, App.destroy);
  }
});

test('Properties: foo', function() {
  // Make sure we injected the service
  ok(component.foo, 'foo is injected');
});
Colin Hanna
  • 41
  • 3
  • The problem is ic-ajax is really meant more for unit tests, and not for integration tests. It's entire goal is for testing a single unit of work. And container injection is definitely out of its scope. – Kingpin2k Jun 23 '14 at 19:00

1 Answers1

2

As I said before, it really lends itself to an integration test since you are testing the container at this point (and not the mini container created by ic-ajax).

Your real test is along the lines of this

test("root lists 3 colors", function(){
  var c = App.__container__.lookup('component:foo-bar');
  ok(c.foo.blah);
});

If you feel guilty about using the container during testing (which you shouldn't) you can create a helper to avoid having to fix it all over the place when/if an api changes in the future.

Ember.Test.registerHelper('containerLookup',
  function(app, look) {
    return app.__container__.lookup(look);
  }
);

Make sure you define this before

App.injectTestHelpers();

and then your test would look like

test("root lists 3 colors", function(){
  var c = containerLookup('component:foo-bar');
  ok(c.foo.blah);
});

http://emberjs.jsbin.com/doxigu/edit

Kingpin2k
  • 47,277
  • 10
  • 78
  • 96