cleanup

Author: suejung-sentry

static/app/views/prevent/preventAI/hooks/useUpdatePreventAIFeature.spec.tsx

@@ -25,6 +25,7 @@ describe('useUpdatePreventAIFeature', () => {
             test_generation: {
               enabled: false,
               triggers: {on_command_phrase: false, on_ready_for_review: false},
+              sensitivity: 'medium',
             },
             bug_prediction: {
               enabled: false,
@@ -45,6 +46,7 @@ describe('useUpdatePreventAIFeature', () => {
           test_generation: {
             enabled: false,
             triggers: {on_command_phrase: false, on_ready_for_review: false},
+            sensitivity: 'medium',
           },
           bug_prediction: {
             enabled: false,
@@ -191,19 +193,19 @@ describe('useUpdatePreventAIFeature', () => {
       expect(feature?.triggers.on_ready_for_review).toBe(true);
     });
 
-    it('should update sensitivity', () => {
+    it('should preserve sensitivity', () => {
       const config = structuredClone(mockOrg.preventAiConfigGithub!);
       const updatedConfig = makePreventAIConfig(config, {
         feature: 'bug_prediction',
-        enabled: true,
+        enabled: false,
         orgName: 'org-1',
         repoName: 'repo-xyz',
-        sensitivity: 'low',
+        trigger: {on_ready_for_review: true},
       });
       const feature =
         updatedConfig.github_organizations?.['org-1']?.repo_overrides?.['repo-xyz']
           ?.bug_prediction;
-      expect(feature?.sensitivity).toBe('low');
+      expect(feature?.sensitivity).toBe('medium');
     });
   });
 });

static/app/views/prevent/preventAI/manageReposPanel.spec.tsx

@@ -35,7 +35,6 @@ describe('ManageReposPanel', () => {
           bug_prediction: {
             enabled: true,
             triggers: {on_command_phrase: true, on_ready_for_review: false},
-            sensitivity: 'medium',
           },
           test_generation: {
             enabled: false,
@@ -44,7 +43,6 @@ describe('ManageReposPanel', () => {
           vanilla: {
             enabled: true,
             triggers: {on_command_phrase: false, on_ready_for_review: false},
-            sensitivity: 'medium',
           },
         },
         repo_overrides: {},
@@ -71,9 +69,9 @@ describe('ManageReposPanel', () => {
 
   it('shows feature toggles with correct initial state', async () => {
     render(<ManageReposPanel {...defaultProps} />, {organization: mockOrganization});
-    expect(await screen.findByLabelText(/Enable PR Review/i)).toBeChecked();
-    expect(await screen.findByLabelText(/Enable Test Generation/i)).not.toBeChecked();
-    expect(await screen.findByLabelText(/Enable Error Prediction/i)).toBeChecked();
+    expect(await screen.findByLabelText(/PR Review/i)).toBeChecked();
+    expect(await screen.findByLabelText(/Test Generation/i)).not.toBeChecked();
+    expect(await screen.findByLabelText(/Error Prediction/i)).toBeChecked();
     expect(
       await screen.findByLabelText(/Auto Run on Opened Pull Requests/i)
     ).not.toBeChecked();
@@ -86,9 +84,9 @@ describe('ManageReposPanel', () => {
       isLoading: true,
     };
     render(<ManageReposPanel {...defaultProps} />, {organization: mockOrganization});
-    expect(await screen.findByLabelText(/Enable PR Review/i)).toBeDisabled();
-    expect(await screen.findByLabelText(/Enable Test Generation/i)).toBeDisabled();
-    expect(await screen.findByLabelText(/Enable Error Prediction/i)).toBeDisabled();
+    expect(await screen.findByLabelText(/PR Review/i)).toBeDisabled();
+    expect(await screen.findByLabelText(/Test Generation/i)).toBeDisabled();
+    expect(await screen.findByLabelText(/Error Prediction/i)).toBeDisabled();
   });
 
   it('shows error message if updateError is present', async () => {
@@ -101,22 +99,6 @@ describe('ManageReposPanel', () => {
     expect(await screen.findByText(/Could not update settings/i)).toBeInTheDocument();
   });
 
-  it('shows sensitivity options when feature is enabled', async () => {
-    render(<ManageReposPanel {...defaultProps} />, {organization: mockOrganization});
-    const prReviewCheckbox = await screen.findByLabelText(/Enable PR Review/i);
-    const errorPredictionCheckbox = await screen.findByLabelText(
-      /Enable Error Prediction/i
-    );
-    expect(prReviewCheckbox).toBeChecked();
-    expect(errorPredictionCheckbox).toBeChecked();
-    expect(
-      await screen.findByTestId(/pr-review-sensitivity-dropdown/i)
-    ).toBeInTheDocument();
-    expect(
-      await screen.findByTestId(/error-prediction-sensitivity-dropdown/i)
-    ).toBeInTheDocument();
-  });
-
   describe('getRepoConfig', () => {
     it('returns repo override config when present', () => {
       const orgConfig: PreventAIOrgConfig = {