dataExport: show 'Data export requires Business Plan' on 404 for SaaS; add test; keep generic/self-hosted fallback

Author: JoshFerge

static/app/components/dataExport.spec.tsx

@@ -160,4 +160,22 @@ describe('DataExport', function () {
       expect(addErrorMessage).toHaveBeenCalledWith('uh oh');
     });
   });
+
+  it('shows upgrade message on 404', async function () {
+    MockApiClient.addMockResponse({
+      url: `/organizations/${mockAuthorizedOrg.slug}/data-export/`,
+      method: 'POST',
+      statusCode: 404,
+    });
+
+    render(<DataExport payload={mockPayload} />, {
+      ...mockContext(mockAuthorizedOrg),
+    });
+
+    await userEvent.click(screen.getByRole('button'));
+
+    await waitFor(() => {
+      expect(addErrorMessage).toHaveBeenCalledWith('Data export requires Business Plan');
+    });
+  });
 });

static/app/components/dataExport.tsx

@@ -5,6 +5,7 @@ import {addErrorMessage, addSuccessMessage} from 'sentry/actionCreators/indicato
 import Feature from 'sentry/components/acl/feature';
 import {Button} from 'sentry/components/core/button';
 import {t} from 'sentry/locale';
+import ConfigStore from 'sentry/stores/configStore';
 import useApi from 'sentry/utils/useApi';
 import useOrganization from 'sentry/utils/useOrganization';
 
@@ -70,11 +71,14 @@ export function useDataExport({
         if (unmountedRef?.current) {
           return;
         }
+        const isSelfHosted = ConfigStore.get('isSelfHosted');
         const message =
-          err?.responseJSON?.detail ??
-          t(
-            "We tried our hardest, but we couldn't export your data. Give it another go."
-          );
+          err?.status === 404 && !isSelfHosted
+            ? t('Data export requires Business Plan')
+            : (err?.responseJSON?.detail ??
+              t(
+                "We tried our hardest, but we couldn't export your data. Give it another go."
+              ));
 
         addErrorMessage(message);
         inProgressCallback?.(false);