Modify tests for new id property

Author: javierbrea

test/acceptance/behaviors-api.spec.js

@@ -27,7 +27,8 @@ describe("behaviors api", () => {
         {
           extendedFrom: null,
           fixtures: ["12e5f429b92f67d4ec2bf90940ec1135", "0dbc954f9d9c9f3f7996c60e63384c9e"],
-          name: "standard"
+          name: "standard",
+          id: "standard"
         },
         {
           extendedFrom: "standard",
@@ -36,7 +37,8 @@ describe("behaviors api", () => {
             "12e5f429b92f67d4ec2bf90940ec1135",
             "0dbc954f9d9c9f3f7996c60e63384c9e"
           ],
-          name: "user2"
+          name: "user2",
+          id: "user2"
         },
         {
           extendedFrom: "standard",
@@ -45,7 +47,8 @@ describe("behaviors api", () => {
             "12e5f429b92f67d4ec2bf90940ec1135",
             "0dbc954f9d9c9f3f7996c60e63384c9e"
           ],
-          name: "dynamic"
+          name: "dynamic",
+          id: "dynamic"
         }
       ]);
     });
@@ -57,7 +60,8 @@ describe("behaviors api", () => {
       expect(response).toEqual({
         extendedFrom: null,
         fixtures: ["12e5f429b92f67d4ec2bf90940ec1135", "0dbc954f9d9c9f3f7996c60e63384c9e"],
-        name: "standard"
+        name: "standard",
+        id: "standard"
       });
     });
   });
@@ -72,7 +76,8 @@ describe("behaviors api", () => {
           "12e5f429b92f67d4ec2bf90940ec1135",
           "0dbc954f9d9c9f3f7996c60e63384c9e"
         ],
-        name: "dynamic"
+        name: "dynamic",
+        id: "dynamic"
       });
     });
   });
@@ -84,7 +89,7 @@ describe("behaviors api", () => {
         simple: false
       });
       expect(response.statusCode).toEqual(404);
-      expect(response.body.message).toEqual('Behavior with name "foo" was not found');
+      expect(response.body.message).toEqual('Behavior with id "foo" was not found');
     });
   });
 });

test/unit/src/Behaviors.spec.js

@@ -55,7 +55,7 @@ describe("Behavior", () => {
     });
 
     it("should register a get router for behavior names", async () => {
-      expect(libMocks.stubs.express.get.getCall(1).args[0]).toEqual("/:name");
+      expect(libMocks.stubs.express.get.getCall(1).args[0]).toEqual("/:id");
     });
   });
 
@@ -64,11 +64,13 @@ describe("Behavior", () => {
       coreInstance.behaviors = {
         collection: [
           {
+            id: "foo",
             name: "foo",
             fixtures: [{ id: "foo-fixture-id-1" }, { id: "foo-fixture-id-2" }],
             extendedFrom: "foo-base-behavior"
           },
           {
+            id: "foo2",
             name: "foo2",
             fixtures: [{ id: "foo-fixture-id-3" }],
             extendedFrom: null
@@ -79,11 +81,13 @@ describe("Behavior", () => {
       behaviors.getCollection({}, resMock);
       expect(resMock.send.getCall(0).args[0]).toEqual([
         {
+          id: "foo",
           name: "foo",
           fixtures: ["foo-fixture-id-1", "foo-fixture-id-2"],
           extendedFrom: "foo-base-behavior"
         },
         {
+          id: "foo2",
           name: "foo2",
           fixtures: ["foo-fixture-id-3"],
           extendedFrom: null
@@ -97,11 +101,13 @@ describe("Behavior", () => {
       coreInstance.behaviors = {
         collection: [
           {
+            id: "foo",
             name: "foo",
             fixtures: [{ id: "foo-fixture-id-1" }, { id: "foo-fixture-id-2" }],
             extendedFrom: "foo-base-behavior"
           },
           {
+            id: "foo2",
             name: "foo2",
             fixtures: [{ id: "foo-fixture-id-3" }],
             extendedFrom: null
@@ -112,12 +118,13 @@ describe("Behavior", () => {
       behaviors.getModel(
         {
           params: {
-            name: "foo"
+            id: "foo"
           }
         },
         resMock
       );
       expect(resMock.send.getCall(0).args[0]).toEqual({
+        id: "foo",
         name: "foo",
         fixtures: ["foo-fixture-id-1", "foo-fixture-id-2"],
         extendedFrom: "foo-base-behavior"
@@ -130,11 +137,13 @@ describe("Behavior", () => {
       coreInstance.behaviors = {
         collection: [
           {
+            id: "foo",
             name: "foo",
             fixtures: [{ id: "foo-fixture-id-1" }, { id: "foo-fixture-id-2" }],
             extendedFrom: "foo-base-behavior"
           },
           {
+            id: "foo2",
             name: "foo2",
             fixtures: [{ id: "foo-fixture-id-3" }],
             extendedFrom: null
@@ -145,7 +154,7 @@ describe("Behavior", () => {
       behaviors.getModel(
         {
           params: {
-            name: "foo3"
+            id: "foo3"
           }
         },
         resMock,