fix(utils): Fix bug introduced by #265 and add test case for remote type

Author: Tronix117

lib/utils.js

@@ -148,7 +148,7 @@ function getTypeFromContext (context) {
 
   const returns = [].concat(context.method.returns)
   for (var i = 0, l = returns.length; i < l; i++) {
-    if (typeof returns[i] === 'object' && returns[i].root === true) continue
+    if (typeof returns[i] !== 'object' || returns[i].root !== true) continue
     return returns[i].type
   }
 }

test/remoteMethods.test.js

@@ -185,6 +185,7 @@ describe('loopback json api remote methods', function () {
             expect(err).to.equal(null)
             expect(res.body).to.be.an('object')
             expect(res.body.data).to.be.an('array').with.lengthOf(1)
+            expect(res.body.data[0].type).to.equal('archives')
             expect(res.body.data[0].id).to.equal(archiveData.id + '')
             expect(res.body.data[0].attributes).to.deep.equal({
               raw: archiveData.raw,
@@ -205,6 +206,7 @@ describe('loopback json api remote methods', function () {
 
         expect(res.body.data).to.be.an('object')
         expect(res.body.data.id).to.equal(postData.id + '')
+        expect(res.body.data.type).to.equal('posts')
         expect(res.body.data.attributes).to.deep.equal({
           title: postData.title,
           content: postData.content