From ba9e33cef6176b92a079071f4c390a8672413e94 Mon Sep 17 00:00:00 2001
From: Anton Khorev <tony29@yandex.ru>
Date: Tue, 1 Jul 2025 19:57:41 +0300
Subject: [PATCH] Add "Change Image" button to user profile

---
 app/views/users/show.html.erb            |  4 ++
 config/locales/en.yml                    |  1 +
 test/system/profile_image_change_test.rb | 74 ++++++++++++++++++++++++
 3 files changed, 79 insertions(+)
 create mode 100644 test/system/profile_image_change_test.rb

diff --git a/app/views/users/show.html.erb b/app/views/users/show.html.erb
index 08cf28bcd..efca2852d 100644
--- a/app/views/users/show.html.erb
+++ b/app/views/users/show.html.erb
@@ -238,6 +238,10 @@
 <% if @user.home_location_name&.strip.present? || @user.company&.strip.present? || !@user.social_links.empty? || owned %>
   <div class="row">
     <div class="col-sm-3">
+      <%= render "sidebar_section", :edit_title => t(".change_image"),
+                                    :edit_path => profile_image_path do %>
+      <% end %>
+
       <%= render "sidebar_section", :present => @user.home_location_name&.strip.present? do %>
         <div class="icon-link mw-100">
           <svg xmlns="http://www.w3.org/2000/svg" width="16" height="16" fill="currentColor" viewBox="0 0 16 16" class="flex-shrink-0">
diff --git a/config/locales/en.yml b/config/locales/en.yml
index d59dd3153..13fa2f928 100644
--- a/config/locales/en.yml
+++ b/config/locales/en.yml
@@ -3076,6 +3076,7 @@ en:
       edit_profile_details: Edit Profile Details
       edit_description: Edit Description
       edit_links: Edit Links
+      change_image: Change Image
       contributions:
         one: "%{count} contribution in the last year"
         other: "%{count} contributions in the last year"
diff --git a/test/system/profile_image_change_test.rb b/test/system/profile_image_change_test.rb
new file mode 100644
index 000000000..51b71fd59
--- /dev/null
+++ b/test/system/profile_image_change_test.rb
@@ -0,0 +1,74 @@
+require "application_system_test_case"
+
+class ProfileImageChangeTest < ApplicationSystemTestCase
+  test "can't change image when unauthorized" do
+    user = create(:user)
+
+    visit user_path(user)
+
+    within_content_body do
+      assert_no_button "Edit Profile Details"
+      assert_no_link "Change Image"
+    end
+  end
+
+  test "can't change image of another user" do
+    user = create(:user)
+    another_user = create(:user)
+
+    sign_in_as(user)
+    visit user_path(another_user)
+
+    within_content_body do
+      assert_no_button "Edit Profile Details"
+      assert_no_link "Change Image"
+    end
+  end
+
+  test "can add and remove image" do
+    user = create(:user)
+
+    sign_in_as(user)
+    visit user_path(user)
+
+    within_content_body do
+      click_on "Edit Profile Details"
+      click_on "Change Image"
+      assert_unchecked_field "Add an image"
+      assert_no_field "Keep the current image"
+      assert_no_field "Remove the current image"
+      assert_no_field "Replace the current image"
+
+      attach_file "Avatar", "test/gpx/fixtures/a.gif"
+
+      assert_checked_field "Add an image"
+
+      click_on "Update Profile"
+    end
+
+    assert_text "Profile image updated."
+
+    within_content_body do
+      click_on "Edit Profile Details"
+      click_on "Change Image"
+      assert_no_field "Add an image"
+      assert_checked_field "Keep the current image"
+      assert_unchecked_field "Remove the current image"
+      assert_unchecked_field "Replace the current image"
+
+      choose "Remove the current image"
+      click_on "Update Profile"
+    end
+
+    assert_text "Profile image updated."
+
+    within_content_body do
+      click_on "Edit Profile Details"
+      click_on "Change Image"
+      assert_unchecked_field "Add an image"
+      assert_no_field "Keep the current image"
+      assert_no_field "Remove the current image"
+      assert_no_field "Replace the current image"
+    end
+  end
+end
-- 
2.39.5