Lint: Black -> Ruff formatter (SatelliteQE#14896)

* Lint: Black -> Ruff formatter

* Address review comments

.pre-commit-config.yaml

@@ -11,15 +11,12 @@ repos:
       exclude: tests/foreman/data/
     - id: check-yaml
     - id: debug-statements
-  - repo: https://github.com/psf/black
-    rev: 22.10.0
-    hooks:
-    - id: black
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: v0.3.0
+    rev: v0.4.1
     hooks:
       - id: ruff
         args: [--fix, --exit-non-zero-on-fix]
+      - id: ruff-format
   - repo: local
     hooks:
       - id: fix-uuids

conftest.py

@@ -1,4 +1,5 @@
 """Global Configurations for py.test runner"""
+
 import pytest
 
 pytest_plugins = [

docs/conf.py

@@ -4,6 +4,7 @@
 http://sphinx-doc.org/config.html
 
 """
+
 import builtins
 import os
 import sys

pyproject.toml

@@ -1,22 +1,12 @@
-[tool.black]
-line-length = 100
-skip-string-normalization = true
-include = '\.pyi?$'
-exclude = '''
-/(
-    \.git
-  | \.hg
-  | \.mypy_cache
-  | \.venv
-  | _build
-  | buck-out
-  | build
-  | dist
-)/
-'''
-
 [tool.ruff]
 target-version = "py311"
+# Allow lines to be as long as 100.
+line-length = 100
+exclude = [".git", ".hg", ".mypy_cache", ".venv", "_build", "buck-out", "build", "dist"]
+
+[tool.ruff.format]
+# Preserve quotes
+quote-style = "preserve"  # TODO: change to "single" when flake8-quotes is enabled
 
 [tool.ruff.lint]
 fixable = ["ALL"]

pytest_fixtures/component/virtwho_config.py

@@ -24,11 +24,7 @@ def org_module(request, default_org, module_sca_manifest_org):
 
 @pytest.fixture
 def org_session(request, session, session_sca):
-    if 'sca' in request.module.__name__.split('.')[-1]:
-        org_session = session_sca
-    else:
-        org_session = session
-    return org_session
+    return session_sca if 'sca' in request.module.__name__.split('.')[-1] else session
 
 
 @pytest.fixture

pytest_fixtures/core/contenthosts.py

@@ -4,6 +4,7 @@
 The functions in this module are read in the pytest_plugins/fixture_markers.py module
 All functions in this module will be treated as fixtures that apply the contenthost mark
 """
+
 from broker import Broker
 import pytest
 

pytest_fixtures/core/xdist.py

@@ -25,10 +25,7 @@ def align_to_satellite(request, worker_id, satellite_factory):
         settings.set("server.hostname", None)
         on_demand_sat = None
 
-        if worker_id in ['master', 'local']:
-            worker_pos = 0
-        else:
-            worker_pos = int(worker_id.replace('gw', ''))
+        worker_pos = 0 if worker_id in ["master", "local"] else int(worker_id.replace("gw", ""))
 
         # attempt to add potential satellites from the broker inventory file
         if settings.server.inventory_filter:

pytest_plugins/capsule_n-minus.py

@@ -19,7 +19,6 @@ def pytest_addoption(parser):
 
 
 def pytest_collection_modifyitems(items, config):
-
     if not config.getoption('n_minus', False):
         return
 

pytest_plugins/requirements/req_updater.py

@@ -3,7 +3,6 @@
 
 
 class ReqUpdater:
-
     # Installed package name as key and its counterpart in requirements file as value
     package_deviates = {
         'Betelgeuse': 'betelgeuse',

pytest_plugins/requirements/update_requirements.py

@@ -1,4 +1,5 @@
 """Plugin enables pytest to notify and update the requirements"""
+
 from .req_updater import ReqUpdater
 
 updater = ReqUpdater()

pytest_plugins/sanity_plugin.py

@@ -1,4 +1,4 @@
-""" A sanity testing plugin to assist in executing robottelo tests as sanity tests smartly
+"""A sanity testing plugin to assist in executing robottelo tests as sanity tests smartly
 
 1. Make installer test to run first which should set the hostname and all other tests then
 should run after that

robottelo/cli/acs.py

@@ -21,6 +21,7 @@
     update          Update an alternate content source.
 
 """
+
 from robottelo.cli.base import Base
 
 

robottelo/cli/activationkey.py

@@ -25,6 +25,7 @@
     subscriptions                 List associated subscriptions
     update                        Update an activation key
 """
+
 from robottelo.cli.base import Base
 
 

robottelo/cli/admin.py

@@ -12,6 +12,7 @@
 Options:
  -h, --help                    Print help
 """
+
 from robottelo.cli.base import Base
 
 

robottelo/cli/ansible.py

@@ -8,6 +8,7 @@
      roles                         Manage ansible roles
      variables                     Manage ansible variables
 """
+
 from robottelo.cli.base import Base
 
 

robottelo/cli/architecture.py

@@ -18,6 +18,7 @@
     remove_operatingsystem        Disassociate a resource
     update                        Update an architecture.
 """
+
 from robottelo.cli.base import Base
 
 

robottelo/cli/arfreport.py

@@ -16,6 +16,7 @@
      list                          List ARF reports
 
 """
+
 from robottelo.cli.base import Base
 
 

robottelo/cli/auth.py

@@ -11,6 +11,7 @@
     logout                        Wipe your credentials
     status                        Information about current connections
 """
+
 from robottelo.cli.base import Base
 
 

robottelo/cli/base.py

@@ -1,4 +1,5 @@
 """Generic base class for cli hammer commands."""
+
 import re
 
 from wait_for import wait_for

robottelo/cli/bootdisk.py

@@ -14,6 +14,7 @@
     host                          Download host image
     subnet                        Download subnet generic image
 """
+
 from robottelo.cli.base import Base
 
 

robottelo/cli/capsule.py

@@ -19,6 +19,7 @@
     refresh-features              Refresh capsule features
     update                        Update a capsule
 """
+
 from robottelo.cli.base import Base
 
 

robottelo/cli/computeprofile.py

@@ -18,6 +18,7 @@
  -h, --help                    Print help
                       Update a compute resource.
 """
+
 from robottelo.cli.base import Base
 
 

robottelo/cli/computeresource.py

@@ -17,6 +17,7 @@
     list                          List all compute resources.
     update                        Update a compute resource.
 """
+
 from robottelo.cli.base import Base
 
 

robottelo/cli/content_credentials.py

@@ -16,6 +16,7 @@
     list                          List content credentials
     update                        Update a content credential
 """
+
 from robottelo.cli.base import Base
 
 

robottelo/cli/content_export.py

@@ -19,6 +19,7 @@
     list                          View content view export histories
 
 """
+
 from robottelo.cli.base import Base
 
 

robottelo/cli/content_import.py

@@ -16,6 +16,7 @@
     version                       Imports a content archive to a content view version
 
 """
+
 from robottelo.cli.base import Base
 
 

robottelo/cli/contentview.py

@@ -33,6 +33,7 @@
 
     -h, --help                    print help
 """
+
 from robottelo.cli import hammer
 from robottelo.cli.base import Base, CLIError
 

robottelo/cli/defaults.py

@@ -15,6 +15,7 @@
     list                          List all the default parameters
     providers                     List all the providers
 """
+
 from robottelo.cli.base import Base
 
 

robottelo/cli/discoveredhost.py

@@ -19,6 +19,7 @@
     reboot                        Reboot a host
     refresh-facts                 Refresh the facts of a host
 """
+
 from robottelo.cli.base import Base
 
 

robottelo/cli/discoveryrule.py

@@ -16,6 +16,7 @@
     list                          List all discovery rules
     update                        Update a rule
 """
+
 from robottelo.cli.base import Base
 
 

robottelo/cli/docker.py

@@ -1,4 +1,5 @@
 """Docker related hammer commands"""
+
 from robottelo.cli.base import Base
 
 

robottelo/cli/domain.py

@@ -18,6 +18,7 @@
     set_parameter                 Create or update parameter for a domain.
     update                        Update a domain.
 """
+
 from robottelo.cli.base import Base
 
 

robottelo/cli/environment.py

@@ -17,6 +17,7 @@
     sc-params                     List all smart class parameters
     update                        Update an environment
 """
+
 from robottelo.cli.base import Base
 
 

robottelo/cli/erratum.py

@@ -13,6 +13,7 @@
  info                          Show an erratum
  list                          List errata
 """
+
 from robottelo.cli.base import Base
 
 

robottelo/cli/fact.py

@@ -12,6 +12,7 @@
 
     list                          List all fact values.
 """
+
 from robottelo.cli.base import Base
 
 

robottelo/cli/file.py

@@ -13,6 +13,7 @@
     info                          Show a file
     list                          List files
 """
+
 from robottelo.cli.base import Base
 
 

robottelo/cli/filter.py

@@ -17,6 +17,7 @@
  list                          List all filters
  update                        Update a filter
 """
+
 from robottelo.cli.base import Base
 
 

robottelo/cli/globalparam.py

@@ -14,6 +14,7 @@
     list                          List all common parameters.
     set                           Set a global parameter.
 """
+
 from robottelo.cli.base import Base
 
 

robottelo/cli/gpgkey.py

@@ -16,6 +16,7 @@
     list                          List GPG Keys
     update                        Update a GPG Key
 """
+
 from robottelo.cli.base import Base
 
 

robottelo/cli/hammer.py

@@ -1,4 +1,5 @@
 """Helpers to interact with hammer command line utility."""
+
 import csv
 import json
 import re

robottelo/cli/host.py

@@ -39,6 +39,7 @@
      update                        Update a host
 
 """
+
 from robottelo.cli.base import Base
 
 

robottelo/cli/host_registration.py

@@ -13,6 +13,7 @@
  -h, --help                    Print help
 
 """
+
 from robottelo.cli.base import Base
 
 

robottelo/cli/hostcollection.py

@@ -23,6 +23,7 @@
  remove-host                   Remove hosts from the host collection
  update                        Update a host collection
 """
+
 from robottelo.cli.base import Base
 
 

robottelo/cli/hostgroup.py

@@ -22,6 +22,7 @@
     set-parameter                 Create or update parameter for a hostgroup
     update                        Update a host group
 """
+
 from robottelo.cli.base import Base
 
 

robottelo/cli/http_proxy.py

@@ -16,6 +16,7 @@
 Options:
  -h, --help                    Print help
 """
+
 from robottelo.cli.base import Base
 
 

robottelo/cli/job_invocation.py

@@ -15,6 +15,7 @@
  rerun                         Rerun the job
 
 """
+
 from robottelo.cli.base import Base