aurora-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bill Farner" <>
Subject Re: Review Request 26308: Fix exit condition for RPC loop, fix test_status_api_failure test.
Date Fri, 03 Oct 2014 04:52:09 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated Oct. 3, 2014, 4:52 a.m.)

Review request for Aurora, Kevin Sweeney and Mark Chu-Carroll.

Repository: aurora


Fixes two problems:

- mocking was incorrect in `test_status_api_failure`.  Turns out that Mock objects were being
passed around and somehow resulted in the test case passing.
- use of `threading.Event()` was broken in  I don't think it's possible
to enter those branches.

Diffs (updated)

  src/main/python/apache/aurora/client/api/ b400cb2dbdb35077fc2c4a6e161c2959a9217317

  src/test/python/apache/aurora/client/api/ 1cbfbf86e903d890baac7d34461109f9beaff442

  src/test/python/apache/aurora/client/cli/ 78f21d2f20cf71fa2dfe0614885d44d2948decd2



./pants src/test/python:all -vxs


Bill Farner

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message