solidus_core 1.2.3 → 1.3.0.beta1

data/spec/models/spree/order_updater_spec.rb

@@ -2,6 +2,7 @@ require 'spec_helper'
 
 module Spree
   describe OrderUpdater, type: :model do
+    let!(:store) { create :store }
     let(:order) { Spree::Order.create }
     let(:updater) { Spree::OrderUpdater.new(order) }
 
@@ -38,14 +39,14 @@ module Spree
         let(:promotion_action) do
           Promotion::Actions::CreateAdjustment.create!({
             calculator: calculator,
-            promotion: promotion,
+            promotion: promotion
           })
         end
 
         before do
           updater.update
           create(:adjustment, source: promotion_action, adjustable: order, order: order)
-          create(:line_item, :order => order, price: 10) # in addition to the two already created
+          create(:line_item, order: order, price: 10) # in addition to the two already created
           updater.update
         end
 
@@ -60,7 +61,7 @@ module Spree
         order.line_items.first.update_columns({
           adjustment_total: 10.05,
           additional_tax_total: 0.05,
-          included_tax_total: 0.05,
+          included_tax_total: 0.05
         })
         updater.update_adjustment_total
         expect(order.adjustment_total).to eq(10.05)
@@ -86,7 +87,6 @@ module Spree
         expect(order.shipment_state).to be_nil
       end
 
-
       ["shipped", "ready", "pending"].each do |state|
         it "is #{state}" do
           create(:shipment, order: order, state: state)
@@ -165,7 +165,6 @@ module Spree
       end
 
       context "order is canceled" do
-
         before do
           order.state = 'canceled'
         end
@@ -181,7 +180,6 @@ module Spree
         end
 
         context "and is paid" do
-
           it "is credit_owed" do
             order.payment_total = 30
             order.total = 30
@@ -190,7 +188,6 @@ module Spree
               updater.update_payment_state
             }.to change { order.payment_state }.to 'credit_owed'
           end
-
         end
 
         context "and payment is refunded" do
@@ -203,7 +200,6 @@ module Spree
           end
         end
       end
-
     end
 
     it "state change" do

data/spec/models/spree/payment_create_spec.rb

@@ -25,13 +25,13 @@ module Spree
     context 'with a new source' do
       let(:attributes) do
         {
-          :amount => 100,
-          :payment_method => payment_method,
-          :source_attributes => {
-            :expiry =>"01 / 99",
-            :number => '1234567890123',
-            :verification_value => '123',
-            :name => 'Foo Bar'
+          amount: 100,
+          payment_method: payment_method,
+          source_attributes: {
+            expiry: "01 / 99",
+            number: '1234567890123',
+            verification_value: '123',
+            name: 'Foo Bar'
           }
         }
       end
@@ -48,9 +48,8 @@ module Spree
           {
             amount: 100,
             payment_method: payment_method,
-            source_attributes: {expiry: "1 / 12"}
+            source_attributes: { expiry: "1 / 12" }
           }
-
         end
         it "errors when payment source not valid" do
           expect(new_payment).not_to be_valid
@@ -71,7 +70,7 @@ module Spree
         {
           source_attributes: {
             existing_card_id: credit_card.id,
-            verification_value: '321',
+            verification_value: '321'
           }
         }
       end
@@ -89,9 +88,9 @@ module Spree
       end
 
       context 'with request_env' do
-        let(:request_env){ {'USER_AGENT' => 'Firefox'} }
+        let(:request_env){ { 'USER_AGENT' => 'Firefox' } }
         it 'sets the request_env on the payment' do
-          expect(new_payment.request_env).to eq({'USER_AGENT' => 'Firefox'})
+          expect(new_payment.request_env).to eq({ 'USER_AGENT' => 'Firefox' })
         end
       end
 

data/spec/models/spree/payment_method/store_credit_spec.rb

@@ -70,7 +70,7 @@ describe Spree::PaymentMethod::StoreCredit do
     let(:auth_code) { auth_event.authorization_code }
     let(:gateway_options) { super().merge(originator: originator) }
 
-    let(:authorized_amount) { capture_amount/100.0 }
+    let(:authorized_amount) { capture_amount / 100.0 }
     let(:auth_event) { create(:store_credit_auth_event, store_credit: store_credit, amount: authorized_amount) }
     let(:store_credit) { create(:store_credit, amount_authorized: authorized_amount) }
     let(:originator) { nil }
@@ -85,7 +85,7 @@ describe Spree::PaymentMethod::StoreCredit do
     end
 
     context 'when unable to authorize the amount' do
-      let(:authorized_amount) { (capture_amount-1)/100 }
+      let(:authorized_amount) { (capture_amount - 1) / 100 }
 
       before do
         allow_any_instance_of(Spree::StoreCredit).to receive_messages(authorize: true)
@@ -256,10 +256,12 @@ describe Spree::PaymentMethod::StoreCredit do
     let(:captured_amount) { 10.0 }
 
     context "capture event found" do
-      let!(:store_credit_event) { create(:store_credit_capture_event,
+      let!(:store_credit_event) {
+        create(:store_credit_capture_event,
                                         authorization_code: auth_code,
                                         amount: captured_amount,
-                                        store_credit: store_credit) }
+                                        store_credit: store_credit)
+      }
 
       it "creates a store credit for the same amount that was captured" do
         expect_any_instance_of(Spree::StoreCredit).to receive(:credit).with(captured_amount, auth_code, store_credit.currency)
@@ -269,10 +271,12 @@ describe Spree::PaymentMethod::StoreCredit do
 
     context "capture event not found" do
       context "auth event found" do
-        let!(:store_credit_event) { create(:store_credit_auth_event,
+        let!(:store_credit_event) {
+          create(:store_credit_auth_event,
                                           authorization_code: auth_code,
                                           amount: captured_amount,
-                                          store_credit: store_credit) }
+                                          store_credit: store_credit)
+        }
 
         it "creates a store credit for the same amount that was captured" do
           expect_any_instance_of(Spree::StoreCredit).to receive(:void).with(auth_code)

data/spec/models/spree/payment_method_spec.rb

@@ -1,6 +1,6 @@
 require 'spec_helper'
 
-describe Spree::PaymentMethod, :type => :model do
+describe Spree::PaymentMethod, type: :model do
   describe ".available" do
     let!(:payment_method_nil_display)  { create(:payment_method, active: true, display_on: nil) }
     let!(:payment_method_both_display) { create(:payment_method, active: true, display_on: 'both') }
@@ -34,7 +34,7 @@ describe Spree::PaymentMethod, :type => :model do
             payment_method_nil_display,
             payment_method_both_display,
             payment_method_front_display,
-            payment_method_back_display,
+            payment_method_back_display
           ]
         )
       end
@@ -64,7 +64,7 @@ describe Spree::PaymentMethod, :type => :model do
                 payment_method_nil_display,
                 payment_method_both_display,
                 store_2_payment_method,
-                no_store_payment_method,
+                no_store_payment_method
               ]
             )
           end
@@ -78,7 +78,7 @@ describe Spree::PaymentMethod, :type => :model do
               payment_method_nil_display,
               payment_method_both_display,
               store_2_payment_method,
-              no_store_payment_method,
+              no_store_payment_method
             ]
           )
         end
@@ -143,5 +143,4 @@ describe Spree::PaymentMethod, :type => :model do
       end
     end
   end
-
 end

data/spec/models/spree/payment_spec.rb

@@ -1,12 +1,13 @@
 require 'spec_helper'
 
-describe Spree::Payment, :type => :model do
-  let(:order) { Spree::Order.create }
+describe Spree::Payment, type: :model do
+  let(:store) { create :store }
+  let(:order) { Spree::Order.create(store: store) }
   let(:refund_reason) { create(:refund_reason) }
 
   let(:gateway) do
-    gateway = Spree::Gateway::Bogus.new(:active => true, :name => 'Bogus gateway')
-    allow(gateway).to receive_messages :source_required => true
+    gateway = Spree::Gateway::Bogus.new(active: true, name: 'Bogus gateway')
+    allow(gateway).to receive_messages source_required: true
     gateway
   end
 
@@ -44,7 +45,6 @@ describe Spree::Payment, :type => :model do
   end
 
   context '.risky' do
-
     let!(:payment_1) { create(:payment, avs_response: 'Y', cvv_response_code: 'M', cvv_response_message: 'Match') }
     let!(:payment_2) { create(:payment, avs_response: 'Y', cvv_response_code: 'M', cvv_response_message: '') }
     let!(:payment_3) { create(:payment, avs_response: 'A', cvv_response_code: 'M', cvv_response_message: 'Match') }
@@ -53,7 +53,6 @@ describe Spree::Payment, :type => :model do
     it 'should not return successful responses' do
       expect(subject.class.risky.to_a).to match_array([payment_3, payment_4])
     end
-
   end
 
   context "#captured_amount" do
@@ -91,7 +90,7 @@ describe Spree::Payment, :type => :model do
       payment.source = Spree::CreditCard.new
       expect(payment).not_to be_valid
       cc_errors = payment.errors['Credit Card']
-      expect(cc_errors).to include("Number can't be blank")
+      expect(cc_errors).to include("Card Number can't be blank")
       expect(cc_errors).to include("Month is not a number")
       expect(cc_errors).to include("Year is not a number")
       expect(cc_errors).to include("Verification Value can't be blank")
@@ -111,7 +110,6 @@ describe Spree::Payment, :type => :model do
       payment.failure
       expect(payment.state).to eql('failed')
     end
-
   end
 
   context 'invalidate' do
@@ -122,7 +120,6 @@ describe Spree::Payment, :type => :model do
     end
 
     context "the payment's source is invalid" do
-
       before(:each) do
         card.year = 2014
         payment.source = card
@@ -131,7 +128,7 @@ describe Spree::Payment, :type => :model do
       it "transitions to invalid" do
         payment.state = 'checkout'
         payment.invalidate
-        expect(payment.state).to eq ('invalid')
+        expect(payment.state).to eq 'invalid'
       end
     end
   end
@@ -210,22 +207,22 @@ describe Spree::Payment, :type => :model do
 
       it "should invalidate if payment method doesnt support source" do
         expect(payment.payment_method).to receive(:supports?).with(payment.source).and_return(false)
-        expect { payment.process!}.to raise_error(Spree::Core::GatewayError)
+        expect { payment.process! }.to raise_error(Spree::Core::GatewayError)
         expect(payment.state).to eq('invalid')
       end
 
       # Regression test for https://github.com/spree/spree/issues/4598
       it "should allow payments with a gateway_customer_profile_id" do
-        allow(payment.source).to receive_messages :gateway_customer_profile_id => "customer_1"
-        expect(payment.payment_method).to receive(:supports?).with(payment.source).and_return(false)
+        payment.source.update!(gateway_customer_profile_id: "customer_1", brand: 'visa')
+        expect(payment.payment_method.provider_class).to receive(:supports?).with('visa').and_return(false)
         expect(payment).to receive(:started_processing!)
         payment.process!
       end
 
       # Another regression test for https://github.com/spree/spree/issues/4598
       it "should allow payments with a gateway_payment_profile_id" do
-        allow(payment.source).to receive_messages :gateway_payment_profile_id => "customer_1"
-        expect(payment.payment_method).to receive(:supports?).with(payment.source).and_return(false)
+        payment.source.update!(gateway_payment_profile_id: "customer_1", brand: 'visa')
+        expect(payment.payment_method.provider_class).to receive(:supports?).with('visa').and_return(false)
         expect(payment).to receive(:started_processing!)
         payment.process!
       end
@@ -240,10 +237,10 @@ describe Spree::Payment, :type => :model do
       end
 
       it "should call authorize on the gateway with the currency code" do
-        allow(payment).to receive_messages :currency => 'GBP'
+        allow(payment).to receive_messages currency: 'GBP'
         expect(payment.payment_method).to receive(:authorize).with(amount_in_cents,
                                                                card,
-                                                               hash_including({:currency => "GBP"})).and_return(success_response)
+                                                               hash_including({ currency: "GBP" })).and_return(success_response)
         payment.authorize!
       end
 
@@ -463,7 +460,7 @@ describe Spree::Payment, :type => :model do
 
         context "if unsuccessful" do
           it "should not make payment complete" do
-            allow(gateway).to receive_messages :capture => failed_response
+            allow(gateway).to receive_messages capture: failed_response
             expect(payment).to receive(:failure)
             expect(payment).not_to receive(:complete)
             expect { payment.capture! }.to raise_error(Spree::Core::GatewayError)
@@ -495,7 +492,7 @@ describe Spree::Payment, :type => :model do
       context "if successful" do
         it "should update the response_code with the authorization from the gateway" do
           # Change it to something different
-          allow(gateway).to receive_messages :cancel => success_response
+          allow(gateway).to receive_messages cancel: success_response
           payment.cancel!
           expect(payment.state).to eq('void')
           expect(payment.response_code).to eq('123')
@@ -504,16 +501,14 @@ describe Spree::Payment, :type => :model do
 
       context "if unsuccessful" do
         it "should not void the payment" do
-          allow(gateway).to receive_messages :cancel => failed_response
+          allow(gateway).to receive_messages cancel: failed_response
           expect { payment.cancel! }.to raise_error(Spree::Core::GatewayError)
           expect(payment.state).to eq('pending')
           expect(payment.response_code).to eq('abc')
         end
       end
-
     end
 
-
     describe "#void_transaction!" do
       before do
         payment.response_code = '123'
@@ -522,7 +517,7 @@ describe Spree::Payment, :type => :model do
 
       context "when profiles are supported" do
         it "should call payment_gateway.void with the payment's response_code" do
-          allow(gateway).to receive_messages :payment_profiles_supported? => true
+          allow(gateway).to receive_messages payment_profiles_supported?: true
           expect(gateway).to receive(:void).with('123', card, anything).and_return(success_response)
           payment.void_transaction!
         end
@@ -530,14 +525,14 @@ describe Spree::Payment, :type => :model do
 
       context "when profiles are not supported" do
         it "should call payment_gateway.void with the payment's response_code" do
-          allow(gateway).to receive_messages :payment_profiles_supported? => false
+          allow(gateway).to receive_messages payment_profiles_supported?: false
           expect(gateway).to receive(:void).with('123', anything).and_return(success_response)
           payment.void_transaction!
         end
       end
 
       it "should log the response" do
-        expect(payment.log_entries).to receive(:create!).with(:details => anything)
+        expect(payment.log_entries).to receive(:create!).with(details: anything)
         payment.void_transaction!
       end
 
@@ -552,7 +547,7 @@ describe Spree::Payment, :type => :model do
 
       context "if unsuccessful" do
         it "should not void the payment" do
-          allow(gateway).to receive_messages :void => failed_response
+          allow(gateway).to receive_messages void: failed_response
           expect(payment).not_to receive(:void)
           expect { payment.void_transaction! }.to raise_error(Spree::Core::GatewayError)
         end
@@ -570,7 +565,6 @@ describe Spree::Payment, :type => :model do
         end
       end
     end
-
   end
 
   context "when already processing" do
@@ -597,7 +591,7 @@ describe Spree::Payment, :type => :model do
     context "raises no error if source is not specified" do
       before do
         payment.source = nil
-        allow(payment.payment_method).to receive_messages(:source_required? => false)
+        allow(payment.payment_method).to receive_messages(source_required?: false)
       end
 
       specify do
@@ -640,7 +634,7 @@ describe Spree::Payment, :type => :model do
     context "not completed payments" do
       it "doesn't update order payment total" do
         expect {
-          Spree::Payment.create(:amount => 100, :order => order)
+          Spree::Payment.create(amount: 100, order: order)
         }.not_to change { order.payment_total }
       end
     end
@@ -661,20 +655,19 @@ describe Spree::Payment, :type => :model do
     end
 
     context "completed orders" do
-      let(:payment_method) { create(:check_payment_method) }
       before { allow(order).to receive_messages completed?: true }
 
       it "updates payment_state and shipments" do
         expect(order.updater).to receive(:update_payment_state)
         expect(order.updater).to receive(:update_shipment_state)
-        Spree::Payment.create!(amount: 100, order: order, payment_method: payment_method)
+        Spree::Payment.create(amount: 100, order: order)
       end
     end
 
     context "when profiles are supported" do
       before do
-        allow(gateway).to receive_messages :payment_profiles_supported? => true
-        allow(payment.source).to receive_messages :has_payment_profile? => false
+        allow(gateway).to receive_messages payment_profiles_supported?: true
+        allow(payment.source).to receive_messages has_payment_profile?: false
       end
 
       context "when there is an error connecting to the gateway" do
@@ -682,10 +675,10 @@ describe Spree::Payment, :type => :model do
           expect(gateway).to receive(:create_profile).and_raise(ActiveMerchant::ConnectionError.new("foo", nil))
           expect do
             Spree::Payment.create(
-              :amount => 100,
-              :order => order,
-              :source => card,
-              :payment_method => gateway
+              amount: 100,
+              order: order,
+              source: card,
+              payment_method: gateway
             )
           end.to raise_error(Spree::Core::GatewayError)
         end
@@ -713,32 +706,31 @@ describe Spree::Payment, :type => :model do
             amount: 100
           )
         end
-
       end
 
       context "when successfully connecting to the gateway" do
         it "should create a payment profile" do
           expect(payment.payment_method).to receive :create_profile
-          payment = Spree::Payment.create(
-            :amount => 100,
-            :order => order,
-            :source => card,
-            :payment_method => gateway
+          Spree::Payment.create(
+            amount: 100,
+            order: order,
+            source: card,
+            payment_method: gateway
           )
         end
       end
     end
 
     context "when profiles are not supported" do
-      before { allow(gateway).to receive_messages :payment_profiles_supported? => false }
+      before { allow(gateway).to receive_messages payment_profiles_supported?: false }
 
       it "should not create a payment profile" do
         expect(gateway).not_to receive :create_profile
-        payment = Spree::Payment.create(
-          :amount => 100,
-          :order => order,
-          :source => card,
-          :payment_method => gateway
+        Spree::Payment.create(
+          amount: 100,
+          order: order,
+          source: card,
+          payment_method: gateway
         )
       end
     end
@@ -747,7 +739,7 @@ describe Spree::Payment, :type => :model do
   describe '#invalidate_old_payments' do
     it 'should not invalidate other payments if not valid' do
       payment.save
-      invalid_payment = Spree::Payment.new(:amount => 100, :order => order, :state => 'invalid', :payment_method => gateway)
+      invalid_payment = Spree::Payment.new(amount: 100, order: order, state: 'invalid', payment_method: gateway)
       invalid_payment.save
       expect(payment.reload.state).to eq('checkout')
     end
@@ -764,13 +756,13 @@ describe Spree::Payment, :type => :model do
     context 'with a new source' do
       let(:params) do
         {
-          :amount => 100,
-          :payment_method => gateway,
-          :source_attributes => {
-            :expiry =>"01 / 99",
-            :number => '1234567890123',
-            :verification_value => '123',
-            :name => 'Spree Commerce'
+          amount: 100,
+          payment_method: gateway,
+          source_attributes: {
+            expiry: "01 / 99",
+            number: '1234567890123',
+            verification_value: '123',
+            name: 'Spree Commerce'
           }
         }
       end
@@ -790,8 +782,8 @@ describe Spree::Payment, :type => :model do
       end
 
       it "errors when payment source not valid" do
-        params = { :amount => 100, :payment_method => gateway,
-          :source_attributes => {:expiry => "1 / 12" }}
+        params = { amount: 100, payment_method: gateway,
+          source_attributes: { expiry: "1 / 12" } }
 
         payment = Spree::Payment.new(params)
         expect(payment).not_to be_valid
@@ -816,8 +808,8 @@ describe Spree::Payment, :type => :model do
         {
           source_attributes: {
             existing_card_id: credit_card.id,
-            verification_value: '321',
-          },
+            verification_value: '321'
+          }
         }
       end
 
@@ -840,8 +832,8 @@ describe Spree::Payment, :type => :model do
       end
 
       it 'sets the request_env on the payment' do
-        payment = order.payments.create!(params.merge(request_env: {'USER_AGENT' => 'Firefox'}))
-        expect(payment.request_env).to eq({'USER_AGENT' => 'Firefox'})
+        payment = order.payments.create!(params.merge(request_env: { 'USER_AGENT' => 'Firefox' }))
+        expect(payment.request_env).to eq({ 'USER_AGENT' => 'Firefox' })
       end
 
       context 'the credit card belongs to a different user' do
@@ -893,7 +885,7 @@ describe Spree::Payment, :type => :model do
 
   # Regression test for https://github.com/spree/spree/issues/2216
   describe "#gateway_options" do
-    before { allow(order).to receive_messages(:last_ip_address => "192.168.1.1") }
+    before { allow(order).to receive_messages(last_ip_address: "192.168.1.1") }
 
     it "contains an IP" do
       expect(payment.gateway_options[:ip]).to eq(order.last_ip_address)
@@ -903,7 +895,7 @@ describe Spree::Payment, :type => :model do
       # Sets the payment's order to a different Ruby object entirely
       payment.order = Spree::Order.find(payment.order_id)
       email = 'foo@example.com'
-      order.update_attributes(:email => email)
+      order.update_attributes(email: email)
       expect(payment.gateway_options[:email]).to eq(email)
     end
   end
@@ -1023,7 +1015,7 @@ describe Spree::Payment, :type => :model do
 
     context "when the locale uses a coma as a decimal separator" do
       before(:each) do
-        I18n.backend.store_translations(:fr, { :number => { :currency => { :format => { :delimiter => ' ', :separator => ',' } } } })
+        I18n.backend.store_translations(:fr, { number: { currency: { format: { delimiter: ' ', separator: ',' } } } })
         I18n.locale = :fr
         subject.amount = amount
       end
@@ -1130,10 +1122,10 @@ describe Spree::Payment, :type => :model do
       expect(payment.state_changes).to be_empty
       expect(payment.process!).to be true
       expect(payment.state_changes.count).to eq(2)
-      changes = payment.state_changes.map { |change| { change.previous_state => change.next_state} }
+      changes = payment.state_changes.map { |change| { change.previous_state => change.next_state } }
       expect(changes).to match_array([
-        {"checkout" => "processing"},
-        { "processing" => "pending"}
+        { "checkout" => "processing" },
+        { "processing" => "pending" }
       ])
     end
   end

data/spec/models/spree/permission_sets/base_spec.rb

@@ -6,7 +6,7 @@ describe Spree::PermissionSets::Base do
 
   describe "activate!" do
     it "raises a not implemented error" do
-      expect{subject}.to raise_error(NotImplementedError)
+      expect{ subject }.to raise_error(NotImplementedError)
     end
   end
 end

data/spec/models/spree/permission_sets/configuration_display.rb

@@ -79,4 +79,3 @@ describe Spree::PermissionSets::ConfigurationDisplay do
     it { is_expected.not_to be_able_to(:admin, Spree::ReturnReason) }
   end
 end
-

data/spec/models/spree/permission_sets/configuration_management_spec.rb

@@ -47,4 +47,3 @@ describe Spree::PermissionSets::ConfigurationManagement do
     it { is_expected.not_to be_able_to(:manage, Spree::ReturnReason) }
   end
 end
-

data/spec/models/spree/permission_sets/dashboard_display_spec.rb

@@ -19,4 +19,3 @@ describe Spree::PermissionSets::DashboardDisplay do
     it { is_expected.not_to be_able_to(:home, :dashboards) }
   end
 end
-

data/spec/models/spree/permission_sets/order_display_spec.rb

@@ -52,4 +52,3 @@ describe Spree::PermissionSets::OrderDisplay do
     it { is_expected.not_to be_able_to(:display, Spree::Refund) }
   end
 end
-

data/spec/models/spree/permission_sets/order_management_spec.rb

@@ -39,4 +39,3 @@ describe Spree::PermissionSets::OrderManagement do
     it { is_expected.not_to be_able_to(:manage, Spree::Refund) }
   end
 end
-

data/spec/models/spree/permission_sets/product_display_spec.rb

@@ -57,4 +57,3 @@ describe Spree::PermissionSets::ProductDisplay do
     it { is_expected.not_to be_able_to(:edit, Spree::Product) }
   end
 end
-

data/spec/models/spree/permission_sets/product_management_spec.rb

@@ -37,4 +37,3 @@ describe Spree::PermissionSets::ProductManagement do
     it { is_expected.not_to be_able_to(:manage, Spree::Classification) }
   end
 end
-

data/spec/models/spree/permission_sets/promotion_display_spec.rb

@@ -37,4 +37,3 @@ describe Spree::PermissionSets::PromotionDisplay do
     it { is_expected.not_to be_able_to(:edit, Spree::Promotion) }
   end
 end
-

data/spec/models/spree/permission_sets/promotion_management_spec.rb

@@ -23,4 +23,3 @@ describe Spree::PermissionSets::PromotionManagement do
     it { is_expected.not_to be_able_to(:manage, Spree::PromotionCategory) }
   end
 end
-

data/spec/models/spree/permission_sets/report_display_spec.rb

@@ -21,4 +21,3 @@ describe Spree::PermissionSets::ReportDisplay do
     it { is_expected.not_to be_able_to(:sales_total, :reports) }
   end
 end
-