c3e6d67f-257b-42f8-9be8-9a892888c795

Committed 18 Jun 2025 03:51AM UTC coverage: 95.786% (-1.6%) from 97.411%

Build # c3e6d67f-257b-42f8-9be8-9a892888c795

Build Type

push

circleci

Committed by

mark-dce

Commit Message

Disable embargo expiration notifications

Per discussions with SCO, they wish to reduce administrative overhead
by eliminating embargo expiration notifications.

We still need to process actual embargo expirations to change the
corresponding ETDs from a restricted to open status.

Therefore, this change leaves the schedule and service code in place
and simply comments out the calls to notification processing so that
we still get nightly expiration processing.

Run Details

2682 of 2800 relevant lines covered (95.79%)

36.47 hits per line

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

97.37

/app/controllers/in_progress_etds_controller.rb

class InProgressEtdsController < ApplicationController
  def new
    authorize! :create, InProgressEtd

    @in_progress_etd = if params[:etd_id]
                         InProgressEtd.find_or_create_by(etd_id: params[:etd_id])
                       else
                         InProgressEtd.find_or_create_by(user_ppid: current_user.ppid)
                       end

    # Now that the record has been created, render the form so student can edit it:
    redirect_to action: :edit, id: @in_progress_etd.id
  end

  # Note: There is no 'create' action because the 'new' action does 'find or create'.

  def edit
    if current_user.nil?
      redirect_to new_user_session_path
    else
      @in_progress_etd = InProgressEtd.find(params[:id])
      authorize! :update, @in_progress_etd
      @in_progress_etd.refresh_from_etd!
      @data = @in_progress_etd.data
    end
  rescue ActiveRecord::RecordNotFound
    redirect_to root_url
  end

  # The Vue.js form uses this action to update the record.
  def update
    @in_progress_etd = InProgressEtd.find(params[:id])
    authorize! :update, @in_progress_etd
    @in_progress_etd.data = prepare_etd_data.to_json

    if @in_progress_etd.save
      @data = @in_progress_etd.data
      render json: { in_progress_etd: @data, lastCompletedStep: current_step(@data), tab_name: tab_name }, status: 200
    else
      render json: { errors: @in_progress_etd.errors.messages }, status: 422
    end
  end

  def destroy
    @in_progress_etd = InProgressEtd.find(params[:id])
    authorize! :update, @in_progress_etd
    @in_progress_etd.destroy

    redirect_to root_url
  end

  private

    def current_step(data)
      saved_data = JSON.parse(data)
      saved_data['currentStep']
    end

    def tab_name
      etd = request.parameters.fetch(:etd)
      etd.fetch(:currentTab, "About Me")
    end

    def prepare_etd_data
      # TODO: strong params
      new_data = request.parameters.fetch(:etd, {})

      # Add the new data to the existing persisted data
      @in_progress_etd.add_data(new_data)
    end
end

1	class InProgressEtdsController < ApplicationController	2✔
2	def new	2✔
3	authorize! :create, InProgressEtd	10✔
4
5	@in_progress_etd = if params[:etd_id]	9✔
6	InProgressEtd.find_or_create_by(etd_id: params[:etd_id])	5✔
7	else
8	InProgressEtd.find_or_create_by(user_ppid: current_user.ppid)	4✔
9	end
10
11	# Now that the record has been created, render the form so student can edit it:
12	redirect_to action: :edit, id: @in_progress_etd.id	9✔
13	end
14
15	# Note: There is no 'create' action because the 'new' action does 'find or create'.
16
17	def edit	2✔
18	if current_user.nil?	13✔
19	redirect_to new_user_session_path	2✔
20	else
21	@in_progress_etd = InProgressEtd.find(params[:id])	11✔
22	authorize! :update, @in_progress_etd	10✔
23	@in_progress_etd.refresh_from_etd!	8✔
24	@data = @in_progress_etd.data	8✔
25	end
26	rescue ActiveRecord::RecordNotFound
27	redirect_to root_url	1✔
28	end
29
30	# The Vue.js form uses this action to update the record.
31	def update	2✔
32	@in_progress_etd = InProgressEtd.find(params[:id])	11✔
33	authorize! :update, @in_progress_etd	11✔
34	@in_progress_etd.data = prepare_etd_data.to_json	9✔
35
36	if @in_progress_etd.save	9✔
37	@data = @in_progress_etd.data	9✔
38	render json: { in_progress_etd: @data, lastCompletedStep: current_step(@data), tab_name: tab_name }, status: 200	9✔
39	else
40	render json: { errors: @in_progress_etd.errors.messages }, status: 422	×
41	end
42	end
43
44	def destroy	2✔
45	@in_progress_etd = InProgressEtd.find(params[:id])	2✔
46	authorize! :update, @in_progress_etd	2✔
47	@in_progress_etd.destroy	1✔
48
49	redirect_to root_url	1✔
50	end
51
52	private	2✔
53
54	def current_step(data)	2✔
55	saved_data = JSON.parse(data)	9✔
56	saved_data['currentStep']	9✔
57	end
58
59	def tab_name	2✔
60	etd = request.parameters.fetch(:etd)	9✔
61	etd.fetch(:currentTab, "About Me")	9✔
62	end
63
64	def prepare_etd_data	2✔
65	# TODO: strong params
66	new_data = request.parameters.fetch(:etd, {})	9✔
67
68	# Add the new data to the existing persisted data
69	@in_progress_etd.add_data(new_data)	9✔
70	end
71	end

curationexperts / laevigata / c3e6d67f-257b-42f8-9be8-9a892888c795

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

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