Merge pull request #1241 from Crown-Commercial-Service/feature/nrmi-39-self-service-user-name-update

NRMI-39

Author: reemi1510

app/controllers/v1/users_controller.rb

@@ -4,4 +4,17 @@ def index
 
     render jsonapi: users
   end
+
+  def update_name
+    user = User.find_by!(auth_id: current_auth_id)
+    name_param = params.dig('_jsonapi', 'name')
+    result = UpdateUser.new(user, name_param).call
+
+    if result.success?
+      render jsonapi: user, status: :ok
+    else
+      render jsonapi_errors: { error: I18n.t('errors.messages.error_updating_user_in_auth0') },
+             status: :unprocessable_entity
+    end
+  end
 end

config/routes.rb

@@ -21,7 +21,11 @@
   get '/check', to: 'check#index', defaults: { format: :json }
 
   namespace :v1, defaults: { format: :json } do
-    resources :users, only: %i[index]
+    resources :users, only: %i[index] do
+      collection do
+        patch :update_name
+      end
+    end
 
     resources :suppliers, only: %i[index]
 

spec/requests/v1/users_spec.rb

@@ -31,4 +31,42 @@
         .with_value(true)
     end
   end
+
+  describe 'PATCH /v1/users/update_name' do
+    it 'updates the name of the current user' do
+      user = FactoryBot.create(:user)
+      stub_auth0_token_request
+      stub_auth0_update_user_request(user)
+
+      patch '/v1/users/update_name',
+            headers: { 'X-Auth-Id' => JWT.encode(user.auth_id, 'test') },
+            params: {
+              _jsonapi: {
+                name: 'New User Name'
+              }
+            }
+
+      expect(response).to be_successful
+      expect(json['data']).to have_id(user.id)
+      expect(json['data'])
+        .to have_attribute(:name)
+        .with_value('New User Name')
+    end
+
+    it 'returns an error if the Auth0 update fails' do
+      user = FactoryBot.create(:user)
+      stub_auth0_token_request
+      stub_auth0_update_user_request_failure(user)
+
+      patch '/v1/users/update_name',
+            headers: { 'X-Auth-Id' => JWT.encode(user.auth_id, 'test') },
+            params: {
+              _jsonapi: {
+                name: 'New User Name'
+              }
+            }
+
+      expect(response.status).to eq 422
+    end
+  end
 end