Body

We have a number of xfail tests in Airflow and they should be fixed as they they might mislead people into thinking those are errors causing CI failures:

• tests/utils/log/test_secrets_masker.py::test_exc_tb Fixed test and remove pytest.mark.xfail for test_exc_tb #23650
• tests/jobs/test_backfill_job.py::test_trigger_controller_dag Use dagrun instead of DagFileProcessor in backfill_job test #28724
• tests/jobs/test_scheduler_job.py::test_task_with_upstream_skip_process_task_instancesb remove xfail test jobs in test_scheduler #23731
• tests/providers/amazon/aws/hooks/test_batch_waiters.py::test_batch_job_waiting Fix the xfail test for batch #23918, Refactor BatchWaitersHook tests #28758
• tests/providers/postgres/hooks/test_postgres.py::test_insert_rows_replace_missing_target_field_arg - Replace pytest.mark.xfail in Postgres tests #23541
• tests/providers/postgres/hooks/test_postgres.py::test_insert_rows_replace_missing_replace_index_arg - Replace pytest.mark.xfail in Postgres tests #23541
• tests/jobs/test_backfill_job.py:150: @pytest.mark.xfail(condition=True, reason="This test is flaky")
• test_missing_assets

  airflow/tests/always/test_project_structure.py

  Lines 377 to 379 in c0b2fcf

  	@pytest.mark.xfail(reason="We did not reach full coverage yet")
  	def test_missing_assets(self):
  	super().test_missing_assets()

  
  Ideally those tests should be turned into regular non-failing tests (unless it makes no sense).

Committer

• I acknowledge that I am a maintainer/committer of the Apache Airflow project.