I’ve noticed that Rails was not properly escaping the name of a table when dropping an index which caused it to generate invalid SQL when the table was named “values”. I’ve reported and then made a patch for it: https://rails.lighthouseapp.com/projects/8994-ruby-on-rails/tickets/4809
The patch is in the issue and you can also see my change in here: http://github.com/pupeno/rails/commit/13f9869896f5b39dd3558a2938ffedb2f0eb9e9d
I wrote a test for it but I’m not sure if that’s the proper style of testing; actually, I’m quite sure it isn’t, but since I need to create a separate table for this test I wasn’t sure what was the best approach. Anyone cares to tell me so I can improve this patch?