From 75f8e30707a023f16246d25489b858d801dd6fd6 Mon Sep 17 00:00:00 2001 From: Hammer Date: Fri, 30 Jan 2026 04:39:26 +0000 Subject: [PATCH] fix: resolve flutter analyze warnings - unused imports, unnecessary casts --- lib/features/clients/presentation/clients_screen.dart | 1 - lib/features/emails/presentation/emails_screen.dart | 1 - test/features/auth/login_screen_test.dart | 1 - test/features/clients/clients_screen_test.dart | 1 - test/features/clients/clients_test.dart | 5 ++--- test/shared/providers/auth_provider_test.dart | 11 +++++------ test/shared/services/api_client_test.dart | 4 +--- 7 files changed, 8 insertions(+), 16 deletions(-) diff --git a/lib/features/clients/presentation/clients_screen.dart b/lib/features/clients/presentation/clients_screen.dart index 89945ca..329a4f7 100644 --- a/lib/features/clients/presentation/clients_screen.dart +++ b/lib/features/clients/presentation/clients_screen.dart @@ -2,7 +2,6 @@ import 'package:flutter/material.dart'; import 'package:flutter_riverpod/flutter_riverpod.dart'; import 'package:go_router/go_router.dart'; import '../../../shared/services/api_client.dart'; -import '../../../shared/providers/auth_provider.dart'; final clientsProvider = FutureProvider.autoDispose .family>, String?>((ref, search) async { diff --git a/lib/features/emails/presentation/emails_screen.dart b/lib/features/emails/presentation/emails_screen.dart index 9137599..e56cbec 100644 --- a/lib/features/emails/presentation/emails_screen.dart +++ b/lib/features/emails/presentation/emails_screen.dart @@ -1,6 +1,5 @@ import 'package:flutter/material.dart'; import 'package:flutter_riverpod/flutter_riverpod.dart'; -import 'package:go_router/go_router.dart'; import 'package:intl/intl.dart'; import '../../../shared/services/api_client.dart'; diff --git a/test/features/auth/login_screen_test.dart b/test/features/auth/login_screen_test.dart index 50eb45c..7dd1687 100644 --- a/test/features/auth/login_screen_test.dart +++ b/test/features/auth/login_screen_test.dart @@ -3,7 +3,6 @@ import 'package:flutter/material.dart'; import 'package:flutter_test/flutter_test.dart'; import 'package:flutter_riverpod/flutter_riverpod.dart'; import 'package:network_app/features/auth/presentation/login_screen.dart'; -import 'package:network_app/shared/providers/auth_provider.dart'; import 'package:network_app/shared/services/api_client.dart'; import 'package:mocktail/mocktail.dart'; diff --git a/test/features/clients/clients_screen_test.dart b/test/features/clients/clients_screen_test.dart index 2d31adc..17cc8e0 100644 --- a/test/features/clients/clients_screen_test.dart +++ b/test/features/clients/clients_screen_test.dart @@ -3,7 +3,6 @@ import 'package:flutter/material.dart'; import 'package:flutter_test/flutter_test.dart'; import 'package:flutter_riverpod/flutter_riverpod.dart'; import 'package:network_app/features/clients/presentation/clients_screen.dart'; -import 'package:network_app/shared/providers/auth_provider.dart'; import 'package:network_app/shared/services/api_client.dart'; import 'package:mocktail/mocktail.dart'; diff --git a/test/features/clients/clients_test.dart b/test/features/clients/clients_test.dart index a1ebff2..fddf8ac 100644 --- a/test/features/clients/clients_test.dart +++ b/test/features/clients/clients_test.dart @@ -1,4 +1,3 @@ -import 'package:flutter/material.dart'; import 'package:flutter_test/flutter_test.dart'; void main() { @@ -53,7 +52,7 @@ void main() { 'company': null, }; - final company = client['company'] as String?; + final company = client['company']; expect(company, isNull); }); @@ -64,7 +63,7 @@ void main() { 'company': '', }; - final company = client['company'] as String?; + final company = client['company']; final hasCompany = company != null && company.isNotEmpty; expect(hasCompany, isFalse); }); diff --git a/test/shared/providers/auth_provider_test.dart b/test/shared/providers/auth_provider_test.dart index 5e6c08b..bbf4a95 100644 --- a/test/shared/providers/auth_provider_test.dart +++ b/test/shared/providers/auth_provider_test.dart @@ -1,4 +1,3 @@ -import 'dart:async'; import 'package:flutter_test/flutter_test.dart'; import 'package:flutter_riverpod/flutter_riverpod.dart'; import 'package:network_app/shared/providers/auth_provider.dart'; @@ -63,7 +62,7 @@ void main() { test('initial state checks session', () async { when(() => mockApiClient.getSession()).thenAnswer((_) async => null); - final notifier = container.read(authStateProvider.notifier); + final _ = container.read(authStateProvider.notifier); // Wait for async initialization await Future.delayed(Duration.zero); @@ -101,7 +100,7 @@ void main() { 'user': {'id': '1', 'email': 'test@test.com'}, }); - final notifier = container.read(authStateProvider.notifier); + final _ = container.read(authStateProvider.notifier); await Future.delayed(Duration.zero); @@ -126,7 +125,7 @@ void main() { 'user': {'id': '1', 'email': 'test@test.com', 'name': 'Test User'}, }); - final notifier = container.read(authStateProvider.notifier); + final _ = container.read(authStateProvider.notifier); await Future.delayed(Duration.zero); @@ -149,7 +148,7 @@ void main() { }); when(() => mockApiClient.signOut()).thenAnswer((_) async {}); - final notifier = container.read(authStateProvider.notifier); + final _ = container.read(authStateProvider.notifier); await Future.delayed(const Duration(milliseconds: 100)); @@ -169,7 +168,7 @@ void main() { password: any(named: 'password'), )).thenThrow(Exception('Invalid credentials')); - final notifier = container.read(authStateProvider.notifier); + final _ = container.read(authStateProvider.notifier); await Future.delayed(Duration.zero); diff --git a/test/shared/services/api_client_test.dart b/test/shared/services/api_client_test.dart index 03fee98..33421e8 100644 --- a/test/shared/services/api_client_test.dart +++ b/test/shared/services/api_client_test.dart @@ -1,6 +1,4 @@ import 'package:flutter_test/flutter_test.dart'; -import 'package:dio/dio.dart'; -import 'package:mocktail/mocktail.dart'; // Unit tests for API client logic (without actual HTTP calls) void main() { @@ -100,7 +98,7 @@ void main() { test('query parameters include values when set', () { final params = {}; - const search = 'John'; + const String? search = 'John'; const String? tag = null; if (search != null) params['search'] = search;