5276: Use using_wait_time 0 instead of evaluate_script.

diff --git a/apps/workbench/test/integration/websockets_test.rb b/apps/workbench/test/integration/websockets_test.rb
index 2b3ba8c41c4e6803d99eb466e6e6957dbc0aa99b..648d59c69000b19cfca089d4aac154da5e2529c7 100644 (file)
--- a/apps/workbench/test/integration/websockets_test.rb
+++ b/apps/workbench/test/integration/websockets_test.rb
@@ -184,11 +184,13 @@ class WebsocketTest < ActionDispatch::IntegrationTest
       end
     end
 
-    # Graph should have appeared (even if it hadn't above). We don't
-    # use capybara's matcher function because we don't want to wait:
-    # we are confirming the graph is visible _immediately_ after the
-    # first data point arrives.
-    assert page.evaluate_script("$('#log_graph_div').is(':visible')")
+    # Graph should have appeared (even if it hadn't above). It's
+    # important not to wait like matchers usually do: we are
+    # confirming the graph is visible _immediately_ after the first
+    # data point arrives.
+    using_wait_time 0 do
+      assert_selector '#log_graph_div', visible: true
+    end
     assert_last_datapoint 'T1-cpu', (((35.39+0.86)/10.0002)/8)
   end