fix: resolve flutter analyze warnings - unused imports, unnecessary casts
This commit is contained in:
@@ -2,7 +2,6 @@ import 'package:flutter/material.dart';
|
|||||||
import 'package:flutter_riverpod/flutter_riverpod.dart';
|
import 'package:flutter_riverpod/flutter_riverpod.dart';
|
||||||
import 'package:go_router/go_router.dart';
|
import 'package:go_router/go_router.dart';
|
||||||
import '../../../shared/services/api_client.dart';
|
import '../../../shared/services/api_client.dart';
|
||||||
import '../../../shared/providers/auth_provider.dart';
|
|
||||||
|
|
||||||
final clientsProvider = FutureProvider.autoDispose
|
final clientsProvider = FutureProvider.autoDispose
|
||||||
.family<List<Map<String, dynamic>>, String?>((ref, search) async {
|
.family<List<Map<String, dynamic>>, String?>((ref, search) async {
|
||||||
|
|||||||
@@ -1,6 +1,5 @@
|
|||||||
import 'package:flutter/material.dart';
|
import 'package:flutter/material.dart';
|
||||||
import 'package:flutter_riverpod/flutter_riverpod.dart';
|
import 'package:flutter_riverpod/flutter_riverpod.dart';
|
||||||
import 'package:go_router/go_router.dart';
|
|
||||||
import 'package:intl/intl.dart';
|
import 'package:intl/intl.dart';
|
||||||
import '../../../shared/services/api_client.dart';
|
import '../../../shared/services/api_client.dart';
|
||||||
|
|
||||||
|
|||||||
@@ -3,7 +3,6 @@ import 'package:flutter/material.dart';
|
|||||||
import 'package:flutter_test/flutter_test.dart';
|
import 'package:flutter_test/flutter_test.dart';
|
||||||
import 'package:flutter_riverpod/flutter_riverpod.dart';
|
import 'package:flutter_riverpod/flutter_riverpod.dart';
|
||||||
import 'package:network_app/features/auth/presentation/login_screen.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:network_app/shared/services/api_client.dart';
|
||||||
import 'package:mocktail/mocktail.dart';
|
import 'package:mocktail/mocktail.dart';
|
||||||
|
|
||||||
|
|||||||
@@ -3,7 +3,6 @@ import 'package:flutter/material.dart';
|
|||||||
import 'package:flutter_test/flutter_test.dart';
|
import 'package:flutter_test/flutter_test.dart';
|
||||||
import 'package:flutter_riverpod/flutter_riverpod.dart';
|
import 'package:flutter_riverpod/flutter_riverpod.dart';
|
||||||
import 'package:network_app/features/clients/presentation/clients_screen.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:network_app/shared/services/api_client.dart';
|
||||||
import 'package:mocktail/mocktail.dart';
|
import 'package:mocktail/mocktail.dart';
|
||||||
|
|
||||||
|
|||||||
@@ -1,4 +1,3 @@
|
|||||||
import 'package:flutter/material.dart';
|
|
||||||
import 'package:flutter_test/flutter_test.dart';
|
import 'package:flutter_test/flutter_test.dart';
|
||||||
|
|
||||||
void main() {
|
void main() {
|
||||||
@@ -53,7 +52,7 @@ void main() {
|
|||||||
'company': null,
|
'company': null,
|
||||||
};
|
};
|
||||||
|
|
||||||
final company = client['company'] as String?;
|
final company = client['company'];
|
||||||
expect(company, isNull);
|
expect(company, isNull);
|
||||||
});
|
});
|
||||||
|
|
||||||
@@ -64,7 +63,7 @@ void main() {
|
|||||||
'company': '',
|
'company': '',
|
||||||
};
|
};
|
||||||
|
|
||||||
final company = client['company'] as String?;
|
final company = client['company'];
|
||||||
final hasCompany = company != null && company.isNotEmpty;
|
final hasCompany = company != null && company.isNotEmpty;
|
||||||
expect(hasCompany, isFalse);
|
expect(hasCompany, isFalse);
|
||||||
});
|
});
|
||||||
|
|||||||
@@ -1,4 +1,3 @@
|
|||||||
import 'dart:async';
|
|
||||||
import 'package:flutter_test/flutter_test.dart';
|
import 'package:flutter_test/flutter_test.dart';
|
||||||
import 'package:flutter_riverpod/flutter_riverpod.dart';
|
import 'package:flutter_riverpod/flutter_riverpod.dart';
|
||||||
import 'package:network_app/shared/providers/auth_provider.dart';
|
import 'package:network_app/shared/providers/auth_provider.dart';
|
||||||
@@ -63,7 +62,7 @@ void main() {
|
|||||||
test('initial state checks session', () async {
|
test('initial state checks session', () async {
|
||||||
when(() => mockApiClient.getSession()).thenAnswer((_) async => null);
|
when(() => mockApiClient.getSession()).thenAnswer((_) async => null);
|
||||||
|
|
||||||
final notifier = container.read(authStateProvider.notifier);
|
final _ = container.read(authStateProvider.notifier);
|
||||||
|
|
||||||
// Wait for async initialization
|
// Wait for async initialization
|
||||||
await Future.delayed(Duration.zero);
|
await Future.delayed(Duration.zero);
|
||||||
@@ -101,7 +100,7 @@ void main() {
|
|||||||
'user': {'id': '1', 'email': 'test@test.com'},
|
'user': {'id': '1', 'email': 'test@test.com'},
|
||||||
});
|
});
|
||||||
|
|
||||||
final notifier = container.read(authStateProvider.notifier);
|
final _ = container.read(authStateProvider.notifier);
|
||||||
|
|
||||||
await Future.delayed(Duration.zero);
|
await Future.delayed(Duration.zero);
|
||||||
|
|
||||||
@@ -126,7 +125,7 @@ void main() {
|
|||||||
'user': {'id': '1', 'email': 'test@test.com', 'name': 'Test User'},
|
'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);
|
await Future.delayed(Duration.zero);
|
||||||
|
|
||||||
@@ -149,7 +148,7 @@ void main() {
|
|||||||
});
|
});
|
||||||
when(() => mockApiClient.signOut()).thenAnswer((_) async {});
|
when(() => mockApiClient.signOut()).thenAnswer((_) async {});
|
||||||
|
|
||||||
final notifier = container.read(authStateProvider.notifier);
|
final _ = container.read(authStateProvider.notifier);
|
||||||
|
|
||||||
await Future.delayed(const Duration(milliseconds: 100));
|
await Future.delayed(const Duration(milliseconds: 100));
|
||||||
|
|
||||||
@@ -169,7 +168,7 @@ void main() {
|
|||||||
password: any(named: 'password'),
|
password: any(named: 'password'),
|
||||||
)).thenThrow(Exception('Invalid credentials'));
|
)).thenThrow(Exception('Invalid credentials'));
|
||||||
|
|
||||||
final notifier = container.read(authStateProvider.notifier);
|
final _ = container.read(authStateProvider.notifier);
|
||||||
|
|
||||||
await Future.delayed(Duration.zero);
|
await Future.delayed(Duration.zero);
|
||||||
|
|
||||||
|
|||||||
@@ -1,6 +1,4 @@
|
|||||||
import 'package:flutter_test/flutter_test.dart';
|
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)
|
// Unit tests for API client logic (without actual HTTP calls)
|
||||||
void main() {
|
void main() {
|
||||||
@@ -100,7 +98,7 @@ void main() {
|
|||||||
test('query parameters include values when set', () {
|
test('query parameters include values when set', () {
|
||||||
final params = <String, dynamic>{};
|
final params = <String, dynamic>{};
|
||||||
|
|
||||||
const search = 'John';
|
const String? search = 'John';
|
||||||
const String? tag = null;
|
const String? tag = null;
|
||||||
|
|
||||||
if (search != null) params['search'] = search;
|
if (search != null) params['search'] = search;
|
||||||
|
|||||||
Reference in New Issue
Block a user