services/api/test/unit/user_test.rb

   1 # Copyright (C) The Arvados Authors. All rights reserved.
   2 #
   3 # SPDX-License-Identifier: AGPL-3.0
   4
   5 require 'test_helper'
   6
   7 class UserTest < ActiveSupport::TestCase
   8   include CurrentApiClient
   9
  10   # The fixture services/api/test/fixtures/users.yml serves as the input for this test case
  11   setup do
  12     # Make sure system_user exists before making "pre-test users" list
  13     system_user
  14   end
  15
  16   %w(a aa a0 aA Aa AA A0).each do |username|
  17     test "#{username.inspect} is a valid username" do
  18       user = User.new(username: username)
  19       assert(user.valid?)
  20     end
  21   end
  22
  23   test "username is not required" do
  24     user = User.new(username: nil)
  25     assert(user.valid?)
  26   end
  27
  28   test "username beginning with numeral is invalid" do
  29     user = User.new(username: "0a")
  30     refute(user.valid?)
  31   end
  32
  33   "\\.-_/!@#$%^&*()[]{}".each_char do |bad_char|
  34     test "username containing #{bad_char.inspect} is invalid" do
  35       user = User.new(username: "bad#{bad_char}username")
  36       refute(user.valid?)
  37     end
  38   end
  39
  40   test "username must be unique" do
  41     user = User.new(username: users(:active).username)
  42     refute(user.valid?)
  43   end
  44
  45   test "non-admin can't update username" do
  46     set_user_from_auth :rominiadmin
  47     user = User.find_by_uuid(users(:rominiadmin).uuid)
  48     user.username = "selfupdate"
  49     assert_not_allowed { user.save }
  50   end
  51
  52   def check_admin_username_change(fixture_name)
  53     set_user_from_auth :admin_trustedclient
  54     user = User.find_by_uuid(users(fixture_name).uuid)
  55     user.username = "newnamefromtest"
  56     assert(user.save)
  57   end
  58
  59   test "admin can set username" do
  60     check_admin_username_change(:active_no_prefs)
  61   end
  62
  63   test "admin can update username" do
  64     check_admin_username_change(:active)
  65   end
  66
  67   test "admin can update own username" do
  68     check_admin_username_change(:admin)
  69   end
  70
  71   def check_new_username_setting(email_name, expect_name)
  72     set_user_from_auth :admin
  73     user = User.create!(email: "#{email_name}@example.org")
  74     assert_equal(expect_name, user.username)
  75   end
  76
  77   test "new username set from e-mail" do
  78     check_new_username_setting("dakota", "dakota")
  79   end
  80
  81   test "new username set from e-mail with leading digits" do
  82     check_new_username_setting("1dakota9", "dakota9")
  83   end
  84
  85   test "new username set from e-mail with punctuation" do
  86     check_new_username_setting("dakota.9", "dakota9")
  87   end
  88
  89   test "new username set from e-mail with leading digits and punctuation" do
  90     check_new_username_setting("1.dakota.z", "dakotaz")
  91   end
  92
  93   test "new username set from e-mail with extra part" do
  94     check_new_username_setting("dakota+arvados", "dakota")
  95   end
  96
  97   test "new username set with deduplication" do
  98     name = users(:active).username
  99     check_new_username_setting(name, "#{name}2")
 100     check_new_username_setting(name, "#{name}3")
 101     # Insert some out-of-order conflicts, to ensure our "sort by
 102     # username, stop when we see a hole" strategy doesn't depend on
 103     # insert order.
 104     check_new_username_setting("#{name}13", "#{name}13")
 105     check_new_username_setting("#{name}5", "#{name}5")
 106     check_new_username_setting(name, "#{name}4")
 107     6.upto(12).each do |n|
 108       check_new_username_setting(name, "#{name}#{n}")
 109     end
 110   end
 111
 112   test "new username set avoiding blacklist" do
 113     Rails.configuration.Users.AutoSetupUsernameBlacklist = ConfigLoader.to_OrderedOptions({"root"=>{}})
 114     check_new_username_setting("root", "root2")
 115   end
 116
 117   test "no username set when no base available" do
 118     check_new_username_setting("_", nil)
 119   end
 120
 121   test "updating username updates repository names" do
 122     set_user_from_auth :admin
 123     user = users(:active)
 124     user.username = "newtestname"
 125     assert(user.save, "username update failed")
 126     {foo: "newtestname/foo", repository2: "newtestname/foo2"}.
 127         each_pair do |repo_sym, expect_name|
 128       assert_equal(expect_name, repositories(repo_sym).name)
 129     end
 130   end
 131
 132   test "admin can clear username when user owns no repositories" do
 133     set_user_from_auth :admin
 134     user = users(:spectator)
 135     user.username = nil
 136     assert(user.save)
 137     assert_nil(user.username)
 138   end
 139
 140   test "admin can't clear username when user owns repositories" do
 141     set_user_from_auth :admin
 142     user = users(:active)
 143     user.username = nil
 144     assert_not_allowed { user.save }
 145     refute_empty(user.errors[:username])
 146   end
 147
 148   test "failed username update doesn't change repository names" do
 149     set_user_from_auth :admin
 150     user = users(:active)
 151     user.username = users(:fuse).username
 152     assert_not_allowed { user.save }
 153     assert_equal("active/foo", repositories(:foo).name)
 154   end
 155
 156   [[false, 'foo@example.com', true, nil],
 157    [false, 'bar@example.com', nil, true],
 158    [true, 'foo@example.com', true, nil],
 159    [true, 'bar@example.com', true, true],
 160    [false, '', nil, nil],
 161    [true, '', true, nil]
 162   ].each do |auto_admin_first_user_config, auto_admin_user_config, foo_should_be_admin, bar_should_be_admin|
 163     # In each case, 'foo' is created first, then 'bar', then 'bar2', then 'baz'.
 164     test "auto admin with auto_admin_first=#{auto_admin_first_user_config} auto_admin=#{auto_admin_user_config}" do
 165
 166       if auto_admin_first_user_config
 167         # This test requires no admin users exist (except for the system user)
 168         act_as_system_user do
 169           users(:admin).update_attributes!(is_admin: false)
 170         end
 171         @all_users = User.where("uuid not like '%-000000000000000'").where(:is_admin => true)
 172         assert_equal 0, @all_users.count, "No admin users should exist (except for the system user)"
 173       end
 174
 175       Rails.configuration.Users.AutoAdminFirstUser = auto_admin_first_user_config
 176       Rails.configuration.Users.AutoAdminUserWithEmail = auto_admin_user_config
 177
 178       # See if the foo user has is_admin
 179       foo = User.new
 180       foo.first_name = 'foo'
 181       foo.email = 'foo@example.com'
 182
 183       act_as_system_user do
 184         foo.save!
 185       end
 186
 187       foo = User.find(foo.id)   # get the user back
 188       assert_equal foo_should_be_admin, foo.is_admin, "is_admin is wrong for user foo"
 189       assert_equal 'foo', foo.first_name
 190
 191       # See if the bar user has is_admin
 192       bar = User.new
 193       bar.first_name = 'bar'
 194       bar.email = 'bar@example.com'
 195
 196       act_as_system_user do
 197         bar.save!
 198       end
 199
 200       bar = User.find(bar.id)   # get the user back
 201       assert_equal bar_should_be_admin, bar.is_admin, "is_admin is wrong for user bar"
 202       assert_equal 'bar', bar.first_name
 203
 204       # A subsequent user with the bar@example.com address should never be
 205       # elevated to admin
 206       bar2 = User.new
 207       bar2.first_name = 'bar2'
 208       bar2.email = 'bar@example.com'
 209
 210       act_as_system_user do
 211         bar2.save!
 212       end
 213
 214       bar2 = User.find(bar2.id)   # get the user back
 215       assert !bar2.is_admin, "is_admin is wrong for user bar2"
 216       assert_equal 'bar2', bar2.first_name
 217
 218       # An ordinary new user should not be elevated to admin
 219       baz = User.new
 220       baz.first_name = 'baz'
 221       baz.email = 'baz@example.com'
 222
 223       act_as_system_user do
 224         baz.save!
 225       end
 226
 227       baz = User.find(baz.id)   # get the user back
 228       assert !baz.is_admin
 229       assert_equal 'baz', baz.first_name
 230
 231     end
 232   end
 233
 234   test "check non-admin active user properties" do
 235     @active_user = users(:active)     # get the active user
 236     assert !@active_user.is_admin, 'is_admin should not be set for a non-admin user'
 237     assert @active_user.is_active, 'user should be active'
 238     assert @active_user.is_invited, 'is_invited should be set'
 239     assert_not_nil @active_user.prefs, "user's preferences should be non-null, but may be size zero"
 240     assert (@active_user.can? :read=>"#{@active_user.uuid}"), "user should be able to read own object"
 241     assert (@active_user.can? :write=>"#{@active_user.uuid}"), "user should be able to write own object"
 242     assert (@active_user.can? :manage=>"#{@active_user.uuid}"), "user should be able to manage own object"
 243
 244     assert @active_user.groups_i_can(:read).size > 0, "active user should be able read at least one group"
 245
 246     # non-admin user cannot manage or write other user objects
 247     @uninvited_user = users(:inactive_uninvited)     # get the uninvited user
 248     assert !(@active_user.can? :read=>"#{@uninvited_user.uuid}")
 249     assert !(@active_user.can? :write=>"#{@uninvited_user.uuid}")
 250     assert !(@active_user.can? :manage=>"#{@uninvited_user.uuid}")
 251   end
 252
 253   test "check admin user properties" do
 254     @admin_user = users(:admin)     # get the admin user
 255     assert @admin_user.is_admin, 'is_admin should be set for admin user'
 256     assert @admin_user.is_active, 'admin user cannot be inactive'
 257     assert @admin_user.is_invited, 'is_invited should be set'
 258     assert_not_nil @admin_user.uuid.size, "user's uuid should be non-null"
 259     assert_not_nil @admin_user.prefs, "user's preferences should be non-null, but may be size zero"
 260     assert @admin_user.identity_url.size > 0, "user's identity url is expected"
 261     assert @admin_user.can? :read=>"#{@admin_user.uuid}"
 262     assert @admin_user.can? :write=>"#{@admin_user.uuid}"
 263     assert @admin_user.can? :manage=>"#{@admin_user.uuid}"
 264
 265     assert @admin_user.groups_i_can(:read).size > 0, "admin active user should be able read at least one group"
 266     assert @admin_user.groups_i_can(:write).size > 0, "admin active user should be able write to at least one group"
 267     assert @admin_user.groups_i_can(:manage).size > 0, "admin active user should be able manage at least one group"
 268
 269     # admin user can also write or manage other users
 270     @uninvited_user = users(:inactive_uninvited)     # get the uninvited user
 271     assert @admin_user.can? :read=>"#{@uninvited_user.uuid}"
 272     assert @admin_user.can? :write=>"#{@uninvited_user.uuid}"
 273     assert @admin_user.can? :manage=>"#{@uninvited_user.uuid}"
 274   end
 275
 276   test "check inactive and uninvited user properties" do
 277     @uninvited_user = users(:inactive_uninvited)     # get the uninvited user
 278     assert !@uninvited_user.is_admin, 'is_admin should not be set for a non-admin user'
 279     assert !@uninvited_user.is_active, 'user should be inactive'
 280     assert !@uninvited_user.is_invited, 'is_invited should not be set'
 281     assert @uninvited_user.can? :read=>"#{@uninvited_user.uuid}"
 282     assert @uninvited_user.can? :write=>"#{@uninvited_user.uuid}"
 283     assert @uninvited_user.can? :manage=>"#{@uninvited_user.uuid}"
 284
 285     assert_equal(@uninvited_user.groups_i_can(:read).sort,
 286                  [@uninvited_user.uuid, groups(:anonymous_group).uuid].sort)
 287     assert_equal(@uninvited_user.groups_i_can(:write),
 288                  [@uninvited_user.uuid])
 289     assert_equal(@uninvited_user.groups_i_can(:manage),
 290                  [@uninvited_user.uuid])
 291   end
 292
 293   test "find user method checks" do
 294     User.all.each do |user|
 295       assert_not_nil user.uuid, "non-null uuid expected for " + user.full_name
 296     end
 297
 298     user = users(:active)     # get the active user
 299
 300     found_user = User.find(user.id)   # find a user by the row id
 301
 302     assert_equal found_user.full_name, user.first_name + ' ' + user.last_name
 303     assert_equal found_user.identity_url, user.identity_url
 304   end
 305
 306   test "full name should not contain spurious whitespace" do
 307     set_user_from_auth :admin
 308
 309     user = User.create ({uuid: 'zzzzz-tpzed-abcdefghijklmno', email: 'foo@example.com' })
 310
 311     assert_equal '', user.full_name
 312
 313     user.first_name = 'John'
 314     user.last_name = 'Smith'
 315
 316     assert_equal user.first_name + ' ' + user.last_name, user.full_name
 317   end
 318
 319   test "create new user" do
 320     set_user_from_auth :admin
 321
 322     @all_users = User.all.to_a
 323
 324     user = User.new
 325     user.first_name = "first_name_for_newly_created_user"
 326     user.save
 327
 328     # verify there is one extra user in the db now
 329     assert_equal @all_users.size+1, User.all.count
 330
 331     user = User.find(user.id)   # get the user back
 332     assert_equal(user.first_name, 'first_name_for_newly_created_user')
 333     assert_not_nil user.uuid, 'uuid should be set for newly created user'
 334     assert_nil user.email, 'email should be null for newly created user, because it was not passed in'
 335     assert_nil user.identity_url, 'identity_url should be null for newly created user, because it was not passed in'
 336
 337     user.first_name = 'first_name_for_newly_created_user_updated'
 338     user.save
 339     user = User.find(user.id)   # get the user back
 340     assert_equal(user.first_name, 'first_name_for_newly_created_user_updated')
 341   end
 342
 343   active_notify_list = ConfigLoader.to_OrderedOptions({"active-notify@example.com"=>{}})
 344   inactive_notify_list = ConfigLoader.to_OrderedOptions({"inactive-notify@example.com"=>{}})
 345   empty_notify_list = ConfigLoader.to_OrderedOptions({})
 346
 347   test "create new user with notifications" do
 348     set_user_from_auth :admin
 349
 350     create_user_and_verify_setup_and_notifications true, active_notify_list, inactive_notify_list, nil, nil
 351     create_user_and_verify_setup_and_notifications true, active_notify_list, empty_notify_list, nil, nil
 352     create_user_and_verify_setup_and_notifications true, empty_notify_list, empty_notify_list, nil, nil
 353     create_user_and_verify_setup_and_notifications false, active_notify_list, inactive_notify_list, nil, nil
 354     create_user_and_verify_setup_and_notifications false, empty_notify_list, inactive_notify_list, nil, nil
 355     create_user_and_verify_setup_and_notifications false, empty_notify_list, empty_notify_list, nil, nil
 356   end
 357
 358   [
 359     # Easy inactive user tests.
 360     [false, empty_notify_list, empty_notify_list, "inactive-none@example.com", false, false, "inactivenone"],
 361     [false, empty_notify_list, empty_notify_list, "inactive-vm@example.com", true, false, "inactivevm"],
 362     [false, empty_notify_list, empty_notify_list, "inactive-repo@example.com", false, true, "inactiverepo"],
 363     [false, empty_notify_list, empty_notify_list, "inactive-both@example.com", true, true, "inactiveboth"],
 364
 365     # Easy active user tests.
 366     [true, active_notify_list, inactive_notify_list, "active-none@example.com", false, false, "activenone"],
 367     [true, active_notify_list, inactive_notify_list, "active-vm@example.com", true, false, "activevm"],
 368     [true, active_notify_list, inactive_notify_list, "active-repo@example.com", false, true, "activerepo"],
 369     [true, active_notify_list, inactive_notify_list, "active-both@example.com", true, true, "activeboth"],
 370
 371     # Test users with malformed e-mail addresses.
 372     [false, empty_notify_list, empty_notify_list, nil, true, true, nil],
 373     [false, empty_notify_list, empty_notify_list, "arvados", true, true, nil],
 374     [false, empty_notify_list, empty_notify_list, "@example.com", true, true, nil],
 375     [true, active_notify_list, inactive_notify_list, "*!*@example.com", true, false, nil],
 376     [true, active_notify_list, inactive_notify_list, "*!*@example.com", false, false, nil],
 377
 378     # Test users with various username transformations.
 379     [false, empty_notify_list, empty_notify_list, "arvados@example.com", false, false, "arvados2"],
 380     [true, active_notify_list, inactive_notify_list, "arvados@example.com", false, false, "arvados2"],
 381     [true, active_notify_list, inactive_notify_list, "root@example.com", true, false, "root2"],
 382     [false, active_notify_list, inactive_notify_list, "root@example.com", true, false, "root2"],
 383     [true, active_notify_list, inactive_notify_list, "roo_t@example.com", false, true, "root2"],
 384     [false, empty_notify_list, empty_notify_list, "^^incorrect_format@example.com", true, true, "incorrectformat"],
 385     [true, active_notify_list, inactive_notify_list, "&4a_d9.@example.com", true, true, "ad9"],
 386     [true, active_notify_list, inactive_notify_list, "&4a_d9.@example.com", false, false, "ad9"],
 387     [false, active_notify_list, inactive_notify_list, "&4a_d9.@example.com", true, true, "ad9"],
 388     [false, active_notify_list, inactive_notify_list, "&4a_d9.@example.com", false, false, "ad9"],
 389   ].each do |active, new_user_recipients, inactive_recipients, email, auto_setup_vm, auto_setup_repo, expect_username|
 390     test "create new user with auto setup #{active} #{email} #{auto_setup_vm} #{auto_setup_repo}" do
 391       set_user_from_auth :admin
 392
 393       Rails.configuration.Users.AutoSetupNewUsers = true
 394
 395       if auto_setup_vm
 396         Rails.configuration.Users.AutoSetupNewUsersWithVmUUID = virtual_machines(:testvm)['uuid']
 397       else
 398         Rails.configuration.Users.AutoSetupNewUsersWithVmUUID = ""
 399       end
 400
 401       Rails.configuration.Users.AutoSetupNewUsersWithRepository = auto_setup_repo
 402
 403       create_user_and_verify_setup_and_notifications active, new_user_recipients, inactive_recipients, email, expect_username
 404     end
 405   end
 406
 407   test "update existing user" do
 408     set_user_from_auth :active    # set active user as current user
 409
 410     @active_user = users(:active)     # get the active user
 411
 412     @active_user.first_name = "first_name_changed"
 413     @active_user.save
 414
 415     @active_user = User.find(@active_user.id)   # get the user back
 416     assert_equal(@active_user.first_name, 'first_name_changed')
 417
 418     # admin user also should be able to update the "active" user info
 419     set_user_from_auth :admin # set admin user as current user
 420     @active_user.first_name = "first_name_changed_by_admin_for_active_user"
 421     @active_user.save
 422
 423     @active_user = User.find(@active_user.id)   # get the user back
 424     assert_equal(@active_user.first_name, 'first_name_changed_by_admin_for_active_user')
 425   end
 426
 427   test "delete a user and verify" do
 428     @active_user = users(:active)     # get the active user
 429     active_user_uuid = @active_user.uuid
 430
 431     set_user_from_auth :admin
 432     @active_user.delete
 433
 434     found_deleted_user = false
 435     User.all.each do |user|
 436       if user.uuid == active_user_uuid
 437         found_deleted_user = true
 438         break
 439       end
 440     end
 441     assert !found_deleted_user, "found deleted user: "+active_user_uuid
 442
 443   end
 444
 445   test "create new user as non-admin user" do
 446     set_user_from_auth :active
 447     assert_not_allowed { User.new.save }
 448   end
 449
 450   test "setup new user" do
 451     set_user_from_auth :admin
 452
 453     email = 'foo@example.com'
 454
 455     user = User.create ({uuid: 'zzzzz-tpzed-abcdefghijklmno', email: email})
 456
 457     vm = VirtualMachine.create
 458
 459     response = user.setup(repo_name: 'foo/testrepo',
 460                           vm_uuid: vm.uuid)
 461
 462     resp_user = find_obj_in_resp response, 'User'
 463     verify_user resp_user, email
 464
 465     group_perm = find_obj_in_resp response, 'Link', 'arvados#group'
 466     verify_link group_perm, 'permission', 'can_read', resp_user[:uuid], nil
 467
 468     repo_perm = find_obj_in_resp response, 'Link', 'arvados#repository'
 469     verify_link repo_perm, 'permission', 'can_manage', resp_user[:uuid], nil
 470
 471     vm_perm = find_obj_in_resp response, 'Link', 'arvados#virtualMachine'
 472     verify_link vm_perm, 'permission', 'can_login', resp_user[:uuid], vm.uuid
 473     assert_equal("foo", vm_perm.properties["username"])
 474   end
 475
 476   test "setup new user with junk in database" do
 477     set_user_from_auth :admin
 478
 479     email = 'foo@example.com'
 480
 481     user = User.create ({uuid: 'zzzzz-tpzed-abcdefghijklmno', email: email})
 482
 483     vm = VirtualMachine.create
 484
 485     response = user.setup(repo_name: 'foo/testrepo',
 486                           vm_uuid: vm.uuid)
 487
 488     resp_user = find_obj_in_resp response, 'User'
 489     verify_user resp_user, email
 490
 491     group_perm = find_obj_in_resp response, 'Link', 'arvados#group'
 492     verify_link group_perm, 'permission', 'can_read', resp_user[:uuid], nil
 493
 494     repo_perm = find_obj_in_resp response, 'Link', 'arvados#repository'
 495     verify_link repo_perm, 'permission', 'can_manage', resp_user[:uuid], nil
 496
 497     vm_perm = find_obj_in_resp response, 'Link', 'arvados#virtualMachine'
 498     verify_link vm_perm, 'permission', 'can_login', resp_user[:uuid], vm.uuid
 499     assert_equal("foo", vm_perm.properties["username"])
 500   end
 501
 502   test "setup new user in multiple steps" do
 503     set_user_from_auth :admin
 504
 505     email = 'foo@example.com'
 506
 507     user = User.create ({uuid: 'zzzzz-tpzed-abcdefghijklmno', email: email})
 508
 509     response = user.setup()
 510
 511     resp_user = find_obj_in_resp response, 'User'
 512     verify_user resp_user, email
 513
 514     group_perm = find_obj_in_resp response, 'Link', 'arvados#group'
 515     verify_link group_perm, 'permission', 'can_read', resp_user[:uuid], nil
 516
 517     # invoke setup again with repo_name
 518     response = user.setup(repo_name: 'foo/testrepo')
 519     resp_user = find_obj_in_resp response, 'User', nil
 520     verify_user resp_user, email
 521     assert_equal user.uuid, resp_user[:uuid], 'expected uuid not found'
 522
 523     group_perm = find_obj_in_resp response, 'Link', 'arvados#group'
 524     verify_link group_perm, 'permission', 'can_read', resp_user[:uuid], nil
 525
 526     repo_perm = find_obj_in_resp response, 'Link', 'arvados#repository'
 527     verify_link repo_perm, 'permission', 'can_manage', resp_user[:uuid], nil
 528
 529     # invoke setup again with a vm_uuid
 530     vm = VirtualMachine.create
 531
 532     response = user.setup(repo_name: 'foo/testrepo',
 533                           vm_uuid: vm.uuid)
 534
 535     resp_user = find_obj_in_resp response, 'User', nil
 536     verify_user resp_user, email
 537     assert_equal user.uuid, resp_user[:uuid], 'expected uuid not found'
 538
 539     group_perm = find_obj_in_resp response, 'Link', 'arvados#group'
 540     verify_link group_perm, 'permission', 'can_read', resp_user[:uuid], nil
 541
 542     repo_perm = find_obj_in_resp response, 'Link', 'arvados#repository'
 543     verify_link repo_perm, 'permission', 'can_manage', resp_user[:uuid], nil
 544
 545     vm_perm = find_obj_in_resp response, 'Link', 'arvados#virtualMachine'
 546     verify_link vm_perm, 'permission', 'can_login', resp_user[:uuid], vm.uuid
 547     assert_equal("foo", vm_perm.properties["username"])
 548   end
 549
 550   def find_obj_in_resp (response_items, object_type, head_kind=nil)
 551     return_obj = nil
 552     response_items.each { |x|
 553       if !x
 554         next
 555       end
 556
 557       if object_type == 'User'
 558         if ArvadosModel::resource_class_for_uuid(x['uuid']) == User
 559           return_obj = x
 560           break
 561         end
 562       else  # looking for a link
 563         if ArvadosModel::resource_class_for_uuid(x['head_uuid']).kind == head_kind
 564           return_obj = x
 565           break
 566         end
 567       end
 568     }
 569     return return_obj
 570   end
 571
 572   def verify_user (resp_user, email)
 573     assert_not_nil resp_user, 'expected user object'
 574     assert_not_nil resp_user['uuid'], 'expected user object'
 575     assert_equal email, resp_user['email'], 'expected email not found'
 576   end
 577
 578   def verify_link (link_object, link_class, link_name, tail_uuid, head_uuid)
 579     assert_not_nil link_object, "expected link for #{link_class} #{link_name}"
 580     assert_not_nil link_object[:uuid],
 581         "expected non-nil uuid for link for #{link_class} #{link_name}"
 582     assert_equal link_class, link_object[:link_class],
 583         "expected link_class not found for #{link_class} #{link_name}"
 584     assert_equal link_name, link_object[:name],
 585         "expected link_name not found for #{link_class} #{link_name}"
 586     assert_equal tail_uuid, link_object[:tail_uuid],
 587         "expected tail_uuid not found for #{link_class} #{link_name}"
 588     if head_uuid
 589       assert_equal head_uuid, link_object[:head_uuid],
 590           "expected head_uuid not found for #{link_class} #{link_name}"
 591     end
 592   end
 593
 594   def create_user_and_verify_setup_and_notifications (active, new_user_recipients, inactive_recipients, email, expect_username)
 595     Rails.configuration.Users.NewUserNotificationRecipients = new_user_recipients
 596     Rails.configuration.Users.NewInactiveUserNotificationRecipients = inactive_recipients
 597
 598     ActionMailer::Base.deliveries = []
 599
 600     can_setup = (Rails.configuration.Users.AutoSetupNewUsers and
 601                  (not expect_username.nil?))
 602     expect_repo_name = "#{expect_username}/#{expect_username}"
 603     prior_repo = Repository.where(name: expect_repo_name).first
 604
 605     user = User.new
 606     user.first_name = "first_name_for_newly_created_user"
 607     user.email = email
 608     user.is_active = active
 609     user.save!
 610     assert_equal(expect_username, user.username)
 611
 612     # check user setup
 613     verify_link_exists(Rails.configuration.Users.AutoSetupNewUsers || active,
 614                        groups(:all_users).uuid, user.uuid,
 615                        "permission", "can_read")
 616
 617     # Check for repository.
 618     if named_repo = (prior_repo or
 619                      Repository.where(name: expect_repo_name).first)
 620       verify_link_exists((can_setup and prior_repo.nil? and
 621                           Rails.configuration.Users.AutoSetupNewUsersWithRepository),
 622                          named_repo.uuid, user.uuid, "permission", "can_manage")
 623     end
 624     # Check for VM login.
 625     if (auto_vm_uuid = Rails.configuration.Users.AutoSetupNewUsersWithVmUUID) != ""
 626       verify_link_exists(can_setup, auto_vm_uuid, user.uuid,
 627                          "permission", "can_login", "username", expect_username)
 628     end
 629
 630     # check email notifications
 631     new_user_email = nil
 632     new_inactive_user_email = nil
 633
 634     new_user_email_subject = "#{Rails.configuration.Users.EmailSubjectPrefix}New user created notification"
 635     if Rails.configuration.Users.AutoSetupNewUsers
 636       new_user_email_subject = (expect_username or active) ?
 637                                  "#{Rails.configuration.Users.EmailSubjectPrefix}New user created and setup notification" :
 638                                  "#{Rails.configuration.Users.EmailSubjectPrefix}New user created, but not setup notification"
 639     end
 640
 641     ActionMailer::Base.deliveries.each do |d|
 642       if d.subject == new_user_email_subject then
 643         new_user_email = d
 644       elsif d.subject == "#{Rails.configuration.Users.EmailSubjectPrefix}New inactive user notification" then
 645         new_inactive_user_email = d
 646       end
 647     end
 648
 649     # both active and inactive user creations should result in new user creation notification mails,
 650     # if the new user email recipients config parameter is set
 651     if not new_user_recipients.empty? then
 652       assert_not_nil new_user_email, 'Expected new user email after setup'
 653       assert_equal Rails.configuration.Users.UserNotifierEmailFrom, new_user_email.from[0]
 654       assert_equal new_user_recipients.stringify_keys.keys.first, new_user_email.to[0]
 655       assert_equal new_user_email_subject, new_user_email.subject
 656     else
 657       assert_nil new_user_email, 'Did not expect new user email after setup'
 658     end
 659
 660     if not active
 661       if not inactive_recipients.empty? then
 662         assert_not_nil new_inactive_user_email, 'Expected new inactive user email after setup'
 663         assert_equal Rails.configuration.Users.UserNotifierEmailFrom, new_inactive_user_email.from[0]
 664         assert_equal inactive_recipients.stringify_keys.keys.first, new_inactive_user_email.to[0]
 665         assert_equal "#{Rails.configuration.Users.EmailSubjectPrefix}New inactive user notification", new_inactive_user_email.subject
 666       else
 667         assert_nil new_inactive_user_email, 'Did not expect new inactive user email after setup'
 668       end
 669     else
 670       assert_nil new_inactive_user_email, 'Expected no inactive user email after setting up active user'
 671     end
 672     ActionMailer::Base.deliveries = []
 673   end
 674
 675   def verify_link_exists link_exists, head_uuid, tail_uuid, link_class, link_name, property_name=nil, property_value=nil
 676     all_links = Link.where(head_uuid: head_uuid,
 677                            tail_uuid: tail_uuid,
 678                            link_class: link_class,
 679                            name: link_name)
 680     assert_equal link_exists, all_links.any?, "Link #{'not' if link_exists} found for #{link_name} #{link_class} #{property_value}"
 681     if link_exists && property_name && property_value
 682       all_links.each do |link|
 683         assert_equal true, all_links.first.properties[property_name].start_with?(property_value), 'Property not found in link'
 684       end
 685     end
 686   end
 687
 688   [
 689     [:active, 'zzzzz-borkd-abcde12345abcde'],
 690     [:active, 'zzzzz-j7d0g-abcde12345abcde'],
 691     [:active, 'zzzzz-tpzed-borkd'],
 692     [:system_user, 'zzzzz-tpzed-abcde12345abcde'],
 693     [:anonymous, 'zzzzz-tpzed-abcde12345abcde'],
 694   ].each do |fixture, new_uuid|
 695     test "disallow update_uuid #{fixture} -> #{new_uuid}" do
 696       u = users(fixture)
 697       orig_uuid = u.uuid
 698       act_as_system_user do
 699         assert_raises do
 700           u.update_uuid(new_uuid: new_uuid)
 701         end
 702       end
 703       # "Successfully aborted orig->new" outcome looks the same as
 704       # "successfully updated new->orig".
 705       assert_update_success(old_uuid: new_uuid,
 706                             new_uuid: orig_uuid,
 707                             expect_owned_objects: fixture == :active)
 708     end
 709   end
 710
 711   [:active, :spectator, :admin].each do |target|
 712     test "update_uuid on #{target} as non-admin user" do
 713       act_as_user users(:active) do
 714         assert_raises(ArvadosModel::PermissionDeniedError) do
 715           users(target).update_uuid(new_uuid: 'zzzzz-tpzed-abcde12345abcde')
 716         end
 717       end
 718     end
 719   end
 720
 721   test "update_uuid to existing uuid" do
 722     u = users(:active)
 723     orig_uuid = u.uuid
 724     new_uuid = users(:admin).uuid
 725     act_as_system_user do
 726       assert_raises do
 727         u.update_uuid(new_uuid: new_uuid)
 728       end
 729     end
 730     u.reload
 731     assert_equal u.uuid, orig_uuid
 732     assert_not_empty Collection.where(owner_uuid: orig_uuid)
 733     assert_not_empty Group.where(owner_uuid: orig_uuid)
 734   end
 735
 736   [
 737     [:active, 'zbbbb-tpzed-abcde12345abcde'],
 738     [:active, 'zzzzz-tpzed-abcde12345abcde'],
 739     [:admin, 'zbbbb-tpzed-abcde12345abcde'],
 740     [:admin, 'zzzzz-tpzed-abcde12345abcde'],
 741   ].each do |fixture, new_uuid|
 742     test "update_uuid #{fixture} to unused uuid #{new_uuid}" do
 743       u = users(fixture)
 744       orig_uuid = u.uuid
 745       act_as_system_user do
 746         u.update_uuid(new_uuid: new_uuid)
 747       end
 748       assert_update_success(old_uuid: orig_uuid,
 749                             new_uuid: new_uuid,
 750                             expect_owned_objects: fixture == :active)
 751     end
 752   end
 753
 754   def assert_update_success(old_uuid:, new_uuid:, expect_owned_objects: true)
 755     [[User, :uuid],
 756      [Link, :head_uuid],
 757      [Link, :tail_uuid],
 758      [Group, :owner_uuid],
 759      [Collection, :owner_uuid],
 760     ].each do |klass, attr|
 761       assert_empty klass.where(attr => old_uuid)
 762       if klass == User || expect_owned_objects
 763         assert_not_empty klass.where(attr => new_uuid)
 764       end
 765     end
 766   end
 767
 768   test "lookup user by email" do
 769     u = User.register({"email" => "active-user@arvados.local", "identity_url" => "different-identity-url"})
 770     active = User.find_by_uuid(users(:active).uuid)
 771     assert_equal active.uuid, u.uuid
 772     assert_equal "active-user@arvados.local", active.email
 773     # identity_url is not updated
 774     assert_equal "https://active-user.openid.local", active.identity_url
 775   end
 776
 777   test "lookup user by alternate email" do
 778     # register method will find that active-user@arvados.local already
 779     # exists and return existing 'active' user.
 780     u = User.register({"email" => "user@parent-company.com",
 781                        "alternate_emails" => ["active-user@arvados.local"],
 782                        "identity_url" => "different-identity-url"})
 783     active = User.find_by_uuid(users(:active).uuid)
 784     assert_equal active.uuid, u.uuid
 785
 786     # email should be updated
 787     assert_equal "user@parent-company.com", active.email
 788
 789     # identity_url is not updated
 790     assert_equal "https://active-user.openid.local", active.identity_url
 791   end
 792
 793   test "register new user" do
 794     u = User.register({"email" => "never-before-seen-user@arvados.local",
 795                        "identity_url" => "different-identity-url",
 796                        "first_name" => "Robert",
 797                        "last_name" => "Baratheon",
 798                        "username" => "bobby"})
 799     nbs = User.find_by_uuid(u.uuid)
 800     assert_equal nbs.uuid, u.uuid
 801     assert_equal "different-identity-url", nbs.identity_url
 802     assert_equal "never-before-seen-user@arvados.local", nbs.email
 803     assert_equal false, nbs.is_admin
 804     assert_equal false , nbs.is_active
 805     assert_equal "bobby", nbs.username
 806     assert_equal "Robert", nbs.first_name
 807     assert_equal "Baratheon", nbs.last_name
 808   end
 809
 810   test "fail when email address is ambiguous" do
 811     User.register({"email" => "active-user@arvados.local"})
 812     u = User.register({"email" => "never-before-seen-user@arvados.local"})
 813     u.email = "active-user@arvados.local"
 814     act_as_system_user do
 815       u.save!
 816     end
 817     assert_raises do
 818       User.register({"email" => "active-user@arvados.local"})
 819     end
 820   end
 821
 822   test "fail lookup without identifiers" do
 823     assert_raises do
 824       User.register({"first_name" => "Robert", "last_name" => "Baratheon"})
 825     end
 826     assert_raises do
 827       User.register({"first_name" => "Robert", "last_name" => "Baratheon", "identity_url" => "", "email" => ""})
 828     end
 829   end
 830
 831   test "user can update name" do
 832     set_user_from_auth :active
 833     user = users(:active)
 834     user.first_name = "MyNewName"
 835     assert user.save
 836   end
 837
 838   test "user cannot update email" do
 839     set_user_from_auth :active
 840     user = users(:active)
 841     user.email = "new-name@example.com"
 842     assert_not_allowed { user.save }
 843   end
 844
 845   test "admin can update email" do
 846     set_user_from_auth :admin
 847     user = users(:active)
 848     user.email = "new-name@example.com"
 849     assert user.save
 850   end
 851
 852 end