Code review comment for lp:~xaav/loggerhead/export-tarball

Revision history for this message
Gavin Panella (allenap) wrote :

Once I'd set up a virtualenv with the right prerequisites, I got the
following error when running the test suite:

Traceback (most recent call last):
  File ".../loggerhead/tests/", line 8, in <module>
    from loggerhead.apps.branch import BranchWSGIApp
  File ".../loggerhead/apps/", line 36, in <module>
    from loggerhead.controllers.download_ui import DownloadUI, DownloadTarballUI
  File ".../loggerhead/controllers/", line 29, in <module>
    from loggerhead.exporter import export_tarball
ImportError: cannot import name export_tarball

After fixing that I got the following error from

Traceback (most recent call last):
  File "/usr/lib/python2.7/dist-packages/testtools/", line 169, in _run_user
    return fn(*args, **kwargs)
  File "/usr/lib/python2.7/dist-packages/testtools/", line 499, in _run_test_method
    return self._get_test_method()()
  File ".../loggerhead/tests/", line 135, in test_download_tarball
    app = self.setUpLoggerhead()
  File ".../loggerhead/tests/", line 47, in setUpLoggerhead
    branch_app = BranchWSGIApp(self.tree.branch, '', **kw).app
AttributeError: 'TestDownloadTarballUI' object has no attribute 'tree'

Obviously this needs some work.

We've been talking about taking more of a "patch pilot" approach in
Launchpad. That seems to mean that one of the core team - fwiw, I
would be happy to do it - would actively help getting this landed,
rather than just reviewing it. Would you like that, or would you
prefer to iterate on your own?

review: Needs Fixing

« Back to merge proposal