Merge pull request #104 from underscore05/richard/fix-non-incremental-id-and-remove-schema-return-type

Fix issue with Remove and Create function not incrementing the ID

Author: fzaninotto

src/introspection/getSchemaFromData.js

@@ -139,9 +139,10 @@ export default (data) => {
                 },
                 {}
             );
+            const { id, ...createFields } = typeFields;
             fields[`create${type.name}`] = {
                 type: typesByName[type.name],
-                args: typeFields,
+                args: createFields,
             };
             fields[`update${type.name}`] = {
                 type: typesByName[type.name],

src/introspection/getSchemaFromData.spec.js

@@ -175,12 +175,6 @@ test('creates three mutation fields per data type', () => {
     const mutations = getSchemaFromData(data).getMutationType().getFields();
     expect(mutations['createPost'].type.name).toEqual(PostType.name);
     expect(mutations['createPost'].args).toEqual([
-        {
-            name: 'id',
-            type: new GraphQLNonNull(GraphQLID),
-            defaultValue: undefined,
-            description: null,
-        },
         {
             name: 'title',
             type: new GraphQLNonNull(GraphQLString),
@@ -238,12 +232,6 @@ test('creates three mutation fields per data type', () => {
     ]);
     expect(mutations['createUser'].type.name).toEqual(UserType.name);
     expect(mutations['createUser'].args).toEqual([
-        {
-            name: 'id',
-            type: new GraphQLNonNull(GraphQLID),
-            defaultValue: undefined,
-            description: null,
-        },
         {
             name: 'name',
             type: new GraphQLNonNull(GraphQLString),

src/resolver/Mutation/create.js

@@ -1,8 +1,7 @@
 export default (entityData = []) => (_, entity) => {
     const newId =
         entityData.length > 0 ? entityData[entityData.length - 1].id + 1 : 0;
-    const newEntity = Object.assign({ id: newId }, entity);
-
+    const newEntity = Object.assign({}, entity, { id: newId });
     entityData.push(newEntity);
     return newEntity;
 };