Bump sphinx-autoapi from 3.0.0 to 3.1.0 (#15114)

(cherry picked from commit 7702a84)

pytest_fixtures/component/repository.py

@@ -131,7 +131,7 @@ class constructor arguments. It can create multiple repos with distro.
     ]
     Then the fixtures loop over it to create multiple repositories.
 
-    :returns: The tuple of distro of repositories(if given) and simplified repos
+    :return: The tuple of distro of repositories(if given) and simplified repos
     """
     _repos = []
     repo_distro = None

requirements-optional.txt

@@ -5,8 +5,8 @@ redis==5.0.4
 pre-commit==3.7.1
 
 # For generating documentation.
-sphinx==7.3.6
-sphinx-autoapi==3.0.0
+sphinx==7.3.7
+sphinx-autoapi==3.1.0
 
 # For 'manage' interactive shell
 manage==0.1.15

robottelo/cli/base.py

@@ -36,7 +36,7 @@ def _handle_response(cls, response, ignore_stderr=None):
         :param response: a result object, returned by :mod:`robottelo.utils.ssh.command`.
         :param ignore_stderr: indicates whether to throw a warning in logs if
             ``stderr`` is not empty.
-        :returns: contents of ``stdout``.
+        :return: contents of ``stdout``.
         :raises robottelo.exceptions.CLIReturnCodeError: If return code is
             different from zero.
         """

robottelo/host_helpers/api_factory.py

@@ -168,7 +168,7 @@ def one_to_one_names(self, name):
             True
 
         :param name: A field name.
-        :returns: A set including both ``name`` and variations on ``name``.
+        :return: A set including both ``name`` and variations on ``name``.
 
         """
         return {f'{name}_name', f'{name}_id'}

robottelo/host_helpers/repository_mixins.py

@@ -588,7 +588,7 @@ def add_item(self, item) -> None:
         Add repository to collection
 
         :param BaseRepository item: Item to add
-        :returns: None
+        :return: None
         """
         if self._repos_info:
             raise RepositoryAlreadyCreated('Repositories already created can not add more')
@@ -605,7 +605,7 @@ def add_items(self, items):
         Add multiple repositories to collection
 
         :param List[BaseRepository] items: Items to add
-        :returns: None
+        :return: None
         """
         for item in items:
             self.add_item(item)

robottelo/host_helpers/satellite_mixins.py

@@ -142,7 +142,7 @@ def upload_manifest(self, org_id, manifest=None, interface='API', timeout=None):
         :type interface: str
         :type timeout: int
 
-        :returns: the manifest upload result
+        :return: the manifest upload result
 
         """
         if not isinstance(manifest, bytes | io.BytesIO) and (
@@ -176,7 +176,7 @@ def is_sca_mode_enabled(self, org_id):
         given organization.
 
         :param str org_id: The unique identifier of the organization to check for SCA mode.
-        :returns: A boolean value indicating whether SCA mode is enabled or not.
+        :return: A boolean value indicating whether SCA mode is enabled or not.
         :rtype: bool
         """
         return self.api.Organization(id=org_id).read().simple_content_access
@@ -187,7 +187,7 @@ def publish_content_view(self, org, repo_list):
         :param str org: The name of the organization to which the content view belongs
         :param list or str repo_list:  A list of repositories or a single repository
 
-        :returns: A dictionary containing the details of the published content view.
+        :return: A dictionary containing the details of the published content view.
         """
         repo = repo_list if isinstance(repo_list, list) else [repo_list]
         content_view = self.api.ContentView(organization=org, repository=repo).create()

robottelo/hosts.py

@@ -339,7 +339,7 @@ def os_distro(self):
     def os_version(self):
         """Get host's OS version information
 
-        :returns: A ``packaging.version.Version`` instance
+        :return: A ``packaging.version.Version`` instance
         """
         return Version(self._os_release['VERSION_ID'])
 
@@ -472,7 +472,7 @@ def download_file(self, file_url, local_path=None, file_name=None):
             provided file will be saved in /tmp/ directory.
         :param str file_name: New name of the Downloaded file else its given from file_url
 
-        :returns: Returns list containing complete file path and name of downloaded file.
+        :return: Returns list containing complete file path and name of downloaded file.
         """
         file_name = PurePath(file_name or file_url).name
         local_path = PurePath(local_path or '/tmp') / file_name
@@ -1497,7 +1497,7 @@ def ping_host(self, host):
         """Check the provisioned host status by pinging the ip of host
 
         :param host: IP address or hostname of the provisioned host
-        :returns: None
+        :return: None
         :raises: : `HostPingFailed` if the host is not pingable
         """
         result = self.execute(

robottelo/utils/datafactory.py

@@ -82,7 +82,7 @@ def generate_strings_list(length=None, exclude_types=None, min_length=3, max_len
         generated list. example: exclude_types=['html', 'cjk']
     :param int min_length: Minimum length to be used in integer generator
     :param int max_length: Maximum length to be used in integer generator
-    :returns: A list of various string types.
+    :return: A list of various string types.
 
     """
     if length is None:

robottelo/utils/report_portal/portal.py

@@ -43,16 +43,12 @@ def __init__(self, rp_url=None, rp_api_key=None, rp_project=None):
 
     @property
     def api_url(self):
-        """Super url of report portal
-        :returns: Base url for API request
-        """
+        """Super url of report portal API."""
         return f'{self.rp_url}/api/v1/{self.rp_project}'
 
     @property
     def headers(self):
-        """The headers for Report Portal Requests.
-        :returns: header for API request
-        """
+        """The headers for Report Portal Requests."""
         return {'Authorization': f'Bearer {self.rp_api_key}'}
 
     def get_launches(

tests/foreman/api/test_notifications.py

@@ -156,7 +156,7 @@ def wait_for_no_long_running_task_mail(target_sat, clean_root_mailbox, long_runn
 def root_mailbox_copy(target_sat, clean_root_mailbox):
     """Parsed local system copy of the Satellite's root user mailbox.
 
-    :returns: :class:`mailbox.mbox` instance
+    :return: :class:`mailbox.mbox` instance
     """
     result = target_sat.execute(f'cat {clean_root_mailbox}')
     assert result.status == 0, f'Could not read mailbox {clean_root_mailbox} on Satellite host.'

tests/foreman/api/test_permission.py

@@ -218,7 +218,7 @@ def give_user_permission(self, perm_name, target_sat):
             searching for the permission with name ``perm_name``.
         :raises: ``requests.exceptions.HTTPError`` if an error occurs when
             updating ``self.user``'s roles.
-        :returns: Nothing.
+        :return: Nothing.
         """
         role = target_sat.api.Role().create()
         permissions = target_sat.api.Permission().search(query={'search': f'name="{perm_name}"'})

tests/foreman/cli/test_oscap.py

@@ -35,7 +35,7 @@ def fetch_scap_and_profile_id(cls, scap_name, sat):
 
         :param scap_name: Scap title
 
-        :returns: scap_id and scap_profile_id
+        :return: scap_id and scap_profile_id
         """
         default_content = sat.cli.Scapcontent.info({'title': scap_name}, output_format='json')
         scap_id = default_content['id']

tests/foreman/longrun/test_oscap.py

@@ -43,7 +43,7 @@ def fetch_scap_and_profile_id(sat, scap_name, scap_profile):
     :param scap_name: Scap title
     :param scap_profile: Scap profile you want to select
 
-    :returns: scap_id and scap_profile_id
+    :return: scap_id and scap_profile_id
     """
 
     default_content = sat.cli.Scapcontent.info({'title': scap_name}, output_format='json')