Added copy and tests

Author: grafitto

packages/client/src/project/entities.ts

@@ -14,9 +14,10 @@ export function getProjectEntities (projectId: string, apiClient: AxiosInstance)
       return wrapEntityArray(entities);
     },
     async put(entity: Entity): Promise<WrappedEntity> {
-      const entityId = entity._id;
-      delete entity._id;
-      const expandedEntity = expandEntity(entity);
+      const _entity = { ...entity }; // Make a shallow copy
+      const entityId = _entity._id;
+      delete _entity._id;
+      const expandedEntity = expandEntity(_entity);
       const res = await apiClient.put<ApiEntity>(
         `/projects/${projectId}/entities/${entityId}`, expandedEntity);
       const returnedEntity = res.data;

packages/client/src/project/tests/__snapshots__/files.test.ts.snap

@@ -113,5 +113,32 @@ describe('Project Entities', () => {
         expect(entity).toMatchSnapshot();
       });
     });
+    describe('put', () => {
+      const userEntity = {
+        data: {
+          name: 'someone 1'
+        },
+        tags: []
+      }
+      let apiClient: AxiosInstance;
+      async function callMockPut (entity: any): Promise<dataHelpers.WrappedEntity> {
+        apiClient = api.getApiClient('http://baseurl', 'somekey');
+        jest.spyOn(apiClient, 'put').mockReturnValue(Promise.resolve({ data: 
+          { entity } }));
+        jest.spyOn(dataHelpers, 'wrapEntity');
+        const project = getProjectApi('project1', apiClient);
+        const ent = await project.entities.put(entity);
+        return ent;
+      }
+      it('should call PUT /projects/:pid/entities/:eid endpoint with entity', async () => {
+        const entity = { ...userEntity };
+        entity['_id'] = 'entity1';
+        const expectedEntity = dataHelpers.expandEntity(entity);
+        const frozen = Object.freeze(entity);
+        await callMockPut(frozen);
+        expect(apiClient.put).toHaveBeenCalledWith(
+          '/projects/project1/entities/entity1', expectedEntity);
+      });
+    });
   }); 
 });