Fix infinity model container so services can be injected on it

[mossymaker]

Injecting services on the extended infinity model was failing because the instance is created without a container. This seems like the right fix?

With the 1.0 API changes, infinityModelLoaded and infinityModelUpdated no longer have the context they did as members of the route. Now, an intermediary (e.g. a service) appears to be required to effect a change in the context of the route or controller, e.g.:

import InfinityModel from 'ember-infinity/lib/infinity-model';

export default InfinityModel.extend({
  infinityModelUpdated({ lastPageLoaded, totalPages }) {
    if (totalPages > lastPageLoaded) {
      // Hide footer until all records are loaded
      set(this, 'memoryStorage.hasActiveInfiniteScroll', true);
    }
  }
});

{{! app/templates/application.hbs}}
{{#unless memoryStorage.hasActiveInfiniteScroll}}
  {{footer-stuff}}
{{/unless}}

[johncalvinyoung]

Not positive how this is connected to other changes.

[snewcomer]

Unrelated but seems like an oversight. Nice catch!

[mossymaker]

Unrelated...I can pull it into another commit/PR if you prefer.

[johncalvinyoung]

This seems like a good idea...but not sure what implications this would have, from a memory- or lifecycle-related perspective.

[snewcomer]

@callovarne What version of ember-infinity are you on? I had thought this was fixed a while back. See the below README link.

https://github.com/ember-infinity/ember-infinity#extending-infinitymodel

[mossymaker]

I'm working off master with 1.2.6. Without giving the container, Ember reports:

Assertion Failed: Attempting to lookup an injected property on an object without a container, ensure that the object was instantiated via a container.

I suspect in pre-1.0, the container was set on the route when it was instantiated by Ember. Since InfinityModel isn't, it seems as though that needs to be done here.

[snewcomer]

@callovarne I generally approve of this PR! Lmk if you have time to add a test.

[mossymaker]

@snewcomer: updated with test.

[snewcomer]

This looks like a great change!