Landscape Client

Merge lp://qastaging/~tribaal/landscape-client/remove-mocker-process into lp://qastaging/~landscape/landscape-client/trunk

  1. remove-mocker-process
  2. Merge into trunk
Proposed by Chris Glass
Status: Merged
Merged at revision: 847
Proposed branch: lp://qastaging/~tribaal/landscape-client/remove-mocker-process
Merge into: lp://qastaging/~landscape/landscape-client/trunk
Diff against target: 60 lines (+16/-15)
1 file modified
landscape/lib/tests/test_process.py (+16/-15)
To merge this branch: bzr merge lp://qastaging/~tribaal/landscape-client/remove-mocker-process
Reviewer Review Type Date Requested Status
🤖 Landscape Builder test results Approve
Free Ekanayaka (community) Approve
Benji York (community) Approve
Review via email: mp+297496@code.qastaging.launchpad.net

Commit message

s/mocker/mock/

Description of the change

Replaced some mocking, etc.

Testing instructions:

To post a comment you must log in.
Revision history for this message
Benji York (benji) wrote :

This branch looks good with one small comment inline.

review: Approve
Revision history for this message
🤖 Landscape Builder (landscape-builder) :
review: Abstain (executing tests)
Revision history for this message
🤖 Landscape Builder (landscape-builder) wrote :

Command: make check
Result: Success
Revno: 843
Branch: lp:~tribaal/landscape-client/remove-mocker-process
Jenkins: https://ci.lscape.net/job/latch-test/5030/

review: Approve (test results)
Revision history for this message
Chris Glass (tribaal) :
Revision history for this message
Free Ekanayaka (free.ekanayaka) wrote :

+1

review: Approve
Revision history for this message
🤖 Landscape Builder (landscape-builder) :
review: Abstain (executing tests)
Revision history for this message
🤖 Landscape Builder (landscape-builder) wrote :

Command: make check
Result: Success
Revno: 846
Branch: lp:~tribaal/landscape-client/remove-mocker-process
Jenkins: https://ci.lscape.net/job/latch-test/5037/

review: Approve (test results)
Revision history for this message
🤖 Landscape Builder (landscape-builder) wrote :
Download full text (178.8 KiB)

The attempt to merge lp:~tribaal/landscape-client/remove-mocker-process into lp:landscape-client failed. Below is the output from the failed tests.

python setup.py build_ext -i
running build_ext
landscape.broker.tests.test_amp
  RemoteBrokerTest
    test_call_if_accepted ... [OK]
    test_call_if_accepted_with_not_accepted ... [OK]
    test_call_on_events ... [OK]
    test_exit ... [OK]
    test_fire_event ... [OK]
    test_get_accepted_message_types ... [OK]
    test_get_server_uuid ... [OK]
    test_is_message_pending ... [OK]
    test_listen_events ... [OK]
    test_method_call_error ... [OK]
    test_ping ... [OK]
    test_register ... [OK]
    test_register_client ... [OK]
    test_register_client_accepted_message_type ... [OK]
    test_reload_configuration ... [OK]
    test_send_message ... [OK]
    test_send_message_with_urgent ... [OK]
    test_stop_clients ... [OK]
  RemoteClientTest
    test_exit ... [OK]
    test_fire_event ... [OK]
    test_message ... [OK]
    test_method_call_error ... [OK]
    test_ping ... [OK]
landscape.broker.tests.test_client
  BrokerClientTest
    test_add ... [OK]
    test_dispatch_message ... [OK]
    test_dispatch_message_with_exception ... [OK]
    test_dispatch_message_with_no_handler ... [OK]
    test_exchange ... [OK]
    test_exchange_logs_errors_and_continues ... [OK]
    test_exchange_on_plugin_without_exchange_method ... [OK]
    test_exit ... [OK]
    test_fire_event ... [OK]
    test_fire_event_with_acceptance_changed ... [OK]
    test_fire_event_with_arguments ... [OK]
    test_fire_event_with_mixed_results ... [OK]
    test_get_named_plugin ... [OK]
...

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: