Use interpolation instead of join in specs

ifad · Sep 22, 2023 · cdd2f04 · cdd2f04
1 parent b7bbcef
commit cdd2f04
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 7 deletions.
diff --git a/spec/chrono_model/adapter/ddl_spec.rb b/spec/chrono_model/adapter/ddl_spec.rb
@@ -7,8 +7,8 @@
   include ChronoTest::Adapter::Helpers
   include ChronoTest::Adapter::Structure
 
-  let(:current) { [ChronoModel::Adapter::TEMPORAL_SCHEMA, table].join('.') }
-  let(:history) { [ChronoModel::Adapter::HISTORY_SCHEMA,  table].join('.') }
+  let(:current) { "#{ChronoModel::Adapter::TEMPORAL_SCHEMA}.#{table}" }
+  let(:history) { "#{ChronoModel::Adapter::HISTORY_SCHEMA}.#{table}" }
 
   def count(table)
     adapter.select_value("SELECT COUNT(*) FROM ONLY #{table}").to_i

diff --git a/spec/chrono_model/time_machine/timestamps_spec.rb b/spec/chrono_model/time_machine/timestamps_spec.rb
@@ -13,7 +13,7 @@
     let(:record) { $t.foo.history.first }
 
     (history_methods + current_methods).each do |attr|
-      describe ['#', attr].join do
+      describe "##{attr}" do
         subject { record.public_send(attr) }
 
         it { is_expected.to be_present }
@@ -27,15 +27,15 @@
     let(:record) { $t.foo }
 
     history_methods.each do |attr|
-      describe ['#', attr].join do
+      describe "##{attr}" do
         subject(:attribute) { record.public_send(attr) }
 
         it { expect { attribute }.to raise_error(NoMethodError) }
       end
     end
 
     current_methods.each do |attr|
-      describe ['#', attr].join do
+      describe "##{attr}" do
         subject { record.public_send(attr) }
 
         it { is_expected.to be_nil }

diff --git a/spec/support/matchers/table.rb b/spec/support/matchers/table.rb
@@ -148,7 +148,7 @@ def has_history_indexes?
                AND tablename  = ?
           SQL
 
-          fqtn = [history_schema, table].join('.')
+          fqtn = "#{history_schema}.#{table}"
 
           expected = [
             "CREATE INDEX index_#{table}_temporal_on_lower_validity ON #{fqtn} USING btree (lower(validity))",
@@ -169,7 +169,7 @@ def has_consistency_constraint?
           binds = {
             conname: connection.timeline_consistency_constraint_name(table),
             connamespace: history_schema,
-            conrelid: [history_schema, table].join('.'),
+            conrelid: "#{history_schema}.#{table}",
             attname: connection.primary_key(table)
           }