FET-1619 If tab specified in query string doesn't exist, app crashes. Should redirect to profile tab

e2e/specs/stateless/profileEditor.spec.ts

@@ -104,6 +104,60 @@ test.describe('profile', () => {
     await expect(profilePage.record('text', 'email')).toHaveText('[email protected]')
     await expect(profilePage.contentHash()).toContainText('ipfs://bafybeic...')
   })
+
+  test('should redirect to profile tab if tab specified in query string does not exist', async ({
+    page,
+    login,
+    makeName,
+    makePageObject,
+  }) => {
+    const name = await makeName({
+      label: 'profile',
+      type: 'legacy',
+      records: await makeRecords(),
+    })
+
+    const profilePage = makePageObject('ProfilePage')
+
+    await profilePage.goto(name)
+    await login.connect()
+
+    const validTabs: Array<String> = [
+      'profile',
+      'records',
+      'ownership',
+      'subnames',
+      'permissions',
+      'more',
+    ] as const
+
+    const generateRandomTab = () => {
+      const characters = 'abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789'
+      const length = 10
+
+      let randomTab = ''
+
+      do {
+        randomTab = Array.from({ length }, () =>
+          characters.charAt(Math.floor(Math.random() * characters.length)),
+        ).join('')
+      } while (validTabs.includes(randomTab))
+
+      return randomTab
+    }
+
+    await page.goto(`/${name}?tab=${generateRandomTab()}`)
+
+    await expect(page).toHaveURL(`/${name}`)
+
+    await page.pause()
+    await expect(profilePage.record('text', 'description')).toHaveText('Hello2')
+    await expect(profilePage.record('text', 'url')).toHaveText('twitter.com')
+    await expect(profilePage.record('address', 'btc')).toHaveText('bc1qj...pwa6n')
+    await expect(profilePage.record('address', 'etcLegacy')).toHaveText('etcLegacy0x3C4...293BC')
+    await expect(profilePage.record('text', 'email')).toHaveText('[email protected]')
+    await expect(profilePage.contentHash()).toContainText('ipfs://bafybeic...')
+  })
 })
 
 test.describe('migrations', () => {

src/components/pages/profile/[name]/Profile.tsx

@@ -177,13 +177,16 @@ const ProfileContent = ({ isSelf, isLoading: parentIsLoading, name }: Props) =>
   // profile.decryptedName fetches labels from NW/subgraph
   // normalisedName fetches labels from localStorage
   useEffect(() => {
+    if (!tab || !tabs.includes(tab)) {
+      router.replace(`/${normalisedName}`)
+    }
     shouldRedirect(router, 'Profile.tsx', '/profile', {
       isSelf,
       name,
       decodedName: profile?.decodedName,
       normalisedName,
     })
-  }, [profile?.decodedName, normalisedName, name, isSelf, router])
+  }, [profile?.decodedName, normalisedName, name, isSelf, router, tab])
 
   // useEffect(() => {
   //   if (shouldShowSuccessPage(transactions)) {
@@ -296,7 +299,7 @@ const ProfileContent = ({ isSelf, isLoading: parentIsLoading, name }: Props) =>
             .with('more', () => (
               <MoreTab name={normalisedName} nameDetails={nameDetails} abilities={abilities.data} />
             ))
-            .exhaustive(),
+            .otherwise(() => <ProfileTab name={normalisedName} nameDetails={nameDetails} />),
         }}
       </Content>
     </>