Code review comment for lp:~rick-rickspencer3/desktopcouch/couchgrid_id_fix
Revision history for this message
| Rick Spencer (rick-rickspencer3) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
35a32c5
(Get the code!)
> Seems OK to me -- no tests, mind, but I am sure they will come :-)
Thanks Stuart ...
1. There is a manual testing facility built directly into couchgrid. If you run the couchgrid module like a program (python CoughGridp.py) you can do the manual test. I ran the manual test and confirmed CouchGrid still worked and that the bug was fixed.
2. I ran all of the tests in desktop records after this change, and they all passed.
3. I'll add an automated test asap to make sure that future similar regressions are caught.
Cheers, Rick