Apollo service is initializes during integration test without its options

[viniciussbs]

So, the base-query-manager mixin, during the #20 development, was using a query manager got with owner.lookup. The Apollo service was not injected by the mixin.

Today the query manager is got from the Apollo service, that's always injected by the mixin. This is causing an error on integration tests because now the service initializes during the test, but the initializer that injects the options into the service doesn't run during the test. Then Apollo client raises an error saying that the network layer needs an endpoint.

How are you handling this? Are you writing integration tests?

[viniciussbs]

@bgentry I've updated the issue with some to references to what I'm talking about.

[viniciussbs]

I might be wrong, but I guess this part of the code was discussed with @dfreeman too, but the discussion was lost due to some rebase or forced push into #20.

[dfreeman]

I believe you could get rid of the initializer entirely if you update the service to do something like:

options: computed(function() {
  const config = getOwner(this).resolveRegistration('config:environment');
  return config.apollo;
})

[bgentry]

Released in v0.4.5, thanks especially to @dfreeman for having all the answers ✌️

[viniciussbs]

Thank you, guys!