[RF] Tests for error handling feature removed

[nickmeinhold]

Fix removed tests

Possibly we just need to use a Store that does pass on actions... seems like the problem is maybe the ‘backend’ of redux not being wired up?

I followed the path of the error for the first test and I think it fails because the AddProblemAction dispatched in middleware when the service throws is never recieved by the relevant reducer

(removed from packages/redfire/test/error-handling/unit-tests/widgets/add_problem_page_widget_test.dart)

testWidgets('problem page appears after error caught',
        (WidgetTester tester) async {
  // The default services harness has mocked out plugins.
  // One of the initial actions attempts to listen to auth state changes
  // which throws in this context as the MockFirebaseAuth has no stub set.
  final harness = AppWidgetHarness.withMockedPlugins(
      ExampleAppState.init(), ServicesHarness());

  await tester.pumpWidget(harness.widget);
  await tester.pump();

  expect(find.byType(ProblemPageView), findsOneWidget);
});

Same kind of probem I think:

Removed from packages/redfire/test/error-handling/unit-tests/widgets/dismiss_problem_widget_test.dart

final problem = ProblemInfo('Problem error message');

testWidgets(
    'is removed from widget tree when ProblemPageData is removed from Store',
    (WidgetTester tester) async {
  final problemPageData = ProblemPageData(problem);
  final state = ExampleAppState.init();
  final updatedState =
      state.copyWith(pages: state.pages.add(problemPageData));
  final store = FakeStore(updatedState);
  final appWidget =
      AppWidgetHarness.withStore(initializedStore: store).widget;

  await tester.pumpWidget(appWidget);
  await tester.pump();

  expect(find.byType(ProblemPageView), findsOneWidget);

  store.updateState(store.state
      .copyWith(pages: store.state.pages.remove(problemPageData)));
  store.updateState(store.state);

  await tester.pumpAndSettle();

  expect(find.byType(ProblemPage), findsNothing);
});

Maybe look at redfire_test and see if there is a different harness that has more of the backend wired up? Maybe we need to add the relevant reducers?