[cleanup]: Remove implicit store injection slated for Ember v4.0.0 (#…

…7667)

packages/-ember-data/addon/setup-container.js

@@ -46,13 +46,6 @@ function initializeStore(application) {
   }
 }
 
-function initializeStoreInjections(application) {
-  let inject = application.inject || application.injection;
-  inject.call(application, 'controller', 'store', 'service:store');
-  inject.call(application, 'route', 'store', 'service:store');
-}
-
 export default function setupContainer(application) {
-  initializeStoreInjections(application);
   initializeStore(application);
 }

packages/-ember-data/app/initializers/ember-data.js

@@ -4,9 +4,6 @@ import setupContainer from 'ember-data/setup-container';
 
 /*
   This code initializes EmberData in an Ember application.
-
-  It ensures that the `store` service is automatically injected
-  as the `store` property on all routes and controllers.
 */
 export default {
   name: 'ember-data',

packages/-ember-data/tests/integration/injection-test.js

@@ -1,11 +1,7 @@
-import Service from '@ember/service';
-
 import { module, test } from 'qunit';
 
 import { setupTest } from 'ember-qunit';
 
-import Model from '@ember-data/model';
-
 module('integration/injection factoryFor enabled', function (hooks) {
   setupTest(hooks);
   let store;
@@ -36,36 +32,3 @@ module('integration/injection factoryFor enabled', function (hooks) {
     assert.equal(modelClass.modelName, 'super-villain', 'expected the factory itself to be returned');
   });
 });
-
-module('integration/injection eager injections', function (hooks) {
-  setupTest(hooks);
-  let store;
-  let Apple = Service.extend();
-
-  hooks.beforeEach(function () {
-    let { owner } = this;
-
-    owner.register('model:foo', Model.extend());
-    owner.register('service:apple', Apple);
-    owner.inject('model:foo', 'apple', 'service:apple');
-
-    store = this.owner.lookup('service:store');
-  });
-
-  test('did inject', async function (assert) {
-    // TODO likely this test should instead test that we can use service injections
-    // on models (e.g. that owner is properly setup for it).
-    assert.expectDeprecation(
-      () => {
-        let foo = store.createRecord('foo');
-        let apple = foo.get('apple');
-        let appleService = this.owner.lookup('service:apple');
-
-        assert.ok(apple, `'model:foo' instance should have an 'apple' property`);
-        assert.ok(apple === appleService, `'model:foo.apple' should be the apple service`);
-        assert.ok(apple instanceof Apple, `'model:foo'.apple should be an instance of 'service:apple'`);
-      },
-      { id: 'implicit-injections', count: 1, when: { ember: '>=3.27.0' } }
-    );
-  });
-});

packages/unpublished-fastboot-test-app/app/routes/index.js

@@ -1,8 +1,11 @@
 import Route from '@ember/routing/route';
+import { inject as service } from '@ember/service';
 
 import { buildTree } from 'ember-simple-tree/utils/tree';
 
 export default class IndexRoute extends Route {
+  @service store;
+
   model() {
     return this.store.findAll('person');
   }

packages/unpublished-fastboot-test-app/app/routes/person/new.js

@@ -1,10 +1,10 @@
 import Route from '@ember/routing/route';
 import { inject as service } from '@ember/service';
 
-export default Route.extend({
-  store: service(),
+export default class PersonNewRoute extends Route {
+  @service store;
 
   model() {
     return this.store.createRecord('person');
-  },
-});
+  }
+}

packages/unpublished-relationship-performance-test-app/app/routes/add-children.js

@@ -1,6 +1,9 @@
 import Route from '@ember/routing/route';
+import { inject as service } from '@ember/service';
 
 export default Route.extend({
+  store: service(),
+
   async model() {
     performance.mark('start-data-generation');
 

packages/unpublished-relationship-performance-test-app/app/routes/destroy.js

@@ -1,8 +1,11 @@
 import Route from '@ember/routing/route';
+import { inject as service } from '@ember/service';
 
 import { all } from 'rsvp';
 
 export default Route.extend({
+  store: service(),
+
   async model() {
     performance.mark('start-data-generation');
     const payload = await fetch('./fixtures/destroy.json').then((r) => r.json());

packages/unpublished-relationship-performance-test-app/app/routes/materialization.js

@@ -1,6 +1,9 @@
 import Route from '@ember/routing/route';
+import { inject as service } from '@ember/service';
 
 export default Route.extend({
+  store: service(),
+
   model() {
     performance.mark('start-find-all');
     return this.store.findAll('car', { reload: true }).then((cars) => {

packages/unpublished-relationship-performance-test-app/app/routes/unload.js

@@ -1,7 +1,10 @@
 import Route from '@ember/routing/route';
 import { run } from '@ember/runloop';
+import { inject as service } from '@ember/service';
 
 export default Route.extend({
+  store: service(),
+
   async model() {
     performance.mark('start-data-generation');
     const payload = await fetch('./fixtures/unload.json').then((r) => r.json());

packages/unpublished-relationship-performance-test-app/app/routes/unused-relationships.js

@@ -1,10 +1,13 @@
 import Route from '@ember/routing/route';
+import { inject as service } from '@ember/service';
 
 /**
  * Measures the performance characteristics of pushing a large payload
  * with tons of relationship data that will not be accessed.
  */
 export default class UnusedRelationshipsRoute extends Route {
+  @service store;
+
   async model() {
     performance.mark('start-data-generation');