draft 1 finished

Author: zancas

f5/bigip/resource.py

@@ -217,7 +217,6 @@ def __init__(self, container):
         self._set_meta_data_uri()
         # Supported versions for each class will be defined here.
         # List can be modified downstream in each sub-class
-        print('about to set the "minimum_version" for %s' % self.__class__.__name__)
         self._meta_data['minimum_version'] = '11.5.0'
         # Commands you can run on a resource or collection, we define it here
         self._meta_data['allowed_commands'] = []
@@ -1014,5 +1013,3 @@ def modify(self, **kwargs):
         raise UnsupportedMethod(
             "%s does not support the modify method" % self.__class__.__name__
         )
-
-    pass

f5/bigip/test/test_resource.py

@@ -781,15 +781,23 @@ def test_modify_raises(self):
     def test_load(self):
         r = Virtual(mock.MagicMock())
         r._meta_data['allowed_lazy_attributes'] = []
-        mockuri = "https://localhost:443/mgmt/tm/ltm/virtual"
-        attrs = {'get.return_value':
-                 MockResponse(
+        attrs = {'get.side_effect':
+                 [MockResponse(
                      {
                          u"generation": 0,
-                         u"selfLink": mockuri,
+                         u"selfLink": "https://localhost:443"
+                            "/mgmt/tm/ltm/virtual",
                          u"kind": u"tm:ltm:virtual:virtualstate"
-                     }
-                 )}
+                     }),
+                  MockResponse(
+                     {
+                         u"generation": 0,
+                         u"selfLink": u'https://localhost'
+                            '/mgmt/tm/ltm/virtual/'
+                            '~Common~modtest1/stats?ver=11.6.0',
+                         u"kind": u"tm:ltm:virtual:virtualstats"
+                     })
+                  ]}
         mock_session = mock.MagicMock(**attrs)
         r._meta_data['bigip']._meta_data =\
             {'icr_session': mock_session,
@@ -801,5 +809,9 @@ def test_load(self):
         x = r.load(partition='Common', name='test_load')
         assert x.selfLink == 'https://localhost:443/mgmt/tm/ltm/virtual'
         assert x._meta_data['allowed_lazy_attributes'] == [Profiles_s, Stats]
-        pp(x.raw)
-        x.stats
+        statsfactory = x.stats
+        assert 'selfLink' not in statsfactory.__dict__
+        statsobj = statsfactory.load()
+        assert statsobj.selfLink ==\
+            u'https://localhost/mgmt/tm/ltm/virtual/'\
+                '~Common~modtest1/stats?ver=11.6.0'

test/functional/tm/ltm/test_virtual.py

@@ -83,6 +83,7 @@ def test_stats(self, request, mgmt_root, setup_device_snapshot):
             request, mgmt_root, 'Common', 'modtest1'
         )
         stats = virtual1.stats.load()
+        pp(stats.raw)
         assert virtual1.cmpEnabled == u'yes'
         assert stats.entries['cmpEnabled']['description'] == u'enabled'
         virtual1.modify(cmpEnabled=u'no')