test: remove redundant profile rename tests from SettingsView.spec.tsx

Author: NoahJenkins

webview-ui/src/components/settings/__tests__/SettingsView.spec.tsx

@@ -681,194 +681,3 @@ describe("SettingsView - Duplicate Commands", () => {
 		)
 	})
 })
-
-describe("SettingsView - Profile Rename Change Detection", () => {
-	beforeEach(() => {
-		vi.clearAllMocks()
-	})
-
-	it("enables save button when profile is renamed", () => {
-		// Render settings view
-		renderSettingsView()
-
-		// Start rename process
-		const renameButton = screen.getByTestId("rename-profile-button")
-		fireEvent.click(renameButton)
-
-		// Enter new name and confirm
-		const renameInput = screen.getByTestId("rename-input")
-		fireEvent.change(renameInput, { target: { value: "newProfileName" } })
-
-		const confirmButton = screen.getByTestId("confirm-rename-button")
-		fireEvent.click(confirmButton)
-
-		// Save button should now be enabled due to change detection
-		const saveButton = screen.getByTestId("save-button")
-		expect(saveButton).not.toBeDisabled()
-	})
-
-	it("sends rename message to vscode when profile is renamed", () => {
-		// Render settings view
-		renderSettingsView()
-
-		// Start rename process
-		const renameButton = screen.getByTestId("rename-profile-button")
-		fireEvent.click(renameButton)
-
-		// Enter new name and confirm
-		const renameInput = screen.getByTestId("rename-input")
-		fireEvent.change(renameInput, { target: { value: "renamedProfile" } })
-
-		const confirmButton = screen.getByTestId("confirm-rename-button")
-		fireEvent.click(confirmButton)
-
-		// Verify that the rename message was sent to vscode with dynamic values
-		// Note: The actual oldName will depend on the current API config name from the extension state
-		expect(vscode.postMessage).toHaveBeenCalledWith(
-			expect.objectContaining({
-				type: "renameApiConfiguration",
-				values: expect.objectContaining({
-					newName: "renamedProfile",
-				}),
-			}),
-		)
-	})
-
-	it("saves changes when save button is clicked after profile rename", () => {
-		// Render settings view
-		renderSettingsView()
-
-		// Start rename process
-		const renameButton = screen.getByTestId("rename-profile-button")
-		fireEvent.click(renameButton)
-
-		// Enter new name and confirm
-		const renameInput = screen.getByTestId("rename-input")
-		fireEvent.change(renameInput, { target: { value: "renamedProfile" } })
-
-		const confirmButton = screen.getByTestId("confirm-rename-button")
-		fireEvent.click(confirmButton)
-
-		// Click save button
-		const saveButton = screen.getByTestId("save-button")
-		fireEvent.click(saveButton)
-
-		// Verify that all settings are saved, including the renamed configuration
-		expect(vscode.postMessage).toHaveBeenCalledWith(
-			expect.objectContaining({
-				type: "upsertApiConfiguration",
-			}),
-		)
-	})
-
-	it("disables save button after successful save following profile rename", () => {
-		// Render settings view
-		renderSettingsView()
-
-		// Start rename process
-		const renameButton = screen.getByTestId("rename-profile-button")
-		fireEvent.click(renameButton)
-
-		// Enter new name and confirm
-		const renameInput = screen.getByTestId("rename-input")
-		fireEvent.change(renameInput, { target: { value: "renamedProfile" } })
-
-		const confirmButton = screen.getByTestId("confirm-rename-button")
-		fireEvent.click(confirmButton)
-
-		// Verify save button is enabled
-		const saveButton = screen.getByTestId("save-button")
-		expect(saveButton).not.toBeDisabled()
-
-		// Click save button
-		fireEvent.click(saveButton)
-
-		// Save button should be disabled again after save
-		expect(saveButton).toBeDisabled()
-	})
-
-	it("shows unsaved changes dialog when attempting to leave after profile rename", () => {
-		// Render with activateTab helper
-		const { onDone } = renderSettingsView()
-
-		// Start rename process
-		const renameButton = screen.getByTestId("rename-profile-button")
-		fireEvent.click(renameButton)
-
-		// Enter new name and confirm to create unsaved changes
-		const renameInput = screen.getByTestId("rename-input")
-		fireEvent.change(renameInput, { target: { value: "renamedProfile" } })
-
-		const confirmButton = screen.getByTestId("confirm-rename-button")
-		fireEvent.click(confirmButton)
-
-		// Try to leave by clicking Done
-		const doneButton = screen.getByText("settings:common.done")
-		fireEvent.click(doneButton)
-
-		// Should show unsaved changes dialog
-		expect(screen.getByText("settings:unsavedChangesDialog.title")).toBeInTheDocument()
-		expect(screen.getByText("settings:unsavedChangesDialog.description")).toBeInTheDocument()
-
-		// Verify onDone was not called yet
-		expect(onDone).not.toHaveBeenCalled()
-	})
-
-	it("allows leaving after discarding changes from profile rename", () => {
-		// Render with activateTab helper
-		const { onDone } = renderSettingsView()
-
-		// Start rename process
-		const renameButton = screen.getByTestId("rename-profile-button")
-		fireEvent.click(renameButton)
-
-		// Enter new name and confirm to create unsaved changes
-		const renameInput = screen.getByTestId("rename-input")
-		fireEvent.change(renameInput, { target: { value: "renamedProfile" } })
-
-		const confirmButton = screen.getByTestId("confirm-rename-button")
-		fireEvent.click(confirmButton)
-
-		// Try to leave by clicking Done
-		const doneButton = screen.getByText("settings:common.done")
-		fireEvent.click(doneButton)
-
-		// Click discard in the dialog
-		const discardButton = screen.getByTestId("alert-dialog-action")
-		fireEvent.click(discardButton)
-
-		// Verify onDone was called
-		expect(onDone).toHaveBeenCalled()
-	})
-
-	it("stays on page when canceling discard dialog after profile rename", () => {
-		// Render with activateTab helper
-		const { onDone } = renderSettingsView()
-
-		// Start rename process
-		const renameButton = screen.getByTestId("rename-profile-button")
-		fireEvent.click(renameButton)
-
-		// Enter new name and confirm to create unsaved changes
-		const renameInput = screen.getByTestId("rename-input")
-		fireEvent.change(renameInput, { target: { value: "renamedProfile" } })
-
-		const confirmButton = screen.getByTestId("confirm-rename-button")
-		fireEvent.click(confirmButton)
-
-		// Try to leave by clicking Done
-		const doneButton = screen.getByText("settings:common.done")
-		fireEvent.click(doneButton)
-
-		// Click cancel in the dialog
-		const cancelButton = screen.getByTestId("alert-dialog-cancel")
-		fireEvent.click(cancelButton)
-
-		// Verify onDone was not called
-		expect(onDone).not.toHaveBeenCalled()
-
-		// Verify save button is still enabled (changes not discarded)
-		const saveButton = screen.getByTestId("save-button")
-		expect(saveButton).not.toBeDisabled()
-	})
-})