X-Git-Url: https://git.arvados.org/arvados.git/blobdiff_plain/fe0d9b8e3a46c610deb95b9a8b501e8c2aad9f54..50128b53da4003912635b03fb27b5be2c5beaca1:/apps/workbench/test/integration/users_test.rb

diff --git a/apps/workbench/test/integration/users_test.rb b/apps/workbench/test/integration/users_test.rb
index 39374e7c70..140cb56127 100644
--- a/apps/workbench/test/integration/users_test.rb
+++ b/apps/workbench/test/integration/users_test.rb
@@ -1,21 +1,20 @@
 require 'integration_helper'
-#require "selenium-webdriver"
-#require 'headless'
 
 class UsersTest < ActionDispatch::IntegrationTest
 
   test "login as active user but not admin" do
-    Capybara.current_driver = Capybara.javascript_driver
+    need_javascript
     visit page_with_token('active_trustedclient')
 
     assert page.has_no_link? 'Users' 'Found Users link for non-admin user'
   end
 
   test "login as admin user and verify active user data" do
-    Capybara.current_driver = Capybara.javascript_driver
+    need_javascript
     visit page_with_token('admin_trustedclient')
 
     # go to Users list page
+    find('#system-menu').click
     click_link 'Users'
 
     # check active user attributes in the list page
@@ -23,55 +22,138 @@ class UsersTest < ActionDispatch::IntegrationTest
       assert (text.include? 'true false'), 'Expected is_active'
     end
 
-    click_link 'zzzzz-tpzed-xurymjxw79nv3jz'
+    find('tr', text: 'zzzzz-tpzed-xurymjxw79nv3jz').
+      find('a', text: 'Show').
+      click
     assert page.has_text? 'Attributes'
-    assert page.has_text? 'Metadata'
+    assert page.has_text? 'Advanced'
     assert page.has_text? 'Admin'
 
     # go to the Attributes tab
     click_link 'Attributes'
     assert page.has_text? 'modified_by_user_uuid'
-    page.within(:xpath, '//a[@data-name="is_active"]') do
+    page.within(:xpath, '//span[@data-name="is_active"]') do
       assert_equal "true", text, "Expected user's is_active to be true"
     end
-    page.within(:xpath, '//a[@data-name="is_admin"]') do
+    page.within(:xpath, '//span[@data-name="is_admin"]') do
       assert_equal "false", text, "Expected user's is_admin to be false"
     end
 
   end
 
   test "create a new user" do
-    Capybara.current_driver = Capybara.javascript_driver
+    need_javascript
+
     visit page_with_token('admin_trustedclient')
 
+    find('#system-menu').click
     click_link 'Users'
 
     assert page.has_text? 'zzzzz-tpzed-d9tiejq69daie8f'
 
-    click_on 'Add a new user'
-    
-    # for now just check that we are back in Users -> List page
-    assert page.has_text? 'zzzzz-tpzed-d9tiejq69daie8f'
+    click_link 'Add a new user'
+
+    within '.modal-content' do
+      find 'label', text: 'Virtual Machine'
+      fill_in "email", :with => "foo@example.com"
+      click_button "Submit"
+      wait_for_ajax
+    end
+
+    visit '/users'
+
+    # verify that the new user showed up in the users page and find
+    # the new user's UUID
+    new_user_uuid =
+      find('tr[data-object-uuid]', text: 'foo@example.com')['data-object-uuid']
+    assert new_user_uuid, "Expected new user uuid not found"
+
+    # go to the new user's page
+    find('tr', text: new_user_uuid).
+      find('a', text: 'Show').
+      click
+
+    assert page.has_text? 'modified_by_user_uuid'
+    page.within(:xpath, '//span[@data-name="is_active"]') do
+      assert_equal "false", text, "Expected new user's is_active to be false"
+    end
+
+    click_link 'Advanced'
+    click_link 'Metadata'
+    assert page.has_text? 'can_login' # make sure page is rendered / ready
+    assert page.has_no_text? 'VirtualMachine:'
+  end
+
+  test "setup the active user" do
+    need_javascript
+    visit page_with_token('admin_trustedclient')
+
+    find('#system-menu').click
+    click_link 'Users'
+
+    # click on active user
+    find('tr', text: 'zzzzz-tpzed-xurymjxw79nv3jz').
+      find('a', text: 'Show').
+      click
+    user_url = page.current_url
+
+    # Setup user
+    click_link 'Admin'
+    assert page.has_text? 'As an admin, you can setup'
+
+    click_link 'Setup shell account for Active User'
+
+    within '.modal-content' do
+      find 'label', text: 'Virtual Machine'
+      click_button "Submit"
+    end
+
+    visit user_url
+    assert page.has_text? 'modified_by_client_uuid'
+
+    click_link 'Advanced'
+    click_link 'Metadata'
+    vm_links = all("a", text: "VirtualMachine:")
+    assert_equal(1, vm_links.size)
+    assert_equal("VirtualMachine: testvm2.shell", vm_links.first.text)
+
+    # Click on Setup button again and this time also choose a VM
+    click_link 'Admin'
+    click_link 'Setup shell account for Active User'
+
+    within '.modal-content' do
+      select("testvm.shell", :from => 'vm_uuid')
+      fill_in "groups", :with => "test group one, test-group-two"
+      click_button "Submit"
+    end
+
+    visit user_url
+    find '#Attributes', text: 'modified_by_client_uuid'
+
+    click_link 'Advanced'
+    click_link 'Metadata'
+    assert page.has_text? 'VirtualMachine: testvm.shell'
+    assert page.has_text? '["test group one", "test-group-two"]'
   end
 
-  #@headless
   test "unsetup active user" do
-    Capybara.current_driver = Capybara.javascript_driver
-    #Capybara.current_driver = :selenium
+    need_javascript
 
     visit page_with_token('admin_trustedclient')
 
+    find('#system-menu').click
     click_link 'Users'
 
-    assert page.has_link? 'zzzzz-tpzed-xurymjxw79nv3jz'
-
     # click on active user
-    click_link 'zzzzz-tpzed-xurymjxw79nv3jz'
+    find('tr', text: 'zzzzz-tpzed-xurymjxw79nv3jz').
+      find('a', text: 'Show').
+      click
+    user_url = page.current_url
 
     # Verify that is_active is set
-    click_link 'Attributes'
+    find('a,button', text: 'Attributes').click
     assert page.has_text? 'modified_by_user_uuid'
-    page.within(:xpath, '//a[@data-name="is_active"]') do
+    page.within(:xpath, '//span[@data-name="is_active"]') do
       assert_equal "true", text, "Expected user's is_active to be true"
     end
 
@@ -79,71 +161,63 @@ class UsersTest < ActionDispatch::IntegrationTest
     click_link 'Admin'
     assert page.has_text? 'As an admin, you can deactivate and reset this user'
 
-    # Click on Deactivate button
+    # unsetup user and verify all the above links are deleted
+    click_link 'Admin'
     click_button 'Deactivate Active User'
 
-    # Click Ok in the confirm dialog
-=begin
-#use with selenium
-    page.driver.browser.switch_to.alert.accept
-    sleep(0.1)
-    popup = page.driver.browser.window_handles.last
-    #popup = page.driver.browser.window_handle
-    page.within_window popup do
-      assert has_text? 'Are you sure you want to deactivate'
-      click_button "OK"
+    if Capybara.current_driver == :selenium
+      sleep(0.1)
+      page.driver.browser.switch_to.alert.accept
+    else
+      # poltergeist returns true for confirm(), so we don't need to accept.
     end
 
-# use with poltergeist driver
-popup = page.driver.window_handles.last
-page.within_window popup do
-  #fill_in "email", :with => "my_email"
-  assert has_text? 'Are you sure you want to deactivate'
-  click_button "OK"
-end
-=end
-
     # Should now be back in the Attributes tab for the user
     assert page.has_text? 'modified_by_user_uuid'
-    page.within(:xpath, '//a[@data-name="is_active"]') do
+    page.within(:xpath, '//span[@data-name="is_active"]') do
       assert_equal "false", text, "Expected user's is_active to be false after unsetup"
     end
-  end
 
-  test "setup the active user" do
-    Capybara.current_driver = Capybara.javascript_driver
-    visit page_with_token('admin_trustedclient')
+    click_link 'Advanced'
+    click_link 'Metadata'
+    assert page.has_no_text? 'VirtualMachine: testvm.shell'
 
-    click_link 'Users'
+    # setup user again and verify links present
+    click_link 'Admin'
+    click_link 'Setup shell account for Active User'
 
-    assert page.has_link? 'zzzzz-tpzed-xurymjxw79nv3jz'
+    within '.modal-content' do
+      select("testvm.shell", :from => 'vm_uuid')
+      click_button "Submit"
+    end
 
-    # click on active user
-    click_link 'zzzzz-tpzed-xurymjxw79nv3jz'
+    visit user_url
+    assert page.has_text? 'modified_by_client_uuid'
 
-    # go to Admin tab
-    click_link 'Admin'
-    assert page.has_text? 'As an admin, you can deactivate and reset this user'
+    click_link 'Advanced'
+    click_link 'Metadata'
+    assert page.has_text? 'VirtualMachine: testvm.shell'
+  end
 
-=begin
-    # Click on Setup button
-    click_button 'Setup Active User'
+  test "test add group button" do
+    need_javascript
 
-    # Click Ok in the confirm dialog
-    sleep(0.1)
+    user_url = "/users/#{api_fixture('users')['active']['uuid']}"
+    visit page_with_token('admin_trustedclient', user_url)
 
-    popup = page.driver.window_handles.last
-    page.within_window popup do
-      assert has_text? 'Are you sure you want to deactivate Active User'
-      fill_in "email", :with => "test@example.com"
-      click_button "Ok"
-    end
+    # Setup user
+    click_link 'Admin'
+    assert page.has_text? 'As an admin, you can setup'
 
-    # Should now be back in the Attributes tab for the user
-    assert page.has_text? 'modified_by_client_uuid'
+    click_link 'Add new group'
 
-    puts "\n\n************* page now = \n#{page.body}"
-=end
-  end
+    within '.modal-content' do
+      fill_in "group_name_input", :with => "test-group-added-in-modal"
+      click_button "Create"
+    end
+    wait_for_ajax
 
+    # Back in the user "Admin" tab
+    assert page.has_text? 'test-group-added-in-modal'
+  end
 end