fix: expected: [<States>], actual [] when unit testing landing page cubit about bloc

[fernan542]

Adding a method in constructor makes things harder to test. The suggested way is to call it via initializer or other way.
Can you provide an accessible repo to inspect it more? Thanks!

[Jeastwood1995]

I've updated it to the following, the handleRequest is now a public method and this will only fire when not instantiated via the test:

blocTest<ConfirmEmailCubit, ConfirmEmailState>(
      'Confirm Email with no params',
      build: () => ConfirmEmailCubit(
        mockNavigatorService,
        {},
        mockLogger,
        testMode: true,
      ),
      seed: () => ConfirmEmailLoading(),
      expect: () => <ConfirmEmailState>[
        ConfirmEmailError(S.current.no_direct_access),
      ],
      act: (ConfirmEmailCubit cubit) => cubit.handleResponse(),
    );

class ConfirmEmailCubit extends Cubit<ConfirmEmailState> {
  final CustomNavigatorService _customNavigatorService;
  final Map<String, dynamic> _queryParams;
  final Logger _logger;
  final bool testMode;

  ConfirmEmailCubit(
    this._customNavigatorService,
    this._queryParams,
    this._logger, {
    this.testMode = false,
  }) : super(ConfirmEmailLoading()) {
    if (!testMode) {
      handleResponse();
    }
  }

  void handleResponse() {
    if (_queryParams.containsKey('code')) {
      _handleConfirmedRedirect();
    } else if (_queryParams.containsKey('error') || _queryParams.containsKey('error_description')) {
      _handleErrorRedirect();
    } else {
      emit(ConfirmEmailError(S.current.no_direct_access));
    }
  }

  void _handleConfirmedRedirect() {
    final code = _queryParams['code'].toString();

    if (code.isEmpty) {
      _logger.warning('No code passed with request');
      emit(ConfirmEmailError(S.current.internal_server_error));
    } else {
      emit(ConfirmEmailConfirmed());
    }
  }

  void _handleErrorRedirect() {
    final errorCode = _queryParams['error'].toString();
    final errorMessage = _queryParams['error_description'].toString();

    if (errorCode.isNotEmpty && errorMessage.isNotEmpty) {
      emit(ConfirmEmailError(errorMessage));
    } else {
      _logger.warning('No error code or message passed with request');
      emit(ConfirmEmailError(S.current.internal_server_error));
    }
  }

  void navigateToLogin() {
    _customNavigatorService.navigateToScreenAndRemoveCurrentScreen(CustomRoutes.login);
  }
}

[felangel]

Hi @Jeastwood1995 👋
Thanks for opening an issue!

Have you resolved this? If not, are you able to share a link to a minimal reproduction sample? Thanks!

[felangel]

Going to close this for now but if it's still an issue please let me know and provide a minimal reproduction sample and I'm happy to take a closer look 👍