[Fixes trilogy-libraries#54] Apply connection configuration when conn…

…ecting
lorint · Jul 19, 2023 · b654761 · b654761
1 parent c85a2b2
commit b654761
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 0 deletions.
diff --git a/lib/active_record/connection_adapters/trilogy_adapter.rb b/lib/active_record/connection_adapters/trilogy_adapter.rb
@@ -297,6 +297,7 @@ def error_number(exception)
 
         def connect
           self.connection = self.class.new_client(@config)
+          configure_connection
         end
 
         def reconnect

diff --git a/test/lib/active_record/connection_adapters/trilogy_adapter_test.rb b/test/lib/active_record/connection_adapters/trilogy_adapter_test.rb
@@ -874,6 +874,12 @@ class Post < ActiveRecord::Base; end
     assert adapter.active?
   end
 
+  test "sql_mode is applied correctly" do
+    adapter = trilogy_adapter(variables: { sql_mode: "STRICT_ALL_TABLES" })
+    current_sql_mode = adapter.execute('SELECT @@SESSION.sql_mode;').rows.first
+    assert_equal current_sql_mode, ["STRICT_ALL_TABLES"]
+  end
+
   def trilogy_adapter_with_connection(connection, **config_overrides)
     ActiveRecord::ConnectionAdapters::TrilogyAdapter
       .new(connection, nil, {}, @configuration.merge(config_overrides))