chore(test): Update linter/style preferences (#409)

- Ensures `celest_lints` is used
- Applies latest linter/style preferences using dart fix
- Re-formats using Dart 3.8

Author: dnys1

.github/workflows/celest_test.yaml

@@ -0,0 +1,37 @@
+name: celest_test
+on:
+  pull_request:
+    paths:
+      - ".github/workflows/celest_test.yaml"
+      - "packages/celest_test/**"
+
+# Prevent duplicate runs due to Graphite
+# https://graphite.dev/docs/troubleshooting#why-are-my-actions-running-twice
+concurrency:
+  group: ${{ github.repository }}-${{ github.workflow }}-${{ github.ref }}-${{ github.ref == 'refs/heads/main' && github.sha || ''}}
+  cancel-in-progress: true
+
+permissions:
+  contents: read
+
+jobs:
+  analyze_and_format:
+    runs-on: ubuntu-latest
+    timeout-minutes: 10
+    steps:
+      - name: Git Checkout
+        uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # 4.2.2
+      - name: Setup Flutter
+        uses: subosito/flutter-action@e938fdf56512cc96ef2f93601a5a40bde3801046 # 2.19.0
+        with:
+          channel: beta
+          cache: true
+      - name: Get Packages
+        working-directory: packages/celest_test
+        run: dart pub upgrade
+      - name: Analyze
+        working-directory: packages/celest_test
+        run: dart analyze
+      - name: Format
+        working-directory: packages/celest_test
+        run: dart format --language-version=latest --set-exit-if-changed .

packages/celest_test/CHANGELOG.md

@@ -1,6 +1,8 @@
-## 0.1.1-wip
+## NEXT
 
 - chore: Update dependencies
+- chore: Re-format with Dart 3.8
+- chore: Update linter/style preferences
 
 ## 0.1.0
 

packages/celest_test/lib/celest_test.dart

@@ -19,13 +19,13 @@ final class CelestTester {
   }) async {
     _logger.fine('Connecting to Celest service: $wsUri');
 
-    var uri = Uri.parse(wsUri);
+    Uri uri = Uri.parse(wsUri);
     if (uri.host case '127.0.0.1' || 'localhost'
         when !kIsWeb && Platform.isAndroid) {
       uri = uri.replace(host: '10.0.2.2');
     }
 
-    final vmService = await vmServiceConnect(uri);
+    final VmService vmService = await vmServiceConnect(uri);
     await vmService.streamListen(EventStreams.kExtension);
 
     final tester = CelestTester._(vmService);
@@ -57,15 +57,12 @@ final class CelestAuthTester {
 
   /// OTP codes sent to users during sign up/sign in.
   Stream<Otp> get onSentOtp async* {
-    await for (final event in _vmService.onExtensionEvent) {
+    await for (final Event event in _vmService.onExtensionEvent) {
       if (event.extensionKind != 'celest.cloud_auth.emailOtpSent') {
         continue;
       }
-      final data = event.extensionData!.data;
-      yield (
-        to: data['to'] as String,
-        code: data['code'] as String,
-      );
+      final Map<String, dynamic> data = event.extensionData!.data;
+      yield (to: data['to'] as String, code: data['code'] as String);
     }
   }
 }

packages/celest_test/lib/src/vm_service_factory.dart

@@ -6,24 +6,21 @@ import 'package:web_socket/web_socket.dart';
 /// An implementation of the `vmServiceConnectUri` method from
 /// `package:vm_service/vm_service_io.dart` which works on Web as well as VM.
 Future<VmService> vmServiceConnect(Uri wsUri, {Log? log}) async {
-  final socket = await WebSocket.connect(wsUri);
+  final WebSocket socket = await WebSocket.connect(wsUri);
 
   final controller = StreamController<dynamic>();
   final streamClosedCompleter = Completer<void>();
 
-  socket.events.listen(
-    (event) {
-      switch (event) {
-        case TextDataReceived():
-          controller.add(event.text);
-        case BinaryDataReceived():
-          controller.add(event.data);
-        case CloseReceived():
-          controller.close();
-      }
-    },
-    onDone: streamClosedCompleter.complete,
-  );
+  socket.events.listen((event) {
+    switch (event) {
+      case TextDataReceived():
+        controller.add(event.text);
+      case BinaryDataReceived():
+        controller.add(event.data);
+      case CloseReceived():
+        controller.close();
+    }
+  }, onDone: streamClosedCompleter.complete);
 
   return VmService.defaultFactory(
     inStream: controller.stream,