Code review comment for lp:~mvo/launchpad/support-timeframe-fix-660433

Revision history for this message
Julian Edwards (julian-edwards) wrote :

167 +# mvo: I would love to use this, but it appears doing this import
168 +# requires python-psycopg2, python-storm, python-transaction,
169 +# python-lazr.restful and now "windmill" that is not packaged
170 +# at this point I give up and let someone else with a proper
171 +# LP environment fix this
172 +#from canonical.testing.layers import DatabaseFunctionalLayer

In an ideal world every test would not use any layers because it's massively quicker to run the tests. However, I can't see any reason that your tests need layers so I think it's fine to remove this commented block entirely.

« Back to merge proposal