pulibrary / tigerdata-app / a3f50227-2424-4d41-8788-b953b9d6c802

coverage: 70.412% (-17.7%) from 88.064%
a3f50227-2424-4d41-8788-b953b9d6c802

push

circleci

web-flow 
Deduplicate department field from requests (#2229)

ref #2147

1 of 4 new or added lines in 1 file covered. (25.0%)

833 existing lines in 48 files now uncovered.

2344 of 3329 relevant lines covered (70.41%)

150.39 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

31.11
/app/controllers/edit_requests_controller.rb
1 
# frozen_string_literal: true
UNCOV
2 
class EditRequestsController < ApplicationController
1✔
UNCOV
3 
  layout "edit_request"
1✔
UNCOV
4 
  before_action :set_breadcrumbs
1✔
5 










UNCOV

6



  before_action :set_request_model, only: %i[edit update]




1✔






UNCOV

7



  before_action :check_access




1✔





8












9



  # GET /edit_requests/1/edit











UNCOV

10



  def edit




1✔






UNCOV

11



    add_breadcrumb(@request_model.project_title, request_path(@request_model))




×








UNCOV

12



    add_breadcrumb("Edit Submitted Request")




×








UNCOV

13



  end










14












15



  # PATCH/PUT /edit_requests/1 or /edit_requests/1.json











UNCOV

16



  def update




1✔






UNCOV

17



    respond_to do |format|




×








UNCOV

18



      if @request_model.update(request_params) && @request_model.valid_to_submit?




×








UNCOV

19



        format.html { redirect_to request_url(@request_model), notice: I18n.t(:successful_update) }




×








UNCOV

20



        format.json { render :show, status: :ok, location: @request_model }




×








UNCOV

21



      else











UNCOV

22



        format.html { render :edit, status: :unprocessable_entity }




×








UNCOV

23



        format.json { render json: @request_model.errors, status: :unprocessable_entity }




×








UNCOV

24



      end











UNCOV

25



    end











UNCOV

26



  end










27













UNCOV

28



  private




1✔





29












30



    # Use callbacks to share common setup or constraints between actions.











UNCOV

31



    def set_request_model




1✔






UNCOV

32



      @princeton_departments = Affiliation.all




×








UNCOV

33



      @request_model = Request.find(params[:id])




×








UNCOV

34



    end










35












36



    # Only allow a list of trusted parameters through.











UNCOV

37



    def request_params




1✔






UNCOV

38



      request_params = params.fetch(:request, {}).permit(:request_title, :project_title, :state, :data_sponsor, :data_manager,




×








UNCOV

39



                                        :description, :project_purpose, :parent_folder, :project_folder, :project_id, :quota,











UNCOV

40



                                        :requested_by, :storage_size, :storage_unit, :number_of_files, :hpc, :smb, :globus,











UNCOV

41



                                        user_roles: [], departments: [])










42













UNCOV

43



      if request_params[:departments].present?




×








UNCOV

44



        request_params[:departments] = request_params[:departments].compact_blank.map { |dep_str| JSON.parse(dep_str) }




×








UNCOV

45



      end











UNCOV

46



      if request_params[:user_roles].present?




×







47



        request_params[:user_roles] = request_params[:user_roles].compact_blank.map { |role_str| JSON.parse(role_str) }




×








UNCOV

48



      end











UNCOV

49



      move_approved_values(request_params)




×








UNCOV

50



    end










51













UNCOV

52



    def move_approved_values(request_params)




1✔






UNCOV

53



      if request_params[:quota].present?




×








UNCOV

54



        request_params[:approved_quota] = request_params.delete(:quota)




×








UNCOV

55



      end











UNCOV

56



      if request_params[:storage_size].present?




×







57



        request_params[:approved_storage_size] = request_params.delete(:storage_size)




×








UNCOV

58



      end











UNCOV

59



      if request_params[:storage_unit].present?




×







60



        request_params[:approved_storage_unit] = request_params.delete(:storage_unit)




×








UNCOV

61



      end











UNCOV

62



      request_params




×








UNCOV

63



    end










64













UNCOV

65



    def set_breadcrumbs




1✔






UNCOV

66



      add_breadcrumb("Dashboard", dashboard_path)




×








UNCOV

67



      add_breadcrumb("Requests", requests_path)




×








UNCOV

68



    end










69













UNCOV

70



    def check_access




1✔






UNCOV

71



      return if user_eligible_to_modify_request?




×







72












73



      # request can not be modified by this user, redirect to the request











UNCOV

74



      flash[:notice] = I18n.t(:no_modify_submitted)




×








UNCOV

75



      redirect_to request_path(@request_model)




×








UNCOV

76



    end










77













UNCOV

78



    def user_eligible_to_modify_request?




1✔





79



      # elevated privs for the current user











UNCOV

80



      if current_user.sysadmin || (current_user.developer && !Rails.env.production?)




×








UNCOV

81



        true




×








UNCOV

82



      else











UNCOV

83



        false




×








UNCOV

84



      end











UNCOV

85



    end











UNCOV

86



end
























    

  • 

    •