14407: Fix "too much recursion" error.
authorTom Clegg <tclegg@veritasgenetics.com>
Thu, 25 Oct 2018 19:52:21 +0000 (15:52 -0400)
committerTom Clegg <tclegg@veritasgenetics.com>
Thu, 25 Oct 2018 19:52:21 +0000 (15:52 -0400)
The recursion error comes from evaluate_script() trying to serialize
the magical jQuery response. We don't need to see that response so
we can use execute_script() instead.

Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tclegg@veritasgenetics.com>

apps/workbench/test/integration_helper.rb

index 224de20f17307ff0f82269e79dcceadf4f1e1158..1af7245a4ea93a4c449be6e7aac65d6d08f24f98 100644 (file)
@@ -103,14 +103,14 @@ module AssertDomEvent
   # DOM element.
   def assert_triggers_dom_event events, target='body'
     magic = 'received-dom-event-' + rand(2**30).to_s(36)
-    page.evaluate_script <<eos
+    page.execute_script <<eos
       $('#{target}').one('#{events}', function() {
         $('body').addClass('#{magic}');
       });
 eos
     yield
     assert_selector "body.#{magic}"
-    page.evaluate_script "$('body').removeClass('#{magic}');";
+    page.execute_script "$('body').removeClass('#{magic}');";
   end
 end