Code review comment for lp:~nskaggs/notes-app/ap-setup-update

Revision history for this message
Martin Pitt (pitti) wrote :

Mostly working now; the AP tests succeed, but autopkgtest fails on having a ton of stderr, due to several exceptions and the fake web server log: http://paste.ubuntu.com/7830998/

I think these are all just from the fake web server, so probably its output should be directed to /dev/null, or perhaps to a log file which you print on a test failure, or just generally put into $ADT_ARTIFACTS/web-server.log?

Alternatively, add the "allow-stderr" restriction.

Thanks!

review: Needs Fixing

« Back to merge proposal