fix(models): improve copyWith methods for AppReview, Engagement, and Report

- Update copyWith methods to use null-aware assignment for most fields
- Add missing fields to copyWith methods in Engagement
- Reorder parameters in Report.copyWith to match field order
- Update related tests to use ValueWrapper for wrapped fields

Author: fulleni

lib/src/models/user_generated_content/app_review.dart

@@ -106,20 +106,25 @@ class AppReview extends Equatable {
 
   /// Creates a copy of this [AppReview] with updated values.
   AppReview copyWith({
+    String? id,
+    String? userId,
     AppReviewFeedback? feedback,
+    DateTime? createdAt,
     DateTime? updatedAt,
     bool? wasStoreReviewRequested,
-    String? feedbackDetails,
+    ValueWrapper<String?>? feedbackDetails,
   }) {
     return AppReview(
-      id: id,
-      userId: userId,
+      id: id ?? this.id,
+      userId: userId ?? this.userId,
       feedback: feedback ?? this.feedback,
-      createdAt: createdAt,
+      createdAt: createdAt ?? this.createdAt,
       updatedAt: updatedAt ?? this.updatedAt,
       wasStoreReviewRequested:
           wasStoreReviewRequested ?? this.wasStoreReviewRequested,
-      feedbackDetails: feedbackDetails ?? this.feedbackDetails,
+      feedbackDetails: feedbackDetails != null
+          ? feedbackDetails.value
+          : this.feedbackDetails,
     );
   }
 }

lib/src/models/user_generated_content/engagement.dart

@@ -1,3 +1,4 @@
+import 'package:core/core.dart';
 import 'package:core/src/enums/engageable_type.dart';
 import 'package:core/src/models/user_generated_content/comment.dart';
 import 'package:core/src/models/user_generated_content/reaction.dart';
@@ -74,15 +75,23 @@ class Engagement extends Equatable {
   ];
 
   /// Creates a copy of this [Engagement] with updated values.
-  Engagement copyWith({Reaction? reaction, Comment? comment}) {
+  Engagement copyWith({
+    String? id,
+    String? userId,
+    String? entityId,
+    EngageableType? entityType,
+    Reaction? reaction,
+    ValueWrapper<Comment?>? comment,
+    DateTime? createdAt,
+  }) {
     return Engagement(
-      id: id,
-      userId: userId,
-      entityId: entityId,
-      entityType: entityType,
+      id: id ?? this.id,
+      userId: userId ?? this.userId,
+      entityId: entityId ?? this.entityId,
+      entityType: entityType ?? this.entityType,
       reaction: reaction ?? this.reaction,
-      comment: comment ?? this.comment,
-      createdAt: createdAt,
+      comment: comment != null ? comment.value : this.comment,
+      createdAt: createdAt ?? this.createdAt,
       updatedAt: DateTime.now(),
     );
   }

lib/src/models/user_generated_content/report.dart

@@ -105,12 +105,12 @@ class Report extends Equatable {
       reporterUserId: reporterUserId ?? this.reporterUserId,
       entityType: entityType ?? this.entityType,
       entityId: entityId ?? this.entityId,
-      reason: reason ?? this.reason,
       status: status ?? this.status,
       additionalComments: additionalComments != null
           ? additionalComments.value
           : this.additionalComments,
       createdAt: createdAt ?? this.createdAt,
+      reason: reason ?? this.reason,
     );
   }
 }

test/src/models/user_generated_content/app_review_test.dart

@@ -63,7 +63,7 @@ void main() {
 
         final updatedReview = negativeReviewWithReason.copyWith(
           updatedAt: newTimestamp,
-          feedbackDetails: newDetails,
+          feedbackDetails: const ValueWrapper(newDetails),
         );
 
         expect(updatedReview.updatedAt, newTimestamp);
@@ -74,6 +74,15 @@ void main() {
         expect(updatedReview.createdAt, negativeReviewWithReason.createdAt);
       });
 
+      test('allows setting feedbackDetails to null', () {
+        final updatedReview = negativeReviewWithReason.copyWith(
+          feedbackDetails: const ValueWrapper(null),
+        );
+
+        expect(updatedReview.feedbackDetails, isNull);
+        expect(updatedReview.id, negativeReviewWithReason.id);
+      });
+
       test('correctly uses initialFeedback in copyWith', () {
         final updatedReview = positiveReview.copyWith(
           feedback: AppReviewFeedback.negative,

test/src/models/user_generated_content/engagement_test.dart

@@ -68,6 +68,32 @@ void main() {
         );
       });
 
+      test('returns a new instance with an updated comment', () {
+        final newComment = Comment(
+          language: languagesFixturesData.first,
+          content: 'This is a new comment.',
+          status: ModerationStatus.resolved,
+        );
+
+        final updatedEngagement = engagementFixture.copyWith(
+          comment: ValueWrapper(newComment),
+        );
+
+        expect(updatedEngagement.comment, newComment);
+        expect(updatedEngagement.id, engagementFixture.id);
+      });
+
+      test('returns a new instance with a null comment', () {
+        // Start with a fixture that has a comment.
+        final updatedEngagement = engagementFixture.copyWith(
+          comment: const ValueWrapper(null),
+        );
+
+        expect(updatedEngagement.comment, isNull);
+        // Verify other fields remain unchanged
+        expect(updatedEngagement.id, engagementFixture.id);
+      });
+
       test(
         'returns a new instance with a new timestamp if no updates provided',
         () {