remove deleteNotebook func and step for afterEach

Author: dididy

zeppelin-web-angular/e2e/tests/notebook/action-bar/action-bar-functionality.spec.ts

@@ -17,8 +17,7 @@ import {
   performLoginIfRequired,
   waitForZeppelinReady,
   PAGES,
-  createTestNotebook,
-  deleteTestNotebook
+  createTestNotebook
 } from '../../../utils';
 
 test.describe('Notebook Action Bar Functionality', () => {
@@ -38,12 +37,6 @@ test.describe('Notebook Action Bar Functionality', () => {
     await page.waitForLoadState('networkidle');
   });
 
-  test.afterEach(async ({ page }) => {
-    if (testNotebook?.noteId) {
-      await deleteTestNotebook(page, testNotebook.noteId);
-    }
-  });
-
   test('should display and allow title editing with tooltip', async ({ page }) => {
     // Then: Title editor should be functional with proper tooltip
     const actionBarUtil = new NotebookActionBarUtil(page);

zeppelin-web-angular/e2e/tests/notebook/keyboard/notebook-keyboard-shortcuts.spec.ts

@@ -19,8 +19,7 @@ import {
   waitForNotebookLinks,
   waitForZeppelinReady,
   PAGES,
-  createTestNotebook,
-  deleteTestNotebook
+  createTestNotebook
 } from '../../../utils';
 
 /**
@@ -60,10 +59,6 @@ test.describe.serial('Comprehensive Keyboard Shortcuts (ShortcutsMap)', () => {
   test.afterEach(async ({ page }) => {
     // Clean up any open dialogs or modals
     await page.keyboard.press('Escape');
-
-    if (testNotebook?.noteId) {
-      await deleteTestNotebook(page, testNotebook.noteId);
-    }
   });
 
   // ===== CORE EXECUTION SHORTCUTS =====

zeppelin-web-angular/e2e/tests/notebook/main/notebook-container.spec.ts

@@ -17,8 +17,7 @@ import {
   performLoginIfRequired,
   waitForZeppelinReady,
   PAGES,
-  createTestNotebook,
-  deleteTestNotebook
+  createTestNotebook
 } from '../../../utils';
 
 test.describe('Notebook Container Component', () => {
@@ -38,12 +37,6 @@ test.describe('Notebook Container Component', () => {
     await page.waitForLoadState('networkidle');
   });
 
-  test.afterEach(async ({ page }) => {
-    if (testNotebook?.noteId) {
-      await deleteTestNotebook(page, testNotebook.noteId);
-    }
-  });
-
   test('should display notebook container with proper structure', async ({ page }) => {
     // Then: Notebook container should be properly structured
     const notebookUtil = new NotebookPageUtil(page);

zeppelin-web-angular/e2e/tests/notebook/paragraph/paragraph-functionality.spec.ts

@@ -18,8 +18,7 @@ import {
   performLoginIfRequired,
   waitForZeppelinReady,
   PAGES,
-  createTestNotebook,
-  deleteTestNotebook
+  createTestNotebook
 } from '../../../utils';
 
 test.describe('Notebook Paragraph Functionality', () => {
@@ -40,12 +39,6 @@ test.describe('Notebook Paragraph Functionality', () => {
     await page.waitForLoadState('networkidle');
   });
 
-  test.afterEach(async ({ page }) => {
-    if (testNotebook?.noteId) {
-      await deleteTestNotebook(page, testNotebook.noteId);
-    }
-  });
-
   test('should display paragraph container with proper structure', async ({ page }) => {
     // Then: Paragraph container should be visible with proper structure
     const paragraphUtil = new NotebookParagraphUtil(page);

zeppelin-web-angular/e2e/tests/notebook/published/published-paragraph.spec.ts

@@ -19,8 +19,7 @@ import {
   waitForNotebookLinks,
   waitForZeppelinReady,
   PAGES,
-  createTestNotebook,
-  deleteTestNotebook
+  createTestNotebook
 } from '../../../utils';
 
 test.describe('Published Paragraph', () => {
@@ -47,12 +46,6 @@ test.describe('Published Paragraph', () => {
     testNotebook = await createTestNotebook(page);
   });
 
-  test.afterEach(async ({ page }) => {
-    if (testNotebook?.noteId) {
-      await deleteTestNotebook(page, testNotebook?.noteId);
-    }
-  });
-
   test.describe('Error Handling', () => {
     test('should show error modal when notebook does not exist', async ({ page }) => {
       const nonExistentIds = testUtil.generateNonExistentIds();

zeppelin-web-angular/e2e/tests/notebook/sidebar/sidebar-functionality.spec.ts

@@ -17,8 +17,7 @@ import {
   performLoginIfRequired,
   waitForZeppelinReady,
   PAGES,
-  createTestNotebook,
-  deleteTestNotebook
+  createTestNotebook
 } from '../../../utils';
 
 test.describe('Notebook Sidebar Functionality', () => {
@@ -43,12 +42,6 @@ test.describe('Notebook Sidebar Functionality', () => {
     await page.waitForLoadState('networkidle');
   });
 
-  test.afterEach(async ({ page }) => {
-    if (testNotebook?.noteId) {
-      await deleteTestNotebook(page, testNotebook.noteId);
-    }
-  });
-
   test('should display navigation buttons', async () => {
     // Then: Navigation buttons should be visible
     await testUtil.verifyNavigationButtons();

zeppelin-web-angular/e2e/tests/share/folder-rename/folder-rename.spec.ts

@@ -18,14 +18,12 @@ import {
   PAGES,
   performLoginIfRequired,
   waitForZeppelinReady,
-  createTestNotebook,
-  deleteTestNotebook
+  createTestNotebook
 } from '../../../utils';
 
 test.describe.serial('Folder Rename', () => {
   let folderRenamePage: FolderRenamePage;
   let folderRenameUtil: FolderRenamePageUtil;
-  let testNotebook: { noteId: string; paragraphId: string };
   let testFolderName: string;
 
   addPageAnnotationBeforeEach(PAGES.SHARE.FOLDER_RENAME);
@@ -40,16 +38,7 @@ test.describe.serial('Folder Rename', () => {
 
     // Create a test notebook with folder structure
     testFolderName = `TestFolder_${Date.now()}`;
-    testNotebook = await createTestNotebook(page, testFolderName);
-    // testFolderName is now the folder that contains the notebook
-  });
-
-  test.afterEach(async ({ page }) => {
-    // Clean up the test notebook and folder
-    if (testNotebook?.noteId) {
-      await deleteTestNotebook(page, testNotebook.noteId);
-      await deleteTestNotebook(page, testFolderName);
-    }
+    await createTestNotebook(page, testFolderName);
   });
 
   test('Given folder exists in notebook list, When hovering over folder, Then context menu should appear', async () => {

zeppelin-web-angular/e2e/tests/share/note-rename/note-rename.spec.ts

@@ -18,8 +18,7 @@ import {
   PAGES,
   performLoginIfRequired,
   waitForZeppelinReady,
-  createTestNotebook,
-  deleteTestNotebook
+  createTestNotebook
 } from '../../../utils';
 
 test.describe('Note Rename', () => {
@@ -45,13 +44,6 @@ test.describe('Note Rename', () => {
     await page.waitForLoadState('networkidle');
   });
 
-  test.afterEach(async ({ page }) => {
-    // Clean up the test notebook after each test
-    if (testNotebook?.noteId) {
-      await deleteTestNotebook(page, testNotebook.noteId);
-    }
-  });
-
   test('Given notebook page is loaded, When checking note title, Then title should be displayed', async () => {
     await noteRenameUtil.verifyTitleIsDisplayed();
   });

zeppelin-web-angular/e2e/tests/share/note-toc/note-toc.spec.ts

@@ -18,8 +18,7 @@ import {
   PAGES,
   performLoginIfRequired,
   waitForZeppelinReady,
-  createTestNotebook,
-  deleteTestNotebook
+  createTestNotebook
 } from '../../../utils';
 
 test.describe('Note Table of Contents', () => {
@@ -53,12 +52,6 @@ test.describe('Note Table of Contents', () => {
     await expect(noteTocPage.tocToggleButton).toBeVisible({ timeout: 10000 });
   });
 
-  test.afterEach(async ({ page }) => {
-    if (testNotebook?.noteId) {
-      await deleteTestNotebook(page, testNotebook.noteId);
-    }
-  });
-
   test('Given notebook page is loaded, When clicking TOC toggle button, Then TOC panel should open', async () => {
     await noteTocUtil.verifyTocPanelOpens();
   });