Merge pull request #293 from marcvberg/hasone-nonnull-field-fix

Hasone nonnull field fix

Author: mikepschneider

packages/appsync-modelgen-plugin/src/__tests__/visitors/appsync-visitor.test.ts

@@ -339,8 +339,9 @@ describe('AppSyncModelVisitor', () => {
       );
       visit(ast, { leave: visitor });
       visitor.generate();
-      const projectFields = visitor.models.Project.fields.map(field => field.name);
-      expect(projectFields).toContain('projectTeamId');
+      const projectTeamIdField = visitor.models.Project.fields.find(field => { return field.name === 'projectTeamId'; });
+      expect(projectTeamIdField).toBeDefined();
+      expect(projectTeamIdField.isNullable).toBeTruthy();
     });
   });
 

packages/appsync-modelgen-plugin/src/visitors/appsync-visitor.ts

@@ -691,7 +691,7 @@ export class AppSyncModelVisitor<
               directives: [],
               type: 'ID',
               isList: false,
-              isNullable: connectionInfo.associatedWith.isNullable,
+              isNullable: field.isNullable,
             });
           } else if (connectionInfo.targetName !== 'id') {
             // Need to remove the field that is targetName