Code review comment for lp:~edwin-grubbs/launchpad/bug-655802-person-vocabulary-timeout

Revision history for this message
Curtis Hovey (sinzui) wrote :

Thanks for working on this. I think this is good to land, but I have a one question/remark. I do not understand why the test is on the LaunchpadFunctionalLayer? I see database objects, but no library objects in the test. I think the test will run on the DatabaseFunctionalLayer.

review: Approve (code)

« Back to merge proposal