From f5b0f6bfce92fa4e7dc0fd20fd4bd1019ab517e1 Mon Sep 17 00:00:00 2001 From: JG Date: Tue, 31 May 2022 15:50:00 +0100 Subject: [PATCH] better test for listing users --- spec/requests/organisations_controller_spec.rb | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/spec/requests/organisations_controller_spec.rb b/spec/requests/organisations_controller_spec.rb index 2cc6a463c..9a45f37f0 100644 --- a/spec/requests/organisations_controller_spec.rb +++ b/spec/requests/organisations_controller_spec.rb @@ -546,6 +546,7 @@ RSpec.describe OrganisationsController, type: :request do context "when viewing a specific organisation users" do let!(:users) { FactoryBot.create_list(:user, 5, organisation: user.organisation) } + let!(:different_org_users) { FactoryBot.create_list(:user, 5) } before do get "/organisations/#{organisation.id}/users", headers:, params: {} @@ -560,12 +561,18 @@ RSpec.describe OrganisationsController, type: :request do expect(page).to have_content("Users") end - it "returns all users" do + it "displays users for this organisation" do expect(page).to have_content(user.email) users.each do |user| expect(page).to have_content(user.email) end end + + it "doesn't display users for other organisations" do + different_org_users.each do |different_org_user| + expect(page).not_to have_content(different_org_user.email) + end + end end