Merge pull request #619 from zancas/feature.test_client_ssl_modification_564

Add modify tests to all profiles.

Author: pjbreaux

f5/bigip/resource.py

@@ -14,6 +14,7 @@
 #   See the License for the specific language governing permissions and
 #   limitations under the License.
 #
+
 """This module provides classes that specify how RESTful resources are handled.
 
 THE MOST IMPORTANT THING TO KNOW ABOUT THIS API IS THAT YOU CAN DIRECTLY INFER
@@ -212,18 +213,6 @@ def _set_meta_data_uri(self):
             self._meta_data['container']._meta_data['uri'] + endpoint + '/'
         self._meta_data['uri'] = final_uri
 
-    def _check_load_parameters(self, **kwargs):
-        """Params given to load should at least satisfy required params.
-
-        :params: kwargs
-        :raises: MissingRequiredReadParameter
-        """
-        rset = self._meta_data['required_load_parameters']
-        check = self._missing_required_parameters(rset, **kwargs)
-        if check:
-            error_message = 'Missing required params: %s' % check
-            raise MissingRequiredReadParameter(error_message)
-
     def _check_create_parameters(self, **kwargs):
         """Params given to create should satisfy required params.
 
@@ -885,6 +874,18 @@ def create(self, **kwargs):
         """
         return self._create(**kwargs)
 
+    def _check_load_parameters(self, **kwargs):
+        """Params given to load should at least satisfy required params.
+
+        :params: kwargs
+        :raises: MissingRequiredReadParameter
+        """
+        rset = self._meta_data['required_load_parameters']
+        check = self._missing_required_parameters(rset, **kwargs)
+        if check:
+            error_message = 'Missing required params: %s' % check
+            raise MissingRequiredReadParameter(error_message)
+
     def _load(self, **kwargs):
         """wrapped with load, override that in a subclass to customize"""
         if 'uri' in self._meta_data:

f5/bigip/test/test_resource.py

@@ -684,7 +684,7 @@ def test_missing_req_param_false(self):
         assert not mrq
 
     def test_check_load_parameters_fail(self):
-        p = PathElement(mock.MagicMock())
+        p = Resource(mock.MagicMock())
         p._meta_data['required_load_parameters'] = set(['FAKELOAD'])
         with pytest.raises(MissingRequiredReadParameter) as RLPEIO:
             p._check_load_parameters(FOOLOAD='FOOVAL')