Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 2 additions & 7 deletions spec/requests/api/v1/event_procedures_request_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -554,14 +554,9 @@
let(:params) { {} }

context "when user is authenticated" do
it "returns ok" do
event_procedure = create(:event_procedure, user_id: user.id)

delete "/api/v1/event_procedures/#{event_procedure.id}", headers: headers
let(:event_procedure) { create(:event_procedure, user_id: user.id) }

expect(response.parsed_body[:message]).to eq("#{event_procedure.class} deleted successfully.")
expect(response).to have_http_status(:ok)
end
include_examples "delete request returns ok", EventProcedure

context "when event_procedure cannot be destroyed" do
it "returns unprocessable_content" do
Expand Down
11 changes: 3 additions & 8 deletions spec/requests/api/v1/hospitals_request_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -169,15 +169,8 @@
let(:http_method) { :delete }
let(:params) { {} }

include_context "when user is not authenticated"

context "when user is authenticated" do
it "returns ok" do
delete path, headers: headers

expect(response.parsed_body[:message]).to eq("#{hospital.class} deleted successfully.")
expect(response).to have_http_status(:ok)
end
include_examples "delete request returns ok", Hospital

context "when hospital cannot be destroyed" do
it "returns unprocessable_content" do
Expand All @@ -199,5 +192,7 @@
end
end
end

include_context "when user is not authenticated"
end
end
5 changes: 1 addition & 4 deletions spec/requests/api/v1/medical_shifts_request_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -385,10 +385,7 @@
let(:params) { {} }

context "when user is authenticated" do
before { delete path, headers: headers }

it { expect(response.parsed_body[:message]).to eq("#{medical_shift.class} deleted successfully.") }
it { expect(response).to have_http_status(:ok) }
include_examples "delete request returns ok", MedicalShift

context "when does not find medical_shifts" do
let(:fake_id) { 9999 }
Expand Down
7 changes: 1 addition & 6 deletions spec/requests/api/v1/patients_request_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -166,12 +166,7 @@
let(:params) { {} }

context "when user is authenticated" do
it "returns ok" do
delete path, headers: headers

expect(response.parsed_body[:message]).to eq("#{patient.class} deleted successfully.")
expect(response).to have_http_status(:ok)
end
include_examples "delete request returns ok", Patient

context "when patient cannot be destroyed" do
it "returns unprocessable_content" do
Expand Down
7 changes: 1 addition & 6 deletions spec/requests/api/v1/procedures_request_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -126,12 +126,7 @@
let(:params) { {} }

context "when user is authenticated" do
it "returns ok" do
delete path, headers: headers

expect(response.parsed_body[:message]).to eq("#{procedure.class} deleted successfully.")
expect(response).to have_http_status(:ok)
end
include_examples "delete request returns ok", Procedure

context "when procedure cannot be destroyed" do
it "returns unprocessable_content" do
Expand Down
10 changes: 10 additions & 0 deletions spec/support/shared_contexts/successful_resource_delete.rb
Original file line number Diff line number Diff line change
@@ -0,0 +1,10 @@
# frozen_string_literal: true

RSpec.shared_examples "delete request returns ok" do |model_class|
it "returns ok" do
send(http_method, path, params: params, headers: headers)

expect(response.parsed_body[:message]).to eq("#{model_class} deleted successfully.")
expect(response).to have_http_status(:ok)
end
end