projectatomic / commissaire-mvp

A lightweight REST interface for upgrading, restarting, and bootstrapping new hosts into an existing Container Management cluster.
http://commissaire.readthedocs.org/en/latest/
GNU General Public License v3.0
15 stars 9 forks source link

[merged] test: Tests for model defaults. #167

Closed ashcrow closed 8 years ago

ashcrow commented 8 years ago

Closes #147.

mbarnes commented 8 years ago

Very nice!

It looks like we could factor out a common helper function next time we add more subset tests, like for a new model class. They all follow the same pattern.

def test_subsets(model_class, kwargs):
mbarnes commented 8 years ago

@rh-atomic-bot r+

rh-atomic-bot commented 8 years ago

:pushpin: Commit e6e71db has been approved by mbarnes

rh-atomic-bot commented 8 years ago

:hourglass: Testing commit e6e71db with merge 41c2981...

rh-atomic-bot commented 8 years ago

:sunny: Test successful - travis Approved by: mbarnes Pushing 41c2981837e60e1d8a4eaef465be53dc3b5eb21b to master...