Merge pull request #32 from digitalsadhu/listepo/relation

Update

Author: listepo

lib/errors.js

@@ -35,7 +35,8 @@ module.exports = function (app, options) {
   remotes.afterError('**', function (ctx, next) {
     var errors = [];
     var statusCode = 500;
-
+    debug('Errors:');
+    debug(ctx.error);
     //validation errors
     if (ctx.error.details && ctx.error.details.messages) {
       errors = Object.keys(ctx.error.details.messages).map(function (key) {

lib/serializer.js

@@ -29,7 +29,7 @@ var parseResource = function (type, data, relations, options) {
   type = inflection.pluralize(type);
   type = caserize(type, options.keyForRelation);
   resource.type = type;
-  var relationships = parseRelations(data, relations);
+  var relationships = parseRelations(data, relations, options);
   if (!_.isEmpty(relationships)) {
     resource.relationships = relationships;
   }
@@ -75,9 +75,9 @@ var parseRelations = function (data, relations, options) {
     if (!_.isUndefined(value)) {
       delete data[fk];
       if (_.isArray(value)) {
-        relationship = makeRelations(type, value);
+        relationship = makeRelations(type, value, options);
       } else {
-        relationship = makeRelation(type, value);
+        relationship = makeRelation(type, value, options);
       }
       name = caserize(name, options.keyForRelation);
       relationships[name] = relationship;
@@ -86,7 +86,8 @@ var parseRelations = function (data, relations, options) {
   return relationships;
 };
 
-var makeRelation = function (type, id) {
+var makeRelation = function (type, id, options) {
+  type = caserize(type, options.keyForRelation);
   var data = {
     type: type,
     id: id
@@ -95,10 +96,10 @@ var makeRelation = function (type, id) {
   return {data: data};
 };
 
-var makeRelations = function (type, ids) {
+var makeRelations = function (type, ids, options) {
   var res = [];
   _.each(ids, function (id) {
-    res.push(makeRelation(type, id));
+    res.push(makeRelation(type, id, options));
   });
   return res;
 };

test/belongsTo.test.js

@@ -55,7 +55,6 @@ describe('loopback json api hasOne relationships', function () {
             expect(res.body).to.be.an('object');
             expect(res.body.links).to.be.an('object');
             expect(res.body.links.self).to.match(/comments\/1\/post/);
-            console.log(res.body.data);
             expect(res.body.data).to.equal(null);
             done();
           });

test/update.test.js

@@ -98,7 +98,6 @@ describe('loopback json api component update method', function () {
         })
         .set('Content-Type', 'application/json')
         .end(function (err, res) {
-          console.log(res.body);
           expect(err).to.equal(null);
           expect(res.body).to.have.deep.property('data.links.self');
           expect(res.body.data.links.self).to.match(/http:\/\/127\.0\.0\.1.*\/posts\/1/);