From 89a73dae7eae0421d7e0f28cc46532200b621e37 Mon Sep 17 00:00:00 2001 From: Bretton Date: Tue, 2 Dec 2025 15:32:42 -0800 Subject: [PATCH] fix: resolve flutter analyze warnings MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - Remove unused imports in test files - Add assertion on unused variable in singleton test - Clean up redaction test file (remove unused mocks) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude --- .../coves_auth_service_redaction_test.dart | 21 ++----------------- .../coves_auth_service_singleton_test.dart | 1 + .../coves_auth_service_validation_test.dart | 1 - 3 files changed, 3 insertions(+), 20 deletions(-) diff --git a/test/services/coves_auth_service_redaction_test.dart b/test/services/coves_auth_service_redaction_test.dart index 553bb4a..4f8a790 100644 --- a/test/services/coves_auth_service_redaction_test.dart +++ b/test/services/coves_auth_service_redaction_test.dart @@ -1,32 +1,15 @@ import 'package:coves_flutter/models/coves_session.dart'; import 'package:coves_flutter/services/coves_auth_service.dart'; -import 'package:dio/dio.dart'; -import 'package:flutter_secure_storage/flutter_secure_storage.dart'; import 'package:flutter_test/flutter_test.dart'; -import 'package:mockito/annotations.dart'; -import 'package:mockito/mockito.dart'; - -import 'coves_auth_service_test.mocks.dart'; /// Tests for sensitive data redaction in CovesAuthService /// /// Verifies that sensitive parameters (tokens) are properly redacted /// from debug logs while preserving useful debugging information. -@GenerateMocks([Dio, FlutterSecureStorage]) void main() { - late CovesAuthService service; - late MockDio mockDio; - late MockFlutterSecureStorage mockStorage; - setUp(() { - mockDio = MockDio(); - mockStorage = MockFlutterSecureStorage(); - - // Create a test instance - service = CovesAuthService.createTestInstance( - dio: mockDio, - storage: mockStorage, - ); + // Reset singleton state before each test + CovesAuthService.resetInstance(); }); tearDown(() { diff --git a/test/services/coves_auth_service_singleton_test.dart b/test/services/coves_auth_service_singleton_test.dart index a443fd4..662a1c4 100644 --- a/test/services/coves_auth_service_singleton_test.dart +++ b/test/services/coves_auth_service_singleton_test.dart @@ -148,6 +148,7 @@ void main() { test('resetInstance() should clear the singleton', () { // Arrange final instance1 = CovesAuthService(dio: mockDio, storage: mockStorage); + expect(instance1, isNotNull); // Verify initial singleton exists // Act CovesAuthService.resetInstance(); diff --git a/test/services/coves_auth_service_validation_test.dart b/test/services/coves_auth_service_validation_test.dart index d8004ad..a027a96 100644 --- a/test/services/coves_auth_service_validation_test.dart +++ b/test/services/coves_auth_service_validation_test.dart @@ -3,7 +3,6 @@ import 'package:dio/dio.dart'; import 'package:flutter_secure_storage/flutter_secure_storage.dart'; import 'package:flutter_test/flutter_test.dart'; import 'package:mockito/annotations.dart'; -import 'package:mockito/mockito.dart'; import 'coves_auth_service_test.mocks.dart'; -- 2.51.2