potiuk commented on a change in pull request #7389: [AIRFLOW-6763] Make systems
tests ready for backport tests
URL: https://github.com/apache/airflow/pull/7389#discussion_r376830835
##########
File path: tests/test_utils/gcp_system_helpers.py
##########
@@ -32,43 +31,6 @@
)
-SKIP_TEST_WARNING = """
-The test is only run when the test is run in environment with GCP-system-tests
enabled
-environment. You can enable it in one of two ways:
-
-* Set GCP_CONFIG_DIR environment variable to point to the GCP configuration
- directory which keeps the {} key.
-* Run this test within automated environment variable workspace where
- config directory is checked out next to the airflow one.
-
-"""
-
-SKIP_LONG_TEST_WARNING = """
-The test is only run when the test is run in with GCP-system-tests enabled
-environment. And environment variable GCP_ENABLE_LONG_TESTS is set to True.
-You can enable it in one of two ways:
-
-* Set GCP_CONFIG_DIR environment variable to point to the GCP configuration
- directory which keeps variables.env file with environment variables to set
- and keys directory which keeps service account keys in .json format and
- set GCP_ENABLE_LONG_TESTS to True
-* Run this test within automated environment variable workspace where
- config directory is checked out next to the airflow one.
-"""
-
-
-LOCAL_EXECUTOR_WARNING = """
Review comment:
Yes. I replaced it with @pytest.markers.backend("postgres", "mysql").
LocalExecutor is the default executor for both Postgres and Mysql and
SequentialExecutor is default (and the only available) for sqlite. By
constraining the tests to run only on Postgres/MySql we automatically set local
executor for unit tests.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
