update to make links object comply with jsonapi

lib/index.js

@@ -4,15 +4,17 @@ var _ = require('lodash');
 module.exports = {
   attributes: {
     modelLinks: function (controller, reverseRouteService, actions) {
-      const result = [];
+      const result = {};
       actions = actions || ['findOne', 'update', 'destroy'];
       _.each(actions, (action) => {
+        let key = action === 'findOne' ? 'self' : action;
         const components = {
           controller: controller + '.' + action,
           args: [this.id]
         };
-        result.push({link: reverseRouteService(components, false), rel: action});
+        result[key] = reverseRouteService(components, true);
       });
+      console.log('result: ', result);
       return result;
     }
   },

package.json

@@ -28,17 +28,17 @@
   "main": "./lib",
   "dependencies": {
     "lodash": "^3.10.1",
-    "sails-util": "^0.10.6"
+    "sails-util": "^0.11.0"
   },
   "optionalDependencies": {
     "sails-reverse-routing": "^1.0.12"
   },
   "devDependencies": {
-    "chai": "1.x.x",
+    "chai": "3.4.1",
     "chai-things": "^0.2.0",
     "coveralls": "^2.11.4",
     "istanbul": "^0.4.0",
-    "mocha": "^1.21.5",
+    "mocha": "^2.3.4",
     "mocha-lcov-reporter": "^1.0.0",
     "rewire": "^2.4.0",
     "sinon": "^1.17.2",

test/index.test.js

@@ -5,7 +5,8 @@ describe('UNIT sails-linking-models', () => {
   describe('modelLinks', () => {
     let reverseRouteService, result;
     before(() => {
-      reverseRouteService = sinon.stub().returns(['link1', 'link2']);
+
+      reverseRouteService = sinon.stub().returns('link1');
     });
 
     describe('without actionsArray passed in...', () => {
@@ -20,23 +21,21 @@ describe('UNIT sails-linking-models', () => {
         });
       });
 
-      it('should return an array of links of length 3', () => {
-        expect(result).to.have.length(3)
-          .and.to.contain.a.thing.with.keys('rel', 'link')
-          .and.to.contain.a.thing.with.property('rel', 'findOne')
-          .and.to.contain.a.thing.with.property('rel', 'update')
-          .and.to.contain.a.thing.with.property('rel', 'destroy');
+
+      it('should return aa links object with the expected properties', () => {
+        expect(result).to.have.property('self')
+        expect(result).to.have.property('update')
+        expect(result).to.have.property('destroy')
       });
     });
     describe('with actionsArray passed in...', () => {
       before(() => {
         result = lib.attributes.modelLinks('controllerName', reverseRouteService, ['action1', 'action2']);
       });
       it("should use the actionsArray parameter", () => {
-        expect(result).to.have.length(2)
-          .and.to.contain.a.thing.with.keys('rel', 'link')
-          .and.to.contain.a.thing.with.property('rel', 'action1')
-          .and.to.contain.a.thing.with.property('rel', 'action2');
+        expect(result).to.have.property('action1')
+        expect(result).to.have.property('action2')
+
       });
     });
   });