fix: partially rolling back commit 310 for flutter, swift, and java platforms (#319)

* fix: rolling back commit 310 for flutter, swift, and java platforms

* fix: short-circuit field rename

* fix: update swift hasMany relationship for compile issues

* fix: remove unnecessary revert logic

* fix: correct field name

* test: add unit tests that we believe are realistic for hasMany/belongsTo

* test: adding snapshot tests corresponding to bad output, waiting on platform test pass to update

* fix: rm duplicate snapshots for v2

* fix: change dart to null safety

* fix: test case 13 for flutter and ios

Co-authored-by: Zeyu Li <[email protected]>

Author: alharris-at

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

@@ -1279,8 +1279,6 @@ class _TagModelType extends ModelType<Tag> {
 class PostTags extends Model {
   static const classType = const _PostTagsModelType();
   final String id;
-  final String postID;
-  final String tagID;
   final Post post;
   final Tag tag;
 
@@ -1293,24 +1291,11 @@ class PostTags extends Model {
   }
 
   const PostTags._internal(
-      {@required this.id,
-      @required this.postID,
-      @required this.tagID,
-      @required this.post,
-      @required this.tag});
+      {@required this.id, @required this.post, @required this.tag});
 
-  factory PostTags(
-      {String id,
-      @required String postID,
-      @required String tagID,
-      @required Post post,
-      @required Tag tag}) {
+  factory PostTags({String id, @required Post post, @required Tag tag}) {
     return PostTags._internal(
-        id: id == null ? UUID.getUUID() : id,
-        postID: postID,
-        tagID: tagID,
-        post: post,
-        tag: tag);
+        id: id == null ? UUID.getUUID() : id, post: post, tag: tag);
   }
 
   bool equals(Object other) {
@@ -1322,8 +1307,6 @@ class PostTags extends Model {
     if (identical(other, this)) return true;
     return other is PostTags &&
         id == other.id &&
-        postID == other.postID &&
-        tagID == other.tagID &&
         post == other.post &&
         tag == other.tag;
   }
@@ -1337,47 +1320,31 @@ class PostTags extends Model {
 
     buffer.write(\\"PostTags {\\");
     buffer.write(\\"id=\\" + \\"$id\\" + \\", \\");
-    buffer.write(\\"postID=\\" + \\"$postID\\" + \\", \\");
-    buffer.write(\\"tagID=\\" + \\"$tagID\\" + \\", \\");
     buffer.write(\\"post=\\" + (post != null ? post.toString() : \\"null\\") + \\", \\");
     buffer.write(\\"tag=\\" + (tag != null ? tag.toString() : \\"null\\"));
     buffer.write(\\"}\\");
 
     return buffer.toString();
   }
 
-  PostTags copyWith(
-      {String id, String postID, String tagID, Post post, Tag tag}) {
+  PostTags copyWith({String id, Post post, Tag tag}) {
     return PostTags(
-        id: id ?? this.id,
-        postID: postID ?? this.postID,
-        tagID: tagID ?? this.tagID,
-        post: post ?? this.post,
-        tag: tag ?? this.tag);
+        id: id ?? this.id, post: post ?? this.post, tag: tag ?? this.tag);
   }
 
   PostTags.fromJson(Map<String, dynamic> json)
       : id = json['id'],
-        postID = json['postID'],
-        tagID = json['tagID'],
         post = json['post'] != null
             ? Post.fromJson(new Map<String, dynamic>.from(json['post']))
             : null,
         tag = json['tag'] != null
             ? Tag.fromJson(new Map<String, dynamic>.from(json['tag']))
             : null;
 
-  Map<String, dynamic> toJson() => {
-        'id': id,
-        'postID': postID,
-        'tagID': tagID,
-        'post': post?.toJson(),
-        'tag': tag?.toJson()
-      };
+  Map<String, dynamic> toJson() =>
+      {'id': id, 'post': post?.toJson(), 'tag': tag?.toJson()};
 
   static final QueryField ID = QueryField(fieldName: \\"postTags.id\\");
-  static final QueryField POSTID = QueryField(fieldName: \\"postID\\");
-  static final QueryField TAGID = QueryField(fieldName: \\"tagID\\");
   static final QueryField POST = QueryField(
       fieldName: \\"post\\",
       fieldType: ModelFieldType(ModelFieldTypeEnum.model,
@@ -1393,16 +1360,6 @@ class PostTags extends Model {
 
     modelSchemaDefinition.addField(ModelFieldDefinition.id());
 
-    modelSchemaDefinition.addField(ModelFieldDefinition.field(
-        key: PostTags.POSTID,
-        isRequired: true,
-        ofType: ModelFieldType(ModelFieldTypeEnum.string)));
-
-    modelSchemaDefinition.addField(ModelFieldDefinition.field(
-        key: PostTags.TAGID,
-        isRequired: true,
-        ofType: ModelFieldType(ModelFieldTypeEnum.string)));
-
     modelSchemaDefinition.addField(ModelFieldDefinition.belongsTo(
         key: PostTags.POST,
         isRequired: true,