spree_backend 3.2.9 → 3.3.0.rc1
Sign up to get free protection for your applications and to get access to all the features.
- checksums.yaml +4 -4
- data/app/assets/javascripts/spree/backend/admin.js +16 -15
- data/app/assets/javascripts/spree/backend/checkouts/edit.js +25 -26
- data/app/assets/javascripts/spree/backend/general_settings.js.coffee +10 -12
- data/app/assets/javascripts/spree/backend/payments/edit.js.coffee +4 -6
- data/app/assets/javascripts/spree/backend/returns/return_item_selection.js +7 -1
- data/app/assets/javascripts/spree/backend/stock_transfer.js.coffee +4 -1
- data/app/assets/javascripts/spree/backend/user_picker.js +3 -1
- data/app/assets/javascripts/spree/backend.js +3 -3
- data/app/controllers/spree/admin/countries_controller.rb +1 -0
- data/app/controllers/spree/admin/log_entries_controller.rb +2 -2
- data/app/controllers/spree/admin/orders/customer_details_controller.rb +5 -6
- data/app/controllers/spree/admin/orders_controller.rb +4 -4
- data/app/controllers/spree/admin/payments_controller.rb +21 -10
- data/app/controllers/spree/admin/products_controller.rb +5 -0
- data/app/controllers/spree/admin/promotion_rules_controller.rb +11 -7
- data/app/controllers/spree/admin/stock_transfers_controller.rb +1 -1
- data/app/controllers/spree/admin/taxons_controller.rb +31 -23
- data/app/helpers/spree/admin/base_helper.rb +1 -1
- data/app/helpers/spree/admin/navigation_helper.rb +10 -4
- data/app/views/spree/admin/adjustments/_adjustments_table.html.erb +2 -2
- data/app/views/spree/admin/general_settings/edit.html.erb +8 -8
- data/app/views/spree/admin/images/_form.html.erb +2 -2
- data/app/views/spree/admin/images/edit.html.erb +2 -2
- data/app/views/spree/admin/images/new.html.erb +1 -1
- data/app/views/spree/admin/option_types/_form.html.erb +2 -2
- data/app/views/spree/admin/option_types/index.html.erb +4 -4
- data/app/views/spree/admin/orders/_line_items.html.erb +40 -38
- data/app/views/spree/admin/orders/_shipment.html.erb +77 -76
- data/app/views/spree/admin/orders/customer_details/_form.html.erb +5 -5
- data/app/views/spree/admin/orders/index.html.erb +16 -16
- data/app/views/spree/admin/payment_methods/_form.html.erb +2 -2
- data/app/views/spree/admin/payments/_list.html.erb +1 -1
- data/app/views/spree/admin/products/_add_stock_form.html.erb +3 -3
- data/app/views/spree/admin/products/_form.html.erb +16 -16
- data/app/views/spree/admin/products/index.html.erb +4 -4
- data/app/views/spree/admin/products/new.html.erb +8 -8
- data/app/views/spree/admin/promotion_categories/index.html.erb +3 -3
- data/app/views/spree/admin/promotions/_form.html.erb +3 -3
- data/app/views/spree/admin/promotions/_promotion_rule.html.erb +1 -1
- data/app/views/spree/admin/promotions/actions/_create_adjustment.html.erb +2 -2
- data/app/views/spree/admin/promotions/actions/_create_item_adjustments.html.erb +2 -2
- data/app/views/spree/admin/promotions/actions/_create_line_items.html.erb +2 -4
- data/app/views/spree/admin/promotions/edit.html.erb +3 -3
- data/app/views/spree/admin/promotions/index.html.erb +6 -6
- data/app/views/spree/admin/promotions/rules/_item_total.html.erb +2 -2
- data/app/views/spree/admin/promotions/rules/_option_value.html.erb +4 -4
- data/app/views/spree/admin/properties/_form.html.erb +2 -2
- data/app/views/spree/admin/properties/index.html.erb +6 -6
- data/app/views/spree/admin/prototypes/index.html.erb +4 -4
- data/app/views/spree/admin/reimbursement_types/_form.html.erb +4 -4
- data/app/views/spree/admin/reimbursements/edit.html.erb +1 -1
- data/app/views/spree/admin/return_authorizations/_form.html.erb +13 -1
- data/app/views/spree/admin/return_index/customer_returns.html.erb +2 -2
- data/app/views/spree/admin/return_index/return_authorizations.html.erb +5 -5
- data/app/views/spree/admin/shared/_content_header.html.erb +2 -2
- data/app/views/spree/admin/shared/_header.html.erb +2 -2
- data/app/views/spree/admin/shared/_index_table_options.html.erb +2 -2
- data/app/views/spree/admin/shared/_order_summary.html.erb +1 -1
- data/app/views/spree/admin/shared/_table_filter.html.erb +1 -1
- data/app/views/spree/admin/shared/_translations.html.erb +0 -1
- data/app/views/spree/admin/shared/_update_order_state.js.erb +1 -1
- data/app/views/spree/admin/shipping_methods/_form.html.erb +10 -10
- data/app/views/spree/admin/state_changes/index.html.erb +3 -3
- data/app/views/spree/admin/states/_form.html.erb +2 -2
- data/app/views/spree/admin/stock_locations/_form.html.erb +2 -2
- data/app/views/spree/admin/stock_locations/_transfer_stock_form.html.erb +4 -4
- data/app/views/spree/admin/stock_transfers/index.html.erb +3 -3
- data/app/views/spree/admin/stock_transfers/new.html.erb +2 -2
- data/app/views/spree/admin/store_credits/index.html.erb +1 -1
- data/app/views/spree/admin/tax_rates/_form.html.erb +2 -2
- data/app/views/spree/admin/taxonomies/_list.html.erb +2 -2
- data/app/views/spree/admin/taxonomies/{edit.erb → edit.html.erb} +0 -0
- data/app/views/spree/admin/taxonomies/index.html.erb +2 -2
- data/app/views/spree/admin/taxons/_form.html.erb +5 -5
- data/app/views/spree/admin/taxons/edit.html.erb +3 -0
- data/app/views/spree/admin/trackers/_form.html.erb +2 -2
- data/app/views/spree/admin/users/_addresses_form.html.erb +2 -2
- data/app/views/spree/admin/users/_form.html.erb +2 -2
- data/app/views/spree/admin/users/_user_page_actions.html.erb +1 -1
- data/app/views/spree/admin/users/index.html.erb +5 -5
- data/app/views/spree/admin/users/items.html.erb +1 -1
- data/app/views/spree/admin/users/orders.html.erb +1 -1
- data/app/views/spree/admin/variants/_form.html.erb +2 -2
- data/app/views/spree/admin/zones/_form.html.erb +2 -2
- data/app/views/spree/layouts/admin.html.erb +3 -3
- data/config/initializers/assets.rb +1 -1
- data/lib/spree/backend/engine.rb +0 -1
- data/spec/controllers/spree/admin/base_controller_spec.rb +46 -0
- data/spec/controllers/spree/admin/customer_returns_controller_spec.rb +185 -0
- data/spec/controllers/spree/admin/general_settings_controller_spec.rb +41 -0
- data/spec/controllers/spree/admin/missing_products_controller_spec.rb +18 -0
- data/spec/controllers/spree/admin/orders/customer_details_controller_spec.rb +199 -0
- data/spec/controllers/spree/admin/orders_controller_spec.rb +296 -0
- data/spec/controllers/spree/admin/payment_methods_controller_spec.rb +64 -0
- data/spec/controllers/spree/admin/payments_controller_spec.rb +97 -0
- data/spec/controllers/spree/admin/products_controller_spec.rb +137 -0
- data/spec/controllers/spree/admin/promotion_actions_controller_spec.rb +21 -0
- data/spec/controllers/spree/admin/promotion_rules_controller_spec.rb +21 -0
- data/spec/controllers/spree/admin/promotions_controller_spec.rb +44 -0
- data/spec/controllers/spree/admin/refunds_controller_spec.rb +32 -0
- data/spec/controllers/spree/admin/reimbursements_controller_spec.rb +74 -0
- data/spec/controllers/spree/admin/reports_controller_spec.rb +42 -0
- data/spec/controllers/spree/admin/resource_controller_spec.rb +252 -0
- data/spec/controllers/spree/admin/return_authorizations_controller_spec.rb +225 -0
- data/spec/controllers/spree/admin/return_index_controller_spec.rb +37 -0
- data/spec/controllers/spree/admin/return_items_controller_spec.rb +27 -0
- data/spec/controllers/spree/admin/shipping_methods_controller_spec.rb +14 -0
- data/spec/controllers/spree/admin/stock_items_controller_spec.rb +19 -0
- data/spec/controllers/spree/admin/stock_locations_controller_spec.rb +41 -0
- data/spec/controllers/spree/admin/stock_transfers_controller_spec.rb +41 -0
- data/spec/controllers/spree/admin/tax_categories_controller_spec.rb +34 -0
- data/spec/controllers/spree/admin/users_controller_spec.rb +161 -0
- data/spec/controllers/spree/admin/variants_controller_spec.rb +32 -0
- data/spec/features/admin/configuration/analytics_tracker_spec.rb +47 -0
- data/spec/features/admin/configuration/countries_spec.rb +24 -0
- data/spec/features/admin/configuration/general_settings_spec.rb +44 -0
- data/spec/features/admin/configuration/payment_methods_spec.rb +63 -0
- data/spec/features/admin/configuration/roles_spec.rb +47 -0
- data/spec/features/admin/configuration/shipping_methods_spec.rb +62 -0
- data/spec/features/admin/configuration/states_spec.rb +68 -0
- data/spec/features/admin/configuration/stock_locations_spec.rb +48 -0
- data/spec/features/admin/configuration/store_credit_categories_spec.rb +50 -0
- data/spec/features/admin/configuration/tax_categories_spec.rb +56 -0
- data/spec/features/admin/configuration/tax_rates_spec.rb +18 -0
- data/spec/features/admin/configuration/zones_spec.rb +39 -0
- data/spec/features/admin/homepage_spec.rb +89 -0
- data/spec/features/admin/locale_spec.rb +31 -0
- data/spec/features/admin/orders/adjustments_promotions_spec.rb +53 -0
- data/spec/features/admin/orders/adjustments_spec.rb +124 -0
- data/spec/features/admin/orders/cancelling_and_resuming_spec.rb +47 -0
- data/spec/features/admin/orders/customer_details_spec.rb +154 -0
- data/spec/features/admin/orders/line_items_spec.rb +51 -0
- data/spec/features/admin/orders/listing_spec.rb +224 -0
- data/spec/features/admin/orders/log_entries_spec.rb +55 -0
- data/spec/features/admin/orders/new_order_spec.rb +186 -0
- data/spec/features/admin/orders/order_details_spec.rb +660 -0
- data/spec/features/admin/orders/payments_spec.rb +244 -0
- data/spec/features/admin/orders/risk_analysis_spec.rb +48 -0
- data/spec/features/admin/orders/shipments_spec.rb +64 -0
- data/spec/features/admin/orders/state_changes_spec.rb +21 -0
- data/spec/features/admin/products/edit/images_spec.rb +86 -0
- data/spec/features/admin/products/edit/products_spec.rb +56 -0
- data/spec/features/admin/products/edit/taxons_spec.rb +41 -0
- data/spec/features/admin/products/edit/variants_spec.rb +56 -0
- data/spec/features/admin/products/option_types_spec.rb +114 -0
- data/spec/features/admin/products/products_spec.rb +445 -0
- data/spec/features/admin/products/properties_spec.rb +147 -0
- data/spec/features/admin/products/prototypes_spec.rb +112 -0
- data/spec/features/admin/products/stock_management_spec.rb +124 -0
- data/spec/features/admin/products/taxonomies_spec.rb +52 -0
- data/spec/features/admin/products/variant_spec.rb +50 -0
- data/spec/features/admin/promotions/adjustments_spec.rb +258 -0
- data/spec/features/admin/promotions/option_value_rule_spec.rb +70 -0
- data/spec/features/admin/promotions/tiered_calculator_spec.rb +70 -0
- data/spec/features/admin/refund_reasons/refund_reasons_spec.rb +57 -0
- data/spec/features/admin/reimbursement_type/edit_reimbursement_type_spec.rb +36 -0
- data/spec/features/admin/reimbursement_type/new_reimbursement_type_spec.rb +62 -0
- data/spec/features/admin/reports_spec.rb +61 -0
- data/spec/features/admin/return_authorization_reasons/return_authorization_reasons_spec.rb +63 -0
- data/spec/features/admin/returns/customer_returns_spec.rb +80 -0
- data/spec/features/admin/returns/return_authorizations_spec.rb +152 -0
- data/spec/features/admin/stock_transfer_spec.rb +91 -0
- data/spec/features/admin/store_credits_spec.rb +93 -0
- data/spec/features/admin/taxons_spec.rb +53 -0
- data/spec/features/admin/users_spec.rb +286 -0
- data/spec/helpers/admin/base_helper_spec.rb +30 -0
- data/spec/helpers/admin/navigation_helper_spec.rb +111 -0
- data/spec/helpers/admin/promotion_rules_helper_spec.rb +12 -0
- data/spec/helpers/admin/stock_movements_helper_spec.rb +30 -0
- data/spec/models/spree/resource_spec.rb +48 -0
- data/spec/routing/admin_path_spec.rb +22 -0
- data/spec/spec_helper.rb +144 -0
- data/spec/support/appear_before_matcher.rb +8 -0
- data/spec/support/ror_ringer.jpeg +0 -0
- data/spec/test_views/spree/admin/submodule/posts/edit.html.erb +1 -0
- data/spec/test_views/spree/admin/submodule/posts/new.html.erb +1 -0
- data/spec/test_views/spree/admin/widgets/edit.html.erb +1 -0
- data/spec/test_views/spree/admin/widgets/new.html.erb +1 -0
- data/spree_backend.gemspec +4 -4
- data/vendor/assets/javascripts/jquery.jstree/jquery.jstree.js +130 -154
- metadata +106 -29
- data/vendor/assets/images/jquery-ui/ui-bg_flat_0_aaaaaa_40x100.png +0 -0
- data/vendor/assets/images/jquery-ui/ui-bg_flat_0_eeeeee_40x100.png +0 -0
- data/vendor/assets/images/jquery-ui/ui-bg_flat_55_ffffff_40x100.png +0 -0
- data/vendor/assets/images/jquery-ui/ui-bg_flat_75_ffffff_40x100.png +0 -0
- data/vendor/assets/images/jquery-ui/ui-bg_glass_65_ffffff_1x400.png +0 -0
- data/vendor/assets/images/jquery-ui/ui-bg_highlight-soft_100_f6f6f6_1x100.png +0 -0
- data/vendor/assets/images/jquery-ui/ui-bg_highlight-soft_25_0073ea_1x100.png +0 -0
- data/vendor/assets/images/jquery-ui/ui-bg_highlight-soft_50_dddddd_1x100.png +0 -0
- data/vendor/assets/images/jquery-ui/ui-icons_0073ea_256x240.png +0 -0
- data/vendor/assets/images/jquery-ui/ui-icons_454545_256x240.png +0 -0
- data/vendor/assets/images/jquery-ui/ui-icons_666666_256x240.png +0 -0
- data/vendor/assets/images/jquery-ui/ui-icons_ff0084_256x240.png +0 -0
- data/vendor/assets/images/jquery-ui/ui-icons_ffffff_256x240.png +0 -0
- data/vendor/assets/stylesheets/jquery-ui.datepicker.css.erb +0 -357
@@ -0,0 +1,185 @@
|
|
1
|
+
require 'spec_helper'
|
2
|
+
|
3
|
+
module Spree
|
4
|
+
module Admin
|
5
|
+
describe CustomerReturnsController, type: :controller do
|
6
|
+
stub_authorization!
|
7
|
+
|
8
|
+
describe "#index" do
|
9
|
+
let(:order) { customer_return.order }
|
10
|
+
let(:customer_return) { create(:customer_return) }
|
11
|
+
|
12
|
+
subject do
|
13
|
+
spree_get :index, { order_id: customer_return.order.to_param }
|
14
|
+
end
|
15
|
+
|
16
|
+
before { subject }
|
17
|
+
|
18
|
+
it "loads the order" do
|
19
|
+
expect(assigns(:order)).to eq order
|
20
|
+
end
|
21
|
+
|
22
|
+
it "loads the customer return" do
|
23
|
+
expect(assigns(:customer_returns)).to include(customer_return)
|
24
|
+
end
|
25
|
+
end
|
26
|
+
|
27
|
+
describe "#new" do
|
28
|
+
let(:order) { create(:shipped_order, line_items_count: 1) }
|
29
|
+
let!(:rma) { create :return_authorization, order: order, return_items: [create(:return_item, inventory_unit: order.inventory_units.first)] }
|
30
|
+
let!(:inactive_reimbursement_type) { create(:reimbursement_type, active: false) }
|
31
|
+
let!(:first_active_reimbursement_type) { create(:reimbursement_type) }
|
32
|
+
let!(:second_active_reimbursement_type) { create(:reimbursement_type) }
|
33
|
+
|
34
|
+
subject do
|
35
|
+
spree_get :new, { order_id: order.to_param }
|
36
|
+
end
|
37
|
+
|
38
|
+
it "loads the order" do
|
39
|
+
subject
|
40
|
+
expect(assigns(:order)).to eq order
|
41
|
+
end
|
42
|
+
|
43
|
+
it "creates a new customer return" do
|
44
|
+
subject
|
45
|
+
expect(assigns(:customer_return)).to_not be_persisted
|
46
|
+
end
|
47
|
+
|
48
|
+
context "with previous customer return" do
|
49
|
+
let(:order) { create(:shipped_order, line_items_count: 4) }
|
50
|
+
let(:rma) { create(:return_authorization, order: order) }
|
51
|
+
|
52
|
+
let!(:rma_return_item) { create(:return_item, return_authorization: rma, inventory_unit: order.inventory_units.first) }
|
53
|
+
let!(:customer_return_return_item) { create(:return_item, return_authorization: rma, inventory_unit: order.inventory_units.last) }
|
54
|
+
|
55
|
+
context "there is a return item associated with an rma but not a customer return" do
|
56
|
+
let!(:previous_customer_return) { create(:customer_return_without_return_items, return_items: [customer_return_return_item]) }
|
57
|
+
|
58
|
+
before do
|
59
|
+
subject
|
60
|
+
end
|
61
|
+
|
62
|
+
it "loads the persisted rma return items" do
|
63
|
+
expect(assigns(:rma_return_items).all? { |return_item| return_item.persisted? }).to eq true
|
64
|
+
end
|
65
|
+
|
66
|
+
it "has one rma return item" do
|
67
|
+
expect(assigns(:rma_return_items)).to include(rma_return_item)
|
68
|
+
end
|
69
|
+
end
|
70
|
+
end
|
71
|
+
end
|
72
|
+
|
73
|
+
describe "#edit" do
|
74
|
+
let(:order) { customer_return.order }
|
75
|
+
let(:customer_return) { create(:customer_return, line_items_count: 3) }
|
76
|
+
|
77
|
+
let!(:accepted_return_item) { customer_return.return_items.order('id').first.tap(&:accept!) }
|
78
|
+
let!(:rejected_return_item) { customer_return.return_items.order('id').second.tap(&:reject!)}
|
79
|
+
let!(:manual_intervention_return_item) { customer_return.return_items.order('id').third.tap(&:require_manual_intervention!) }
|
80
|
+
|
81
|
+
subject do
|
82
|
+
spree_get :edit, { order_id: order.to_param, id: customer_return.to_param }
|
83
|
+
end
|
84
|
+
|
85
|
+
before do
|
86
|
+
subject
|
87
|
+
end
|
88
|
+
|
89
|
+
it "loads the order" do
|
90
|
+
expect(assigns(:order)).to eq order
|
91
|
+
end
|
92
|
+
|
93
|
+
it "loads the customer return" do
|
94
|
+
expect(assigns(:customer_return)).to eq customer_return
|
95
|
+
end
|
96
|
+
|
97
|
+
it "loads the accepted return items" do
|
98
|
+
expect(assigns(:accepted_return_items)).to eq [accepted_return_item]
|
99
|
+
end
|
100
|
+
|
101
|
+
it "loads the rejected return items" do
|
102
|
+
expect(assigns(:rejected_return_items)).to eq [rejected_return_item]
|
103
|
+
end
|
104
|
+
|
105
|
+
it "loads the return items that require manual intervention" do
|
106
|
+
expect(assigns(:manual_intervention_return_items)).to eq [manual_intervention_return_item]
|
107
|
+
end
|
108
|
+
|
109
|
+
it "loads the return items that are still pending" do
|
110
|
+
expect(assigns(:pending_return_items)).to eq []
|
111
|
+
end
|
112
|
+
|
113
|
+
it "loads the reimbursements that are still pending" do
|
114
|
+
expect(assigns(:pending_reimbursements)).to eq []
|
115
|
+
end
|
116
|
+
end
|
117
|
+
|
118
|
+
describe "#create" do
|
119
|
+
let(:order) { create(:shipped_order, line_items_count: 1) }
|
120
|
+
let!(:return_authorization) { create :return_authorization, order: order, return_items: [create(:return_item, inventory_unit: order.inventory_units.shipped.last)] }
|
121
|
+
|
122
|
+
subject do
|
123
|
+
spree_post :create, customer_return_params
|
124
|
+
end
|
125
|
+
|
126
|
+
context "valid customer return" do
|
127
|
+
let(:stock_location) { order.shipments.last.stock_location }
|
128
|
+
|
129
|
+
let!(:customer_return_params) do
|
130
|
+
{
|
131
|
+
order_id: order.to_param,
|
132
|
+
customer_return: {
|
133
|
+
stock_location_id: stock_location.id,
|
134
|
+
return_items_attributes: {
|
135
|
+
"0" => {
|
136
|
+
id: return_authorization.return_items.first.id,
|
137
|
+
returned: "1",
|
138
|
+
"pre_tax_amount"=>"15.99",
|
139
|
+
inventory_unit_id: order.inventory_units.shipped.last.id
|
140
|
+
}
|
141
|
+
}
|
142
|
+
}
|
143
|
+
}
|
144
|
+
end
|
145
|
+
|
146
|
+
it "creates a customer return" do
|
147
|
+
expect{ subject }.to change { Spree::CustomerReturn.count }.by(1)
|
148
|
+
end
|
149
|
+
|
150
|
+
it "redirects to the index page" do
|
151
|
+
subject
|
152
|
+
expect(response).to redirect_to(spree.edit_admin_order_customer_return_path(order, assigns(:customer_return)))
|
153
|
+
end
|
154
|
+
end
|
155
|
+
|
156
|
+
context "invalid customer return" do
|
157
|
+
let!(:customer_return_params) do
|
158
|
+
{
|
159
|
+
order_id: order.to_param,
|
160
|
+
customer_return: {
|
161
|
+
stock_location_id: "",
|
162
|
+
return_items_attributes: {
|
163
|
+
"0" => {
|
164
|
+
returned: "1",
|
165
|
+
"pre_tax_amount"=>"15.99",
|
166
|
+
inventory_unit_id: order.inventory_units.shipped.last.id
|
167
|
+
}
|
168
|
+
}
|
169
|
+
}
|
170
|
+
}
|
171
|
+
end
|
172
|
+
|
173
|
+
it "doesn't create a customer return" do
|
174
|
+
expect{ subject }.to_not change { Spree::CustomerReturn.count }
|
175
|
+
end
|
176
|
+
|
177
|
+
it "renders the new page" do
|
178
|
+
subject
|
179
|
+
expect(response).to render_template(:new)
|
180
|
+
end
|
181
|
+
end
|
182
|
+
end
|
183
|
+
end
|
184
|
+
end
|
185
|
+
end
|
@@ -0,0 +1,41 @@
|
|
1
|
+
require 'spec_helper'
|
2
|
+
|
3
|
+
describe Spree::Admin::GeneralSettingsController, type: :controller do
|
4
|
+
let(:user) { create(:user) }
|
5
|
+
|
6
|
+
before do
|
7
|
+
allow(controller).to receive_messages spree_current_user: user
|
8
|
+
user.spree_roles << Spree::Role.find_or_create_by(name: 'admin')
|
9
|
+
end
|
10
|
+
|
11
|
+
describe '#clear_cache' do
|
12
|
+
subject { spree_post :clear_cache }
|
13
|
+
|
14
|
+
shared_examples 'a HTTP 204 response' do
|
15
|
+
it 'grant access to users with an admin role' do
|
16
|
+
subject
|
17
|
+
expect(response.status).to eq(204)
|
18
|
+
end
|
19
|
+
end
|
20
|
+
|
21
|
+
context 'when no callback' do
|
22
|
+
it_behaves_like 'a HTTP 204 response'
|
23
|
+
end
|
24
|
+
|
25
|
+
context 'when callback implemented' do
|
26
|
+
Spree::Admin::GeneralSettingsController.class_eval do
|
27
|
+
custom_callback(:clear_cache).after :foo
|
28
|
+
|
29
|
+
def foo
|
30
|
+
# Make a call to Akamai, CloudFlare, etc invalidation....
|
31
|
+
end
|
32
|
+
end
|
33
|
+
|
34
|
+
before do
|
35
|
+
expect(controller).to receive(:foo).once
|
36
|
+
end
|
37
|
+
|
38
|
+
it_behaves_like 'a HTTP 204 response'
|
39
|
+
end
|
40
|
+
end
|
41
|
+
end
|
@@ -0,0 +1,18 @@
|
|
1
|
+
require 'spec_helper'
|
2
|
+
# This test exists in this file because in the standard admin/products_controller spec
|
3
|
+
# There is the stub_authorization call. This call is not triggered for this test because
|
4
|
+
# the load_resource filter in Spree::Admin::ResourceController is prepended to the filter chain
|
5
|
+
# this means this call is triggered before the authorize_admin call and in this case
|
6
|
+
# the load_resource filter halts the request meaning authorize_admin is not called at all.
|
7
|
+
describe Spree::Admin::ProductsController, type: :controller do
|
8
|
+
stub_authorization!
|
9
|
+
|
10
|
+
# Regression test for GH #538
|
11
|
+
it "cannot find a non-existent product" do
|
12
|
+
spree_get :edit, id: "non-existent-product"
|
13
|
+
expect(response).to redirect_to(spree.admin_products_path)
|
14
|
+
expect(flash[:error]).to eql("Product is not found")
|
15
|
+
end
|
16
|
+
end
|
17
|
+
|
18
|
+
|
@@ -0,0 +1,199 @@
|
|
1
|
+
require "spec_helper"
|
2
|
+
require "cancan"
|
3
|
+
require "spree/testing_support/bar_ability"
|
4
|
+
|
5
|
+
describe Spree::Admin::Orders::CustomerDetailsController, type: :controller do
|
6
|
+
|
7
|
+
context "with authorization" do
|
8
|
+
stub_authorization!
|
9
|
+
|
10
|
+
let(:user) { mock_model(Spree.user_class) }
|
11
|
+
|
12
|
+
let(:order) do
|
13
|
+
mock_model(
|
14
|
+
Spree::Order,
|
15
|
+
total: 100,
|
16
|
+
number: "R123456789",
|
17
|
+
billing_address: mock_model(Spree::Address)
|
18
|
+
)
|
19
|
+
end
|
20
|
+
|
21
|
+
before do
|
22
|
+
allow(Spree::Order).to receive_message_chain(:includes, find_by!: order)
|
23
|
+
end
|
24
|
+
|
25
|
+
describe '#update' do
|
26
|
+
let(:attributes) do
|
27
|
+
{
|
28
|
+
order_id: order.number,
|
29
|
+
order: {
|
30
|
+
email: "",
|
31
|
+
use_billing: "",
|
32
|
+
bill_address_attributes: { firstname: 'john' },
|
33
|
+
ship_address_attributes: { firstname: 'john' },
|
34
|
+
user_id: user.id.to_s
|
35
|
+
},
|
36
|
+
guest_checkout: 'true'
|
37
|
+
}
|
38
|
+
end
|
39
|
+
|
40
|
+
def send_request(params = {})
|
41
|
+
spree_put :update, params
|
42
|
+
end
|
43
|
+
|
44
|
+
context 'using guest checkout' do
|
45
|
+
context 'having valid parameters' do
|
46
|
+
before do
|
47
|
+
allow(order).to receive_messages(update_attributes: true)
|
48
|
+
allow(order).to receive_messages(next: false)
|
49
|
+
allow(order).to receive_messages(address?: false)
|
50
|
+
allow(order).to receive_messages(refresh_shipment_rates: true)
|
51
|
+
end
|
52
|
+
|
53
|
+
context 'having successful response' do
|
54
|
+
before { send_request(attributes) }
|
55
|
+
it { expect(response).to have_http_status(302) }
|
56
|
+
it { expect(response).to redirect_to(edit_admin_order_url(order)) }
|
57
|
+
end
|
58
|
+
|
59
|
+
context 'with correct method flow' do
|
60
|
+
it { expect(order).to receive(:update_attributes).with(ActionController::Parameters.new(attributes[:order]).permit(permitted_order_attributes)) }
|
61
|
+
it { expect(order).to_not receive(:next) }
|
62
|
+
it { expect(order).to receive(:address?) }
|
63
|
+
it 'does refresh the shipment rates with all shipping methods' do
|
64
|
+
expect(order).to receive(:refresh_shipment_rates).
|
65
|
+
with(Spree::ShippingMethod::DISPLAY_ON_BACK_END)
|
66
|
+
end
|
67
|
+
it { expect(controller).to receive(:load_order).and_call_original }
|
68
|
+
it { expect(controller).to receive(:guest_checkout?).twice.and_call_original }
|
69
|
+
it { expect(controller).to_not receive(:load_user).and_call_original }
|
70
|
+
after { send_request(attributes) }
|
71
|
+
end
|
72
|
+
end
|
73
|
+
|
74
|
+
context 'having invalid parameters' do
|
75
|
+
before do
|
76
|
+
allow(order).to receive_messages(update_attributes: false)
|
77
|
+
end
|
78
|
+
|
79
|
+
context 'having failure response' do
|
80
|
+
before { send_request(attributes) }
|
81
|
+
it { expect(response).to render_template(:edit) }
|
82
|
+
end
|
83
|
+
|
84
|
+
context 'with correct method flow' do
|
85
|
+
it { expect(order).to receive(:update_attributes).with(ActionController::Parameters.new(attributes[:order]).permit(permitted_order_attributes)) }
|
86
|
+
it { expect(controller).to receive(:load_order).and_call_original }
|
87
|
+
it { expect(controller).to receive(:guest_checkout?).and_call_original }
|
88
|
+
it { expect(controller).to_not receive(:load_user).and_call_original }
|
89
|
+
after { send_request(attributes) }
|
90
|
+
end
|
91
|
+
end
|
92
|
+
end
|
93
|
+
|
94
|
+
context 'without using guest checkout' do
|
95
|
+
let(:changed_attributes) { attributes.merge(guest_checkout: 'false') }
|
96
|
+
|
97
|
+
context 'having valid parameters' do
|
98
|
+
before do
|
99
|
+
allow(Spree.user_class).to receive(:find_by).and_return(user)
|
100
|
+
allow(order).to receive_messages(update_attributes: true)
|
101
|
+
allow(order).to receive_messages(next: false)
|
102
|
+
allow(order).to receive_messages(address?: false)
|
103
|
+
allow(order).to receive_messages(refresh_shipment_rates: true)
|
104
|
+
allow(order).to receive_messages(associate_user!: true)
|
105
|
+
allow(controller).to receive(:guest_checkout?).and_return(false)
|
106
|
+
allow(order).to receive(:associate_user!)
|
107
|
+
end
|
108
|
+
|
109
|
+
context 'having successful response' do
|
110
|
+
before { send_request(changed_attributes) }
|
111
|
+
it { expect(response).to have_http_status(302) }
|
112
|
+
it { expect(response).to redirect_to(edit_admin_order_url(order)) }
|
113
|
+
end
|
114
|
+
|
115
|
+
context 'with correct method flow' do
|
116
|
+
it { expect(order).to receive(:update_attributes).with(ActionController::Parameters.new(attributes[:order]).permit(permitted_order_attributes)) }
|
117
|
+
it { expect(order).to receive(:associate_user!).with(user, order.email.blank?) }
|
118
|
+
it { expect(order).to_not receive(:next) }
|
119
|
+
it { expect(order).to receive(:address?) }
|
120
|
+
it 'does refresh the shipment rates with all shipping methods' do
|
121
|
+
expect(order).to receive(:refresh_shipment_rates).
|
122
|
+
with(Spree::ShippingMethod::DISPLAY_ON_BACK_END)
|
123
|
+
end
|
124
|
+
it { expect(controller).to receive(:load_order).and_call_original }
|
125
|
+
it { expect(controller).to receive(:guest_checkout?).twice.and_call_original }
|
126
|
+
it { expect(controller).to receive(:load_user).and_call_original }
|
127
|
+
after { send_request(changed_attributes) }
|
128
|
+
end
|
129
|
+
end
|
130
|
+
|
131
|
+
context 'having invalid parameters' do
|
132
|
+
before do
|
133
|
+
allow(Spree.user_class).to receive(:find_by).and_return(false)
|
134
|
+
allow(controller).to receive(:guest_checkout?).and_return(false)
|
135
|
+
end
|
136
|
+
|
137
|
+
context 'having failure response' do
|
138
|
+
before { send_request(changed_attributes) }
|
139
|
+
it { expect(response).to render_template(:edit) }
|
140
|
+
end
|
141
|
+
|
142
|
+
context 'with correct method flow' do
|
143
|
+
it { expect(order).to_not receive(:update_attributes).with(ActionController::Parameters.new(attributes[:order]).permit(permitted_order_attributes)) }
|
144
|
+
it { expect(controller).to receive(:load_order).and_call_original }
|
145
|
+
it { expect(controller).to receive(:guest_checkout?).and_call_original }
|
146
|
+
it { expect(controller).to receive(:load_user).and_call_original }
|
147
|
+
after { send_request(changed_attributes) }
|
148
|
+
end
|
149
|
+
end
|
150
|
+
|
151
|
+
describe '#load_user' do
|
152
|
+
context 'having valid parameters' do
|
153
|
+
before do
|
154
|
+
allow(Spree.user_class).to receive(:find_by).and_return(user)
|
155
|
+
allow(order).to receive_messages(update_attributes: true)
|
156
|
+
allow(order).to receive_messages(next: false)
|
157
|
+
allow(order).to receive_messages(address?: false)
|
158
|
+
allow(order).to receive_messages(refresh_shipment_rates: true)
|
159
|
+
allow(order).to receive_messages(associate_user!: true)
|
160
|
+
allow(controller).to receive(:guest_checkout?).and_return(false)
|
161
|
+
allow(order).to receive(:associate_user!)
|
162
|
+
end
|
163
|
+
|
164
|
+
it "expects to assign user" do
|
165
|
+
send_request(changed_attributes)
|
166
|
+
expect(assigns[:user]).to eq(user)
|
167
|
+
end
|
168
|
+
|
169
|
+
context 'with correct method flow' do
|
170
|
+
it { expect(Spree.user_class).to receive(:find_by).with(id: user.id.to_s).and_return(user) }
|
171
|
+
after { send_request(changed_attributes) }
|
172
|
+
end
|
173
|
+
end
|
174
|
+
|
175
|
+
context 'with invalid parameters' do
|
176
|
+
before do
|
177
|
+
allow(Spree.user_class).to receive(:find_by).and_return(nil)
|
178
|
+
allow(controller).to receive(:guest_checkout?).and_return(false)
|
179
|
+
end
|
180
|
+
|
181
|
+
it "expects to not assign user" do
|
182
|
+
send_request(changed_attributes)
|
183
|
+
expect(assigns[:user]).to_not eq(user)
|
184
|
+
end
|
185
|
+
|
186
|
+
context 'with correct method flow' do
|
187
|
+
it { expect(Spree.user_class).to receive(:find_by).with(id: user.id.to_s).and_return(nil) }
|
188
|
+
it "expects user class to receive find_by with email" do
|
189
|
+
expect(Spree.user_class).to receive(:find_by).
|
190
|
+
with(email: changed_attributes[:order][:email]).and_return(nil)
|
191
|
+
end
|
192
|
+
after { send_request(changed_attributes) }
|
193
|
+
end
|
194
|
+
end
|
195
|
+
end
|
196
|
+
end
|
197
|
+
end
|
198
|
+
end
|
199
|
+
end
|