Merge lp://staging/~thumper/launchpad/remove-mock-loggers into lp://staging/launchpad

Proposed by Tim Penhey
Status: Merged
Approved by: Tim Penhey
Approved revision: not available
Merged at revision: 12141
Proposed branch: lp://staging/~thumper/launchpad/remove-mock-loggers
Merge into: lp://staging/launchpad
Prerequisite: lp://staging/~thumper/launchpad/move-test-loggers
Diff against target: 3928 lines (+723/-974)
60 files modified
lib/canonical/launchpad/scripts/tests/test_librarian_apache_log_parser.py (+4/-4)
lib/canonical/librarian/ftests/test_gc.py (+20/-34)
lib/lp/archivepublisher/tests/process-in-batches.txt (+6/-8)
lib/lp/archivepublisher/tests/publishing-meta-data-files.txt (+3/-3)
lib/lp/archiveuploader/tests/__init__.py (+0/-38)
lib/lp/archiveuploader/tests/nascentupload-announcements.txt (+122/-121)
lib/lp/archiveuploader/tests/nascentupload-packageset.txt (+5/-5)
lib/lp/archiveuploader/tests/nascentupload.txt (+24/-21)
lib/lp/archiveuploader/tests/nascentuploadfile.txt (+14/-15)
lib/lp/archiveuploader/tests/static-translations.txt (+4/-4)
lib/lp/archiveuploader/tests/test_buildduploads.py (+2/-3)
lib/lp/archiveuploader/tests/test_dscfile.py (+7/-10)
lib/lp/archiveuploader/tests/test_nascentupload.py (+2/-2)
lib/lp/archiveuploader/tests/test_nascentupload_documentation.py (+4/-4)
lib/lp/archiveuploader/tests/test_ppauploadprocessor.py (+0/-2)
lib/lp/archiveuploader/tests/test_recipeuploads.py (+1/-1)
lib/lp/archiveuploader/tests/test_uploadprocessor.py (+7/-28)
lib/lp/archiveuploader/uploadprocessor.py (+1/-1)
lib/lp/blueprints/doc/spec-mail-exploder.txt (+2/-2)
lib/lp/bugs/doc/checkwatches-cli-switches.txt (+5/-5)
lib/lp/bugs/doc/checkwatches.txt (+1/-1)
lib/lp/bugs/doc/sourceforge-remote-products.txt (+0/-1)
lib/lp/code/model/tests/test_recipebuilder.py (+15/-16)
lib/lp/code/model/tests/test_revision.py (+3/-3)
lib/lp/code/model/tests/test_revisionauthor.py (+2/-2)
lib/lp/code/model/tests/test_sourcepackagerecipebuild.py (+1/-1)
lib/lp/code/scripts/tests/test_revisionkarma.py (+3/-3)
lib/lp/codehosting/tests/test_rewrite.py (+4/-4)
lib/lp/hardwaredb/doc/hwdb.txt (+3/-3)
lib/lp/registry/doc/standing.txt (+2/-3)
lib/lp/registry/tests/test_mlists.py (+21/-23)
lib/lp/services/apachelogparser/tests/test_apachelogparser.py (+11/-11)
lib/lp/services/job/tests/test_runner.py (+13/-23)
lib/lp/services/log/logger.py (+32/-18)
lib/lp/services/scripts/tests/test_cronscript_enabled.py (+2/-4)
lib/lp/soyuz/browser/tests/distroseriesqueue-views.txt (+3/-3)
lib/lp/soyuz/doc/build-failedtoupload-workflow.txt (+3/-3)
lib/lp/soyuz/doc/distroarchseriesbinarypackage.txt (+8/-9)
lib/lp/soyuz/doc/distroseriesqueue-ddtp-tarball.txt (+45/-50)
lib/lp/soyuz/doc/distroseriesqueue-debian-installer.txt (+11/-9)
lib/lp/soyuz/doc/distroseriesqueue-dist-upgrader.txt (+48/-48)
lib/lp/soyuz/doc/distroseriesqueue-notify.txt (+50/-49)
lib/lp/soyuz/doc/distroseriesqueue-translations.txt (+20/-20)
lib/lp/soyuz/doc/distroseriesqueue.txt (+7/-6)
lib/lp/soyuz/doc/package-cache.txt (+22/-25)
lib/lp/soyuz/doc/soyuz-set-of-uploads.txt (+44/-44)
lib/lp/soyuz/scripts/tests/test_changeoverride.py (+36/-36)
lib/lp/soyuz/scripts/tests/test_copypackage.py (+27/-27)
lib/lp/soyuz/scripts/tests/test_populatearchive.py (+5/-5)
lib/lp/soyuz/scripts/tests/test_queue.py (+2/-2)
lib/lp/soyuz/scripts/tests/test_soyuzscript.py (+1/-1)
lib/lp/soyuz/scripts/tests/test_sync_source.py (+4/-5)
lib/lp/soyuz/tests/test_publishing.py (+1/-5)
lib/lp/testing/logger.py (+0/-67)
lib/lp/testing/mail_helpers.py (+2/-2)
lib/lp/testing/tests/test_mocklogger.py (+0/-74)
lib/lp/translations/doc/poexport-language-pack.txt (+20/-20)
lib/lp/translations/doc/poexport-request-productseries.txt (+7/-8)
lib/lp/translations/doc/poexport-request.txt (+7/-24)
lib/lp/translations/doc/poimport.txt (+4/-5)
To merge this branch: bzr merge lp://staging/~thumper/launchpad/remove-mock-loggers
Reviewer Review Type Date Requested Status
Tim Penhey (community) Approve
Review via email: mp+44298@code.staging.launchpad.net

Commit message

[no-qa] [r=thumper][ui=none][bug=619108] Remove all MockLoggers and replace with one of FakeLogger, BufferLogger or DevNullLogger.

To post a comment you must log in.
Revision history for this message
Tim Penhey (thumper) wrote :

Self review.

review: Approve
Revision history for this message
Tim Penhey (thumper) wrote :

Just counted, 11 logging classes deleted.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.