autofix RSpec/MetadataStyle

Author: modosc

Diff for: .rubocop_todo.yml

@@ -1,18 +1,11 @@
 # This configuration was generated by
 # `rubocop --auto-gen-config`
-# on 2025-04-04 23:07:22 UTC using RuboCop version 1.75.1.
+# on 2025-04-04 23:09:43 UTC using RuboCop version 1.75.1.
 # The point is for the user to remove these configuration records
 # one by one as the offenses are removed from the code base.
 # Note that changes in the inspected code, or installation of new
 # versions of RuboCop, may require this file to be generated again.
 
-# Offense count: 81
-# This cop supports safe autocorrection (--autocorrect).
-# Configuration parameters: EnforcedStyle.
-# SupportedStyles: hash, symbol
-RSpec/MetadataStyle:
-  Enabled: false
-
 # Offense count: 1
 # This cop supports unsafe autocorrection (--autocorrect-all).
 Rails/ActiveSupportOnLoad:

Diff for: spec/controllers/widgets_controller_spec.rb

@@ -2,7 +2,7 @@
 
 require "spec_helper"
 
-RSpec.describe WidgetsController, versioning: true do
+RSpec.describe WidgetsController, :versioning do
   before { request.env["REMOTE_ADDR"] = "127.0.0.1" }
 
   after { RequestStore.store[:paper_trail] = nil }

Diff for: spec/models/animal_spec.rb

@@ -2,7 +2,7 @@
 
 require "spec_helper"
 
-RSpec.describe Animal, versioning: true do
+RSpec.describe Animal, :versioning do
   it "baseline test setup" do
     expect(described_class.new).to be_versioned
     expect(described_class.inheritance_column).to eq("species")

Diff for: spec/models/article_spec.rb

@@ -2,7 +2,7 @@
 
 require "spec_helper"
 
-RSpec.describe Article, versioning: true do
+RSpec.describe Article, :versioning do
   describe ".create" do
     it "also creates a version record" do
       expect { described_class.create }.to(

Diff for: spec/models/book_spec.rb

@@ -2,7 +2,7 @@
 
 require "spec_helper"
 
-RSpec.describe Book, versioning: true do
+RSpec.describe Book, :versioning do
   context "with :has_many :through" do
     it "store version on source <<" do
       book = described_class.create(title: "War and Peace")

Diff for: spec/models/boolit_spec.rb

@@ -3,7 +3,7 @@
 require "spec_helper"
 require "support/custom_json_serializer"
 
-RSpec.describe Boolit, versioning: true do
+RSpec.describe Boolit, :versioning do
   let(:boolit) { described_class.create! }
 
   before { boolit.update!(name: FFaker::Name.name) }

Diff for: spec/models/callback_modifier_spec.rb

@@ -2,7 +2,7 @@
 
 require "spec_helper"
 
-RSpec.describe CallbackModifier, versioning: true do
+RSpec.describe CallbackModifier, :versioning do
   describe "paper_trail_on_destroy" do
     it "adds :destroy to paper_trail_options[:on]" do
       modifier = NoArgDestroyModifier.create!(some_content: FFaker::Lorem.sentence)

Diff for: spec/models/car_spec.rb

@@ -5,15 +5,15 @@
 RSpec.describe Car do
   it { is_expected.to be_versioned }
 
-  describe "changeset", versioning: true do
+  describe "changeset", :versioning do
     it "has the expected keys (see issue 738)" do
       car = described_class.create!(name: "Alice")
       car.update(name: "Bob")
       assert_includes car.versions.last.changeset.keys, "name"
     end
   end
 
-  describe "attributes and accessors", versioning: true do
+  describe "attributes and accessors", :versioning do
     it "reifies attributes that are not AR attributes" do
       car = described_class.create name: "Pinto", color: "green"
       car.update color: "yellow"

Diff for: spec/models/cat_spec.rb

@@ -2,7 +2,7 @@
 
 require "spec_helper"
 
-RSpec.describe Cat, versioning: true do
+RSpec.describe Cat, :versioning do
   describe "#descends_from_active_record?" do
     it "returns false, meaning that Cat is an STI subclass" do
       expect(described_class.descends_from_active_record?).to eq(false)

Diff for: spec/models/custom_primary_key_record_spec.rb

@@ -6,7 +6,7 @@
   it { is_expected.to be_versioned }
 
   describe "#versions" do
-    it "returns instances of CustomPrimaryKeyRecordVersion", versioning: true do
+    it "returns instances of CustomPrimaryKeyRecordVersion", :versioning do
       custom_primary_key_record = described_class.create!
       custom_primary_key_record.update!(name: "bob")
       version = custom_primary_key_record.versions.last

Diff for: spec/models/document_spec.rb

@@ -2,7 +2,7 @@
 
 require "spec_helper"
 
-RSpec.describe Document, versioning: true do
+RSpec.describe Document, :versioning do
   describe "have_a_version_with matcher" do
     it "works with custom versions association" do
       document = described_class.create!(name: "Foo")

Diff for: spec/models/family/celebrity_family_spec.rb

@@ -3,7 +3,7 @@
 require "spec_helper"
 
 module Family
-  RSpec.describe CelebrityFamily, versioning: true do
+  RSpec.describe CelebrityFamily, :versioning do
     describe "#joins" do
       it "works on an STI model" do
         described_class.create!

Diff for: spec/models/foo_widget_spec.rb

@@ -3,7 +3,7 @@
 require "spec_helper"
 require "support/performance_helpers"
 
-RSpec.describe(FooWidget, versioning: true) do
+RSpec.describe(FooWidget, :versioning) do
   context "with a subclass" do
     let(:foo) { described_class.create }
 

Diff for: spec/models/fruit_spec.rb

@@ -3,7 +3,7 @@
 require "spec_helper"
 
 if ENV["DB"] == "postgres" && JsonVersion.table_exists?
-  RSpec.describe Fruit, versioning: true do
+  RSpec.describe Fruit, :versioning do
     describe "have_a_version_with_changes matcher" do
       it "works with Fruit because Fruit uses JsonVersion" do
         # As of PT 9.0.0, with_version_changes only supports json(b) columns,
@@ -17,7 +17,7 @@
       end
     end
 
-    describe "queries of versions", versioning: true do
+    describe "queries of versions", :versioning do
       let!(:fruit) { described_class.create(name: "Apple", mass: 1, color: "green") }
 
       before do

Diff for: spec/models/gadget_spec.rb

@@ -7,7 +7,7 @@
 
   it { is_expected.to be_versioned }
 
-  describe "updates", versioning: true do
+  describe "updates", :versioning do
     it "generates a version for updates" do
       expect { gadget.update_attribute(:name, "Hammer") }.to(change { gadget.versions.size }.by(1))
     end

Diff for: spec/models/gizmo_spec.rb

@@ -3,7 +3,7 @@
 require "spec_helper"
 require "support/performance_helpers"
 
-RSpec.describe Gizmo, versioning: true do
+RSpec.describe Gizmo, :versioning do
   context "with a persisted record" do
     it "does not use the gizmo `updated_at` as the version's `created_at`" do
       gizmo = described_class.create(name: "Fred", created_at: 1.day.ago)

Diff for: spec/models/joined_version_spec.rb

@@ -2,7 +2,7 @@
 
 require "spec_helper"
 
-RSpec.describe JoinedVersion, versioning: true do
+RSpec.describe JoinedVersion, :versioning do
   let(:widget) { Widget.create!(name: FFaker::Name.name) }
   let(:version) { described_class.first }
 

Diff for: spec/models/json_version_spec.rb

@@ -5,7 +5,7 @@
 # The `json_versions` table tests postgres' `json` data type. So, that
 # table is only created when testing against postgres.
 if JsonVersion.table_exists?
-  RSpec.describe JsonVersion, versioning: true do
+  RSpec.describe JsonVersion, :versioning do
     it "includes the VersionConcern module" do
       expect(described_class).to include(PaperTrail::VersionConcern)
     end
@@ -30,7 +30,7 @@
         end
       end
 
-      context "with valid arguments", versioning: true do
+      context "with valid arguments", :versioning do
         it "locates versions according to their `object` contents" do
           fruit = Fruit.create!(name: "apple")
           expect(fruit.versions.length).to eq(1)
@@ -72,7 +72,7 @@
         end
       end
 
-      context "with valid arguments", versioning: true do
+      context "with valid arguments", :versioning do
         it "finds versions according to their `object_changes` contents" do
           fruit = Fruit.create!(name: "apple")
           fruit.update!(name: "banana", color: "red")

Diff for: spec/models/kitchen/banana_spec.rb

@@ -7,7 +7,7 @@ module Kitchen
     it { is_expected.to be_versioned }
 
     describe "#versions" do
-      it "returns instances of Kitchen::BananaVersion", versioning: true do
+      it "returns instances of Kitchen::BananaVersion", :versioning do
         banana = described_class.create!
         expect(banana.versions.first).to be_a(Kitchen::BananaVersion)
       end

Diff for: spec/models/legacy_widget_spec.rb

@@ -2,7 +2,7 @@
 
 require "spec_helper"
 
-RSpec.describe LegacyWidget, versioning: true do
+RSpec.describe LegacyWidget, :versioning do
   describe "#custom_version" do
     it "knows which version it came from" do
       widget = described_class.create(name: "foo", version: 2)

Diff for: spec/models/management_spec.rb

@@ -2,7 +2,7 @@
 
 require "spec_helper"
 
-RSpec.describe(Management, versioning: true) do
+RSpec.describe(Management, :versioning) do
   it "utilises the base_class for STI classes having no type column" do
     expect(described_class.inheritance_column).to eq("type")
     expect(described_class.columns.map(&:name)).not_to include("type")

Diff for: spec/models/no_object_spec.rb

@@ -2,7 +2,7 @@
 
 require "spec_helper"
 
-RSpec.describe NoObject, versioning: true do
+RSpec.describe NoObject, :versioning do
   it "still creates version records" do
     n = described_class.create!(letter: "A")
     a = n.versions.last.attributes

Diff for: spec/models/not_on_update_spec.rb

@@ -3,7 +3,7 @@
 require "spec_helper"
 
 RSpec.describe NotOnUpdate do
-  describe "#save_with_version", versioning: true do
+  describe "#save_with_version", :versioning do
     let!(:record) { described_class.create! }
 
     it "creates a version, regardless" do

Diff for: spec/models/on/create_spec.rb

@@ -4,7 +4,7 @@
 require_dependency "on/create"
 
 module On
-  RSpec.describe Create, versioning: true do
+  RSpec.describe Create, :versioning do
     describe "#versions" do
       it "only have a version for the create event" do
         record = described_class.create(name: "Alice")

Diff for: spec/models/on/destroy_spec.rb

@@ -4,7 +4,7 @@
 require_dependency "on/destroy"
 
 module On
-  RSpec.describe Destroy, versioning: true do
+  RSpec.describe Destroy, :versioning do
     describe "#versions" do
       it "only creates one version record, for the destroy event" do
         record = described_class.create(name: "Alice")

Diff for: spec/models/on/empty_array_spec.rb

@@ -4,7 +4,7 @@
 require_dependency "on/empty_array"
 
 module On
-  RSpec.describe EmptyArray, versioning: true do
+  RSpec.describe EmptyArray, :versioning do
     describe "#create" do
       it "does not create any version records" do
         record = described_class.create(name: "Alice")

Diff for: spec/models/on/touch_spec.rb

@@ -4,7 +4,7 @@
 require_dependency "on/create"
 
 module On
-  RSpec.describe Touch, versioning: true do
+  RSpec.describe Touch, :versioning do
     describe "#create" do
       it "does not create a version" do
         record = described_class.create(name: "Alice")

Diff for: spec/models/on/update_spec.rb

@@ -4,7 +4,7 @@
 require_dependency "on/update"
 
 module On
-  RSpec.describe Update, versioning: true do
+  RSpec.describe Update, :versioning do
     describe "#versions" do
       it "only creates one version record, for the update event" do
         record = described_class.create(name: "Alice")

Diff for: spec/models/order_spec.rb

@@ -2,7 +2,7 @@
 
 require "spec_helper"
 
-RSpec.describe Order, versioning: true do
+RSpec.describe Order, :versioning do
   context "when the record destroyed" do
     it "creates a version record for association" do
       customer = Customer.create!

Diff for: spec/models/person_spec.rb

@@ -6,7 +6,7 @@
 #
 # - has a dozen associations of various types
 # - has a custom serializer, TimeZoneSerializer, for its `time_zone` attribute
-RSpec.describe Person, versioning: true do
+RSpec.describe Person, :versioning do
   describe "#time_zone" do
     it "returns an ActiveSupport::TimeZone" do
       person = described_class.new(time_zone: "Samoa")

Diff for: spec/models/pet_spec.rb

@@ -3,7 +3,7 @@
 require "spec_helper"
 require "rails/generators"
 
-RSpec.describe Pet, versioning: true do
+RSpec.describe Pet, :versioning do
   it "baseline test setup" do
     expect(described_class.new).to be_versioned
   end

Diff for: spec/models/plant_spec.rb

@@ -2,7 +2,7 @@
 
 require "spec_helper"
 
-RSpec.describe Plant, versioning: true do
+RSpec.describe Plant, :versioning do
   it "baseline test setup" do
     expect(described_class.new).to be_versioned
     expect(described_class.inheritance_column).to eq("species")

Diff for: spec/models/post_spec.rb

@@ -3,7 +3,7 @@
 require "spec_helper"
 
 # The `Post` model uses a custom version class, `PostVersion`
-RSpec.describe Post, versioning: true do
+RSpec.describe Post, :versioning do
   it "inserts records into the correct table, post_versions" do
     post = described_class.create
     expect(PostVersion.count).to(eq(1))

Diff for: spec/models/skipper_spec.rb

@@ -2,10 +2,10 @@
 
 require "spec_helper"
 
-RSpec.describe Skipper, versioning: true do
+RSpec.describe Skipper, :versioning do
   it { is_expected.to be_versioned }
 
-  describe "#update!", versioning: true do
+  describe "#update!", :versioning do
     context "when updating a skipped attribute" do
       let(:t1) { Time.zone.local(2015, 7, 15, 20, 34, 0) }
       let(:t2) { Time.zone.local(2015, 7, 15, 20, 34, 30) }

Diff for: spec/models/song_spec.rb

@@ -2,7 +2,7 @@
 
 require "spec_helper"
 
-RSpec.describe(Song, versioning: true) do
+RSpec.describe(Song, :versioning) do
   describe "#joins" do
     it "sets event to 'create' when a new record is created" do
       described_class.create!

Diff for: spec/models/thing_spec.rb

@@ -3,7 +3,7 @@
 require "spec_helper"
 
 RSpec.describe Thing do
-  describe "#versions", versioning: true do
+  describe "#versions", :versioning do
     let(:thing) { described_class.create! }
 
     it "applies the scope option" do

Diff for: spec/models/translation_spec.rb

@@ -2,7 +2,7 @@
 
 require "spec_helper"
 
-RSpec.describe Translation, versioning: true do
+RSpec.describe Translation, :versioning do
   context "with non-US translations" do
     it "not change the number of versions" do
       described_class.create!(headline: "Headline")

Diff for: spec/models/vegetable_spec.rb

@@ -5,7 +5,7 @@
 
 if ENV["DB"] == "postgres" && JsonbVersion.table_exists?
   RSpec.describe Vegetable do
-    describe "queries of versions", versioning: true do
+    describe "queries of versions", :versioning do
       let!(:vegetable) { described_class.create(name: "Veggie", mass: 1, color: "green") }
 
       before do