Merge branch 'main' into feature/fet-2911-update-ensjs-version-in-manager-v3

Author: laurgk

e2e/specs/stateful/profile.spec.ts

@@ -1,4 +1,5 @@
 /* eslint-disable no-await-in-loop */
+/* eslint-disable no-restricted-syntax */
 import { expect } from '@playwright/test'
 
 import { test } from '../../../playwright/index.js'
@@ -49,6 +50,12 @@ const profiles = [
         value: '0xFc5...7acf0',
         fullValue: '0xFc5958B4B6F9a06D21E06429c8833f865577acf0',
       },
+      {
+        type: 'address',
+        key: 'sui',
+        value: '0x123...31231',
+        fullValue: '0x1231231231231231231231231231231231231231231231231231231231231231',
+      },
       {
         type: 'other',
         key: 'avatar',

…ateless/addressPageErrorHandling.spec.ts e2e/specs/stateless/addressPage.spec.tse2e/specs/stateless/addressPageErrorHandling.spec.ts renamed to e2e/specs/stateless/addressPage.spec.ts e2e/specs/stateless/addressPageErrorHandling.spec.ts renamed to e2e/specs/stateless/addressPage.spec.ts

@@ -176,6 +176,30 @@ test.describe('Address page error handling', () => {
     expect(requestCount).toBeGreaterThanOrEqual(3) // At least 2 failures + 1 success
   })
 
+  test('should display no-profile-snippet when no primary name set with invalid resolver', async ({
+    page,
+    login,
+    makeName,
+    makePageObject,
+  }) => {
+    test.slow()
+
+    // SETUP: Create name with invalid resolver (triggers ContractFunctionExecutionError)
+    // Deliberately NOT setting as primary name to test no-primary-name scenario
+    const name = await createUserName(makeName, 'no-primary-invalid-resolver')
+    await setInvalidResolver(page, name, login, makePageObject)
+
+    // ACTION: Navigate to address page
+    const userAddress = getUserAddress()
+    await page.goto(`/${userAddress}`)
+
+    // ASSERT: Page loads gracefully showing no-profile-snippet (not profile-snippet)
+    await expect(page.getByTestId('no-profile-snippet')).toBeVisible()
+
+    // ASSERT: Names list loads despite invalid resolver on owned name
+    await expect(page.getByTestId('names-list')).toBeVisible({ timeout: NAMES_LIST_TIMEOUT })
+  })
+
   test.afterEach(async () => {
     // Clean up: reset primary name after each test
     await setPrimaryName(walletClient, {

e2e/specs/stateless/advancedEditor.spec.ts

@@ -25,6 +25,11 @@ test('should be able to maintain state when returning from transaction modal whe
         { coin: 'SOL', value: 'HN7cABqLq46Es1jh92dQQisAq662SmxELLLsHHe4YWrH' },
         { coin: 'ETH', value: '0xbec1C7C11F2Fa9AB24b9E49122D26e721766DAF6' },
         { coin: 'BTC', value: '1PzAJcFtEiXo9UGtRU6iqXQKj8NXtcC7DE' },
+        {
+          coin: 'SUI',
+          // eslint-disable-next-line no-restricted-syntax -- SUI addresses are 64 hex chars, not private keys
+          value: '0x1231231231231231231231231231231231231231231231231231231231231231',
+        },
       ],
       contentHash: 'ipfs://bafybeico3uuyj3vphxpvbowchdwjlrlrh62awxscrnii7w7flu5z6fk77y',
       abi: await encodeAbi({ encodeAs: 'json', data: dummyABI }),
@@ -51,6 +56,10 @@ test('should be able to maintain state when returning from transaction modal whe
   await expect(recordsPage.getRecordValue('address', 'BTC')).toHaveText(
     '1PzAJcFtEiXo9UGtRU6iqXQKj8NXtcC7DE',
   )
+  await expect(recordsPage.getRecordValue('address', 'SUI')).toHaveText(
+    // eslint-disable-next-line no-restricted-syntax -- SUI addresses are 64 hex chars, not private keys
+    '0x1231231231231231231231231231231231231231231231231231231231231231',
+  )
   await expect(recordsPage.getRecordValue('contentHash')).toHaveText(
     'ipfs://bafybeico3uuyj3vphxpvbowchdwjlrlrh62awxscrnii7w7flu5z6fk77y',
   )
@@ -71,6 +80,10 @@ test('should be able to maintain state when returning from transaction modal whe
   await expect(await advancedEditor.recordInput('address', 'BTC')).toHaveValue(
     '1PzAJcFtEiXo9UGtRU6iqXQKj8NXtcC7DE',
   )
+  await expect(await advancedEditor.recordInput('address', 'SUI')).toHaveValue(
+    // eslint-disable-next-line no-restricted-syntax -- SUI addresses are 64 hex chars, not private keys
+    '0x1231231231231231231231231231231231231231231231231231231231231231',
+  )
   await expect(await advancedEditor.recordInput('contentHash')).toHaveValue(
     'ipfs://bafybeico3uuyj3vphxpvbowchdwjlrlrh62awxscrnii7w7flu5z6fk77y',
   )
@@ -79,34 +92,37 @@ test('should be able to maintain state when returning from transaction modal whe
   await advancedEditor.recordClearButton('text', 'text').then((button) => button.click())
   await advancedEditor.recordClearButton('address', 'SOL').then((button) => button.click())
   await advancedEditor.recordClearButton('address', 'ETH').then((button) => button.click())
+  await advancedEditor.recordClearButton('address', 'SUI').then((button) => button.click())
   await advancedEditor.recordInput('contentHash').then((input) => input.fill(''))
   await advancedEditor.recordInput('abi').then((input) => input.fill(''))
 
   await advancedEditor.saveButton.click()
 
   // Validate transaction display item
-  await expect(transactionModal.displayItem('update')).toHaveText('5 records')
+  await expect(transactionModal.displayItem('update')).toHaveText('6 records')
 
   await transactionModal.backButton.click()
 
   // Validate inputs have been rebuilt correctly
   await expect(await advancedEditor.recordComponent('text', 'text')).toHaveCount(0)
   await expect(await advancedEditor.recordComponent('address', 'SOL')).toHaveCount(0)
   await expect(await advancedEditor.recordComponent('address', 'ETH')).toHaveCount(0)
+  await expect(await advancedEditor.recordComponent('address', 'SUI')).toHaveCount(0)
   await expect(await advancedEditor.recordInput('contentHash')).toHaveValue('')
   await expect(await advancedEditor.recordInput('abi')).toHaveValue('')
 
   await advancedEditor.saveButton.click()
 
   // Validate transaction display item
-  await expect(transactionModal.displayItem('update')).toHaveText('5 records')
+  await expect(transactionModal.displayItem('update')).toHaveText('6 records')
 
   await transactionModal.autoComplete()
 
   // Validate change in records
   await expect(recordsPage.getRecordButton('text', 'text')).toHaveCount(0)
   await expect(recordsPage.getRecordButton('address', 'SOL')).toHaveCount(0)
   await expect(recordsPage.getRecordButton('address', 'ETH')).toHaveCount(0)
+  await expect(recordsPage.getRecordButton('address', 'SUI')).toHaveCount(0)
   await expect(recordsPage.getRecordButton('contentHash')).toHaveCount(0)
   await expect(recordsPage.getRecordButton('abi')).toHaveCount(0)
 })

e2e/specs/stateless/box.spec.ts

@@ -112,10 +112,10 @@ test('should be able to extend multiple names (including names in grace preiod)
     const label = name.replace('.eth', '')
     await addresPage.search(label)
     await expect(addresPage.getNameRow(name)).toBeVisible({ timeout: 5000 })
-    const newTs = await addresPage.getTimestamp(name)
-    expect(newTs).not.toBe(timestampDict[name])
-    // Allow 1 day tolerance for block timestamp rounding
-    expect(Math.abs(newTs - timestampDict[name] - 31536000000 * 3)).toBeLessThanOrEqual(86400000)
+    await expect(await addresPage.getTimestamp(name)).not.toBe(timestampDict[name])
+    // Allow up to 3 days tolerance for leap year differences across 3 years
+    const ts = await addresPage.getTimestamp(name)
+    expect(Math.abs(ts - timestampDict[name] - 31536000000 * 3)).toBeLessThanOrEqual(86400000 * 3)
   }
 })
 
@@ -181,7 +181,7 @@ test('should be able to extend a single unwrapped name from profile', async ({
     await extendNamesModal.getExtendButton.click()
     await transactionModal.autoComplete()
     const newTimestamp = await profilePage.getExpiryTimestamp()
-    // Allow 1 day tolerance for block timestamp rounding
+    // Allow 1 day tolerance for leap year differences
     expect(Math.abs(newTimestamp - timestamp - 31536000000)).toBeLessThanOrEqual(86400000)
   })
 })
@@ -244,7 +244,7 @@ test('should be able to extend a single unwrapped name in grace period from prof
     await transactionModal.autoComplete()
 
     const newTimestamp = await profilePage.getExpiryTimestamp()
-    // Allow 1 day tolerance for block timestamp rounding
+    // Allow 1 day tolerance for leap year differences
     expect(Math.abs(newTimestamp - timestamp - 31536000000)).toBeLessThanOrEqual(86400000)
   })
 })
@@ -308,7 +308,7 @@ test('should be able to extend a single unwrapped name in grace period from prof
     const transactionModal = makePageObject('TransactionModal')
     await transactionModal.autoComplete()
     const newTimestamp = await profilePage.getExpiryTimestamp()
-    // Allow 1 day tolerance for block timestamp rounding
+    // Allow 1 day tolerance for leap year differences
     expect(Math.abs(newTimestamp - timestamp - 31536000000)).toBeLessThanOrEqual(86400000)
   })
 })
@@ -1151,4 +1151,51 @@ test.describe('Wrapped Name Renewal with Referrer', () => {
     // Note: Bulk renewals should use the legacy bulk renewal contract
     // which doesn't support referrers
   })
+
+  test('should extend wrapped name in grace period with referrer', async ({
+    page,
+    login,
+    makeName,
+    makePageObject,
+  }) => {
+    const name = await makeName({
+      label: 'wrapped-grace-referrer',
+      type: 'wrapped',
+      owner: 'user',
+      duration: -24 * 60 * 60, // Expired 1 day ago
+    })
+
+    const referrerAddress = '0x1234567890123456789012345678901234567890'
+
+    const profilePage = makePageObject('ProfilePage')
+    const transactionModal = makePageObject('TransactionModal')
+
+    await profilePage.goto(name)
+    await login.connect()
+
+    // Verify name is in grace period
+    await expect(page.getByText(`${name} has expired`)).toBeVisible()
+
+    // Add referrer to URL and navigate
+    await page.goto(`/${name}?referrer=${referrerAddress}`)
+
+    // Click extend button
+    await profilePage.getExtendButton.click()
+
+    // Set extension and proceed
+    await expect(page.getByTestId('plus-minus-control-label')).toHaveText('1 year')
+    await page.locator('button:has-text("Next")').click()
+
+    // Complete transaction
+    await transactionModal.confirm()
+
+    await expect(page.getByText('Your "Extend names" transaction was successful')).toBeVisible({
+      timeout: 10000,
+    })
+
+    // Verify referrer included in the transaction calldata
+    const latestTransaction = await publicClient.getTransaction({ blockTag: 'latest', index: 0 })
+    const referrerHex = addressToBytes32(referrerAddress)
+    expect(latestTransaction.input).toContain(referrerHex.slice(2)) // Remove '0x' prefix for comparison
+  })
 })