Clean up really odd examples of Field::Deferred

thoughtbot · Jul 9, 2020 · fd26c5c · fd26c5c
1 parent d49cbad
commit fd26c5c
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 14 deletions.
diff --git a/spec/lib/fields/deferred_spec.rb b/spec/lib/fields/deferred_spec.rb
@@ -7,16 +7,19 @@
   describe "#permitted_attribute" do
     context "when given a `foreign_key` option" do
       it "returns the value given" do
-        deferred = Administrate::Field::Deferred.
-          new(Administrate::Field::BelongsTo.with_options(foreign_key: :bar))
+        deferred = Administrate::Field::Deferred.new(
+          Administrate::Field::BelongsTo,
+          foreign_key: :bar,
+        )
         expect(deferred.permitted_attribute(LineItem, :foo)).to eq(:bar)
       end
     end
 
     context "when not given a `foreign_key` option" do
       it "delegates to the backing class" do
-        deferred = Administrate::Field::Deferred.
-          new(Administrate::Field::String)
+        deferred = Administrate::Field::Deferred.new(
+          Administrate::Field::String
+        )
         allow(Administrate::Field::String).to receive(:permitted_attribute)
 
         deferred.permitted_attribute(LineItem, :foo)
@@ -30,10 +33,14 @@
   describe "#searchable?" do
     context "when given a `searchable` option" do
       it "returns the value given" do
-        searchable_deferred = Administrate::Field::Deferred.
-          new(double(searchable?: false), searchable: true)
-        unsearchable_deferred = Administrate::Field::Deferred.
-          new(double(searchable?: true), searchable: false)
+        searchable_deferred = Administrate::Field::Deferred.new(
+          double(searchable?: false),
+          searchable: true,
+        )
+        unsearchable_deferred = Administrate::Field::Deferred.new(
+          double(searchable?: true),
+          searchable: false,
+        )
 
         expect(searchable_deferred.searchable?).to eq(true)
         expect(unsearchable_deferred.searchable?).to eq(false)
@@ -42,10 +49,12 @@
 
     context "when not given a `searchable` option" do
       it "falls back to the default of the deferred class" do
-        searchable_deferred = Administrate::Field::Deferred.
-          new(double(searchable?: true))
-        unsearchable_deferred = Administrate::Field::Deferred.
-          new(double(searchable?: false))
+        searchable_deferred = Administrate::Field::Deferred.new(
+          double(searchable?: true),
+        )
+        unsearchable_deferred = Administrate::Field::Deferred.new(
+          double(searchable?: false),
+        )
 
         expect(searchable_deferred.searchable?).to eq(true)
         expect(unsearchable_deferred.searchable?).to eq(false)

diff --git a/spec/lib/fields/has_one_spec.rb b/spec/lib/fields/has_one_spec.rb
@@ -26,8 +26,10 @@
   describe ".permitted_attribute" do
     context "with custom class_name" do
       it "returns attributes from correct dashboard" do
-        field = Administrate::Field::Deferred.new(Administrate::Field::HasOne.
-            with_options(class_name: :product_meta_tag))
+        field = Administrate::Field::Deferred.new(
+          Administrate::Field::HasOne,
+          class_name: :product_meta_tag,
+        )
 
         field_name = "product_meta_tag"
         attributes = field.permitted_attribute(Product, field_name)