Retry refute_has

ftes · ftes · commit 7bd49bd55da6 · 2025-06-30T14:23:41.000+02:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -5,6 +5,10 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [tbd]
+### Fixed
+- `refute_has/3` add retry, don't fail if element initially found
+
 ## [0.7.0] 2025-06-16
 ### Added
 - Dialog handling
diff --git a/lib/phoenix_test/playwright.ex b/lib/phoenix_test/playwright.ex
@@ -608,9 +608,11 @@ defmodule PhoenixTest.Playwright do
 
   @doc false
   def refute_has(conn, selector, opts) do
-    if found?(conn, selector, opts) do
-      flunk("Found element #{selector} #{inspect(opts)}")
-    end
+    retry(fn ->
+      if found?(conn, selector, opts) do
+        flunk("Found element #{selector} #{inspect(opts)}")
+      end
+    end)
 
     conn
   end
diff --git a/test/phoenix_test/playwright_test.exs b/test/phoenix_test/playwright_test.exs
@@ -703,6 +703,13 @@ defmodule PhoenixTest.PlaywrightTest do
         |> refute_has(".multiple_links", count: 2)
       end
     end
+
+    test "retries if element initially visible", %{conn: conn} do
+      conn
+      |> visit("/live/index")
+      |> click_button("Button with push navigation")
+      |> refute_has("h1", text: "main page")
+    end
   end
 
   describe "refute_has/3" do
