Browse Source

Fix tests

pull/2606/head
Manny Dinssa 2 years ago
parent
commit
28eeb18b8c
  1. 6
      spec/requests/merge_requests_controller_spec.rb

6
spec/requests/merge_requests_controller_spec.rb

@ -491,7 +491,7 @@ RSpec.describe MergeRequestsController, type: :request do
end end
context "with a merged request" do context "with a merged request" do
let(:merge_request) { create(:merge_request, request_merged: true, total_users: 34, total_schemes: 12, total_stock_owners: 8, total_managing_agents: 5) } let(:merge_request) { create(:merge_request, request_merged: true, total_users: 34, total_schemes: 12, total_stock_owners: 8, total_managing_agents: 5) }
it "shows saved users count and doesn't have links to view merge outcomes" do it "shows saved users count and doesn't have links to view merge outcomes" do
expect(merge_request.status).to eq("request_merged") expect(merge_request.status).to eq("request_merged")
@ -506,7 +506,7 @@ RSpec.describe MergeRequestsController, type: :request do
end end
it "shows stock owners and managing agents count and doesn't have links to view merge outcomes" do it "shows stock owners and managing agents count and doesn't have links to view merge outcomes" do
expect(merge_request.status).to eq("processing") expect(merge_request.status).to eq("request_merged")
expect(page).not_to have_link("View", href: relationship_outcomes_merge_request_path(merge_request)) expect(page).not_to have_link("View", href: relationship_outcomes_merge_request_path(merge_request))
expect(page).to have_content("8 stock owners") expect(page).to have_content("8 stock owners")
expect(page).to have_content("5 managing agents") expect(page).to have_content("5 managing agents")
@ -514,7 +514,7 @@ RSpec.describe MergeRequestsController, type: :request do
end end
context "with a processing request" do context "with a processing request" do
let(:merge_request) { create(:merge_request, processing: true, total_users: 51, total_schemes: 33, total_stock_owners: 15, total_managing_agents: 20) } let(:merge_request) { create(:merge_request, processing: true, total_users: 51, total_schemes: 33, total_stock_owners: 15, total_managing_agents: 20) }
it "shows saved users count and doesn't have links to view merge outcomes" do it "shows saved users count and doesn't have links to view merge outcomes" do
expect(merge_request.status).to eq("processing") expect(merge_request.status).to eq("processing")

Loading…
Cancel
Save