[ARVADOS] updated: 38dea652d11a2a562623d80328d270707ed44887

git at public.curoverse.com git at public.curoverse.com
Thu Apr 10 16:44:52 EDT 2014


Summary of changes:
 apps/workbench/test/integration/users_test.rb |   19 ++++++++++++-------
 1 files changed, 12 insertions(+), 7 deletions(-)

       via  38dea652d11a2a562623d80328d270707ed44887 (commit)
      from  de3768fdd71eb58cf040d72b0d989deeb0bb75ae (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 38dea652d11a2a562623d80328d270707ed44887
Author: radhika chippada <radhika at curoverse.com>
Date:   Thu Apr 10 16:44:17 2014 -0400

    Fix the users test to get the created user id

diff --git a/apps/workbench/test/integration/users_test.rb b/apps/workbench/test/integration/users_test.rb
index aa5fdcc..22b92c0 100644
--- a/apps/workbench/test/integration/users_test.rb
+++ b/apps/workbench/test/integration/users_test.rb
@@ -67,15 +67,20 @@ class UsersTest < ActionDispatch::IntegrationTest
 
     # verify that the new user showed up in the users page
     assert page.has_text? 'foo at example.com'
-=begin
-    page.within(:xpath, '//tr[@data-object-uuid][1]') do
-      assert (text.include? 'foo at example.com false'), 'Expected email'
-      new_user_uuid = text.split[0]
 
-      # go to the new user's page
-      click_link new_user_uuid
+    new_user_uuid = nil
+    all("tr").each do |elem|
+      if elem.text.include? 'foo at example.com'
+        new_user_uuid = elem.text.split[0]
+        break
+      end
     end
 
+    assert new_user_uuid, "Expected new user uuid not found"
+
+    # go to the new user's page
+    click_link new_user_uuid
+
     assert page.has_text? 'modified_by_user_uuid'
     page.within(:xpath, '//a[@data-name="is_active"]') do
       assert_equal "false", text, "Expected new user's is_active to be false"
@@ -84,7 +89,7 @@ class UsersTest < ActionDispatch::IntegrationTest
     click_link 'Metadata'
     assert page.has_text? '(Repository: test_repo)'
     assert !(page.has_text? '(VirtualMachine:)')
-=end
+
     headless.stop
   end
 

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list