fix(investigation-rule): capture exception instead of error toast (#86017)

Author: obostjancic

static/app/components/dynamicSampling/investigationRule.spec.tsx

@@ -164,7 +164,7 @@ describe('InvestigationRule', function () {
     expect(getRule).toHaveBeenCalledTimes(0);
   });
 
-  it('does not render when there is an unknown error but shows an error', async function () {
+  it('does not render when there is an unknown error', async function () {
     initComponentEnvironment({hasRule: false});
     const getRule = MockApiClient.addMockResponse({
       url: '/organizations/org-slug/dynamic-sampling/custom-rules/',
@@ -178,7 +178,7 @@ describe('InvestigationRule', function () {
     );
     await expectNotToRender();
 
-    expect(addErrorMessage).toHaveBeenCalledTimes(1);
+    expect(addErrorMessage).toHaveBeenCalledTimes(0);
     // check we did call the endpoint to check if a rule exists
     expect(getRule).toHaveBeenCalledTimes(1);
   });

static/app/components/dynamicSampling/investigationRule.tsx

@@ -1,5 +1,6 @@
 import {useEffect} from 'react';
 import styled from '@emotion/styled';
+import * as Sentry from '@sentry/react';
 import moment from 'moment-timezone';
 
 import {addErrorMessage, addSuccessMessage} from 'sentry/actionCreators/indicator';
@@ -216,9 +217,12 @@ function InvestigationRuleCreationInternal(props: PropsInternal) {
     if (isBreakingRequestError) {
       const msg = t('Unable to fetch investigation rule');
       handleXhrErrorResponse(msg, error);
-      addErrorMessage(msg);
+      Sentry.withScope(scope => {
+        scope.setExtra('query', query);
+        Sentry.captureException(error);
+      });
     }
-  }, [isBreakingRequestError, error]);
+  }, [isBreakingRequestError, error, query]);
 
   if (isLoading || isBreakingRequestError) {
     return null;
@@ -256,30 +260,32 @@ function InvestigationRuleCreationInternal(props: PropsInternal) {
     <Tooltip
       isHoverable
       title={
-        isTransactionQueryMissing
-          ? tct(
-              'If you filter by [code:event.type:transaction] we can adjust your sampling priorities, increasing the odds of getting matching events. [link:Learn more.]',
-              {
-                code: <code />,
-                link: (
-                  <ExternalLink href="https://docs.sentry.io/product/performance/retention-priorities/#investigation-mode" />
-                ),
-              }
-            )
-          : tct(
-              'We can find more events that match your search query by adjusting your sampling priorities for an hour, increasing the odds of getting matching events. [link:Learn more.]',
-              {
-                link: (
-                  <ExternalLink href="https://docs.sentry.io/product/performance/retention-priorities/#investigation-mode" />
-                ),
-              }
-            )
+        isBreakingRequestError
+          ? t('Search query unsupported.')
+          : isTransactionQueryMissing
+            ? tct(
+                'If you filter by [code:event.type:transaction] we can adjust your sampling priorities, increasing the odds of getting matching events. [link:Learn more.]',
+                {
+                  code: <code />,
+                  link: (
+                    <ExternalLink href="https://docs.sentry.io/product/performance/retention-priorities/#investigation-mode" />
+                  ),
+                }
+              )
+            : tct(
+                'We can find more events that match your search query by adjusting your sampling priorities for an hour, increasing the odds of getting matching events. [link:Learn more.]',
+                {
+                  link: (
+                    <ExternalLink href="https://docs.sentry.io/product/performance/retention-priorities/#investigation-mode" />
+                  ),
+                }
+              )
       }
     >
       <Button
         {...props.buttonProps}
         priority={isLikelyMoreNeeded ? 'primary' : 'default'}
-        disabled={isTransactionQueryMissing}
+        disabled={isTransactionQueryMissing || isBreakingRequestError}
         onClick={() => createInvestigationRule({organization, projects, query})}
         icon={<IconStack />}
       >