chore: upgrade dio to 5.0.0 (#32)

* chore: upgrade dio to 5.0.0

* add test randomization seed

* update semantic PR workflow

* upgrade dependencies, run codegen

* fix tests that rely on specific dio messaging

* update version of test_track in pubspec

.github/workflows/semantic-pull-request.yml

@@ -1,12 +1,26 @@
 name: Semantic PR
 
-on: [pull_request_target]
+on: [pull_request]
+
+permissions:
+  pull-requests: read
 
 jobs:
-  validate_title:
+  build:
     runs-on: ubuntu-latest
     steps:
       - name: Validate PR title
-        uses: amannn/action-semantic-pull-request@v4
+        uses: amannn/[email protected]
+        with:
+          types:
+            feat
+            fix
+            docs
+            style
+            refactor
+            test
+            ci
+            chore
+          validateSingleCommit: true
         env:
-          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+          GITHUB_TOKEN: ${{ github.token }}

.github/workflows/validate.yml

@@ -24,8 +24,8 @@ jobs:
         working-directory: packages/${{ inputs.package-name }}
 
     steps:
-      - uses: actions/checkout@v2.3.5
-      - uses: dart-lang/setup-dart@v1.3
+      - uses: actions/checkout@v3.3.0
+      - uses: dart-lang/setup-dart@v1.4
 
       - name: Install dependencies
         run: dart pub get
@@ -60,8 +60,8 @@ jobs:
           working-directory: packages/${{ inputs.package-name }}
 
       steps:
-        - uses: actions/checkout@v2.3.5
-        - uses: dart-lang/setup-dart@v1.3
+        - uses: actions/checkout@v3.3.0
+        - uses: dart-lang/setup-dart@v1.4
 
         - name: Install dependencies
           run: |

packages/test_track/lib/src/domain/vary/run_ab.dart

@@ -37,11 +37,11 @@ class RunAb {
       );
     }
 
-    final _trueVariant = trueVariant ?? 'true';
+    trueVariant ??= 'true';
     final unsortedVariants = List.of(split.variants);
     final sortedFalseVariants = unsortedVariants
       ..sort((v1, v2) => v1.name.compareTo(v2.name))
-      ..removeWhere((v) => v.name == _trueVariant);
+      ..removeWhere((v) => v.name == trueVariant);
 
     final falseVariant = sortedFalseVariants.firstOrNull?.name;
 
@@ -53,7 +53,7 @@ class RunAb {
     );
 
     return RunAbResult(
-      _trueVariant == runVaryResult.variant,
+      trueVariant == runVaryResult.variant,
       runVaryResult.visitor,
     );
   }

packages/test_track/lib/src/models/app_version_build.freezed.dart

@@ -1,7 +1,7 @@
 // coverage:ignore-file
 // GENERATED CODE - DO NOT MODIFY BY HAND
 // ignore_for_file: type=lint
-// ignore_for_file: unused_element, deprecated_member_use, deprecated_member_use_from_same_package, use_function_type_syntax_for_parameters, unnecessary_const, avoid_init_to_null, invalid_override_different_default_values_named, prefer_expression_function_bodies, annotate_overrides, invalid_annotation_target
+// ignore_for_file: unused_element, deprecated_member_use, deprecated_member_use_from_same_package, use_function_type_syntax_for_parameters, unnecessary_const, avoid_init_to_null, invalid_override_different_default_values_named, prefer_expression_function_bodies, annotate_overrides, invalid_annotation_target, unnecessary_question_mark
 
 part of 'app_version_build.dart';
 
@@ -34,39 +34,42 @@ mixin _$AppVersionBuild {
 abstract class $AppVersionBuildCopyWith<$Res> {
   factory $AppVersionBuildCopyWith(
           AppVersionBuild value, $Res Function(AppVersionBuild) then) =
-      _$AppVersionBuildCopyWithImpl<$Res>;
+      _$AppVersionBuildCopyWithImpl<$Res, AppVersionBuild>;
+  @useResult
   $Res call({String appName, String version, String buildTimestamp});
 }
 
 /// @nodoc
-class _$AppVersionBuildCopyWithImpl<$Res>
+class _$AppVersionBuildCopyWithImpl<$Res, $Val extends AppVersionBuild>
     implements $AppVersionBuildCopyWith<$Res> {
   _$AppVersionBuildCopyWithImpl(this._value, this._then);
 
-  final AppVersionBuild _value;
   // ignore: unused_field
-  final $Res Function(AppVersionBuild) _then;
+  final $Val _value;
+  // ignore: unused_field
+  final $Res Function($Val) _then;
 
+  @pragma('vm:prefer-inline')
   @override
   $Res call({
-    Object? appName = freezed,
-    Object? version = freezed,
-    Object? buildTimestamp = freezed,
+    Object? appName = null,
+    Object? version = null,
+    Object? buildTimestamp = null,
   }) {
     return _then(_value.copyWith(
-      appName: appName == freezed
+      appName: null == appName
           ? _value.appName
           : appName // ignore: cast_nullable_to_non_nullable
               as String,
-      version: version == freezed
+      version: null == version
           ? _value.version
           : version // ignore: cast_nullable_to_non_nullable
               as String,
-      buildTimestamp: buildTimestamp == freezed
+      buildTimestamp: null == buildTimestamp
           ? _value.buildTimestamp
           : buildTimestamp // ignore: cast_nullable_to_non_nullable
               as String,
-    ));
+    ) as $Val);
   }
 }
 
@@ -77,36 +80,35 @@ abstract class _$$_AppVersionBuildCopyWith<$Res>
           _$_AppVersionBuild value, $Res Function(_$_AppVersionBuild) then) =
       __$$_AppVersionBuildCopyWithImpl<$Res>;
   @override
+  @useResult
   $Res call({String appName, String version, String buildTimestamp});
 }
 
 /// @nodoc
 class __$$_AppVersionBuildCopyWithImpl<$Res>
-    extends _$AppVersionBuildCopyWithImpl<$Res>
+    extends _$AppVersionBuildCopyWithImpl<$Res, _$_AppVersionBuild>
     implements _$$_AppVersionBuildCopyWith<$Res> {
   __$$_AppVersionBuildCopyWithImpl(
       _$_AppVersionBuild _value, $Res Function(_$_AppVersionBuild) _then)
-      : super(_value, (v) => _then(v as _$_AppVersionBuild));
-
-  @override
-  _$_AppVersionBuild get _value => super._value as _$_AppVersionBuild;
+      : super(_value, _then);
 
+  @pragma('vm:prefer-inline')
   @override
   $Res call({
-    Object? appName = freezed,
-    Object? version = freezed,
-    Object? buildTimestamp = freezed,
+    Object? appName = null,
+    Object? version = null,
+    Object? buildTimestamp = null,
   }) {
     return _then(_$_AppVersionBuild(
-      appName: appName == freezed
+      appName: null == appName
           ? _value.appName
           : appName // ignore: cast_nullable_to_non_nullable
               as String,
-      version: version == freezed
+      version: null == version
           ? _value.version
           : version // ignore: cast_nullable_to_non_nullable
               as String,
-      buildTimestamp: buildTimestamp == freezed
+      buildTimestamp: null == buildTimestamp
           ? _value.buildTimestamp
           : buildTimestamp // ignore: cast_nullable_to_non_nullable
               as String,
@@ -143,22 +145,20 @@ class _$_AppVersionBuild implements _AppVersionBuild {
     return identical(this, other) ||
         (other.runtimeType == runtimeType &&
             other is _$_AppVersionBuild &&
-            const DeepCollectionEquality().equals(other.appName, appName) &&
-            const DeepCollectionEquality().equals(other.version, version) &&
-            const DeepCollectionEquality()
-                .equals(other.buildTimestamp, buildTimestamp));
+            (identical(other.appName, appName) || other.appName == appName) &&
+            (identical(other.version, version) || other.version == version) &&
+            (identical(other.buildTimestamp, buildTimestamp) ||
+                other.buildTimestamp == buildTimestamp));
   }
 
   @JsonKey(ignore: true)
   @override
-  int get hashCode => Object.hash(
-      runtimeType,
-      const DeepCollectionEquality().hash(appName),
-      const DeepCollectionEquality().hash(version),
-      const DeepCollectionEquality().hash(buildTimestamp));
+  int get hashCode =>
+      Object.hash(runtimeType, appName, version, buildTimestamp);
 
   @JsonKey(ignore: true)
   @override
+  @pragma('vm:prefer-inline')
   _$$_AppVersionBuildCopyWith<_$_AppVersionBuild> get copyWith =>
       __$$_AppVersionBuildCopyWithImpl<_$_AppVersionBuild>(this, _$identity);
 

packages/test_track/lib/src/models/app_visitor_config.freezed.dart

@@ -1,7 +1,7 @@
 // coverage:ignore-file
 // GENERATED CODE - DO NOT MODIFY BY HAND
 // ignore_for_file: type=lint
-// ignore_for_file: unused_element, deprecated_member_use, deprecated_member_use_from_same_package, use_function_type_syntax_for_parameters, unnecessary_const, avoid_init_to_null, invalid_override_different_default_values_named, prefer_expression_function_bodies, annotate_overrides, invalid_annotation_target
+// ignore_for_file: unused_element, deprecated_member_use, deprecated_member_use_from_same_package, use_function_type_syntax_for_parameters, unnecessary_const, avoid_init_to_null, invalid_override_different_default_values_named, prefer_expression_function_bodies, annotate_overrides, invalid_annotation_target, unnecessary_question_mark
 
 part of 'app_visitor_config.dart';
 
@@ -33,42 +33,46 @@ mixin _$AppVisitorConfig {
 abstract class $AppVisitorConfigCopyWith<$Res> {
   factory $AppVisitorConfigCopyWith(
           AppVisitorConfig value, $Res Function(AppVisitorConfig) then) =
-      _$AppVisitorConfigCopyWithImpl<$Res>;
+      _$AppVisitorConfigCopyWithImpl<$Res, AppVisitorConfig>;
+  @useResult
   $Res call({List<Split> splits, Visitor visitor});
 
   $VisitorCopyWith<$Res> get visitor;
 }
 
 /// @nodoc
-class _$AppVisitorConfigCopyWithImpl<$Res>
+class _$AppVisitorConfigCopyWithImpl<$Res, $Val extends AppVisitorConfig>
     implements $AppVisitorConfigCopyWith<$Res> {
   _$AppVisitorConfigCopyWithImpl(this._value, this._then);
 
-  final AppVisitorConfig _value;
   // ignore: unused_field
-  final $Res Function(AppVisitorConfig) _then;
+  final $Val _value;
+  // ignore: unused_field
+  final $Res Function($Val) _then;
 
+  @pragma('vm:prefer-inline')
   @override
   $Res call({
-    Object? splits = freezed,
-    Object? visitor = freezed,
+    Object? splits = null,
+    Object? visitor = null,
   }) {
     return _then(_value.copyWith(
-      splits: splits == freezed
+      splits: null == splits
           ? _value.splits
           : splits // ignore: cast_nullable_to_non_nullable
               as List<Split>,
-      visitor: visitor == freezed
+      visitor: null == visitor
           ? _value.visitor
           : visitor // ignore: cast_nullable_to_non_nullable
               as Visitor,
-    ));
+    ) as $Val);
   }
 
   @override
+  @pragma('vm:prefer-inline')
   $VisitorCopyWith<$Res> get visitor {
     return $VisitorCopyWith<$Res>(_value.visitor, (value) {
-      return _then(_value.copyWith(visitor: value));
+      return _then(_value.copyWith(visitor: value) as $Val);
     });
   }
 }
@@ -80,6 +84,7 @@ abstract class _$$_AppVisitorConfigCopyWith<$Res>
           _$_AppVisitorConfig value, $Res Function(_$_AppVisitorConfig) then) =
       __$$_AppVisitorConfigCopyWithImpl<$Res>;
   @override
+  @useResult
   $Res call({List<Split> splits, Visitor visitor});
 
   @override
@@ -88,26 +93,24 @@ abstract class _$$_AppVisitorConfigCopyWith<$Res>
 
 /// @nodoc
 class __$$_AppVisitorConfigCopyWithImpl<$Res>
-    extends _$AppVisitorConfigCopyWithImpl<$Res>
+    extends _$AppVisitorConfigCopyWithImpl<$Res, _$_AppVisitorConfig>
     implements _$$_AppVisitorConfigCopyWith<$Res> {
   __$$_AppVisitorConfigCopyWithImpl(
       _$_AppVisitorConfig _value, $Res Function(_$_AppVisitorConfig) _then)
-      : super(_value, (v) => _then(v as _$_AppVisitorConfig));
-
-  @override
-  _$_AppVisitorConfig get _value => super._value as _$_AppVisitorConfig;
+      : super(_value, _then);
 
+  @pragma('vm:prefer-inline')
   @override
   $Res call({
-    Object? splits = freezed,
-    Object? visitor = freezed,
+    Object? splits = null,
+    Object? visitor = null,
   }) {
     return _then(_$_AppVisitorConfig(
-      splits: splits == freezed
+      splits: null == splits
           ? _value._splits
           : splits // ignore: cast_nullable_to_non_nullable
               as List<Split>,
-      visitor: visitor == freezed
+      visitor: null == visitor
           ? _value.visitor
           : visitor // ignore: cast_nullable_to_non_nullable
               as Visitor,
@@ -128,6 +131,7 @@ class _$_AppVisitorConfig implements _AppVisitorConfig {
   final List<Split> _splits;
   @override
   List<Split> get splits {
+    if (_splits is EqualUnmodifiableListView) return _splits;
     // ignore: implicit_dynamic_type
     return EqualUnmodifiableListView(_splits);
   }
@@ -146,18 +150,17 @@ class _$_AppVisitorConfig implements _AppVisitorConfig {
         (other.runtimeType == runtimeType &&
             other is _$_AppVisitorConfig &&
             const DeepCollectionEquality().equals(other._splits, _splits) &&
-            const DeepCollectionEquality().equals(other.visitor, visitor));
+            (identical(other.visitor, visitor) || other.visitor == visitor));
   }
 
   @JsonKey(ignore: true)
   @override
   int get hashCode => Object.hash(
-      runtimeType,
-      const DeepCollectionEquality().hash(_splits),
-      const DeepCollectionEquality().hash(visitor));
+      runtimeType, const DeepCollectionEquality().hash(_splits), visitor);
 
   @JsonKey(ignore: true)
   @override
+  @pragma('vm:prefer-inline')
   _$$_AppVisitorConfigCopyWith<_$_AppVisitorConfig> get copyWith =>
       __$$_AppVisitorConfigCopyWithImpl<_$_AppVisitorConfig>(this, _$identity);