diff --git a/.rubocop_todo.yml b/.rubocop_todo.yml
index d8be25b156..a51c63ddfc 100644
--- a/.rubocop_todo.yml
+++ b/.rubocop_todo.yml
@@ -160,7 +160,6 @@ Metrics/ModuleLength:
     - 'app/helpers/spree/admin/orders_helper.rb'
     - 'app/models/spree/order/checkout.rb'
     - 'app/models/spree/payment/processing.rb'
-    - 'engines/order_management/spec/services/order_management/subscriptions/summarizer_spec.rb'
     - 'engines/order_management/spec/services/order_management/subscriptions/summary_spec.rb'
     - 'engines/order_management/spec/services/order_management/subscriptions/validator_spec.rb'
     - 'engines/order_management/spec/services/order_management/subscriptions/variants_list_spec.rb'
diff --git a/engines/order_management/spec/services/order_management/subscriptions/summarizer_spec.rb b/engines/order_management/spec/services/order_management/subscriptions/summarizer_spec.rb
index a442a76f6f..052306a9ac 100644
--- a/engines/order_management/spec/services/order_management/subscriptions/summarizer_spec.rb
+++ b/engines/order_management/spec/services/order_management/subscriptions/summarizer_spec.rb
@@ -2,145 +2,141 @@
 
 require 'spec_helper'
 
-module OrderManagement
-  module Subscriptions
-    RSpec.describe Summarizer do
-      let(:order) { create(:order) }
-      let(:summarizer) { OrderManagement::Subscriptions::Summarizer.new }
+RSpec.describe OrderManagement::Subscriptions::Summarizer do
+  let(:order) { create(:order) }
+  let(:summarizer) { described_class.new }
 
-      before { allow(JobLogger.logger).to receive(:info) }
+  before { allow(JobLogger.logger).to receive(:info) }
 
-      describe "#summary_for" do
-        let(:order) { double(:order, distributor_id: 123) }
+  describe "#summary_for" do
+    let(:order) { double(:order, distributor_id: 123) }
 
-        context "when a summary for the order's distributor doesn't already exist" do
-          it "initializes a new summary object, and returns it" do
-            expect(summarizer.instance_variable_get(:@summaries).count).to be 0
-            summary = summarizer.__send__(:summary_for, order)
-            expect(summary.shop_id).to be 123
-            expect(summarizer.instance_variable_get(:@summaries).count).to be 1
-          end
-        end
+    context "when a summary for the order's distributor doesn't already exist" do
+      it "initializes a new summary object, and returns it" do
+        expect(summarizer.instance_variable_get(:@summaries).count).to be 0
+        summary = summarizer.__send__(:summary_for, order)
+        expect(summary.shop_id).to be 123
+        expect(summarizer.instance_variable_get(:@summaries).count).to be 1
+      end
+    end
 
-        context "when a summary for the order's distributor already exists" do
-          let(:summary) { double(:summary) }
+    context "when a summary for the order's distributor already exists" do
+      let(:summary) { double(:summary) }
 
-          before do
-            summarizer.instance_variable_set(:@summaries, 123 => summary)
-          end
-
-          it "returns the existing summary object" do
-            expect(summarizer.instance_variable_get(:@summaries).count).to be 1
-            expect(summarizer.__send__(:summary_for, order)).to eq summary
-            expect(summarizer.instance_variable_get(:@summaries).count).to be 1
-          end
-        end
+      before do
+        summarizer.instance_variable_set(:@summaries, 123 => summary)
       end
 
-      describe "recording events" do
-        let(:order) { double(:order) }
-        let(:summary) { double(:summary) }
-        before { allow(summarizer).to receive(:summary_for).with(order) { summary } }
-
-        describe "#record_order" do
-          it "requests a summary for the order and calls #record_order on it" do
-            expect(summary).to receive(:record_order).with(order).once
-            summarizer.record_order(order)
-          end
-        end
-
-        describe "#record_success" do
-          it "requests a summary for the order and calls #record_success on it" do
-            expect(summary).to receive(:record_success).with(order).once
-            summarizer.record_success(order)
-          end
-        end
-
-        describe "#record_issue" do
-          it "requests a summary for the order and calls #record_issue on it" do
-            expect(order).to receive(:id)
-            expect(summary).to receive(:record_issue).with(:type, order, "message").once
-            summarizer.record_issue(:type, order, "message")
-          end
-        end
-
-        describe "#record_and_log_error" do
-          before do
-            allow(order).to receive(:number) { "123" }
-          end
-
-          context "when errors exist on the order" do
-            before do
-              allow(order).to receive(:errors) {
-                double(:errors, any?: true, full_messages: ["Some error"])
-              }
-            end
-
-            it "sends error info to rails logger and calls #record_issue with an error message" do
-              expect(summarizer).to receive(:record_issue).with(:processing,
-                                                                order, "Errors: Some error")
-              summarizer.record_and_log_error(:processing, order)
-            end
-          end
-
-          context "when no errors exist on the order" do
-            before do
-              allow(order).to receive(:errors) { double(:errors, any?: false) }
-            end
-
-            it "falls back to calling record_issue" do
-              expect(summarizer).to receive(:record_issue).with(:processing, order)
-              summarizer.record_and_log_error(:processing, order)
-            end
-          end
-        end
-
-        describe "#record_subscription_issue" do
-          let(:subscription) { double(:subscription, shop_id: 1) }
-
-          before do
-            allow(summarizer).to receive(:summary_for_shop_id).
-              with(subscription.shop_id) { summary }
-          end
-
-          it "records a subscription issue" do
-            expect(summary).to receive(:record_subscription_issue).with(subscription).once
-            summarizer.record_subscription_issue(subscription)
-          end
-        end
-      end
-
-      describe "#send_placement_summary_emails" do
-        let(:summary1) { double(:summary) }
-        let(:summary2) { double(:summary) }
-        let(:summaries) { { 1 => summary1, 2 => summary2 } }
-        let(:mail_mock) { double(:mail, deliver_now: true) }
-
-        before do
-          summarizer.instance_variable_set(:@summaries, summaries)
-        end
-
-        it "sends a placement summary email for each summary" do
-          expect(SubscriptionMailer).to receive(:placement_summary_email).twice { mail_mock }
-          summarizer.send_placement_summary_emails
-        end
-      end
-
-      describe "#send_confirmation_summary_emails" do
-        let(:summary1) { double(:summary) }
-        let(:summary2) { double(:summary) }
-        let(:summaries) { { 1 => summary1, 2 => summary2 } }
-        let(:mail_mock) { double(:mail, deliver_now: true) }
-
-        before do
-          summarizer.instance_variable_set(:@summaries, summaries)
-        end
-
-        it "sends a placement summary email for each summary" do
-          expect(SubscriptionMailer).to receive(:confirmation_summary_email).twice { mail_mock }
-          summarizer.send_confirmation_summary_emails
-        end
+      it "returns the existing summary object" do
+        expect(summarizer.instance_variable_get(:@summaries).count).to be 1
+        expect(summarizer.__send__(:summary_for, order)).to eq summary
+        expect(summarizer.instance_variable_get(:@summaries).count).to be 1
       end
     end
   end
+
+  describe "recording events" do
+    let(:order) { double(:order) }
+    let(:summary) { double(:summary) }
+    before { allow(summarizer).to receive(:summary_for).with(order) { summary } }
+
+    describe "#record_order" do
+      it "requests a summary for the order and calls #record_order on it" do
+        expect(summary).to receive(:record_order).with(order).once
+        summarizer.record_order(order)
+      end
+    end
+
+    describe "#record_success" do
+      it "requests a summary for the order and calls #record_success on it" do
+        expect(summary).to receive(:record_success).with(order).once
+        summarizer.record_success(order)
+      end
+    end
+
+    describe "#record_issue" do
+      it "requests a summary for the order and calls #record_issue on it" do
+        expect(order).to receive(:id)
+        expect(summary).to receive(:record_issue).with(:type, order, "message").once
+        summarizer.record_issue(:type, order, "message")
+      end
+    end
+
+    describe "#record_and_log_error" do
+      before do
+        allow(order).to receive(:number) { "123" }
+      end
+
+      context "when errors exist on the order" do
+        before do
+          allow(order).to receive(:errors) {
+            double(:errors, any?: true, full_messages: ["Some error"])
+          }
+        end
+
+        it "sends error info to rails logger and calls #record_issue with an error message" do
+          expect(summarizer).to receive(:record_issue).with(:processing,
+                                                            order, "Errors: Some error")
+          summarizer.record_and_log_error(:processing, order)
+        end
+      end
+
+      context "when no errors exist on the order" do
+        before do
+          allow(order).to receive(:errors) { double(:errors, any?: false) }
+        end
+
+        it "falls back to calling record_issue" do
+          expect(summarizer).to receive(:record_issue).with(:processing, order)
+          summarizer.record_and_log_error(:processing, order)
+        end
+      end
+    end
+
+    describe "#record_subscription_issue" do
+      let(:subscription) { double(:subscription, shop_id: 1) }
+
+      before do
+        allow(summarizer).to receive(:summary_for_shop_id).
+          with(subscription.shop_id) { summary }
+      end
+
+      it "records a subscription issue" do
+        expect(summary).to receive(:record_subscription_issue).with(subscription).once
+        summarizer.record_subscription_issue(subscription)
+      end
+    end
+  end
+
+  describe "#send_placement_summary_emails" do
+    let(:summary1) { double(:summary) }
+    let(:summary2) { double(:summary) }
+    let(:summaries) { { 1 => summary1, 2 => summary2 } }
+    let(:mail_mock) { double(:mail, deliver_now: true) }
+
+    before do
+      summarizer.instance_variable_set(:@summaries, summaries)
+    end
+
+    it "sends a placement summary email for each summary" do
+      expect(SubscriptionMailer).to receive(:placement_summary_email).twice { mail_mock }
+      summarizer.send_placement_summary_emails
+    end
+  end
+
+  describe "#send_confirmation_summary_emails" do
+    let(:summary1) { double(:summary) }
+    let(:summary2) { double(:summary) }
+    let(:summaries) { { 1 => summary1, 2 => summary2 } }
+    let(:mail_mock) { double(:mail, deliver_now: true) }
+
+    before do
+      summarizer.instance_variable_set(:@summaries, summaries)
+    end
+
+    it "sends a placement summary email for each summary" do
+      expect(SubscriptionMailer).to receive(:confirmation_summary_email).twice { mail_mock }
+      summarizer.send_confirmation_summary_emails
+    end
+  end
 end