add doc string and fix unit tests

Author: gwang111

src/sagemaker/inference_recommender/inference_recommender_mixin.py

@@ -616,6 +616,7 @@ def _search_recommendation(self, recommendation_list, inference_recommendation_i
         )
 
     def _add_client_type_tag(self, tags, client_type):
+        """Tagging for Inference Recommender and Deployment Recommendations"""
         client_type_tag = {"Key": "ClientType", "Value": client_type}
         tags = tags.append(client_type_tag) if tags else [client_type_tag]
         return tags

src/sagemaker/model.py

@@ -1738,10 +1738,10 @@ def _create_sagemaker_model(self, *args, **kwargs):  # pylint: disable=unused-ar
 
         Args:
             args: Positional arguments coming from the caller. This class does not require
-                any but will specifically look for Tags (3rd arg positionally) if specified
+                any but will look for tags in the 3rd parameter.
 
             kwargs: Keyword arguments coming from the caller. This class does not require
-                any so they are ignored.
+                any but will search for tags if not in args.
         """
         if self.algorithm_arn:
             # When ModelPackage is created using an algorithm_arn we need to first
@@ -1763,13 +1763,17 @@ def _create_sagemaker_model(self, *args, **kwargs):  # pylint: disable=unused-ar
         self._ensure_base_name_if_needed(model_package_name.split("/")[-1])
         self._set_model_name_if_needed()
 
+        # If tags are in args, it must be the 3rd param
+        # If not, then check kwargs and set to either tags or None
+        tags = args[2] if len(args) >= 3 else kwargs.get('tags')
+
         self.sagemaker_session.create_model(
             self.name,
             self.role,
             container_def,
             vpc_config=self.vpc_config,
             enable_network_isolation=self.enable_network_isolation(),
-            tags=args[2],
+            tags=tags,
         )
 
     def _ensure_base_name_if_needed(self, base_name):