test(actions): Unskip E2E test (#22959)

PostHog · Jun 13, 2024 · 6290dae · 6290dae
1 parent bd6f570
commit 6290dae
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/cypress/e2e/actions.cy.ts b/cypress/e2e/actions.cy.ts
@@ -40,16 +40,14 @@ describe('Action Events', () => {
         cy.get('[data-attr=trend-element-subject-1] span').should('contain', actionName)
     })
 
-    // FIXME: This test fails after the 3000 rework, as the input field for new actions
-    // doesn't get cleared
-    it.skip('Notifies when an action event with this name already exists', () => {
+    it('Notifies when an action event with this name already exists', () => {
         createAction(actionName)
         navigateToActionsTab()
         createAction(actionName)
         // Oh noes, there already is an action with name `actionName`
         cy.contains('Action with this name already exists').should('exist')
         // Let's see it
-        cy.contains('Click here to edit').click()
+        cy.contains('Edit it here').click()
         // We should now be seeing the action from "Create action"
         cy.get('[data-attr=edit-action-url-input]').should('have.value', Cypress.config().baseUrl)
     })

diff --git a/frontend/src/scenes/actions/actionEditLogic.tsx b/frontend/src/scenes/actions/actionEditLogic.tsx
@@ -90,7 +90,10 @@ export const actionEditLogic = kea<actionEditLogicType>([
 
                         lemonToast.error(
                             <>
-                                Action with this name already exists. <Link to={urls.action(dupeId)}>Edit it</Link>
+                                Action with this name already exists.{' '}
+                                <Link to={urls.action(dupeId)} target="_blank">
+                                    Edit it here
+                                </Link>
                             </>
                         )