airflow: Missing unit tests for some modules in providers package
Description
We have a rule that every file in the providers
directory should have tests. This is true in many cases, but there are minor exceptions.
https://github.com/apache/airflow/blob/master/tests/test_project_structure.py#L29-L50
-
tests/providers/amazon/aws/hooks/test_athena.py
-
tests/providers/apache/cassandra/sensors/test_record.py
-
tests/providers/apache/cassandra/sensors/test_table.py
-
tests/providers/apache/hdfs/sensors/test_web_hdfs.py
-
tests/providers/apache/hive/operators/test_vertica_to_hive.py
-
tests/providers/apache/pig/operators/test_pig.py
-
tests/providers/apache/spark/hooks/test_spark_jdbc_script.py
-
tests/providers/cncf/kubernetes/operators/test_kubernetes_pod.py
-
tests/providers/google/cloud/operators/test_datastore.py
-
tests/providers/google/cloud/operators/test_sql_to_gcs.py
-
tests/providers/google/cloud/sensors/test_bigquery.py
-
tests/providers/google/cloud/utils/test_field_sanitizer.py
-
tests/providers/google/cloud/utils/test_field_validator.py
-
tests/providers/google/cloud/utils/test_mlengine_operator_utils.py
-
tests/providers/google/cloud/utils/test_mlengine_prediction_summary.py
-
tests/providers/jenkins/hooks/test_jenkins.py
-
tests/providers/microsoft/azure/sensors/test_azure_cosmos.py
-
tests/providers/microsoft/mssql/hooks/test_mssql.py
-
tests/providers/oracle/operators/test_oracle.py
-
tests/providers/qubole/hooks/test_qubole.py
-
tests/providers/samba/hooks/test_samba.py
-
tests/providers/sqlite/operators/test_sqlite.py
-
tests/providers/yandex/hooks/test_yandex.py
I would be happy if someone would complete the missing tests.
If you decide to finish this ticket you don’t have to do all the work yourself. One PR can only deal with a single file and it’s ok.
Are you wondering how to start contributing to this project? Start by reading our contributor guide
Use case / motivation
Raising test coverage facilitates refactoring and prevents regression.
For the contributor, this is an opportunity to learn more about the structure of the project and to learn better good practices in writing tests in Python.
Related Issues
N/A
About this issue
- Original URL
- State: closed
- Created 4 years ago
- Reactions: 2
- Comments: 19 (18 by maintainers)
Commits related to this issue
- [AIRFLOW-8278] Added tests for yandex hooks #8278 — committed to subkanthi/airflow by deleted user 4 years ago
- [AIRFLOW-8278] Fixed lint errors for yandex hooks tests #8278 — committed to subkanthi/airflow by deleted user 4 years ago
- [AIRFLOW-8278] Fixed lint errors for yandex hooks tests #8278 — committed to subkanthi/airflow by deleted user 4 years ago
- [AIRFLOW-8278] Fixed flake8 errors #8278 — committed to subkanthi/airflow by deleted user 4 years ago
If no one is working on the test_mssql.py, I can take work it.
tests/providers/microsoft/mssql/hooks/test_mssql.py
@rootcss Wouldn’t you be willing to give some love for this ticket? https://github.com/apache/airflow/issues/8189