Remove unused Test Execution Review mentions

canonical · Oct 16, 2024 · 5050737 · 5050737
1 parent 3e158d5
commit 5050737
Show file tree

Hide file tree

Showing 3 changed files with 0 additions and 100 deletions.
diff --git a/frontend/lib/models/test_execution.dart b/frontend/lib/models/test_execution.dart
@@ -24,16 +24,6 @@ class TestExecution with _$TestExecution {
 
   factory TestExecution.fromJson(Map<String, Object?> json) =>
       _$TestExecutionFromJson(json);
-
-  static Object updateReviewDecisionRequestData(
-    String reviewComment,
-    List<TestExecutionReviewDecision> reviewDecision,
-  ) {
-    return {
-      reviewCommentJsonKey: reviewComment,
-      reviewDecisionJsonKey: reviewDecision,
-    };
-  }
 }
 
 enum TestExecutionStatus {
@@ -120,62 +110,3 @@ enum TestExecutionStatus {
     }
   }
 }
-
-enum TestExecutionReviewDecision {
-  @JsonValue('REJECTED')
-  rejected,
-  @JsonValue('APPROVED_INCONSISTENT_TEST')
-  approvedInconsistentTest,
-  @JsonValue('APPROVED_UNSTABLE_PHYSICAL_INFRA')
-  approvedUnstablePhysicalInfra,
-  @JsonValue('APPROVED_CUSTOMER_PREREQUISITE_FAIL')
-  approvedCustomerPrerequisiteFail,
-  @JsonValue('APPROVED_FAULTY_HARDWARE')
-  approvedFaultyHardware,
-  @JsonValue('APPROVED_ALL_TESTS_PASS')
-  approvedAllTestsPass;
-
-  String get name {
-    switch (this) {
-      case rejected:
-        return 'Reject';
-      case approvedInconsistentTest:
-        return 'Approve (inconsistent test definition)';
-      case approvedUnstablePhysicalInfra:
-        return 'Approve (unstable physical infrastructure)';
-      case approvedCustomerPrerequisiteFail:
-        return 'Approve (customer provided prerequisite failing)';
-      case approvedFaultyHardware:
-        return 'Approve (faulty hardware)';
-      case approvedAllTestsPass:
-        return 'Approve (all tests pass)';
-    }
-  }
-
-  bool get isDeprecated {
-    switch (this) {
-      case approvedFaultyHardware:
-      case approvedUnstablePhysicalInfra:
-        return true;
-      default:
-        return false;
-    }
-  }
-
-  String toJson() {
-    switch (this) {
-      case rejected:
-        return 'REJECTED';
-      case approvedInconsistentTest:
-        return 'APPROVED_INCONSISTENT_TEST';
-      case approvedUnstablePhysicalInfra:
-        return 'APPROVED_UNSTABLE_PHYSICAL_INFRA';
-      case approvedCustomerPrerequisiteFail:
-        return 'APPROVED_CUSTOMER_PREREQUISITE_FAIL';
-      case approvedFaultyHardware:
-        return 'APPROVED_FAULTY_HARDWARE';
-      case approvedAllTestsPass:
-        return 'APPROVED_ALL_TESTS_PASS';
-    }
-  }
-}
diff --git a/frontend/lib/providers/artefact_builds.dart b/frontend/lib/providers/artefact_builds.dart
@@ -14,21 +14,6 @@ class ArtefactBuilds extends _$ArtefactBuilds {
     return await api.getArtefactBuilds(artefactId);
   }
 
-  Future<void> changeReviewDecision(
-    int testExecutionId,
-    String reviewComment,
-    List<TestExecutionReviewDecision> reviewDecision,
-  ) async {
-    final api = ref.read(apiProvider);
-    final testExecution = await api.changeTestExecutionReview(
-      testExecutionId,
-      reviewDecision,
-      reviewComment,
-    );
-
-    await _updateTestExecutions({testExecutionId}, (_) => testExecution);
-  }
-
   Future<void> rerunTestExecutions(Set<int> testExecutionIds) async {
     final api = ref.read(apiProvider);
     final rerunRequests = await api.rerunTestExecutions(testExecutionIds);

diff --git a/frontend/lib/repositories/api_repository.dart b/frontend/lib/repositories/api_repository.dart
@@ -8,7 +8,6 @@ import '../models/environment_issue.dart';
 import '../models/environment_review.dart';
 import '../models/family_name.dart';
 import '../models/rerun_request.dart';
-import '../models/test_execution.dart';
 import '../models/test_issue.dart';
 import '../models/test_result.dart';
 import '../models/test_event.dart';
@@ -47,21 +46,6 @@ class ApiRepository {
     return artefactBuilds;
   }
 
-  Future<TestExecution> changeTestExecutionReview(
-    int testExecutionId,
-    List<TestExecutionReviewDecision> reviewDecision,
-    String reviewComment,
-  ) async {
-    final response = await dio.patch(
-      '/v1/test-executions/$testExecutionId',
-      data: TestExecution.updateReviewDecisionRequestData(
-        reviewComment,
-        reviewDecision,
-      ),
-    );
-    return TestExecution.fromJson(response.data);
-  }
-
   Future<List<TestResult>> getTestExecutionResults(int testExecutionId) async {
     final response =
         await dio.get('/v1/test-executions/$testExecutionId/test-results');