[CHORE] Refactor away from createStore test helper

From emberjs#6166 This refactors some tests so that they don't rely on the `createStore` test helper. `createStore` was helpful in the past but is less so now that we have [nice APIs for dependency injection](https://guides.emberjs.com/release/applications/dependency-injection/). --- One test was removed from `unit/store/adapter-interop - Store working with a Adapter` because I don't think it was correct. Here was the test: https://github.com/emberjs/data/blob/39182f324d5777e4a75c4a582f0791631e37d8de/packages/-ember-data/tests/unit/store/adapter-interop-test.js#L28-L32 This test was, at one time, testing that it was possible to pass a factory as the `adapter` property to `Store`. This feature was removed in emberjs#3191 but the test continued passing because of the way the `createStore` works. https://github.com/emberjs/data/blob/39182f324d5777e4a75c4a582f0791631e37d8de/packages/-ember-data/tests/helpers/store.js#L57-L60
HeroicEric · Aug 19, 2019 · 5c8c793 · 5c8c793
1 parent 963645c
commit 5c8c793
Show file tree

Hide file tree

Showing 12 changed files with 1,340 additions and 1,454 deletions.
diff --git a/packages/-ember-data/tests/helpers/store.js b/packages/-ember-data/tests/helpers/store.js
@@ -107,7 +107,3 @@ export default function setupStore(options) {
 }
 
 export { setupStore };
-
-export function createStore(options) {
-  return setupStore(options).store;
-}
diff --git a/packages/-ember-data/tests/integration/record-arrays/adapter-populated-record-array-test.js b/packages/-ember-data/tests/integration/record-arrays/adapter-populated-record-array-test.js
@@ -1,35 +1,33 @@
 import { run } from '@ember/runloop';
 import { Promise } from 'rsvp';
-import { setupStore, createStore } from 'dummy/tests/helpers/store';
-
+import { setupTest } from 'ember-qunit';
 import { module, test } from 'qunit';
 
-import DS from 'ember-data';
-
-let store;
+import Adapter from '@ember-data/adapter';
+import Model, { attr } from '@ember-data/model';
 
-const Person = DS.Model.extend({
-  name: DS.attr('string'),
+const Person = Model.extend({
+  name: attr('string'),
   toString() {
     return `<Person#${this.get('id')}>`;
   },
 });
 
-const adapter = DS.Adapter.extend({
-  deleteRecord() {
-    return Promise.resolve();
-  },
-});
+module('integration/record-arrays/adapter_populated_record_array - AdapterPopulatedRecordArray', function(hooks) {
+  setupTest(hooks);
 
-module('integration/record-arrays/adapter_populated_record_array - DS.AdapterPopulatedRecordArray', function(hooks) {
   hooks.beforeEach(function() {
-    store = createStore({
-      adapter: adapter,
-      person: Person,
-    });
+    this.owner.register('model:person', Person);
   });
 
   test('when a record is deleted in an adapter populated record array, it should be removed', function(assert) {
+    this.owner.register('adapter:application', Adapter.extend({
+      deleteRecord() {
+        return Promise.resolve();
+      },
+    }));
+
+    let store = this.owner.lookup('service:store');
     let recordArray = store.recordArrayManager.createAdapterPopulatedRecordArray('person', null);
 
     let payload = {
@@ -70,6 +68,7 @@ module('integration/record-arrays/adapter_populated_record_array - DS.AdapterPop
   });
 
   test('stores the metadata off the payload', function(assert) {
+    let store = this.owner.lookup('service:store');
     let recordArray = store.recordArrayManager.createAdapterPopulatedRecordArray('person', null);
 
     let payload = {
@@ -109,6 +108,7 @@ module('integration/record-arrays/adapter_populated_record_array - DS.AdapterPop
   });
 
   test('stores the links off the payload', function(assert) {
+    let store = this.owner.lookup('service:store');
     let recordArray = store.recordArrayManager.createAdapterPopulatedRecordArray('person', null);
 
     let payload = {
@@ -152,6 +152,7 @@ module('integration/record-arrays/adapter_populated_record_array - DS.AdapterPop
   });
 
   test('recordArray.replace() throws error', function(assert) {
+    let store = this.owner.lookup('service:store');
     let recordArray = store.recordArrayManager.createAdapterPopulatedRecordArray('person', null);
 
     assert.throws(
@@ -164,8 +165,8 @@ module('integration/record-arrays/adapter_populated_record_array - DS.AdapterPop
   });
 
   test('pass record array to adapter.query regardless of arity', function(assert) {
-    let env = setupStore({ person: Person });
-    let store = env.store;
+    let store = this.owner.lookup('service:store');
+    let adapter = store.adapterFor('application');
 
     let payload = {
       data: [
@@ -174,14 +175,14 @@ module('integration/record-arrays/adapter_populated_record_array - DS.AdapterPop
       ],
     };
 
-    env.adapter.query = function(store, type, query) {
+    adapter.query = function(store, type, query) {
       // Due to #6232, we now expect 5 arguments regardless of arity
       assert.equal(arguments.length, 5);
       return payload;
     };
 
     return store.query('person', {}).then(recordArray => {
-      env.adapter.query = function(store, type, query, _recordArray) {
+      adapter.query = function(store, type, query, _recordArray) {
         assert.equal(arguments.length, 5);
         return payload;
       };
@@ -190,8 +191,8 @@ module('integration/record-arrays/adapter_populated_record_array - DS.AdapterPop
   });
 
   test('pass record array to adapter.query regardless of arity', function(assert) {
-    let env = setupStore({ person: Person });
-    let store = env.store;
+    let store = this.owner.lookup('service:store');
+    let adapter = store.adapterFor('application');
 
     let payload = {
       data: [
@@ -214,14 +215,14 @@ module('integration/record-arrays/adapter_populated_record_array - DS.AdapterPop
       return superCreateAdapterPopulatedRecordArray.apply(this, arguments);
     };
 
-    env.adapter.query = function(store, type, query) {
+    adapter.query = function(store, type, query) {
       // Due to #6232, we now expect 5 arguments regardless of arity
       assert.equal(arguments.length, 5);
       return payload;
     };
 
     return store.query('person', actualQuery).then(recordArray => {
-      env.adapter.query = function(store, type, query, _recordArray) {
+      adapter.query = function(store, type, query, _recordArray) {
         assert.equal(arguments.length, 5);
         return payload;
       };
@@ -239,8 +240,8 @@ module('integration/record-arrays/adapter_populated_record_array - DS.AdapterPop
   });
 
   test('loadRecord re-syncs internalModels recordArrays', function(assert) {
-    let env = setupStore({ person: Person });
-    let store = env.store;
+    let store = this.owner.lookup('service:store');
+    let adapter = store.adapterFor('application');
 
     let payload = {
       data: [
@@ -249,7 +250,7 @@ module('integration/record-arrays/adapter_populated_record_array - DS.AdapterPop
       ],
     };
 
-    env.adapter.query = function(store, type, query, recordArray) {
+    adapter.query = function(store, type, query, recordArray) {
       return payload;
     };
 
@@ -282,18 +283,18 @@ module('integration/record-arrays/adapter_populated_record_array - DS.AdapterPop
     assert.expect(8);
 
     let queryPromise, queryArr, findPromise, findArray;
-    let env = setupStore({ person: Person });
-    let store = env.store;
+    let store = this.owner.lookup('service:store');
+    let adapter = store.adapterFor('application');
     let array = [{ id: '1', type: 'person', attributes: { name: 'Scumbag Dale' } }];
 
     // resemble server side filtering
-    env.adapter.query = function(store, type, query, recordArray) {
+    adapter.query = function(store, type, query, recordArray) {
       return { data: array.slice(query.slice) };
     };
 
     // implement findAll to further test that query updates won't muddle
     // with the non-query record arrays
-    env.adapter.findAll = function(store, type, sinceToken) {
+    adapter.findAll = function(store, type, sinceToken) {
       return { data: array.slice(0) };
     };
 

diff --git a/packages/-ember-data/tests/integration/record-arrays/peeked-records-test.js b/packages/-ember-data/tests/integration/record-arrays/peeked-records-test.js
@@ -1,24 +1,26 @@
+import Adapter from '@ember-data/adapter';
+import Model, { attr } from '@ember-data/model';
 import { run } from '@ember/runloop';
-import { createStore } from 'dummy/tests/helpers/store';
+import { setupTest } from 'ember-qunit';
 import { module, test } from 'qunit';
-import DS from 'ember-data';
 import { get } from '@ember/object';
 import { watchProperties } from '../../helpers/watch-property';
 
 let store;
 
-const Person = DS.Model.extend({
-  name: DS.attr('string'),
+const Person = Model.extend({
+  name: attr('string'),
   toString() {
     return `<Person#${this.get('id')}>`;
   },
 });
 
 module('integration/peeked-records', function(hooks) {
+  setupTest(hooks);
+
   hooks.beforeEach(function() {
-    store = createStore({
-      person: Person,
-    });
+    this.owner.register('model:person', Person);
+    store = this.owner.lookup('service:store');
   });
 
   test('repeated calls to peekAll in separate run-loops works as expected', function(assert) {

diff --git a/packages/-ember-data/tests/unit/adapters/rest-adapter/group-records-for-find-many-test.js b/packages/-ember-data/tests/unit/adapters/rest-adapter/group-records-for-find-many-test.js
@@ -1,32 +1,33 @@
 import { run } from '@ember/runloop';
 import { Promise as EmberPromise } from 'rsvp';
-import { createStore } from 'dummy/tests/helpers/store';
+import { setupTest } from 'ember-qunit';
 
 import { module, test } from 'qunit';
 
-import DS from 'ember-data';
+import Model from '@ember-data/model';
+import RESTAdapter from '@ember-data/adapter/rest';
 
-let GroupsAdapter, store, requests;
+let store, requests;
 let maxLength;
 let lengths;
 
 module('unit/adapters/rest_adapter/group_records_for_find_many_test - DS.RESTAdapter#groupRecordsForFindMany', function(
   hooks
 ) {
+  setupTest(hooks);
+
   hooks.beforeEach(function() {
     maxLength = -1;
     requests = [];
     lengths = [];
 
-    GroupsAdapter = DS.RESTAdapter.extend({
+    this.owner.register('adapter:application', RESTAdapter.extend({
       coalesceFindRequests: true,
 
       findRecord(store, type, id, snapshot) {
         return { id };
       },
-    });
 
-    GroupsAdapter.reopen({
       ajax(url, type, options) {
         requests.push({
           url,
@@ -46,16 +47,11 @@ module('unit/adapters/rest_adapter/group_records_for_find_many_test - DS.RESTAda
         let testRecords = options.data.ids.map(id => ({ id }));
         return EmberPromise.resolve({ testRecords: testRecords });
       },
-    });
+    }));
 
-    store = createStore({
-      adapter: GroupsAdapter,
-      testRecord: DS.Model.extend(),
-    });
-  });
+    this.owner.register('model:test-record', Model.extend());
 
-  hooks.afterEach(function() {
-    run(store, 'destroy');
+    store = this.owner.lookup('service:store');
   });
 
   test('groupRecordsForFindMany - findMany', function(assert) {