Deprecate User subscription nested attributes

2 files changed, 20 insertions, 0 deletions

diff --git a/app/decorators/models/solidus_subscriptions/spree/user/have_many_subscriptions.rb b/app/decorators/models/solidus_subscriptions/spree/user/have_many_subscriptions.rb
index 561e5a1..1fb4cdf 100644
--- a/app/decorators/models/solidus_subscriptions/spree/user/have_many_subscriptions.rb
+++ b/app/decorators/models/solidus_subscriptions/spree/user/have_many_subscriptions.rb
@@ -14,6 +14,14 @@ module SolidusSubscriptions
 
           base.accepts_nested_attributes_for :subscriptions
         end
+
+        def subscriptions_attributes=(params)
+          ::Spree::Deprecation.warn(
+            'Creating or updating subscriptions through Spree::User nested attributes is deprecated. ' \
+            'Please use subscriptions APIs directly.'
+          )
+          super
+        end
       end
     end
   end
diff --git a/spec/decorators/models/solidus_subscriptions/spree/user/have_many_subscriptions_spec.rb b/spec/decorators/models/solidus_subscriptions/spree/user/have_many_subscriptions_spec.rb
index 66bbe61..a76f130 100644
--- a/spec/decorators/models/solidus_subscriptions/spree/user/have_many_subscriptions_spec.rb
+++ b/spec/decorators/models/solidus_subscriptions/spree/user/have_many_subscriptions_spec.rb
@@ -5,4 +5,16 @@ RSpec.describe SolidusSubscriptions::Spree::User::HaveManySubscriptions, type: :
 
   it { is_expected.to have_many :subscriptions }
   it { is_expected.to accept_nested_attributes_for :subscriptions }
+
+  describe '#subscriptions_attributes=' do
+    it 'throws a deprecation warning' do
+      allow(::Spree::Deprecation).to receive(:warn)
+
+      subject.subscriptions_attributes = [{ interval_length: 2 }]
+
+      expect(::Spree::Deprecation)
+        .to have_received(:warn)
+        .with(/Creating or updating subscriptions through Spree::User nested attributes is deprecated/)
+    end
+  end
 end