Feat/Usuario edita e deleta categoria de trabalho

 Co-authored-by: Lucas Vasques <[email protected]>

Author: rozbr96

app/assets/images/edit-icon.svg

@@ -1,10 +1,11 @@
 class ProfileJobCategoriesController < ApplicationController
-  before_action :authenticate_user!, only: %w[new create]
+  before_action :authenticate_user!, only: %w[new create edit destroy update]
   before_action :check_if_job_categories_exist, only: %w[new create]
+  before_action :set_profile_job_category, only: %w[edit destroy update]
+  before_action :authorize, only: %w[edit destroy update]
 
   def new
     @profile_job_category = current_user.profile.profile_job_categories.build
-    flash.now[:notice] = t('.inform_user_edit_is_unavailable')
   end
 
   def create
@@ -18,6 +19,18 @@ def create
     end
   end
 
+  def edit; end
+
+  def destroy
+    @profile_job_category.destroy!
+    redirect_to current_user.profile, notice: t('.success')
+  end
+
+  def update
+    @profile_job_category.update! profile_job_category_params
+    redirect_to current_user.profile, notice: t('.success')
+  end
+
   private
 
   def profile_job_category_params
@@ -30,4 +43,14 @@ def check_if_job_categories_exist
     redirect_to profile_path(current_user.profile),
                 notice: t('.check_if_job_categories_exist.error')
   end
+
+  def set_profile_job_category
+    @profile_job_category = ProfileJobCategory.find params[:id]
+  end
+
+  def authorize
+    return if current_user.profile == @profile_job_category.profile
+
+    redirect_to current_user.profile, alert: t('alerts.unauthorized')
+  end
 end

app/assets/images/x-icon.svg

@@ -0,0 +1,23 @@
+<% if @profile_job_category.errors.any? %>
+  <% @profile_job_category.errors.full_messages.each do |msg| %>
+    <p class="alert alert-warning">
+      <%= msg %>
+    </p>
+  <% end %>
+<% end %>
+<%= form_with model: @profile_job_category do |f| %>
+  <% if should_render %>
+    <div class="mb-3">
+      <%= f.label :job_category_id, class: 'form-label' %>
+      <%= f.collection_select :job_category_id, JobCategory.order(:name), :id, :name, {prompt: 'Selecione uma categoria'}, class: 'form-select' %>
+    </div>
+  <% end %>
+  <div class="mb-3">
+    <%= f.label :description, class: 'form-label' %>
+    <%= f.text_area :description, class: 'form-control' %>
+  </div>
+  <div class="mb-3">
+    <%= f.submit t('save_btn'), class: 'btn btn-primary' %>
+    <%= link_to t('return_btn'), profile_path(current_user.profile), class: 'btn btn-secondary' %>
+  </div>
+<% end %>

app/controllers/profile_job_categories_controller.rb

@@ -0,0 +1,2 @@
+<h1><%= t('.form_title') %></h1>
+<%= render partial: 'form', locals: { should_render: false } %>

app/views/profile_job_categories/_form.html.erb

@@ -1,22 +1,2 @@
-<h1><%= t('content.new_form_title') %></h1>
-<% if @profile_job_category.errors.any? %>
-  <% @profile_job_category.errors.full_messages.each do |msg| %>
-    <p class="alert alert-warning">
-      <%= msg %>
-    </p>
-  <% end %>
-<% end %>
-<%= form_with model: @profile_job_category do |f| %>
-  <div class="mb-3">
-    <%= f.label :job_category_id, class: 'form-label' %>
-    <%= f.collection_select :job_category_id, JobCategory.order(:name), :id, :name, {prompt: 'Selecione uma categoria'}, class: 'form-select' %>
-  </div>
-  <div class="mb-3">
-    <%= f.label :description, class: 'form-label' %>
-    <%= f.text_area :description, class: 'form-control' %>
-  </div>
-  <div class="mb-3">
-    <%= f.submit t('save_btn'), class: 'btn btn-primary' %>
-    <%= link_to t('return_btn'), profile_path(current_user.profile), class: 'btn btn-secondary' %>
-  </div>
-<% end %>
+<h1><%= t('.form_title') %></h1>
+<%= render partial: 'form', locals: { should_render: true } %>

app/views/profile_job_categories/edit.html.erb

@@ -13,7 +13,7 @@
     <div class="border py-1 px-2 rounded mb-3">
       <div class="d-flex gap-1">
         <%= image_tag 'person-fill', width: '28rem', class: 'mb-2' %>
-        <h3><%= PersonalInfo.model_name.human %></h3>  
+        <h3><%= PersonalInfo.model_name.human %></h3>
       </div>
       <%= render 'personal_info', personal_info: personal_info, user_is_owner: user_is_owner %>
       <%= link_to 'Editar Informações Pessoais', edit_user_profile_path, class:'btn btn-secondary mx-4 mb-3' if user_is_owner %>
@@ -27,7 +27,7 @@
         <h3><%= ProfessionalInfo.model_name.human %></h3>
       </div>
     <% end %>
-    
+
     <p>
       <%= link_to 'Adicionar Experiência Profissional', new_user_profile_professional_info_path, class:'btn btn-primary' if user_is_owner %>
     </p>
@@ -64,7 +64,7 @@
       </p>
     <% end %>
   </div>
-  
+
   <div class="border py-1 px-2 rounded mb-3">
     <% if profile.education_infos.visibles_list.any? || user_is_owner %>
       <div class="d-flex gap-2">
@@ -77,7 +77,7 @@
     </p>
       <% profile.education_infos.each do |education_info| %>
         <% if education_info.visibility || user_is_owner %>
-          <h4>  
+          <h4>
             <%= EducationInfo.human_attribute_name(:institution) %>: <%= education_info.institution %>
           </h4>
           <ul>
@@ -104,22 +104,36 @@
   </div>
 </div>
 <div id='profile-job-categories' class="border py-1 px-2 rounded mb-3">
-  <% if profile.profile_job_categories.any? || user_is_owner %> 
+  <% if profile.profile_job_categories.any? || user_is_owner %>
     <div class="d-flex gap-1">
       <%= image_tag 'person-working', width: '28rem', class: 'mb-2 mx-1' %>
       <h3><%= t('content.section_title') %></h3>
     </div>
-  <% end %> 
+  <% end %>
 
   <%= link_to t('links.new_profile_job_category_path'), new_profile_job_category_path, class:'btn btn-primary' if user_is_owner %>
   <div class='mt-2'>
     <% profile.profile_job_categories.each do |profile_job_category| %>
       <p><strong><%= profile_job_category.job_category.name %></strong></p>
-      <% if profile_job_category.description.present? %>
-        <p><%= profile_job_category.description %></p>
-      <% else %>
-        <p><%= t('content.profile_job_category_without_description') %></p>
-      <% end %>
+      <div class='d-flex row'>
+        <div class='d-flex column col-6 align-text-top'>
+          <% if profile_job_category.description.present? %>
+            <p><%= profile_job_category.description %></p>
+          <% else %>
+            <p><%= t('content.profile_job_category_without_description') %></p>
+          <% end %>
+        </div>
+        <% if profile.user == current_user %>
+          <div class='d-flex column col-6'>
+            <%= button_to edit_profile_job_category_path(profile_job_category), method: :get, class:'btn btn-sm btn-edit-category' do %>
+              <%= image_tag 'edit-icon', width: '20rem', height: '20rem' %>
+            <% end %>
+            <%= button_to profile_job_category_path(profile_job_category), method: :delete, class:'btn btn-sm btn-delete-category' do %>
+              <%= image_tag 'x-icon', width: '20rem', height: '20rem' %>
+            <% end %>
+          </div>
+        <% end %>
+      </div>
     <% end %>
   </div>
 </div>

app/views/profile_job_categories/new.html.erb

@@ -1,7 +1,7 @@
 pt-BR:
   activerecord:
     models:
-      profile_job_category: 
+      profile_job_category:
         one: Categoria de trabalho
         other: Categorias de trabalho
     attributes:
@@ -13,11 +13,18 @@ pt-BR:
     create:
       success: Categoria de trabalho adicionada com sucesso!
       error: Não foi possível adicionar a categoria de trabalho.
+    destroy:
+      success: Categoria de trabalho deletada com sucesso!
     new:
       inform_user_edit_is_unavailable: 'Atenção: Os dados cadastrados não poderão ser alterados após salvar. Preencha com cuidado.'
       check_if_job_categories_exist:
         error: Essa página não está disponível no momento, entre em contato com o administrador.
-  links:        
+      form_title: Cadastrar Categoria de Trabalho no Perfil
+    edit:
+      form_title: Editar Categoria de Trabalho no Perfil
+    update:
+      success: Categoria de trabalho editada com sucesso!
+  links:
     new_profile_job_category_path: Adicionar nova categoria de trabalho
   content:
     profile_job_category_without_description: Não foi adicionada uma descrição para essa categoria.

app/views/profiles/_profile_details.html.erb

@@ -67,7 +67,7 @@
     resources :education_infos, shallow: true, only: %i[new create edit update]
   end
 
-  resources :profile_job_categories, only: %i[new create]
+  resources :profile_job_categories, only: %i[new create edit destroy update]
 
   namespace :api do
     namespace :v1 do

config/locales/models/profile_job_category.pt-BR.yml

@@ -0,0 +1,31 @@
+require 'rails_helper'
+
+describe 'Usuário remove categoria de trabalho' do
+  it 'mas precisa estar logado' do
+    user = create(:user, full_name: 'Zelda Hyrule')
+    job_category = create(:job_category, name: 'Luta com espada')
+    job_category_two = create(:job_category, name: 'Magia')
+    ProfileJobCategory.create!(job_category:, profile: user.profile, description: 'Eu sou muito bom nessa categoria')
+    ProfileJobCategory.create!(job_category: job_category_two, profile: user.profile)
+
+    delete profile_job_category_path job_category
+
+    expect(response).to redirect_to new_user_session_path
+    expect(flash[:alert]).to eq 'Para continuar, faça login ou registre-se.'
+  end
+
+  it 'mas precisa ser dono da categoria' do
+    user = create(:user, full_name: 'Zelda Hyrule')
+    another_user = create(:user)
+    job_category = create(:job_category, name: 'Luta com espada')
+    job_category_two = create(:job_category, name: 'Magia')
+    ProfileJobCategory.create!(job_category:, profile: user.profile, description: 'Eu sou muito bom nessa categoria')
+    ProfileJobCategory.create!(job_category: job_category_two, profile: user.profile)
+
+    login_as another_user
+    delete profile_job_category_path job_category
+
+    expect(response).to redirect_to profile_path(another_user.profile)
+    expect(flash[:alert]).to eq 'Você não têm permissão para realizar essa ação.'
+  end
+end

config/routes.rb

@@ -0,0 +1,31 @@
+require 'rails_helper'
+
+describe 'Usuário remove categoria de trabalho' do
+  it 'mas precisa estar logado' do
+    user = create(:user, full_name: 'Zelda Hyrule')
+    job_category = create(:job_category, name: 'Luta com espada')
+    job_category_two = create(:job_category, name: 'Magia')
+    ProfileJobCategory.create!(job_category:, profile: user.profile, description: 'Eu sou muito bom nessa categoria')
+    ProfileJobCategory.create!(job_category: job_category_two, profile: user.profile)
+
+    patch profile_job_category_path job_category
+
+    expect(response).to redirect_to new_user_session_path
+    expect(flash[:alert]).to eq 'Para continuar, faça login ou registre-se.'
+  end
+
+  it 'mas precisa ser dono da categoria' do
+    user = create(:user, full_name: 'Zelda Hyrule')
+    another_user = create(:user)
+    job_category = create(:job_category, name: 'Luta com espada')
+    job_category_two = create(:job_category, name: 'Magia')
+    ProfileJobCategory.create!(job_category:, profile: user.profile, description: 'Eu sou muito bom nessa categoria')
+    ProfileJobCategory.create!(job_category: job_category_two, profile: user.profile)
+
+    login_as another_user
+    patch profile_job_category_path job_category
+
+    expect(response).to redirect_to profile_path(another_user.profile)
+    expect(flash[:alert]).to eq 'Você não têm permissão para realizar essa ação.'
+  end
+end

spec/requests/profile_job_categories/user_deletes_job_category_spec.rb

@@ -76,17 +76,6 @@
     expect(page).to have_current_path profile_path(user.profile)
   end
 
-  it 'e é alertado no formulário sobre não poder editar ou remover os dados no futuro' do
-    user = create(:user)
-    create(:job_category)
-
-    login_as user
-    visit new_profile_job_category_path
-
-    alert_text = 'Atenção: Os dados cadastrados não poderão ser alterados após salvar. Preencha com cuidado.'
-    expect(page).to have_content alert_text
-  end
-
   it 'e vê um link para retornar à página de meu perfil' do
     user = create(:user)
     create(:job_category)

spec/requests/profile_job_categories/user_edits_job_category_spec.rb

@@ -0,0 +1,38 @@
+require 'rails_helper'
+
+describe 'Usuário deleta categoria de trabalho' do
+  it 'do seu perfil com sucesso' do
+    user = create(:user, full_name: 'Zelda Hyrule')
+    job_category = create(:job_category, name: 'Luta com espada')
+    job_category_two = create(:job_category, name: 'Magia')
+    ProfileJobCategory.create!(job_category:, profile: user.profile, description: 'Eu sou muito bom nessa categoria')
+    ProfileJobCategory.create!(job_category: job_category_two, profile: user.profile)
+
+    login_as user
+    visit profile_path(user.profile)
+    page.all('.btn-delete-category').to_a.second.click
+
+    expect(ProfileJobCategory.count).to eq 1
+    expect(page).to have_content 'Eu sou muito bom nessa categoria'
+    expect(page).to have_content 'Categoria de trabalho deletada com sucesso!'
+    expect(page).to have_content 'Luta com espada'
+    expect(page).not_to have_content 'Magia'
+  end
+
+  it 'e não consegue ver o botão em outro perfil' do
+    user = create(:user, full_name: 'Zelda Hyrule')
+    other_user = create(:user)
+    job_category = create(:job_category, name: 'Luta com espada')
+    job_category_two = create(:job_category, name: 'Magia')
+    ProfileJobCategory.create!(job_category:, profile: user.profile, description: 'Eu sou muito bom nessa categoria')
+    ProfileJobCategory.create!(job_category: job_category_two, profile: user.profile)
+
+    login_as other_user
+    visit profile_path(user.profile)
+
+    expect(page).to have_content 'Zelda Hyrule'
+    expect(page).to have_content 'Luta com espada'
+    expect(page).to have_content 'Magia'
+    expect(page.all('.btn-delete-category').count).to eq 0
+  end
+end