-
Notifications
You must be signed in to change notification settings - Fork 21.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove unneccessary special case for money in quoting #16037
Conversation
Tests are broken |
Updated |
@@ -45,14 +45,6 @@ def test_quote_float_infinity | |||
assert_equal "'Infinity'", @conn.quote(infinity, c) | |||
end | |||
|
|||
def test_quote_cast_numeric | |||
fixnum = 666 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removing this test doesn't mean that now integer are not being quoted when the column type is a string? I guess this would cause an SQL error. Does we have tests to ensure it is working?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, I don't think there's a test for this. It's perfectly valid to provide a number for a string column, so I felt like we would be testing PG not Rails. I'll provide a test if you think we should, though.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hmm, if it is perfect valid why we were quoting? I think is better to add a test to make sure it works.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No clue why were quoting. I'll add a test...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Test added.
This should be good, now. |
Remove unneccessary special case for money in quoting
No description provided.