From f5a2bcf46b2f84aa995af6c0a9f4789b653289dd Mon Sep 17 00:00:00 2001
From: storywithoutend <storywithoutend@gmail.com>
Date: Tue, 29 Oct 2024 22:36:36 +0800
Subject: [PATCH] fix onchain not owner e2e test

---
 e2e/specs/stateless/_importName.spec.ts | 26 ++++++++++++-------------
 src/hooks/useEventTracker.ts            |  2 --
 2 files changed, 13 insertions(+), 15 deletions(-)

diff --git a/e2e/specs/stateless/_importName.spec.ts b/e2e/specs/stateless/_importName.spec.ts
index 6b9c554cf..f60566b6c 100644
--- a/e2e/specs/stateless/_importName.spec.ts
+++ b/e2e/specs/stateless/_importName.spec.ts
@@ -2,17 +2,17 @@ import { expect } from '@playwright/test'
 
 import { test } from '../../../playwright'
 
-// When testing locally, reducers will be run twice
-const reducerEventCount = process.env.CI ? 1 : 2
+// When testing locally, reducers will be run twice because of strict mode
+const strictModeEventCount = process.env.CI ? 1 : 2
 
-test.only('should allow claim (owned by user)', async ({
+test('should allow claim (owned by user)', async ({
   page,
   login,
   accounts,
   makePageObject,
   consoleListener,
 }) => {
-  console.log(reducerEventCount)
+  console.log(strictModeEventCount)
   const name = 'swagabc.xyz'
   await consoleListener.initialize({
     regex: new RegExp(
@@ -69,7 +69,7 @@ test.only('should allow claim (owned by user)', async ({
   await expect(importPage.heading).toHaveText('Claim your domain')
 
   await test.step('should fire DNS import tracking event: dns_selected_import_type', async () => {
-    await expect(consoleListener.getMessages()).toHaveLength(reducerEventCount)
+    await expect(consoleListener.getMessages()).toHaveLength(strictModeEventCount)
 
     await expect(consoleListener.getMessages().toString()).toMatch(
       new RegExp(`dns_selected_import_type.*?${name}`),
@@ -149,8 +149,8 @@ test('should allow import (not owned by user)', async ({
     regex: new RegExp(
       `Event triggered on local development.*?(${[
         'search_selected_dns',
-        'import_type_selected_dns',
-        'verify_ownership_started_dns',
+        'dns_selected_import_type',
+        'dns_verified_ownership',
       ].join('|')})`,
     ),
   })
@@ -190,11 +190,11 @@ test('should allow import (not owned by user)', async ({
   await expect(importPage.nextButton).toBeEnabled({ timeout: 15000 })
   await importPage.nextButton.click()
 
-  await test.step('should fire DNS import tracking event: import_type_selected_dns', async () => {
-    await expect(consoleListener.getMessages()).toHaveLength(1)
+  await test.step('should fire DNS import tracking event: dns_selected_import_type', async () => {
+    await expect(consoleListener.getMessages()).toHaveLength(strictModeEventCount)
 
     await expect(consoleListener.getMessages().toString()).toMatch(
-      new RegExp(`import_type_selected_dns.*?${name}`),
+      new RegExp(`dns_selected_import_type.*?${name}`),
     )
     consoleListener.clearMessages()
   })
@@ -211,10 +211,10 @@ test('should allow import (not owned by user)', async ({
 
   await importPage.nextButton.click()
 
-  await test.step('should fire DNS import tracking event: verify_ownership_started_dns', async () => {
-    await expect(consoleListener.getMessages()).toHaveLength(1)
+  await test.step('should fire DNS import tracking event: dns_verified_ownership', async () => {
+    await expect(consoleListener.getMessages()).toHaveLength(strictModeEventCount)
 
-    await expect(consoleListener.getMessages().toString()).toContain('verify_ownership_started_dns')
+    await expect(consoleListener.getMessages().toString()).toContain('dns_verified_ownership')
     consoleListener.clearMessages()
   })
 
diff --git a/src/hooks/useEventTracker.ts b/src/hooks/useEventTracker.ts
index 2bc960368..b7257638d 100644
--- a/src/hooks/useEventTracker.ts
+++ b/src/hooks/useEventTracker.ts
@@ -43,7 +43,6 @@ type DefaultEvent = {
     | 'register_started'
     | 'register_started_box'
     | 'register_wallet_opened'
-    | 'verify_ownership_started_dns'
     | 'claim_domain_started_dns'
     | 'commit_wallet_opened_dns'
     | 'register_started_dns'
@@ -84,7 +83,6 @@ export const useEventTracker = () => {
             'register_started',
             'register_started_box',
             'register_wallet_opened',
-            'verify_ownership_started_dns',
             'claim_domain_started_dns',
             'commit_wallet_opened_dns',
             'register_started_dns',