Fix tests

Author: aidanharan

lib/active_record/connection_adapters/sqlserver/database_statements.rb

@@ -78,14 +78,15 @@ def delete(arel, name = nil, binds = [])
 
           # Add `SELECT @@ROWCOUNT` to the end of the SQL to get the number of affected rows. This is needed because SQL Server does not return the number of affected rows in the same way as other databases.
           sql = intent.processed_sql.present? ? intent.processed_sql : intent.raw_sql
+          ensure_writes_are_allowed(sql) if write_query?(sql)
           intent.processed_sql = "#{sql}; SELECT @@ROWCOUNT AS AffectedRows"
 
           affected_rows(raw_execute(intent))
         end
 
         # Executes the update statement and returns the number of rows affected.
         def update(arel, name = nil, binds = [])
-          # Clear query cache if the connection pool is configured to do so. 
+          # Clear query cache if the connection pool is configured to do so.
           if pool.dirties_query_cache
             ActiveRecord::Base.clear_query_caches_for_current_thread
           end
@@ -97,6 +98,7 @@ def update(arel, name = nil, binds = [])
 
           # Add `SELECT @@ROWCOUNT` to the end of the SQL to get the number of affected rows. This is needed because SQL Server does not return the number of affected rows in the same way as other databases.
           sql = intent.processed_sql.present? ? intent.processed_sql : intent.raw_sql
+          ensure_writes_are_allowed(sql) if write_query?(sql)
           intent.processed_sql = "#{sql}; SELECT @@ROWCOUNT AS AffectedRows"
 
           affected_rows(raw_execute(intent))

test/support/query_assertions.rb

@@ -14,7 +14,7 @@ def assert_queries_count(count = nil, include_schema: false, &block)
           # End of monkey-patch
 
           if count
-            assert_equal count, queries.size, "#{queries.size} instead of #{count} queries were executed#{queries.empty? ? '' : ". Queries:\n\n#{queries.join("\n\n")}"}"
+            assert_equal count, queries.size, "#{queries.size} instead of #{count} queries were executed#{". Queries:\n\n#{queries.join("\n\n")}" unless queries.empty?}"
           else
             assert_operator queries.size, :>=, 1, "1 or more queries expected, but none were executed"
           end