Skip to content

Make sure that Arel.sql would work for update_column_in_batches

Lin Jen-Shin requested to merge test-update_column_in_batches-with-sql-mysql into master

What does this MR do?

Add a test for update_column_in_batches with Arel.sql

Why was this MR needed?

We don't seem to test this before. Also note that if we're passing '1+1' as a literal string to update_column_in_batches, it would end up with '1+1'.to_i therefore 1!

Arel should really raise an exception but it doesn't!

So we can't really determine if we should wrap with Arel.sql or it's really meant to be a string.

Merge request reports