fix: handle invalid URLs in PermissionsSummary component

Updated the hostname extraction logic in the PermissionsSummary component to gracefully handle invalid URLs by returning the original string or an empty string. This prevents potential errors when processing URLs. Additionally, removed outdated tests related to SDK URL status from AccountConnect.test.tsx to streamline the test suite.

Author: abretonc7s

app/components/UI/PermissionsSummary/PermissionsSummary.tsx

@@ -80,10 +80,15 @@ const PermissionsSummary = ({
   const providerConfig = useSelector(selectProviderConfig);
   const chainId = useSelector(selectEvmChainId);
 
-  const hostname = useMemo(
-    () => new URL(currentPageInformation.url).hostname,
-    [currentPageInformation.url],
-  );
+  const hostname = useMemo(() => {
+    try {
+      return new URL(currentPageInformation.url).hostname;
+    } catch (e) {
+      // Handle invalid URL, default to the original string or an empty string
+      return currentPageInformation.url || '';
+    }
+  }, [currentPageInformation.url]);
+
   const networkInfo = useNetworkInfo(hostname);
 
   // if network switch, we get the chain name from the customNetworkInformation

app/components/Views/AccountConnect/AccountConnect.test.tsx

@@ -171,65 +171,6 @@ describe('AccountConnect', () => {
     expect(toJSON()).toMatchSnapshot();
   });
 
-  describe('Renders different screens based on SDK URL status', () => {
-    it('should render SingleConnect screen when isSdkUrlUnknown is true', () => {
-      const mockPropsForUnknownUrl = {
-        route: {
-          params: {
-            hostInfo: {
-              metadata: {
-                id: 'mockId',
-                // Using an invalid/unknown format for origin
-                origin: '',
-              },
-              permissions: {
-                eth_accounts: {
-                  parentCapability: 'eth_accounts',
-                },
-              },
-            },
-            permissionRequestId: 'test',
-          },
-        },
-      };
-
-      const { getByTestId } = renderWithProvider(
-        <AccountConnect {...mockPropsForUnknownUrl} />,
-        { state: mockInitialState },
-      );
-
-      expect(getByTestId('connect-account-modal')).toBeDefined();
-    });
-
-    it('should render PermissionsSummary screen when isSdkUrlUnknown is false', () => {
-      const mockPropsForKnownUrl = {
-        route: {
-          params: {
-            hostInfo: {
-              metadata: {
-                id: 'mockId',
-                // Using a valid URL format
-                origin: 'https://example.com',
-              },
-              permissions: {
-                eth_accounts: {
-                  parentCapability: 'eth_accounts',
-                },
-              },
-            },
-            permissionRequestId: 'test',
-          },
-        },
-      };
-
-      const { getByTestId } = renderWithProvider(
-        <AccountConnect {...mockPropsForKnownUrl} />,
-        { state: mockInitialState },
-      );
-
-      expect(getByTestId('permission-summary-container')).toBeDefined();
-    });
-  });
 
   describe('AccountConnectMultiSelector handlers', () => {
     it('invokes onPrimaryActionButtonPress property and renders permissions summary', async () => {