Address review comments

tests/foreman/api/test_errata.py

@@ -123,7 +123,7 @@ def _validate_errata_counts(host, errata_type, expected_value, timeout=120):
  sleep(5)
  else:
  pytest.fail(
- 'Host {} contains {} {} errata, but expected to contain ' '{} of them'.format(
+ 'Host {} contains {} {} errata, but expected to contain {} of them'.format(
  host.name,
  host.content_facet_attributes['errata_counts'][errata_type],
  errata_type,
@@ -149,7 +149,7 @@ def _fetch_available_errata(host, expected_amount=None, timeout=120):
  errata = host.errata()
  else:
  pytest.fail(
- 'Host {} contains {} available errata, but expected to ' 'contain {} of them'.format(
+ 'Host {} contains {} available errata, but expected to contain {} of them'.format(
  host.name,
  len(errata['results']),
  expected_amount if not None else 'No expected_amount provided',

tests/foreman/api/test_notifications.py

@@ -239,8 +239,8 @@ def long_running_task(target_sat):
  f"SET start_at = {sql_date_2_days_ago}, "
  f" started_at = {sql_date_2_days_ago}, "
  f" state_updated_at = {sql_date_2_days_ago} "
- f"WHERE id='{job['task']['id']}';\nEOF\n\" "
- )
+ f"WHERE id=\'{job['task']['id']}\';\nEOF\n\" "
+ ) # fmt: skip # skip formatting to avoid breaking the SQL query
  assert 'UPDATE 1' in result.stdout, f'Failed to age task {job["task"]["id"]}: {result.stderr}'
 
  yield job

tests/foreman/ui/test_ansible.py

@@ -183,7 +183,7 @@ def _finalize():
  result = rhel_contenthost.execute(command)
  assert result.status == 0, f'Failed to register host: {result.stderr}'
  target_host = rhel_contenthost.nailgun_host
- default_value = '["test"]'
+ default_value = '[\"test\"]' # fmt: skip
  parameter_type = 'array'
  with target_sat.ui_session() as session:
  session.organization.select(org_name=module_org.name)