refactor: create separate BLoCs for each screen with comprehensive tests
- Created 8 separate BLoCs (Home, Library, BookDetails, AddBook, Scanner, Categories, Wishlist, Settings) - Each BLoC has its own event, state, and bloc files - Added 70 comprehensive tests covering all BLoC functionality - All tests passing (70/70) - Fixed linting issues and updated deprecated APIs - Improved code organization and maintainability
This commit is contained in:
22
bookshelf_flutter/lib/bloc/home/home_bloc.dart
Normal file
22
bookshelf_flutter/lib/bloc/home/home_bloc.dart
Normal file
@@ -0,0 +1,22 @@
|
||||
import 'package:flutter_bloc/flutter_bloc.dart';
|
||||
import '../../models/models.dart';
|
||||
import 'home_event.dart';
|
||||
import 'home_state.dart';
|
||||
|
||||
class HomeBloc extends Bloc<HomeEvent, HomeState> {
|
||||
HomeBloc() : super(HomeState.initial()) {
|
||||
on<LoadHomeData>(_onLoadHomeData);
|
||||
on<NavigateToScreen>(_onNavigateToScreen);
|
||||
}
|
||||
|
||||
void _onLoadHomeData(LoadHomeData event, Emitter<HomeState> emit) {
|
||||
emit(state.copyWith(
|
||||
currentScreen: AppScreen.library,
|
||||
isLoading: false,
|
||||
));
|
||||
}
|
||||
|
||||
void _onNavigateToScreen(NavigateToScreen event, Emitter<HomeState> emit) {
|
||||
emit(state.copyWith(currentScreen: event.screen));
|
||||
}
|
||||
}
|
||||
14
bookshelf_flutter/lib/bloc/home/home_event.dart
Normal file
14
bookshelf_flutter/lib/bloc/home/home_event.dart
Normal file
@@ -0,0 +1,14 @@
|
||||
import '../../models/models.dart';
|
||||
|
||||
abstract class HomeEvent {
|
||||
const HomeEvent();
|
||||
}
|
||||
|
||||
class LoadHomeData extends HomeEvent {
|
||||
const LoadHomeData();
|
||||
}
|
||||
|
||||
class NavigateToScreen extends HomeEvent {
|
||||
final AppScreen screen;
|
||||
const NavigateToScreen(this.screen);
|
||||
}
|
||||
39
bookshelf_flutter/lib/bloc/home/home_state.dart
Normal file
39
bookshelf_flutter/lib/bloc/home/home_state.dart
Normal file
@@ -0,0 +1,39 @@
|
||||
import 'package:equatable/equatable.dart';
|
||||
import '../../models/models.dart';
|
||||
|
||||
class HomeState extends Equatable {
|
||||
final AppScreen currentScreen;
|
||||
final bool isLoading;
|
||||
final String? errorMessage;
|
||||
|
||||
const HomeState({
|
||||
this.currentScreen = AppScreen.library,
|
||||
this.isLoading = false,
|
||||
this.errorMessage,
|
||||
});
|
||||
|
||||
factory HomeState.initial() {
|
||||
return const HomeState(
|
||||
isLoading: true,
|
||||
);
|
||||
}
|
||||
|
||||
HomeState copyWith({
|
||||
AppScreen? currentScreen,
|
||||
bool? isLoading,
|
||||
String? errorMessage,
|
||||
}) {
|
||||
return HomeState(
|
||||
currentScreen: currentScreen ?? this.currentScreen,
|
||||
isLoading: isLoading ?? this.isLoading,
|
||||
errorMessage: errorMessage ?? this.errorMessage,
|
||||
);
|
||||
}
|
||||
|
||||
@override
|
||||
List<Object?> get props => [
|
||||
currentScreen,
|
||||
isLoading,
|
||||
errorMessage,
|
||||
];
|
||||
}
|
||||
Reference in New Issue
Block a user