[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