style(Style/OpenStructUse): manual

Author: antonsavitskiy

.rubocop_todo.yml

@@ -828,13 +828,6 @@ Style/MultilineBlockChain:
     - 'spec/mongoid/changeable_spec.rb'
     - 'spec/mongoid/document_spec.rb'
 
-# Offense count: 4
-Style/OpenStructUse:
-  Exclude:
-    - 'lib/mongoid/indexable.rb'
-    - 'spec/rails/controller_extension/controller_runtime_spec.rb'
-    - 'spec/support/rails_mock.rb'
-
 # Offense count: 17
 # Configuration parameters: AllowedMethods.
 # AllowedMethods: respond_to_missing?

lib/mongoid/indexable.rb

@@ -17,6 +17,8 @@ module Indexable
 
     module ClassMethods
 
+      IndexSpec = Struct.new(:fields, :key, keyword_init: true)
+
       # Send the actual index creation comments to the MongoDB driver
       #
       # @example Create the indexes for the class.
@@ -109,7 +111,7 @@ def index(spec, options = nil)
       #
       # @return [ Specification ] The found specification.
       def index_specification(index_hash, index_name = nil)
-        index = OpenStruct.new(fields: index_hash.keys, key: index_hash)
+        index = IndexSpec.new(fields: index_hash.keys, key: index_hash)
         index_specifications.detect do |spec|
           spec == index || (index_name && index_name == spec.name)
         end

spec/rails/controller_extension/controller_runtime_spec.rb

@@ -41,6 +41,10 @@ def clear_metric!
 
   describe 'Collector' do
 
+    before do
+      stub_const('EventPayload', Struct.new(:duration, keyword_init: true))
+    end
+
     it 'stores the metric in thread-safe manner' do
       clear_metric!
       expect(collector.runtime).to eq(0)
@@ -50,7 +54,7 @@ def clear_metric!
 
     it 'sets metric on both succeeded and failed' do
       instance = collector.new
-      event_payload = OpenStruct.new duration: 42
+      event_payload = EventPayload.new(duration: 42)
 
       clear_metric!
       instance.succeeded event_payload

spec/support/rails_mock.rb

@@ -19,11 +19,14 @@ def eager_load!; end
   end
 end
 
+ConfigPaths = Struct.new(:expanded, keyword_init: true)
+AppConfig = Struct.new(:paths, keyword_init: true)
+
 Rails.env = 'development'
 Rails.root = Pathname.new('.')
 Rails.logger = Logger.new($stdout)
 Rails.application = Rails::Application
 
-Rails.application.config = OpenStruct.new(
-  paths: { 'app/models' => OpenStruct.new(expanded: ['app/models']) }
+Rails.application.config = AppConfig.new(
+  paths: { 'app/models' => ConfigPaths.new(expanded: ['app/models']) }
 )