+
+ <%= check_box_tag :has_account, '1', true, id: 'has_account_checkbox', class: 'mt-1',
+ data: { login_spl_target: "checkbox", action: "login-spl#toggle" }%>
+ <%= label_tag :has_account, I18n.t('spl.user.have_account'), class: 'text-sm text-neutral-700' %>
+
+
+
+
+
+
+ <%= I18n.t("spl.user.sent_to", phone: phone_e164) if phone_e164.present? %>
+
+
+
+ <%= f.label :spl_auth_code, I18n.t("spl.user.code_label"), class: "text-xs text-neutral-600 block mb-2" %>
+ <%= f.text_field :spl_auth_code, class: "text-input w-full" %>
+
+
+ <%= f.submit I18n.t("buttons.verify_button"), class: "btn-primary w-full" %>
+ <% end %>
+<% end %>
diff --git a/app/views/spl/_otp_registration_form.html.erb b/app/views/spl/_otp_registration_form.html.erb
new file mode 100644
index 0000000..6ac44ee
--- /dev/null
+++ b/app/views/spl/_otp_registration_form.html.erb
@@ -0,0 +1,18 @@
+<%= turbo_frame_tag "otp_registration_form" do %>
+ <%= render partial: "spree/shared/error_messages", locals: { target: user } %>
+ <%= form_with scope: :user, url: register_loyalty_account_account_profile_path, method: :post, data: { turbo_frame: "otp_registration_form" } do |f| %>
+
+
+ <%= I18n.t("spl.user.sent_to", phone: phone_e164) if phone_e164.present? %>
+
+
+
+ <%= f.label :spl_auth_code, I18n.t("spl.user.code_label"), class: "text-xs text-neutral-600 block mb-2" %>
+ <%= f.text_field :spl_auth_code, class: "text-input w-full" %>
+
+
+ <%= hidden_field_tag user.phone %>
+
+ <%= f.submit I18n.t("buttons.verify_button"), class: "btn-primary w-full" %>
+ <% end %>
+<% end %>
diff --git a/app/views/spree/checkout/_spl_coupons.html.erb b/app/views/spree/checkout/_spl_coupons.html.erb
new file mode 100644
index 0000000..f435677
--- /dev/null
+++ b/app/views/spree/checkout/_spl_coupons.html.erb
@@ -0,0 +1,32 @@
+
+
+
<%= I18n.t('spl.coupons.find.checkout_list') if @coupons.present? %>
+
+ <% @coupons&.each do |coupon| %>
+ <%= coupon["typeCustomerName"] %>
+ <% if coupon.dig("couponType", "autoLoad") == "A" %>
+ <% if coupon["autoLoad"] %>
+ <%= button_to I18n.t('spl.coupons.deactivate'),
+ deactivate_coupon_coupons_path(
+ order: @order,
+ token: @order.token,
+ coupon_code: coupon["code"]
+ ),
+ method: :post,
+ data: { turbo: true },
+ class: "min-h-full ml-3 btn-primary !px-4 !py-3" %>
+ <% else %>
+ <%= button_to I18n.t('spl.coupons.activate'),
+ activate_coupon_coupons_path(
+ order: @order,
+ token: @order.token,
+ coupon_code: coupon["code"]
+ ),
+ method: :post,
+ data: { turbo: true },
+ class: "min-h-full ml-3 btn-primary !px-4 !py-3" %>
+ <% end %>
+ <% end %>
+ <% end %>
+
+
diff --git a/app/views/spree/checkout/activate_coupon.turbo_stream.erb b/app/views/spree/checkout/activate_coupon.turbo_stream.erb
new file mode 100644
index 0000000..aa53e8b
--- /dev/null
+++ b/app/views/spree/checkout/activate_coupon.turbo_stream.erb
@@ -0,0 +1,4 @@
+<%= turbo_stream.replace "checkout_coupons" do %>
+ <%= render partial: "spree/checkout/spl_coupons",
+ locals: { coupons: @coupons, order: @order } %>
+<% end %>
\ No newline at end of file
diff --git a/app/views/spree/checkout/deactivate_coupon.turbo_stream.erb b/app/views/spree/checkout/deactivate_coupon.turbo_stream.erb
new file mode 100644
index 0000000..03de1fa
--- /dev/null
+++ b/app/views/spree/checkout/deactivate_coupon.turbo_stream.erb
@@ -0,0 +1,4 @@
+<%= turbo_stream.replace "checkout_coupons" do %>
+ <%= render partial: "spree/checkout/spl_coupons",
+ locals: { coupons: @coupons, order: @order } %>
+<% end %>
diff --git a/config/initializers/spree_spl.rb b/config/initializers/spree_spl.rb
index f1f4273..f1919c3 100644
--- a/config/initializers/spree_spl.rb
+++ b/config/initializers/spree_spl.rb
@@ -17,6 +17,10 @@
Spl::Spree::Api::AccountControllerDecorator
)
+ ::Spree::CheckoutController.prepend(
+ Spl::Spree::Storefront::CheckoutControllerDecorator
+ )
+
::Spree::Adjustable::AdjustmentsUpdater.prepend(
Spree::Adjustable::AdjustmentsUpdaterDecorator
)
diff --git a/config/locales/cs.yml b/config/locales/cs.yml
index 739c2ab..00fd0bc 100644
--- a/config/locales/cs.yml
+++ b/config/locales/cs.yml
@@ -2,8 +2,46 @@ cs:
order:
loyalty_card_missing: K objednávce není přiřazena žádná věrnostní karta
spl:
+ user:
+ card_number: The club card number has been assigned to the account
+ card_active: SPL card activation
+ have_account: Do you have loyalty club account?
+ accept_yc_terms_label: I accept the terms and conditions
+ sent_to: The code was sent to %{phone}
+ code_label: One-time code
+ errors:
+ must_accept_yc_terms: You must accept the loyalty club terms and conditions to continue
+ invalid_phone: Invalid phone number
card_validation:
errors:
card_not_active: Karta není aktivní
card_not_match: Karta neodpovídá číslu karty uživatele
wrong_owner: Karta je přiřazena jinému uživateli
+ coupons:
+ find:
+ checkout_list: Your SPL coupons
+ activate: Activate
+ deactivate: Deactivate
+ profile:
+ connect_button: Connect to your loyalty account
+ errors:
+ generic: We couldn't connect your loyalty account. Please try again.
+ auth_temporary_blocked: Too many failed attempts. Please try again later.
+ auth_failed: Authentication failed.
+ token_expired: Your session has expired. Please sign in again.
+ not_authorized: You are not authorized to perform this action.
+ coupon_cannot_be_modified: This coupon cannot be modified.
+ coupon_expired: This coupon has expired.
+ coupon_is_not_valid: This coupon is not valid.
+ coupon_already_used: This coupon has already been used.
+ temporary_blocked: Temporarily blocked due to too many attempts. Please try again later.
+ person_not_found: Person not found.
+ unknow_error: NEZNÁMÁ_CHYBA
+ buttons:
+ connect_account: Connect loyalty account
+ send_log_in_code: Send one-time code for log in
+ send_registration_code: Send one-time code for registration
+ verify_button: Verify phone number
+ user:
+ card_number: The club card number has been assigned to the account
+ card_active: SPL card activation
diff --git a/config/locales/en.yml b/config/locales/en.yml
index 6438221..3c3759d 100644
--- a/config/locales/en.yml
+++ b/config/locales/en.yml
@@ -2,11 +2,43 @@ en:
order:
loyalty_card_missing: There is no loyalty card assigned to order
spl:
+ user:
+ card_number: The club card number has been assigned to the account
+ card_active: SPL card activation
+ have_account: Do you have loyalty club account?
+ accept_yc_terms_label: I accept the terms and conditions
+ sent_to: The code was sent to %{phone}
+ code_label: One-time code
+ errors:
+ must_accept_yc_terms: You must accept the loyalty club terms and conditions to continue
+ invalid_phone: Invalid phone number
card_validation:
errors:
card_not_active: Card is not active
card_not_match: Card doesnt match user's card number
wrong_owner: Card assigned to a different user
- user:
- card_number: The club card number has been assigned to the account
- card_active: SPL card activation
+ coupons:
+ find:
+ checkout_list: Your SPL coupons
+ activate: Activate
+ deactivate: Deactivate
+ profile:
+ connect_button: Connect to your loyalty account
+ errors:
+ generic: We couldn't connect your loyalty account. Please try again.
+ auth_temporary_blocked: Too many failed attempts. Please try again later.
+ auth_failed: Authentication failed.
+ token_expired: Your session has expired. Please sign in again.
+ not_authorized: You are not authorized to perform this action.
+ coupon_cannot_be_modified: This coupon cannot be modified.
+ coupon_expired: This coupon has expired.
+ coupon_is_not_valid: This coupon is not valid.
+ coupon_already_used: This coupon has already been used.
+ temporary_blocked: Temporarily blocked due to too many attempts. Please try again later.
+ person_not_found: Person not found.
+ unknow_error: UNKNOWN_ERROR
+ buttons:
+ connect_account: Connect loyalty account
+ send_log_in_code: Send one-time code for log in
+ send_registration_code: Send one-time code for registration
+ verify_button: Verify phone number
diff --git a/config/locales/pl.yml b/config/locales/pl.yml
index e3012c0..4cd854e 100644
--- a/config/locales/pl.yml
+++ b/config/locales/pl.yml
@@ -2,11 +2,45 @@ pl:
order:
loyalty_card_missing: Do zamówienia nie przypisano karty lojalnościowej
spl:
+ user:
+ card_number: Numer karty klubowej został przypisany do konta
+ card_active: Aktywacja karty SPL
+ have_account: Czy masz konto w klubie lojalnościowym?
+ accept_yc_terms_label: Akceptuje regulamin
+ sent_to: Wysłano kod na numer %{phone}
+ code_label: Jednorazowy kod
+ errors:
+ must_accept_yc_terms: Musisz zaakceptować regulamin klubu lojalnościowego
+ invalid_phone: Podaj poprawny numer telefonu
+ coupons:
+ find:
+ checkout_list: Twoje kupony lojalnościowe
card_validation:
errors:
card_not_active: Karta nie jest aktywna
card_not_match: Karta nie jest zgodna z numerem karty użytkownika
wrong_owner: Karta przypisana innemu użytkownikowi
- user:
- card_number: Numer karty klubowej został przypisany do konta
- card_active: Aktywacja karty SPL
+ coupons:
+ find:
+ checkout_list: Twoje kupony lojalnościowe
+ activate: Aktywuj
+ deactivate: Dezaktywuj
+ errors:
+ generic: Nie udało się wykonać akcji. Spróbuj ponownie.
+ auth_temporary_blocked: Zbyt wiele nieudanych prób. Spróbuj ponownie później.
+ auth_failed: Uwierzytelnienie nie powiodło się.
+ token_expired: Twoja sesja wygasła. Zaloguj się ponownie.
+ not_authorized: Nie masz uprawnień do wykonania tej operacji.
+ coupon_cannot_be_modified: Ten kupon nie może zostać zmodyfikowany.
+ coupon_expired: Ten kupon wygasł.
+ coupon_is_not_valid: Ten kupon jest nieprawidłowy.
+ coupon_already_used: Ten kupon został już wykorzystany.
+ temporary_blocked: Tymczasowo zablokowano z powodu zbyt wielu prób. Spróbuj ponownie później.
+ person_not_found: Nie znaleziono użytkownika
+ unknow_error: NIEZNANY_BŁĄD
+ buttons:
+ connect_account: Połącz z kontem lojalnościowym
+ send_log_in_code: Wyślij jednorazowy kod do logowania
+ send_registration_code: Wyślij jednorazowy kod do rejestracji
+ verify_button: Zweryfikuj numer telefonu
+
diff --git a/config/locales/sk.yml b/config/locales/sk.yml
index dd62daf..2d0ea11 100644
--- a/config/locales/sk.yml
+++ b/config/locales/sk.yml
@@ -2,8 +2,46 @@ sk:
order:
loyalty_card_missing: K objednávke nie je priradená žiadna vernostná karta
spl:
+ user:
+ card_number: The club card number has been assigned to the account
+ card_active: SPL card activation
+ have_account: Do you have loyalty club account?
+ accept_yc_terms_label: I accept the terms and conditions
+ sent_to: The code was sent to %{phone}
+ code_label: One-time code
+ errors:
+ must_accept_yc_terms: You must accept the loyalty club terms and conditions to continue
+ invalid_phone: Invalid phone number
card_validation:
errors:
card_not_active: Karta nie je aktívna
card_not_match: Karta nezodpovedá číslu karty používateľa
wrong_owner: Karta je priradená inému používateľovi
+ coupons:
+ find:
+ checkout_list: Your SPL coupons
+ activate: Activate
+ deactivate: Deactivate
+ profile:
+ connect_button: Connect to your loyalty account
+ errors:
+ generic: We couldn't connect your loyalty account. Please try again.
+ auth_temporary_blocked: Too many failed attempts. Please try again later.
+ auth_failed: Authentication failed.
+ token_expired: Your session has expired. Please sign in again.
+ not_authorized: You are not authorized to perform this action.
+ coupon_cannot_be_modified: This coupon cannot be modified.
+ coupon_expired: This coupon has expired.
+ coupon_is_not_valid: This coupon is not valid.
+ coupon_already_used: This coupon has already been used.
+ temporary_blocked: Temporarily blocked due to too many attempts. Please try again later.
+ person_not_found: Person not found.
+ unknow_error: NEZNÁMA_CHYBA
+ buttons:
+ connect_account: Connect loyalty account
+ send_log_in_code: Send one-time code for log in
+ send_registration_code: Send one-time code for registration
+ verify_button: Verify phone number
+ user:
+ card_number: The club card number has been assigned to the account
+ card_active: SPL card activation
diff --git a/config/routes.rb b/config/routes.rb
index 1f06ae8..7be4eff 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -16,4 +16,16 @@
end
end
end
+ resource :checkout, as: 'coupons', controller: :checkout, only: %i[show create update] do
+ post :activate_coupon, to: 'checkout#activate_coupon'
+ post :deactivate_coupon, to: 'checkout#deactivate_coupon'
+ end
+ namespace :account do
+ resource :profile, controller: :profile, only: [:edit, :update] do
+ patch :registration_code
+ post :register_loyalty_account
+ patch :login_code
+ post :connect_loyalty_account
+ end
+ end
end
diff --git a/lib/generators/spl/install/install_generator.rb b/lib/generators/spl/install/install_generator.rb
new file mode 100644
index 0000000..ebaa5e3
--- /dev/null
+++ b/lib/generators/spl/install/install_generator.rb
@@ -0,0 +1,33 @@
+# frozen_string_literal: true
+
+module Spl
+ module Generators
+ class InstallGenerator < Rails::Generators::Base
+ source_root File.expand_path('templates', __dir__)
+
+ desc 'Install SPL Stimulus controller'
+
+ def copy_stimulus_controller
+ destination = 'app/javascript/controllers/login_spl_controller.js'
+
+ if File.exist?(destination)
+ say_status :skip, "#{destination} already exists", :yellow
+ else
+ copy_file 'login_spl_controller.js', destination
+ say_status :create, destination, :green
+ end
+ end
+
+ def ensure_controllers_autoload
+ controllers_index = 'app/javascript/controllers/index.js'
+
+ unless File.exist?(controllers_index)
+ say_status :warning, 'Stimulus controllers index.js not found', :yellow
+ return
+ end
+
+ say_status :info, 'Stimulus controllers are auto-loaded', :blue
+ end
+ end
+ end
+end
diff --git a/lib/generators/spl/install/templates/login_spl_controller.js b/lib/generators/spl/install/templates/login_spl_controller.js
new file mode 100644
index 0000000..8e233ab
--- /dev/null
+++ b/lib/generators/spl/install/templates/login_spl_controller.js
@@ -0,0 +1,22 @@
+// This file is generated using the spree-spl gem.
+import { Controller } from "@hotwired/stimulus"
+
+export default class extends Controller {
+ static targets = ["checkbox", "loginForm", "registrationForm"]
+
+ connect() {
+ this.toggle()
+ }
+
+ toggle() {
+ if (!this.hasCheckboxTarget) return
+
+ if (this.checkboxTarget.checked) {
+ this.loginFormTarget.classList.remove("hidden")
+ this.registrationFormTarget.classList.add("hidden")
+ } else {
+ this.loginFormTarget.classList.add("hidden")
+ this.registrationFormTarget.classList.remove("hidden")
+ }
+ }
+}
diff --git a/lib/spree/spl/configuration.rb b/lib/spree/spl/configuration.rb
new file mode 100644
index 0000000..4b58aa0
--- /dev/null
+++ b/lib/spree/spl/configuration.rb
@@ -0,0 +1,22 @@
+# frozen_string_literal: true
+
+module Spree
+ module Spl
+ class Configuration
+ attr_accessor :error_reporter
+
+ def initialize
+ @error_reporter = ->(message, extra = {}) { Rails.logger.warn("[SPL] #{message} #{extra.inspect}") }
+ end
+ end
+
+ def self.config = (@config ||= Configuration.new)
+ def self.configure = yield(config)
+
+ def self.report_error(message, extra = {})
+ config.error_reporter&.call(message, extra)
+ rescue StandardError => e
+ Rails.logger.error("[SPL] error_reporter failed: #{e.class}: #{e.message}")
+ end
+ end
+end
diff --git a/spec/controllers/checkout_controller_decorator_spec.rb b/spec/controllers/checkout_controller_decorator_spec.rb
index 8dd7f41..ad55ddd 100644
--- a/spec/controllers/checkout_controller_decorator_spec.rb
+++ b/spec/controllers/checkout_controller_decorator_spec.rb
@@ -2,127 +2,184 @@
require 'rails_helper'
-describe Spree::CheckoutController, type: :controller do
- let(:store) do
- create(:store, private_metadata: {
- 'spl_url' => 'https://fake-spl.example.com',
- 'spl_mode' => 'test',
- 'spl_pos_key' => 'fake-pos-key',
- 'spl_api_user' => 'fake-api-user',
- 'spl_prg_code' => 'fake-prg-code',
- 'spl_sale_url' => 'https://fake-spl.example.com/sale',
- 'spl_api_token' => 'fake-token',
- 'spl_place_code' => 'fake-place',
- 'spl_partner_code' => 'fake-partner',
- 'spl_signature_seed' => 'fake-seed'
- })
- end
+RSpec.describe Spl::Spree::Storefront::CheckoutControllerDecorator, type: :controller do
+ controller(ActionController::Base) do
+ before_action :set_order
+
+ def checkout_path
+ '/checkout'
+ end
+
+ def show
+ render plain: 'ok'
+ end
+
+ # Provided by decorator; defined here so Rails can dispatch
+
+ private
- let!(:state) { create(:state, country: create(:country_us), name: 'Gdansk', abbr: 'GDA') }
- let(:user) { create(:user, public_metadata: { 'spl_no_card' => '1234567890123', 'spl_card_active' => 'true' }) }
-
- let(:order) do
- create(
- :order_with_totals,
- store: store,
- user: user,
- email: 'test@example.com',
- public_metadata: {
- 'spl_no_card' => '1234567890123',
- 'spl_card_active' => 'true'
- }
- )
+ def set_order
+ @order = Spree::Order.first
+ end
end
before do
- allow(controller).to receive_messages(current_store: store, try_spree_current_user: user, spree_current_user: user,
- spree_signup_path: '/signup', spree_login_path: '/login')
-
- stub_request(:any, /fake-spl\.example\.com/).to_return(
- status: 200,
- body: {
- 'errorCode' => '0',
- 'response' => {
- 'basket' => order.line_items.map do |li|
- {
- 'pos' => li.id,
- 'discounts' => [
- {
- 'name' => 'TEST'
- }
- ],
- 'discountGross' => 5.0
- }
- end
- }
- }.to_json,
- headers: { 'Content-Type' => 'application/json' }
- )
-
- allow_any_instance_of(Spree::Order).to receive(:recalculate).and_return(true)
+ controller.class.prepend described_class
+ routes.draw do
+ get '/checkout', to: 'anonymous#show'
+ post '/checkout/activate_coupon', to: 'anonymous#activate_coupon'
+ post '/checkout/deactivate_coupon', to: 'anonymous#deactivate_coupon'
+ end
+ allow(request).to receive(:url).and_return('http://test.host/checkout')
+ end
+
+ let(:country) { create(:country) }
+ let!(:store) { create(:store, default_country: country) }
+
+ let!(:user) { create(:user) }
+ let!(:order) do
+ create(:order,
+ store: store,
+ user: user)
end
describe '#promotion_switcher' do
- context 'creates a new SPL adjustment' do
- it 'with SPL source_type' do
- post :update, params: { state: 'address', token: order.token }
+ it "calls PromotionSwitcherService with confirm=false when URL does not include 'confirm'" do
+ switcher = instance_double('PromotionSwitcherService', call: true)
+ expect(PromotionSwitcherService).to receive(:new).with(order, false).and_return(switcher)
- expect(order.line_items.last.adjustments.last.source_type).to eq('SPL')
- end
+ allow(Spl::Coupons::GetCouponsService)
+ .to receive(:new).with(user, store)
+ .and_return(instance_double('Spl::Coupons::GetCouponsService', call: []))
- it 'with amount from discountGross' do
- post :update, params: { state: 'address', token: order.token }
+ get :show
+ expect(response).to have_http_status(:ok)
+ end
- expect(order.line_items.last.adjustments.last.amount).to eq(-5.0)
- end
+ it "calls PromotionSwitcherService with confirm=true when URL includes 'confirm'" do
+ allow_any_instance_of(ActionDispatch::Request)
+ .to receive(:url)
+ .and_return('http://test.host/checkout/confirm')
+
+ switcher = instance_double('PromotionSwitcherService', call: true)
+ expect(PromotionSwitcherService).to receive(:new).with(order, true).and_return(switcher)
+
+ allow(Spl::Coupons::GetCouponsService)
+ .to receive(:new).with(user, store)
+ .and_return(instance_double('Spl::Coupons::GetCouponsService', call: []))
+
+ get :show
+ expect(response).to have_http_status(:ok)
end
+ end
- context 'when SPL API response has no line_items (unhappy path)' do
- before do
- stub_request(:any, /fake-spl\.example\.com/).to_return(
- status: 200,
- body: {
- 'errorCode' => '0',
- 'response' => {
- 'basket' => []
- }
- }.to_json,
- headers: { 'Content-Type' => 'application/json' }
- )
- end
+ describe '#load_user_coupons' do
+ it 'loads coupons for user/store and assigns @coupons' do
+ allow(PromotionSwitcherService)
+ .to receive(:new)
+ .and_return(instance_double('PromotionSwitcherService', call: true))
- it 'does NOT create SPL adjustments' do
- expect do
- post :update, params: { state: 'address', token: order.token }
- end.not_to(change { order.line_items.first.adjustments.count })
- end
+ coupons_service = instance_double('Spl::Coupons::GetCouponsService', call: %w[c1 c2])
+ expect(Spl::Coupons::GetCouponsService)
+ .to receive(:new).with(user, store).and_return(coupons_service)
+
+ get :show
+
+ expect(response).to have_http_status(:ok)
+ expect(assigns(:coupons)).to eq(%w[c1 c2])
end
- context 'when SPL API response has no discounts (unhappy path)' do
- before do
- stub_request(:any, /fake-spl\.example\.com/).to_return(
- status: 200,
- body: {
- 'errorCode' => '0',
- 'response' => {
- 'basket' => order.line_items.map do |li|
- {
- 'pos' => li.id,
- 'discounts' => [],
- 'discountGross' => 0.0
- }
- end
- }
- }.to_json,
- headers: { 'Content-Type' => 'application/json' }
- )
+ it 'does not run load_user_coupons as a before_action for activate_coupon' do
+ allow(PromotionSwitcherService)
+ .to receive(:new)
+ .and_return(instance_double('PromotionSwitcherService', call: true))
+
+ # This will still be called (inside #activate_coupon), but we can ensure it is not called twice.
+ coupons_service = instance_double('Spl::Coupons::GetCouponsService', call: [])
+ expect(Spl::Coupons::GetCouponsService).to receive(:new).with(user, store).once.and_return(coupons_service)
+
+ allow(Spl::Coupons::ActivateCouponService)
+ .to receive(:new)
+ .and_return(instance_double('Spl::Coupons::ActivateCouponService', call: true))
+
+ post :activate_coupon, params: { coupon_code: 'YOLO_90' }, format: :html
+ end
+ end
+
+ describe '#activate_coupon' do
+ let(:activate) { instance_double(Spl::Coupons::ActivateCouponService, call: true) }
+
+ before do
+ allow(controller).to receive(:try_spree_current_user).and_return(user)
+ end
+
+ context 'when coupon is valid' do
+ it 'calls ActivateCouponService, reloads coupons, and redirects (HTML)' do
+ expect(Spl::Coupons::ActivateCouponService)
+ .to receive(:new).with(user, store, 'YOLO_90').and_return(activate)
+
+ coupons_service = instance_double('Spl::Coupons::GetCouponsService', call: %w[YOLO_90])
+ expect(Spl::Coupons::GetCouponsService)
+ .to receive(:new).with(user, store).and_return(coupons_service)
+
+ post :activate_coupon, params: { coupon_code: 'YOLO_90' }, format: :html
+
+ expect(assigns(:coupons)).to eq(%w[YOLO_90])
+ expect(response).to have_http_status(:found)
+ expect(response.location).to include('/checkout')
end
+ end
+
+ context 'when coupon is invalid' do
+ it 'still redirects even if ActivateCouponService raises' do
+ expect(Spl::Coupons::ActivateCouponService).to receive(:new).with(user, store, 'YOLO_90').and_return(activate)
+ expect(activate).to receive(:call).and_raise(Spl::Coupons::ActivateCouponService::ActivateCouponServiceError,
+ 'some error')
- it 'does NOT create SPL adjustments' do
expect do
- post :update, params: { state: 'address', token: order.token }
- end.not_to(change { order.line_items.first.adjustments.count })
+ post :activate_coupon, params: { coupon_code: 'YOLO_90' }, format: :html
+ end.to raise_error('some error')
+
+ expect(response).to have_http_status(:found)
+ expect(response.location).to include('/checkout')
end
end
end
+
+ describe '#deactivate_coupon' do
+ before do
+ allow(controller).to receive(:try_spree_current_user).and_return(user)
+ end
+
+ it 'calls DeactivateCouponService, reloads coupons, and redirects (HTML)' do
+ deactivate = instance_double('Spl::Coupons::DeactivateCouponService', call: true)
+ expect(Spl::Coupons::DeactivateCouponService)
+ .to receive(:new).with(user, store, 'YOLO_90').and_return(deactivate)
+
+ coupons_service = instance_double('Spl::Coupons::GetCouponsService', call: %w[YOLO_90])
+ expect(Spl::Coupons::GetCouponsService)
+ .to receive(:new).with(user, store).and_return(coupons_service)
+
+ post :deactivate_coupon, params: { coupon_code: 'YOLO_90' }, format: :html
+
+ expect(assigns(:coupons)).to eq(%w[YOLO_90])
+ expect(response).to have_http_status(:found)
+ expect(response.location).to include('/checkout')
+ end
+
+ it 'still redirects even if DeactivateCouponService raises' do
+ deactivate = instance_double('Spl::Coupons::DeactivateCouponService')
+ expect(Spl::Coupons::DeactivateCouponService).to receive(:new).with(user, store, 'YOLO_90').and_return(deactivate)
+ expect(deactivate).to receive(:call).and_raise(
+ Spl::Coupons::DeactivateCouponService::DeactivateCouponServiceError, 'some error'
+ )
+
+ expect do
+ post :deactivate_coupon, params: { coupon_code: 'YOLO_90' }, format: :html
+ end.to raise_error('some error')
+
+ expect(response).to have_http_status(:found)
+ expect(response.location).to include('/checkout')
+ end
+ end
end
diff --git a/spec/controllers/spl/spree/account/profile_controller_decorator_spec.rb b/spec/controllers/spl/spree/account/profile_controller_decorator_spec.rb
new file mode 100644
index 0000000..51b8d69
--- /dev/null
+++ b/spec/controllers/spl/spree/account/profile_controller_decorator_spec.rb
@@ -0,0 +1,304 @@
+# frozen_string_literal: true
+
+require 'rails_helper'
+
+RSpec.describe Spree::Account::ProfileController, type: :controller do
+ before do
+ stub_const('Spl::SendOtpService::SplSendOtpError', Class.new(StandardError)) unless defined?(Spl::SendOtpService::SplSendOtpError)
+ end
+
+ let(:country) { create(:country) }
+ let(:store) { create(:store, default_country: country) }
+ let!(:user) { create(:user, public_metadata: {}) }
+
+ let(:terms_accepted) { 'true' }
+ let(:phone_valid) { true }
+ let(:phone_value) { '+48500600700' }
+
+ let(:phone_double) do
+ instance_double(
+ 'PhoneParserService',
+ valid?: phone_valid,
+ country_code: '+48',
+ national_number: '500600700',
+ e164: '+48500600700'
+ )
+ end
+
+ let(:turbo_stream_helper) { instance_double('TurboStreamHelper') }
+
+ before do
+ allow(controller).to receive(:current_store).and_return(store)
+ allow(controller).to receive(:try_spree_current_user).and_return(user)
+ allow(controller).to receive(:login_code_params).and_return(
+ ActionController::Parameters.new(phone: phone_value, accept_yc_terms: terms_accepted).permit!
+ )
+ allow(controller).to receive(:phone_parser).and_return(phone_double)
+ allow(controller).to receive(:turbo_stream).and_return(turbo_stream_helper)
+ allow(turbo_stream_helper).to receive(:replace).and_return('