From 6062800cf941ec6334f2297c61ac58b3b6604f74 Mon Sep 17 00:00:00 2001
From: Anton Khorev <tony29@yandex.ru>
Date: Wed, 4 Jun 2025 04:26:10 +0300
Subject: [PATCH] Add "Edit Company" button to user profile

---
 app/views/users/show.html.erb              |  4 ++-
 config/locales/en.yml                      |  1 +
 test/system/profile_company_change_test.rb | 39 +++++++++++++++++++---
 3 files changed, 39 insertions(+), 5 deletions(-)

diff --git a/app/views/users/show.html.erb b/app/views/users/show.html.erb
index 926ac2ff2..5405d7821 100644
--- a/app/views/users/show.html.erb
+++ b/app/views/users/show.html.erb
@@ -258,7 +258,9 @@
         </div>
       <% end %>
 
-      <%= render "sidebar_section", :present => @user.company&.strip.present? do %>
+      <%= render "sidebar_section", :present => @user.company&.strip.present?,
+                                    :edit_title => t(".edit_company"),
+                                    :edit_path => profile_company_path 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">
             <title><%= t ".company" %></title>
diff --git a/config/locales/en.yml b/config/locales/en.yml
index 13fa2f928..ad00b3767 100644
--- a/config/locales/en.yml
+++ b/config/locales/en.yml
@@ -3077,6 +3077,7 @@ en:
       edit_description: Edit Description
       edit_links: Edit Links
       change_image: Change Image
+      edit_company: Edit Company
       contributions:
         one: "%{count} contribution in the last year"
         other: "%{count} contributions in the last year"
diff --git a/test/system/profile_company_change_test.rb b/test/system/profile_company_change_test.rb
index 3a2034007..6e060398a 100644
--- a/test/system/profile_company_change_test.rb
+++ b/test/system/profile_company_change_test.rb
@@ -1,19 +1,39 @@
 require "application_system_test_case"
 
 class ProfileCompanyChangeTest < ApplicationSystemTestCase
-  test "User can change company" do
+  test "can't change company when unauthorized" do
     user = create(:user)
-    sign_in_as(user)
 
     visit user_path(user)
 
     within_content_body do
-      assert_no_text :all, "Company"
+      assert_no_button "Edit Profile Details"
+      assert_no_link "Edit Company"
     end
+  end
+
+  test "can't change company of another user" do
+    user = create(:user)
+    another_user = create(:user)
 
-    visit profile_company_path
+    sign_in_as(user)
+    visit user_path(another_user)
 
     within_content_body do
+      assert_no_button "Edit Profile Details"
+      assert_no_link "Edit Company"
+    end
+  end
+
+  test "can change company" do
+    user = create(:user)
+
+    sign_in_as(user)
+    visit user_path(user)
+
+    within_content_body do
+      click_on "Edit Profile Details"
+      click_on "Edit Company"
       fill_in "Company", :with => "Test Co."
       click_on "Update Profile"
     end
@@ -22,6 +42,17 @@ class ProfileCompanyChangeTest < ApplicationSystemTestCase
 
     within_content_body do
       assert_text :all, "Company Test Co."
+
+      click_on "Edit Profile Details"
+      click_on "Edit Company"
+      fill_in "Company", :with => "Test More Co."
+      click_on "Update Profile"
+    end
+
+    assert_text "Profile company updated."
+
+    within_content_body do
+      assert_text :all, "Company Test More Co."
     end
   end
 end
-- 
2.39.5