move _check_load_parameters

Author: zancas

f5/bigip/resource.py

@@ -212,18 +212,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 +873,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')