aspidites/ViCE — pull request #19 by Gallaecio (2012-07-20 20:43:52+00:00)
Title: rename() method for colums and kargs

Hi,

I gave it a try and wrote a method to rename a column.

For simplicity (and lack of any knowledge around SQL Alchemy), I used SQL Alchemy Migrate. I hope you do not mind this additional dependency, but if you do I can try to approach the issue again without it (should not be that hard studying Migrate’s source code).

About the name of the function, I chose just “rename” (as opposed to “renameColumn”) because I think it could be nice in the future to define the method with a variable number of properties (kargs) and just do one thing or the other depending on the given parameters.

I had never written a test before, so I decided to write one before actually writting the method, so I could check whether it was working or not. I like the fact that you are pro unit tests. I took the chance to fix an issue that was causing a couple of failed tests here (kargs argument syntax), I hope it was fine.

Note: this commit covers one of the requested features in the issue #9. It would be great if you could specify some more “requirements” there :)