You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "potiuk (via GitHub)" <gi...@apache.org> on 2024/06/01 10:10:43 UTC
Re: [PR] Update integration tests docs and add "Writing Integration Tests" guide [airflow]
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org