From bd6e36f6b3133516982fe62ec43c7a16aca86831 Mon Sep 17 00:00:00 2001 From: Aidan Haran Date: Thu, 13 Jun 2024 16:14:07 +0100 Subject: [PATCH] Fix missing assertion warning --- test/cases/execute_procedure_test_sqlserver.rb | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/test/cases/execute_procedure_test_sqlserver.rb b/test/cases/execute_procedure_test_sqlserver.rb index 9fa8df0df..d7f58965c 100644 --- a/test/cases/execute_procedure_test_sqlserver.rb +++ b/test/cases/execute_procedure_test_sqlserver.rb @@ -42,12 +42,16 @@ class ExecuteProcedureTestSQLServer < ActiveRecord::TestCase assert_equal date_base.change(usec: 0), date_proc.change(usec: 0) end + def transaction_with_procedure_and_return + ActiveRecord::Base.transaction do + connection.execute_procedure("my_getutcdate") + return + end + end + it 'test deprecation with transaction return when executing procedure' do - assert_deprecated(ActiveRecord.deprecator) do - ActiveRecord::Base.transaction do - connection.execute_procedure("my_getutcdate") - return - end + assert_not_deprecated(ActiveRecord.deprecator) do + transaction_with_procedure_and_return end end end