Mark transaction as written if non-read SQL performed (#897)

Co-authored-by: Aidan Haran <aharan@fusioneer.com>

Author: aidanharan

lib/active_record/connection_adapters/sqlserver/database_statements.rb

@@ -17,6 +17,7 @@ def execute(sql, name = nil)
           end
 
           materialize_transactions
+          mark_transaction_written_if_write(sql)
 
           if id_insert_table_name = query_requires_identity_insert?(sql)
             with_identity_insert_enabled(id_insert_table_name) { do_execute(sql, name) }
@@ -31,6 +32,7 @@ def exec_query(sql, name = "SQL", binds = [], prepare: false)
           end
 
           materialize_transactions
+          mark_transaction_written_if_write(sql)
 
           sp_executesql(sql, name, binds, prepare: prepare)
         end
@@ -291,6 +293,7 @@ def set_identity_insert(table_name, enable = true)
 
         def do_execute(sql, name = "SQL")
           materialize_transactions
+          mark_transaction_written_if_write(sql)
 
           log(sql, name) { raw_connection_do(sql) }
         end

test/cases/execute_procedure_test_sqlserver.rb

@@ -41,4 +41,13 @@ class ExecuteProcedureTestSQLServer < ActiveRecord::TestCase
     date_base = connection.select_value("select GETUTCDATE()")
     assert_equal date_base.change(usec: 0), date_proc.change(usec: 0)
   end
+
+  it 'test deprecation with transaction return when executing procedure' do
+    assert_deprecated do
+      ActiveRecord::Base.transaction do
+        connection.execute_procedure("my_getutcdate")
+        return
+      end
+    end
+  end
 end