refactor: changed toggle to next at multipliers

pull/231/head
RuiAlonso 3 years ago
parent 8d17ee8e67
commit db177b1afe

@ -19,7 +19,7 @@ class MultipliersBehavior extends Component
final multipliers = parent.children.whereType<Multiplier>(); final multipliers = parent.children.whereType<Multiplier>();
for (final multiplier in multipliers) { for (final multiplier in multipliers) {
multiplier.bloc.toggle(state.multiplier); multiplier.bloc.next(state.multiplier);
} }
} }
}); });

@ -16,7 +16,7 @@ class MultiplierCubit extends Cubit<MultiplierState> {
); );
/// Event added when the game current multiplier changes. /// Event added when the game current multiplier changes.
void toggle(int multiplier) { void next(int multiplier) {
if (state.equalsTo(multiplier)) { if (state.equalsTo(multiplier)) {
if (state.spriteState == MultiplierSpriteState.dimmed) { if (state.spriteState == MultiplierSpriteState.dimmed) {
emit(state.copyWith(spriteState: MultiplierSpriteState.lit)); emit(state.copyWith(spriteState: MultiplierSpriteState.lit));

@ -86,7 +86,7 @@ class MultipliersGame extends BallGame with KeyboardEvents {
} }
for (final multiplier in multipliers) { for (final multiplier in multipliers) {
multiplier.bloc.toggle(currentMultiplier); multiplier.bloc.next(currentMultiplier);
} }
return KeyEventResult.handled; return KeyEventResult.handled;

@ -9,9 +9,9 @@ void main() {
'MultiplierCubit', 'MultiplierCubit',
() { () {
blocTest<MultiplierCubit, MultiplierState>( blocTest<MultiplierCubit, MultiplierState>(
'emits [lit] when toggle dimmed with same multiplier value', "emits [lit] when 'next' on dimmed with same multiplier value",
build: () => MultiplierCubit(MultiplierValue.x2), build: () => MultiplierCubit(MultiplierValue.x2),
act: (bloc) => bloc.toggle(2), act: (bloc) => bloc.next(2),
expect: () => [ expect: () => [
isA<MultiplierState>() isA<MultiplierState>()
..having( ..having(
@ -23,13 +23,13 @@ void main() {
); );
blocTest<MultiplierCubit, MultiplierState>( blocTest<MultiplierCubit, MultiplierState>(
'emits [dimmed] when toggle lit with different multiplier value', "emits [dimmed] when 'next' on lit with different multiplier value",
build: () => MultiplierCubit(MultiplierValue.x2), build: () => MultiplierCubit(MultiplierValue.x2),
seed: () => MultiplierState( seed: () => MultiplierState(
value: MultiplierValue.x2, value: MultiplierValue.x2,
spriteState: MultiplierSpriteState.lit, spriteState: MultiplierSpriteState.lit,
), ),
act: (bloc) => bloc.toggle(3), act: (bloc) => bloc.next(3),
expect: () => [ expect: () => [
isA<MultiplierState>() isA<MultiplierState>()
..having( ..having(
@ -41,20 +41,20 @@ void main() {
); );
blocTest<MultiplierCubit, MultiplierState>( blocTest<MultiplierCubit, MultiplierState>(
'emits nothing when toggle lit with same multiplier value', "emits nothing when 'next' on lit with same multiplier value",
build: () => MultiplierCubit(MultiplierValue.x2), build: () => MultiplierCubit(MultiplierValue.x2),
seed: () => MultiplierState( seed: () => MultiplierState(
value: MultiplierValue.x2, value: MultiplierValue.x2,
spriteState: MultiplierSpriteState.lit, spriteState: MultiplierSpriteState.lit,
), ),
act: (bloc) => bloc.toggle(2), act: (bloc) => bloc.next(2),
expect: () => <MultiplierState>[], expect: () => <MultiplierState>[],
); );
blocTest<MultiplierCubit, MultiplierState>( blocTest<MultiplierCubit, MultiplierState>(
'emits nothing when toggle dimmed with different multiplier value', "emits nothing when 'next' on dimmed with different multiplier value",
build: () => MultiplierCubit(MultiplierValue.x2), build: () => MultiplierCubit(MultiplierValue.x2),
act: (bloc) => bloc.toggle(3), act: (bloc) => bloc.next(3),
expect: () => <MultiplierState>[], expect: () => <MultiplierState>[],
); );
}, },

@ -47,7 +47,7 @@ void main() {
); );
flameBlocTester.testGameWidget( flameBlocTester.testGameWidget(
'calls toggle once per each multiplier when GameBloc emit state', "calls 'next' once per each multiplier when GameBloc emit state",
setUp: (game, tester) async { setUp: (game, tester) async {
final multiplierCubit = MockMultiplierCubit(); final multiplierCubit = MockMultiplierCubit();
final behavior = MultipliersBehavior(); final behavior = MultipliersBehavior();
@ -85,7 +85,7 @@ void main() {
for (final multiplier in multipliers) { for (final multiplier in multipliers) {
verify( verify(
() => multiplier.bloc.toggle(any()), () => multiplier.bloc.next(any()),
).called(1); ).called(1);
} }
}, },

Loading…
Cancel
Save