From 4d3fc7786738e12a0baec345fd93e3b5da5635fb Mon Sep 17 00:00:00 2001
From: Eric Schultz <eric@wwahammy.com>
Date: Mon, 2 Apr 2018 13:42:34 -0500
Subject: [PATCH] Stripe can now be created when org doesn't have email address

---
 lib/stripe_account.rb           |  2 +-
 spec/lib/stripe_account_spec.rb | 45 +++++++++++++++++++++++++++++++--
 2 files changed, 44 insertions(+), 3 deletions(-)

diff --git a/lib/stripe_account.rb b/lib/stripe_account.rb
index 9e3bd609..eaa236da 100644
--- a/lib/stripe_account.rb
+++ b/lib/stripe_account.rb
@@ -23,7 +23,7 @@ module StripeAccount
 		ParamValidation.new({:np => np}, {:np => {:required=> true, :is_a => Nonprofit}})
 		params = {
 				managed: true,
-				email: np['email'],
+				email: np['email'].present? ? np['email'] : np.roles.nonprofit_admins.order('created_at ASC').first.user.email,
 				business_name: np['name'],
 				legal_entity: {
 						type: 'company',
diff --git a/spec/lib/stripe_account_spec.rb b/spec/lib/stripe_account_spec.rb
index af38282d..87519a6a 100644
--- a/spec/lib/stripe_account_spec.rb
+++ b/spec/lib/stripe_account_spec.rb
@@ -7,7 +7,7 @@ describe StripeAccount do
   let(:stripe_helper) { StripeMock.create_test_helper }
   before(:each) { StripeMock.start}
   after(:each) { StripeMock.stop}
-  let!(:nonprofit) { force_create(:nonprofit)}
+  let(:nonprofit) { force_create(:nonprofit)}
 
   describe '.find_or_create' do
     describe 'param validation' do
@@ -92,7 +92,7 @@ describe StripeAccount do
       })
     end
 
-    describe 'saves properly' do
+    describe 'saves properly without org email' do
 
       let!(:result) { StripeAccount.create(nonprofit)}
 
@@ -118,6 +118,47 @@ describe StripeAccount do
         expect(np.stripe_account_id).to eq result
       end
     end
+
+    describe 'saves properly without org email' do
+
+      before(:each){
+        nonprofit.email = nil
+        nonprofit.save!
+
+        role
+      }
+
+      let(:admin_role_email) { "email_user@email.email"}
+      let(:user) { force_create(:user, email: admin_role_email)}
+      let(:role) { force_create(:role, user: user, host: nonprofit, name: :nonprofit_admin)}
+
+
+      let(:result) { StripeAccount.create(nonprofit)}
+
+      it 'returns a Stripe acct id' do
+
+        expect(result).to_not be_blank
+      end
+      it 'sets the Account values on Stripe' do
+        saved_account = Stripe::Account.retrieve(result)
+        expect(saved_account['managed']).to eq true
+        expect(saved_account['business_name']).to eq (nonprofit.name)
+        expect(saved_account['email']).to eq (admin_role_email)
+        expect(saved_account['business_url']).to eq (nonprofit.website)
+        expect(saved_account['legal_entity']['type']).to eq ("company")
+        expect(saved_account['legal_entity']['address']['city']).to eq (nonprofit.city)
+        expect(saved_account['legal_entity']['address']['state']).to eq (nonprofit.state_code)
+        expect(saved_account['legal_entity']['business_name']).to eq (nonprofit.name)
+        expect(saved_account['product_description']).to eq ('Nonprofit donations')
+        expect(saved_account['transfer_schedule']['interval']).to eq('manual')
+      end
+
+      it 'updates the nonprofit itself' do
+        result
+        np = Nonprofit.find(nonprofit.id)
+        expect(np.stripe_account_id).to eq result
+      end
+    end
   end
 
 end