Landscape Client

Code review comment for lp:~simpoir/landscape-client/mocker_to_mock_test_facade

  1. mocker_to_mock_test_facade
  2. Merge into trunk
Revision history for this message
Данило Шеган (danilo) wrote :

You should make use of the wraps=os.dup2 in mock.patch("os.dup2") call. I've got a few other inline suggestions regarding naming.

review: Approve

« Back to merge proposal