Adds generic error handling

packages/core/src/js/feedback/FeedbackForm.tsx

@@ -55,14 +55,17 @@ export class FeedbackForm extends React.Component<FeedbackFormProps, FeedbackFor
     };
 
     // eslint-disable-next-line @typescript-eslint/no-floating-promises
-    checkInternetConnection(() => {
+    checkInternetConnection(() => { // onConnected
       onFormClose();
       this.setState({ isVisible: false });
       captureFeedback(userFeedback);
       Alert.alert(text.successMessageText);
-    }, () => {
+    }, () => { // onDisconnected
       Alert.alert(text.errorTitle, text.networkError);
       logger.error(`Feedback form submission failed: ${text.networkError}`);
+    }, () => { // onError
+      Alert.alert(text.errorTitle, text.genericError);
+      logger.error(`Feedback form submission failed: ${text.genericError}`);
     });
   };
 

packages/core/src/js/feedback/FeedbackForm.types.ts

@@ -116,6 +116,11 @@ export interface FeedbackTextConfiguration {
    * Message when there is a network error
    */
   networkError?: string;
+
+  /**
+   * Message when there is a network error
+   */
+  genericError?: string;
 }
 
 /**

packages/core/src/js/feedback/defaults.ts

@@ -18,6 +18,7 @@ const FORM_ERROR = 'Please fill out all required fields.';
 const EMAIL_ERROR = 'Please enter a valid email address.';
 const SUCCESS_MESSAGE_TEXT = 'Thank you for your report!';
 const CONNECTIONS_ERROR_TEXT = 'Unable to send Feedback due to network issues.';
+const GENERIC_ERROR_TEXT = 'Unable to send feedback due to an unexpected error.';
 
 export const defaultConfiguration: Partial<FeedbackFormProps> = {
   // FeedbackCallbacks
@@ -56,4 +57,5 @@ export const defaultConfiguration: Partial<FeedbackFormProps> = {
   emailError: EMAIL_ERROR,
   successMessageText: SUCCESS_MESSAGE_TEXT,
   networkError: CONNECTIONS_ERROR_TEXT,
+  genericError: GENERIC_ERROR_TEXT,
 };

packages/core/src/js/feedback/utils.ts

@@ -1,4 +1,8 @@
-export const checkInternetConnection = async (onConnected: () => void, onDisconnected: () => void): Promise<void> => {
+export const checkInternetConnection = async (
+  onConnected: () => void,
+  onDisconnected: () => void,
+  onError: () => void,
+): Promise<void> => {
   try {
     const response = await fetch('https://sentry.io', { method: 'HEAD' });
     if (response.ok) {
@@ -7,7 +11,7 @@ export const checkInternetConnection = async (onConnected: () => void, onDisconn
       onDisconnected();
     }
   } catch (error) {
-    onDisconnected();
+    onError();
   }
 };
 

packages/core/test/feedback/FeedbackForm.test.tsx

@@ -44,11 +44,12 @@ const defaultProps: FeedbackFormProps = {
   emailError: 'The email address is not valid.',
   successMessageText: 'Feedback success',
   networkError: 'Network error',
+  genericError: 'Generic error',
 };
 
 describe('FeedbackForm', () => {
   beforeEach(() => {
-    (checkInternetConnection as jest.Mock).mockImplementation((onConnected, _) => {
+    (checkInternetConnection as jest.Mock).mockImplementation((onConnected, _onDisconnected, _onError) => {
       onConnected();
     });
   });
@@ -138,7 +139,7 @@ describe('FeedbackForm', () => {
   });
 
   it('shows an error message when there is no network connection', async () => {
-    (checkInternetConnection as jest.Mock).mockImplementationOnce((_, onDisconnected) => {
+    (checkInternetConnection as jest.Mock).mockImplementationOnce((_onConnected, onDisconnected, _onError) => {
       onDisconnected();
     });
 
@@ -155,6 +156,24 @@ describe('FeedbackForm', () => {
     });
   });
 
+  it('shows an error message when there is a generic connection', async () => {
+    (checkInternetConnection as jest.Mock).mockImplementationOnce((_onConnected, _onDisconnected, onError) => {
+      onError();
+    });
+
+    const { getByPlaceholderText, getByText } = render(<FeedbackForm {...defaultProps} />);
+
+    fireEvent.changeText(getByPlaceholderText(defaultProps.namePlaceholder), 'John Doe');
+    fireEvent.changeText(getByPlaceholderText(defaultProps.emailPlaceholder), '[email protected]');
+    fireEvent.changeText(getByPlaceholderText(defaultProps.messagePlaceholder), 'This is a feedback message.');
+
+    fireEvent.press(getByText(defaultProps.submitButtonLabel));
+
+    await waitFor(() => {
+      expect(Alert.alert).toHaveBeenCalledWith(defaultProps.errorTitle, defaultProps.networkError);
+    });
+  });
+
   it('calls onFormClose when the form is submitted successfully', async () => {
     const { getByPlaceholderText, getByText } = render(<FeedbackForm {...defaultProps} />);