potiuk commented on code in PR #39986: URL: https://github.com/apache/airflow/pull/39986#discussion_r1623194473
########## contributing-docs/testing/integration_tests.rst: ########## @@ -18,28 +18,32 @@ Airflow Integration Tests ========================= -Some of the tests in Airflow are integration tests. These tests require ``airflow`` Docker -image and extra images with integrations (such as ``celery``, ``mongodb``, etc.). -The integration tests are all stored in the ``tests/integration`` folder. +Integration tests in Airflow check the interactions between Airflow components and external services +that could run as separate Docker containers, without a connecting to an external API on the internet. +These tests require ``airflow`` Docker image and extra images with integrations (such as ``celery``, ``mongodb``, etc.). +The integration tests are all stored in the ``tests/integration`` folder, and similarly to the unit tests they all run +using `pytest <http://doc.pytest.org/en/latest/>`_ . Review Comment: ```suggestion using `pytest <http://doc.pytest.org/en/latest/>`_, but they are skipped by default unless `--integration` flag is passed to pytest. ``` ########## contributing-docs/testing/integration_tests.rst: ########## @@ -18,28 +18,32 @@ Airflow Integration Tests ========================= -Some of the tests in Airflow are integration tests. These tests require ``airflow`` Docker -image and extra images with integrations (such as ``celery``, ``mongodb``, etc.). -The integration tests are all stored in the ``tests/integration`` folder. +Integration tests in Airflow check the interactions between Airflow components and external services +that could run as separate Docker containers, without a connecting to an external API on the internet. Review Comment: ```suggestion that could run as separate Docker containers, without connecting to an external API on the internet. ``` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
