Add specs

Author: jonny5

spec/requests/partners_requests_spec.rb

@@ -468,6 +468,22 @@
       end
     end
 
+    context "csv file with a partner email already in another organization" do
+      let(:file) { fixture_file_upload("partners_with_six_fields.csv", "text/csv") }
+
+      before do
+        other_org = create(:organization)
+        create(:partner, name: "Existing Partner", email: "[email protected]", organization: other_org)
+      end
+
+      subject { post import_csv_partners_path, params: { file: file } }
+
+      it "shows an error for the partner in another organization" do
+        subject
+        expect(flash[:error]).to match(/has already been taken/)
+      end
+    end
+
     context "csv file with wrong headers" do
       let(:file) { fixture_file_upload("wrong_headers.csv", "text/csv") }
       subject { post import_csv_partners_path, params: { file: file } }

spec/services/partner_create_service_spec.rb

@@ -30,6 +30,23 @@
       end
     end
 
+    context 'when the partner email already exists in a different organization' do
+      let(:other_organization) { create(:organization) }
+
+      before do
+        create(:partner, email: partner_attrs[:email], organization: other_organization)
+      end
+
+      it 'should contain an error about the partner being with a different organization' do
+        result = subject
+        expect(result.errors[:email]).to include("has already been taken")
+      end
+
+      it 'should not create a new partner' do
+        expect { subject }.not_to change { Partner.count }
+      end
+    end
+
     context 'when the arguments are valid' do
       it 'should create a new partner record with the organization provided' do
         expect { subject }.to change { organization.partners.count }.by(1)
@@ -103,4 +120,3 @@
     end
   end
 end
-