From d47c2c794f4c5c1c7480610db50b7218707a1686 Mon Sep 17 00:00:00 2001 From: Ian Cordasco Date: Wed, 26 Oct 2016 11:58:56 -0500 Subject: [PATCH] Add tests for our Region and RegionManager classes Minimalistic though they might be, it makes sense to test everything we can think of, including these two classes in cratonclient.v1.regions. If we build up our testing from first principles there are things we don't need to test about these classes. For example, we've already written a number of tests around both cratonclient.crud.CRUDManager and cratonclient.crud.Resource so those tests need not be duplicated here. Change-Id: Ia5c0c8c55d78aa5c78e063163bc803525f96ed34 --- cratonclient/tests/unit/test_regions.py | 38 +++++++++++++++++++++++++ 1 file changed, 38 insertions(+) create mode 100644 cratonclient/tests/unit/test_regions.py diff --git a/cratonclient/tests/unit/test_regions.py b/cratonclient/tests/unit/test_regions.py new file mode 100644 index 0000000..969f596 --- /dev/null +++ b/cratonclient/tests/unit/test_regions.py @@ -0,0 +1,38 @@ +# Licensed under the Apache License, Version 2.0 (the "License"); you may +# not use this file except in compliance with the License. You may obtain +# a copy of the License at +# +# http://www.apache.org/licenses/LICENSE-2.0 +# +# Unless required by applicable law or agreed to in writing, software +# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT +# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the +# License for the specific language governing permissions and limitations +# under the License. +"""Tests for `cratonclient.v1.regions` module.""" + +from cratonclient import crud +from cratonclient.tests import base +from cratonclient.v1 import regions + +import mock + + +class TestRegion(base.TestCase): + """Tests for the Region Resource.""" + + def test_is_a_resource_instance(self): + """Verify that a Region instance is an instance of a Resource.""" + manager = mock.Mock() + self.assertIsInstance(regions.Region(manager, {}), + crud.Resource) + + +class TestRegionManager(base.TestCase): + """Tests for the RegionManager class.""" + + def test_is_a_crudclient(self): + """Verify our RegionManager is a CRUDClient.""" + session = mock.Mock() + region_mgr = regions.RegionManager(session, '') + self.assertIsInstance(region_mgr, crud.CRUDClient)