diff options
author | Marius Balteanu <marius.balteanu@zitec.com> | 2022-05-27 22:14:59 +0000 |
---|---|---|
committer | Marius Balteanu <marius.balteanu@zitec.com> | 2022-05-27 22:14:59 +0000 |
commit | 3ce719b120dcf5b29e9b5ea0c12911f0d8bda439 (patch) | |
tree | fd0e72bae81f946119ac2914d101321c347a4a2e /test/integration/api_test | |
parent | 630b71975f37d1ca4516c6be27b80aa50c020549 (diff) | |
download | redmine-3ce719b120dcf5b29e9b5ea0c12911f0d8bda439.tar.gz redmine-3ce719b120dcf5b29e9b5ea0c12911f0d8bda439.zip |
Fix internal server error when trying to retrieve AnonymousUser's information via Users API (#37157).
Patch by Yuichi HARADA.
git-svn-id: https://svn.redmine.org/redmine/trunk@21606 e93f8b46-1217-0410-a6f0-8f06a7374b81
Diffstat (limited to 'test/integration/api_test')
-rw-r--r-- | test/integration/api_test/users_test.rb | 18 |
1 files changed, 18 insertions, 0 deletions
diff --git a/test/integration/api_test/users_test.rb b/test/integration/api_test/users_test.rb index 26473356a..8444500d9 100644 --- a/test/integration/api_test/users_test.rb +++ b/test/integration/api_test/users_test.rb @@ -96,6 +96,24 @@ class Redmine::ApiTest::UsersTest < Redmine::ApiTest::Base assert_select 'user avatar_url', :text => %r|\Ahttps://gravatar.com/avatar/\h{32}\?default=robohash| end + test "GET /users/:id.xml should not return avatar_url when not set email address" do + user = User.find(2) + user.email_addresses.delete_all + assert_equal 'jsmith', user.login + assert_nil user.mail + + Redmine::Configuration.with 'avatar_server_url' => 'https://gravatar.com' do + with_settings :gravatar_enabled => '1', :gravatar_default => 'robohash' do + get '/users/2.xml' + end + end + + assert_response :success + assert_select 'user id', :text => '2' + assert_select 'user login', :text => 'jsmith' + assert_select 'user avatar_url', :count => 0 + end + test "GET /users/:id.json should return the user" do get '/users/2.json' |