Fix rerun filtered environments doesn't take into consideration review filters (#222)

* Fix rerun filtered environments doesn't take into consideration review filters

* Add rejected filter option for environments

* Remove commented out code

Author: omar-selo

frontend/lib/models/filters.dart

@@ -122,8 +122,11 @@ final emptyEnvironmentReviewFilters = FiltersGroup<EnvironmentReview>(
   filters: [
     Filter<EnvironmentReview>(
       name: 'Review status',
-      extractOption: (te) =>
-          te.reviewDecision.isEmpty ? 'Undecided' : 'Reviewed',
+      extractOption: (er) => switch (er.reviewDecision) {
+        [] => 'Undecided',
+        [EnvironmentReviewDecision.rejected] => 'Rejected',
+        [...] => 'Approved',
+      },
     ),
   ],
 );

frontend/lib/ui/artefact_page/artefact_page_body.dart

@@ -58,7 +58,10 @@ class ArtefactPageBody extends ConsumerWidget {
               testExecutions: filteredCombination.map((comb) => comb.$2),
             ),
             const Spacer(),
-            const RerunFilteredEnvironmentsButton(),
+            RerunFilteredEnvironmentsButton(
+              filteredTestExecutions:
+                  filteredCombination.map((comb) => comb.$2),
+            ),
           ],
         ),
         NonBlockingProviderPreloader(

frontend/lib/ui/artefact_page/rerun_filtered_environments_button.dart

@@ -5,23 +5,22 @@ import 'package:intersperse/intersperse.dart';
 
 import '../../models/test_execution.dart';
 import '../../providers/artefact_builds.dart';
-import '../../providers/filtered_test_executions.dart';
 import '../../routing.dart';
 import '../spacing.dart';
 
 class RerunFilteredEnvironmentsButton extends ConsumerWidget {
-  const RerunFilteredEnvironmentsButton({super.key});
+  const RerunFilteredEnvironmentsButton({
+    super.key,
+    required this.filteredTestExecutions,
+  });
+
+  final Iterable<TestExecution> filteredTestExecutions;
 
   @override
   Widget build(BuildContext context, WidgetRef ref) {
     final pageUri = GoRouterState.of(context).uri;
     final artefactId = AppRoutes.artefactIdFromUri(pageUri);
 
-    final filteredTestExecutions = ref
-        .watch(filteredTestExecutionsProvider(pageUri))
-        .requireValue
-        .toList();
-
     handlePress() => showDialog(
           context: context,
           builder: (_) => _ConfirmationDialog(
@@ -46,7 +45,7 @@ class _ConfirmationDialog extends ConsumerWidget {
     required this.artefactId,
   });
 
-  final List<TestExecution> filteredTestExecutions;
+  final Iterable<TestExecution> filteredTestExecutions;
   final int artefactId;
 
   @override