[BUG] [v2.3.1] Invalid copyWith override

[zs-dima]

v2.3.1 still has Invalid copyWith override issue

@freezed
class Union with _$Union {
  factory Union.first(int value) = _UnionFirst;
  factory Union.second(double? value) = _UnionSecond;
  factory Union.third(Syting value) = _UnionThird;
* * *

Error: The parameter 'text' of the method '_$$_SearchTextChangedCopyWith.call' has type 'String', which does not match the corresponding type, 'String?', in the overridden method, '$SearchEventCopyWith.call'.
lib/…/***_bloc.freezed.dart:239
Change to a supertype of 'String?', or, for a covariant parameter, a subtype.
  $Res call({String text});
                    ^
: Context: This is the overridden method ('call').
lib/…/***_bloc.freezed.dart:70
  $Res call({String? text});
       ^
: Error: The parameter 'text' of the method '_$$_SearchTextChangedCopyWith.call' has type 'String', which does not match the corresponding type, 'String?', in the overridden method, '$SearchEventCopyWith.call'.

related issues:
#814
#811

[rrousselGit]

Could you share a real example?

Your error doesn't match your example.

[fryette]

@rrousselGit
Seems I able to reproduce an issue with 2.3.1
@zs-dima example generate normal freezed file without any issues

But after some modifications...

@freezed
class Union with _$Union {
  factory Union.eventOne({
    required int count,
    required String? id,
    required String? name,
  }) = UnionOne;

  factory Union.eventTwo({
    required int? count,
    required String id,
    required String name,
  }) = UnionTwo;
}

It generate not compilable freezed.dart with incorrect override

abstract class _$$UnionTwoCopyWith<$Res> implements $UnionCopyWith<$Res> {
  factory _$$UnionTwoCopyWith(
          _$UnionTwo value, $Res Function(_$UnionTwo) then) =
      __$$UnionTwoCopyWithImpl<$Res>;
  @override
  @useResult
  $Res call({int? count, String id, String name}); // error here
}

_'_$$UnionTwoCopyWith.call' ('$Res Function({int? count, String id, String name})') isn't a valid override of '$UnionCopyWith.call' ('$Res Function({int count, String? id, String? name})')._

Also I still have some issues with nested models, but as far as I see it's related to the same issue that I mentioned above

[rrousselGit]

Fixed.

So many edge-cases... At least we're moving forward.