[unified_analytics] Deprecate flutterWasmDryRun (#2306)

Author: srujzs

pkgs/unified_analytics/CHANGELOG.md

@@ -1,3 +1,6 @@
+## 8.0.13
+- Deprecated `Event.flutterWasmDryRun` in favor of `Event.flutterWasmDryRunPackage`.
+
 ## 8.0.12
 - Added `Event.plugins` to track new analyzer plugins
 

pkgs/unified_analytics/lib/src/constants.dart

@@ -87,7 +87,7 @@ const int kMaxLogFileSize = 25 * (1 << 20);
 const String kLogFileName = 'dart-flutter-telemetry.log';
 
 /// The current version of the package, should be in line with pubspec version.
-const String kPackageVersion = '8.0.12';
+const String kPackageVersion = '8.0.13';
 
 /// The minimum length for a session.
 const int kSessionDurationMinutes = 30;

pkgs/unified_analytics/lib/src/enums.dart

@@ -103,6 +103,8 @@ enum DashEvent {
     description: 'Provides information about flutter commands that ran',
     toolOwner: DashTool.flutterTool,
   ),
+  @Deprecated("Use 'flutterWasmDryRunPackage' instead. This will be removed in "
+      'a future version.')
   flutterWasmDryRun(
     label: 'wasm_dry_run',
     description: 'Information for a dart2wasm dry run invoked from Flutter',

pkgs/unified_analytics/lib/src/event.dart

@@ -660,6 +660,8 @@ final class Event {
           },
         );
 
+  @Deprecated("Use 'flutterWasmDryRunPackage' instead. This will be removed in "
+      'a future version.')
   Event.flutterWasmDryRun({
     required String result,
     required int exitCode,

pkgs/unified_analytics/pubspec.yaml

@@ -5,7 +5,7 @@ description: >-
 # LINT.IfChange
 # When updating this, keep the version consistent with the changelog and the
 # value in lib/src/constants.dart.
-version: 8.0.12
+version: 8.0.13
 # LINT.ThenChange(lib/src/constants.dart)
 repository: https://github.com/dart-lang/tools/tree/main/pkgs/unified_analytics
 issue_tracker: https://github.com/dart-lang/tools/issues?q=is%3Aissue+is%3Aopen+label%3Apackage%3Aunified_analytics

pkgs/unified_analytics/test/event_test.dart

@@ -477,47 +477,6 @@ void main() {
     expect(constructedEvent.eventData.length, 4);
   });
 
-  test('Event.flutterWasmDryRun constructed no findings', () {
-    Event generateEventNoFindings() => Event.flutterWasmDryRun(
-          result: 'success',
-          exitCode: 123,
-        );
-
-    final constructedEvent1 = generateEventNoFindings();
-
-    expect(generateEventNoFindings, returnsNormally);
-    expect(constructedEvent1.eventName, DashEvent.flutterWasmDryRun);
-    expect(constructedEvent1.eventData['result'], 'success');
-    expect(constructedEvent1.eventData['exitCode'], 123);
-    expect(constructedEvent1.eventData.length, 2);
-  });
-  test('Event.flutterWasmDryRun constructed with findings', () {
-    Event generateEventNoFindings() => Event.flutterWasmDryRun(
-          result: 'success',
-          exitCode: 123,
-        );
-
-    final constructedEvent1 = generateEventNoFindings();
-
-    expect(generateEventNoFindings, returnsNormally);
-    expect(constructedEvent1.eventName, DashEvent.flutterWasmDryRun);
-    expect(constructedEvent1.eventData['result'], 'success');
-    expect(constructedEvent1.eventData['exitCode'], 123);
-    expect(constructedEvent1.eventData.length, 2);
-
-    Event generateEventFindings() => Event.flutterWasmDryRun(
-        result: 'success', exitCode: 123, findingsSummary: '1,2,3');
-
-    final constructedEvent2 = generateEventFindings();
-
-    expect(generateEventFindings, returnsNormally);
-    expect(constructedEvent2.eventName, DashEvent.flutterWasmDryRun);
-    expect(constructedEvent2.eventData['result'], 'success');
-    expect(constructedEvent2.eventData['exitCode'], 123);
-    expect(constructedEvent2.eventData['findings'], '1,2,3');
-    expect(constructedEvent2.eventData.length, 3);
-  });
-
   test('Event.flutterWasmDryRunPackage constructed', () {
     Event generateEvent() => Event.flutterWasmDryRunPackage(
             result: 'success',