From 03a3f2c446cb16ae62dc2cad2c9ac5a758a48292 Mon Sep 17 00:00:00 2001
From: luca-landa <lucalanda@hotmail.it>
Date: Fri, 26 Feb 2021 15:46:21 +0100
Subject: Set default config currency to subscription

Also add a spec that covers manual subscription creation from admin panel.
---
 app/models/solidus_subscriptions/subscription.rb   |  5 +++
 spec/features/admin/subscriptions_spec.rb          | 47 +++++++++++++++++++++-
 .../solidus_subscriptions/subscription_spec.rb     |  6 +++
 3 files changed, 56 insertions(+), 2 deletions(-)

diff --git a/app/models/solidus_subscriptions/subscription.rb b/app/models/solidus_subscriptions/subscription.rb
index 3190ab6..9b3ff68 100644
--- a/app/models/solidus_subscriptions/subscription.rb
+++ b/app/models/solidus_subscriptions/subscription.rb
@@ -33,6 +33,7 @@ module SolidusSubscriptions
     accepts_nested_attributes_for :line_items, allow_destroy: true, reject_if: ->(p) { p[:quantity].blank? }
 
     before_validation :set_payment_method
+    before_validation :set_currency
     before_create :generate_guest_token
     after_create :emit_event_for_creation
     before_update :update_actionable_date_if_interval_changed
@@ -311,6 +312,10 @@ module SolidusSubscriptions
       end
     end
 
+    def set_currency
+      self.currency ||= ::Spree::Config[:currency]
+    end
+
     def generate_guest_token
       self.guest_token ||= loop do
         random_token = SecureRandom.urlsafe_base64(nil, false)
diff --git a/spec/features/admin/subscriptions_spec.rb b/spec/features/admin/subscriptions_spec.rb
index 53228ec..1d9f698 100644
--- a/spec/features/admin/subscriptions_spec.rb
+++ b/spec/features/admin/subscriptions_spec.rb
@@ -1,18 +1,61 @@
 RSpec.describe 'Subscriptions admin' do
   stub_authorization!
 
+  let(:shipping_address_fieldset) { 'Shipping Address' }
+  let(:billing_address_fieldset) { 'Billing Address' }
+
   it 'Updating a subscription' do
     subscription = create(:subscription, :with_shipping_address, :with_billing_address)
 
     visit spree.admin_path
     click_link 'Subscriptions'
     find('.fa-edit').click
-    fill_in 'subscription[shipping_address_attributes][zipcode]', with: '33166'
-    fill_in 'subscription[billing_address_attributes][zipcode]', with: '33167'
+    within_fieldset(shipping_address_fieldset) do
+      fill_in 'Zip Code', with: '33166'
+    end
+    within_fieldset(billing_address_fieldset) do
+      fill_in 'Zip Code', with: '33167'
+    end
     click_button 'Update'
     subscription.reload
 
     expect(subscription.shipping_address.zipcode).to eq('33166')
     expect(subscription.billing_address.zipcode).to eq('33167')
   end
+
+  it 'Creates a subscription' do
+    variant = create(:variant, subscribable: true)
+    create(:user)
+    create(:store)
+
+    visit spree.admin_path
+    click_link 'Subscriptions'
+    click_link 'New Subscription'
+    fill_in 'Actionable date', with: '01/01/2020'
+    fill_in 'Interval length', with: 2
+    fill_in 'End date', with: '01/03/2020'
+
+    [shipping_address_fieldset, billing_address_fieldset].each do |fieldset|
+      within_fieldset(fieldset) do
+        name_input_label = if Spree.solidus_gem_version >= Gem::Version.new('2.11.0')
+                             'Name'
+                           else
+                             'First Name'
+                           end
+
+        fill_in name_input_label, with: 'John Doe'
+        fill_in 'Street Address', with: 'Street Address'
+        fill_in 'City', with: 'City'
+        fill_in 'Zip Code', with: '33166'
+        fill_in 'Phone', with: '1234567890'
+      end
+    end
+
+    select variant.name, from: 'Subscribable'
+    fill_in 'Quantity', with: 1
+
+    expect { click_on 'Create' }.to change { SolidusSubscriptions::Subscription.count }.by(1)
+
+    expect(page).to have_text('Subscription has been successfully created!')
+  end
 end
diff --git a/spec/models/solidus_subscriptions/subscription_spec.rb b/spec/models/solidus_subscriptions/subscription_spec.rb
index 6acc1b3..b9bfe56 100644
--- a/spec/models/solidus_subscriptions/subscription_spec.rb
+++ b/spec/models/solidus_subscriptions/subscription_spec.rb
@@ -31,6 +31,12 @@ RSpec.describe SolidusSubscriptions::Subscription, type: :model do
 
       expect(subscription.guest_token).to be_present
     end
+
+    it 'sets default config currency if not given' do
+      subscription = create(:subscription, currency: nil)
+
+      expect(subscription.currency).to eq(Spree::Config.currency)
+    end
   end
 
   describe 'updating a subscription' do
-- 
cgit v1.2.3