fix: correct typo in variable name and simplify e2e test navigation

Co-authored-by: Raubzeug <[email protected]>

Author: Copilot

src/containers/Node/Node.tsx

@@ -69,21 +69,21 @@ export function Node() {
     const isStorageNode = node?.Roles?.find((el) => el === STORAGE_ROLE);
 
     const {activeTab, nodeTabs} = React.useMemo(() => {
-        let actulaNodeTabs = isStorageNode
+        let actualNodeTabs = isStorageNode
             ? NODE_TABS
             : NODE_TABS.filter((el) => el.id !== 'storage');
         if (isDiskPagesAvailable) {
-            actulaNodeTabs = actulaNodeTabs.filter((el) => el.id !== 'structure');
+            actualNodeTabs = actualNodeTabs.filter((el) => el.id !== 'structure');
         }
         // Filter out threads tab if there's no thread data in the API response
         if (!node?.Threads || node.Threads.length === 0) {
-            actulaNodeTabs = actulaNodeTabs.filter((el) => el.id !== 'threads');
+            actualNodeTabs = actualNodeTabs.filter((el) => el.id !== 'threads');
         }
 
         const actualActiveTab =
-            actulaNodeTabs.find(({id}) => id === activeTabId) ?? actulaNodeTabs[0];
+            actualNodeTabs.find(({id}) => id === activeTabId) ?? actualNodeTabs[0];
 
-        return {activeTab: actualActiveTab, nodeTabs: actulaNodeTabs};
+        return {activeTab: actualActiveTab, nodeTabs: actualNodeTabs};
     }, [isStorageNode, isDiskPagesAvailable, activeTabId, node?.Threads]);
 
     const tenantName = node?.Tenants?.[0] || tenantNameFromQuery?.toString();

tests/suites/nodes/nodes.test.ts

@@ -214,27 +214,9 @@ test.describe('Test Node Page Threads Tab', async () => {
             });
         });
 
-        const nodesPage = new NodesPage(page);
-        await nodesPage.goto();
-
-        // Get first node ID to navigate to
-        const paginatedTable = new ClusterNodesTable(page);
-        await paginatedTable.waitForTableToLoad();
-        await paginatedTable.waitForTableData();
-
-        // Click on first node to navigate to node page
-        const firstRowLink = page
-            .locator('.ydb-paginated-table__table tbody tr:first-child a')
-            .first();
-        await firstRowLink.click();
-
-        // Wait for navigation to complete
-        await page.waitForURL(/\/node\/\d+/);
-
-        const nodeId = await page.url().match(/\/node\/(\d+)/)?.[1];
-        expect(nodeId).toBeDefined();
-
-        const nodePage = new NodePage(page, nodeId!);
+        // Navigate directly to node page
+        const nodePage = new NodePage(page, '1');
+        await nodePage.goto();
         await nodePage.waitForNodePageLoad();
 
         // Verify threads tab is not visible
@@ -270,27 +252,9 @@ test.describe('Test Node Page Threads Tab', async () => {
             });
         });
 
-        const nodesPage = new NodesPage(page);
-        await nodesPage.goto();
-
-        // Get first node ID to navigate to
-        const paginatedTable = new ClusterNodesTable(page);
-        await paginatedTable.waitForTableToLoad();
-        await paginatedTable.waitForTableData();
-
-        // Click on first node to navigate to node page
-        const firstRowLink = page
-            .locator('.ydb-paginated-table__table tbody tr:first-child a')
-            .first();
-        await firstRowLink.click();
-
-        // Wait for navigation to complete
-        await page.waitForURL(/\/node\/\d+/);
-
-        const nodeId = await page.url().match(/\/node\/(\d+)/)?.[1];
-        expect(nodeId).toBeDefined();
-
-        const nodePage = new NodePage(page, nodeId!);
+        // Navigate directly to node page
+        const nodePage = new NodePage(page, '1');
+        await nodePage.goto();
         await nodePage.waitForNodePageLoad();
 
         // Verify threads tab is visible
@@ -325,27 +289,9 @@ test.describe('Test Node Page Threads Tab', async () => {
             });
         });
 
-        const nodesPage = new NodesPage(page);
-        await nodesPage.goto();
-
-        // Get first node ID to navigate to
-        const paginatedTable = new ClusterNodesTable(page);
-        await paginatedTable.waitForTableToLoad();
-        await paginatedTable.waitForTableData();
-
-        // Click on first node to navigate to node page
-        const firstRowLink = page
-            .locator('.ydb-paginated-table__table tbody tr:first-child a')
-            .first();
-        await firstRowLink.click();
-
-        // Wait for navigation to complete
-        await page.waitForURL(/\/node\/\d+/);
-
-        const nodeId = await page.url().match(/\/node\/(\d+)/)?.[1];
-        expect(nodeId).toBeDefined();
-
-        const nodePage = new NodePage(page, nodeId!);
+        // Navigate directly to node page
+        const nodePage = new NodePage(page, '1');
+        await nodePage.goto();
         await nodePage.waitForNodePageLoad();
 
         // Verify threads tab is not visible