Landscape Client

Merge lp://qastaging/~chad.smith/landscape-client/mocker-to-mock-cephusage into lp://qastaging/~landscape/landscape-client/trunk

  1. mocker-to-mock-cephusage
  2. Merge into trunk
Proposed by Chad Smith
Status: Merged
Approved by: Chad Smith
Approved revision: 843
Merged at revision: 843
Proposed branch: lp://qastaging/~chad.smith/landscape-client/mocker-to-mock-cephusage
Merge into: lp://qastaging/~landscape/landscape-client/trunk
Diff against target: 34 lines (+7/-6)
1 file modified
landscape/monitor/tests/test_cephusage.py (+7/-6)
To merge this branch: bzr merge lp://qastaging/~chad.smith/landscape-client/mocker-to-mock-cephusage
Reviewer Review Type Date Requested Status
Chad Smith Abstain
Benji York (community) Approve
Geoff Teale (community) Approve
Review via email: mp+297488@code.qastaging.launchpad.net

Commit message

Use mock instead of mocker for cephusage unit tests.

Description of the change

Mock our logging check for cephusage.

To post a comment you must log in.
Revision history for this message
Geoff Teale (tealeg) wrote :

+1 Chadwick.

review: Approve
Revision history for this message
Benji York (benji) wrote :

Looks great.

review: Approve
Revision history for this message
Chad Smith (chad.smith) wrote :

Not only do I like the branch, but I like the author. +1

Revision history for this message
Chad Smith (chad.smith) :
review: Abstain

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.

Subscribers

People subscribed via source and target branches

to all changes: