-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31350/
-----------------------------------------------------------

(Updated Feb. 24, 2015, 10:52 p.m.)


Review request for Aurora.


Changes
-------

Review changes
    
* use pytest.raises to ignore the exception
* start with a non-empty cluster
* assert clusters content instead of size (as far as possible, given what the 
class offers)


Repository: aurora


Description
-------

Fix clusters.patch contextmanager cleanup

Otherwise one failing testcase can affect the entire suite.


Diffs (updated)
-----

  src/main/python/apache/aurora/common/clusters.py 
c4b7fefca30313b281808478bf23158a9b7fbf85 
  src/test/python/apache/aurora/common/test_clusters.py 
1bd696e9cd28d87d0cac68b33ab043407d796b61 

Diff: https://reviews.apache.org/r/31350/diff/


Testing
-------

./pants test.pytest --no-fast src/test/python/apache/aurora/common::


Thanks,

Stephan Erb

Reply via email to