[ARVADOS] updated: 1.2.0-256-g62f6c1f0b

Git user git at public.curoverse.com
Thu Oct 25 15:54:00 EDT 2018


Summary of changes:
 apps/workbench/test/integration_helper.rb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

       via  62f6c1f0be5ebd4911ca0891e2b68db4c1e07a9a (commit)
      from  8aeabdc8bbcc545e07e1a0569f1c26583d89d68d (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 62f6c1f0be5ebd4911ca0891e2b68db4c1e07a9a
Author: Tom Clegg <tclegg at veritasgenetics.com>
Date:   Thu Oct 25 15:52:21 2018 -0400

    14407: Fix "too much recursion" error.
    
    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 at veritasgenetics.com>

diff --git a/apps/workbench/test/integration_helper.rb b/apps/workbench/test/integration_helper.rb
index 224de20f1..1af7245a4 100644
--- a/apps/workbench/test/integration_helper.rb
+++ b/apps/workbench/test/integration_helper.rb
@@ -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
 

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list