Merge pull request #90 from efx/auxillary-refactors

migrate tests for other entities

Author: Turbo87

__testfixtures__/fourteen-testing-api/setup-adapter-test.input.js

@@ -0,0 +1,15 @@
+import { expect } from 'chai';
+import { describe, it } from 'mocha';
+import { setupTest } from 'ember-mocha';
+
+describe('Unit | Adapter | rental', function() {
+  setupTest('adapter:rental', {
+    needs: ['service:store']
+  });
+
+  // Replace this with your real tests.
+  it('exists', function() {
+    let adapter = this.subject();
+    expect(adapter).to.be.ok;
+  });
+});

__testfixtures__/fourteen-testing-api/setup-adapter-test.output.js

@@ -0,0 +1,13 @@
+import { expect } from 'chai';
+import { describe, it } from 'mocha';
+import { setupTest } from 'ember-mocha';
+
+describe('Unit | Adapter | rental', function() {
+  setupTest();
+
+  // Replace this with your real tests.
+  it('exists', function() {
+    let adapter = this.owner.lookup('adapter:rental');
+    expect(adapter).to.be.ok;
+  });
+});

__testfixtures__/fourteen-testing-api/setup-serializer-test.input.js

@@ -0,0 +1,17 @@
+import { expect } from 'chai';
+import { describe, it } from 'mocha';
+import { setupTest } from 'ember-mocha';
+
+describe('Unit | Serializer | rental', function() {
+  setupTest('serializer:rental', {
+    needs: ['service:store']
+  });
+
+  // Replace this with your real tests.
+  it('exists', function() {
+    let serializer = this.subject();
+
+    expect(serializer).to.be.ok;
+  });
+
+});

__testfixtures__/fourteen-testing-api/setup-serializer-test.output.js

@@ -0,0 +1,15 @@
+import { expect } from 'chai';
+import { describe, it } from 'mocha';
+import { setupTest } from 'ember-mocha';
+
+describe('Unit | Serializer | rental', function() {
+  setupTest();
+
+  // Replace this with your real tests.
+  it('exists', function() {
+    let serializer = this.owner.lookup('service:store').serializerFor('rental');
+
+    expect(serializer).to.be.ok;
+  });
+
+});

fourteen-testing-api.js

@@ -54,8 +54,7 @@ module.exports = function(file, api) {
       && nodePath.expression.arguments.length === 2
       && j.Literal.check(nodePath.expression.arguments[0])
       && (
-        nodePath.expression.arguments[0].value.startsWith('route:')
-        || nodePath.expression.arguments[0].value.startsWith('controller:')
+        /^(adapter|controller|route|service|serializer|transform):/.test(nodePath.expression.arguments[0].value)
       )
     return isSetupNeeds ;
   }
@@ -472,7 +471,7 @@ module.exports = function(file, api) {
         subjectType = split[0];
         subjectName = split[1];
       }
-      let isSingletonSubject = ['model', 'component'].indexOf(subjectType) === -1;
+      let isSingletonSubject = !['model', 'component', 'serializer'].includes(subjectType);
 
       // if we don't have `options` and the type is a singleton type
       // use `this.owner.lookup(subject)`
@@ -486,6 +485,22 @@ module.exports = function(file, api) {
             [subject]
           )
         );
+      } else if (subjectType === 'serializer') {
+        p.replace(
+              j.callExpression(
+                j.memberExpression(
+                  j.callExpression(
+                    j.memberExpression(
+                      j.memberExpression(j.thisExpression(), j.identifier('owner')),
+                      j.identifier('lookup')
+                    ),
+                    [j.literal('service:store')]
+                  ),
+                  j.identifier('serializerFor')
+                ),
+                [j.literal(subjectName)].filter(Boolean)
+              )
+        );
       } else if (subjectType === 'model') {
         p.replace(
               j.callExpression(