test/unit/ardb_tests.rb in ardb-0.29.1 vs test/unit/ardb_tests.rb in ardb-0.29.2

@@ -1,5 +1,7 @@
+# frozen_string_literal: true
+
 require "assert"
 require "ardb"
 
 require "logger"
 require "ardb/adapter_spy"
@@ -112,11 +114,14 @@
 
     should "set ActiveRecord::Base attributes" do
       subject.init
 
       assert_equal subject.config.logger, ActiveRecord::Base.logger
-      assert_equal subject.config.default_timezone, ActiveRecord::Base.default_timezone
+      assert_equal(
+        subject.config.default_timezone,
+        ActiveRecord::Base.default_timezone,
+      )
     end
 
     should "optionally establish an AR connection" do
       assert_nil @ardb_adapter
       subject.init
@@ -130,11 +135,13 @@
     end
 
     should "raise a not initialized error using its adapter before init" do
       subject.reset_adapter
       assert_raises(NotInitializedError){ subject.adapter }
-      assert_raises(NotInitializedError){ subject.escape_like_pattern(Factory.string) }
+      assert_raises(NotInitializedError) do
+        subject.escape_like_pattern(Factory.string)
+      end
 
       subject.init
       assert_nothing_raised{ subject.adapter }
       assert_nothing_raised{ subject.escape_like_pattern(Factory.string) }
     end
@@ -169,11 +176,11 @@
         :port,
         :username,
         :password,
         :pool,
         :checkout_timeout,
-        :min_messages
+        :min_messages,
       ]
       assert_equal exp, subject::ACTIVERECORD_ATTRS
     end
 
     should "know its default migrations path" do
@@ -197,23 +204,31 @@
     setup do
       @config = @config_class.new
     end
     subject{ @config }
 
-    should have_accessors *Ardb::Config::ACTIVERECORD_ATTRS
+    should have_accessors(*Ardb::Config::ACTIVERECORD_ATTRS)
     should have_accessors :default_timezone, :logger, :root_path
     should have_readers :schema_format
     should have_writers :migrations_path, :schema_path
     should have_imeths :activerecord_connect_hash, :validate!
 
     should "default its attributs" do
       assert_equal :utc, subject.default_timezone
       assert_instance_of Logger, subject.logger
       assert_equal ENV["PWD"], subject.root_path
-      exp = File.expand_path(@config_class::DEFAULT_MIGRATIONS_PATH, subject.root_path)
+      exp =
+        File.expand_path(
+          @config_class::DEFAULT_MIGRATIONS_PATH,
+          subject.root_path,
+        )
       assert_equal exp, subject.migrations_path
-      exp = File.expand_path(@config_class::DEFAULT_SCHEMA_PATH, subject.root_path)
+      exp =
+        File.expand_path(
+          @config_class::DEFAULT_SCHEMA_PATH,
+          subject.root_path,
+        )
       assert_equal exp, subject.schema_path
       assert_equal @config_class::RUBY_SCHEMA_FORMAT, subject.schema_format
     end
 
     should "allow reading/writing its paths" do
@@ -248,15 +263,15 @@
       subject.schema_format = new_schema_format
       assert_equal new_schema_format.to_sym, subject.schema_format
     end
 
     should "know its activerecord connection hash" do
-      attrs_and_values = @config_class::ACTIVERECORD_ATTRS.map do |attr_name|
-        value = [Factory.string,  nil].sample
+      attrs_and_values = @config_class::ACTIVERECORD_ATTRS.map{ |attr_name|
+        value = [Factory.string, nil].sample
         subject.send("#{attr_name}=", value)
-        [attr_name.to_s, value] if !value.nil?
-      end.compact
+        [attr_name.to_s, value] unless value.nil?
+      }.compact
       assert_equal Hash[attrs_and_values], subject.activerecord_connect_hash
     end
 
     should "raise errors with invalid attribute values using `validate!`" do
       subject.adapter  = Factory.string
@@ -283,11 +298,11 @@
         :default_timezone,
         :logger,
         :root_path,
         :schema_format,
         :migrations_path,
-        :schema_path
+        :schema_path,
       ]
       attrs.each do |attr_name|
         subject.send("#{attr_name}=", Factory.string)
       end
 
@@ -322,19 +337,19 @@
         "sqlite",
         "sqlite3",
         "postgresql",
         "postgres",
         "mysql",
-        "mysql2"
+        "mysql2",
       ]
       assert_equal exp, subject::VALID_ADAPTERS
     end
 
     should "build an adapter specific class using the passed config" do
       adapter_key, exp_adapter_class = [
         ["sqlite",     Ardb::Adapter::Sqlite],
         ["postgresql", Ardb::Adapter::Postgresql],
-        ["mysql",      Ardb::Adapter::Mysql]
+        ["mysql",      Ardb::Adapter::Mysql],
       ].sample
       @config.adapter = adapter_key
 
       adapter = subject.new(@config)
       assert_instance_of exp_adapter_class, adapter