apache / apache/airflow

Add test to verify parameters match between operator and decorator

Open
#48,448 10 comments 0 reactions 0 assignees View on GitHub
area:providers good first issue kind:meta provider:cncf-kubernetes
Dominant language
Python
Stars
46.9k
Forks
17.8k
Avg merge
2d 10h
Merged PRs (30d)
483

Description

### Body

We shouldn't have notes like:
https://github.com/apache/airflow/blob/6508f1242c1b336728d7a256ebb8597214bbcf90/providers/cncf/kubernetes/src/airflow/providers/cncf/kubernetes/operators/pod.py#L236-L237

this is very easy to miss both by PR author and the reviewer

**The task:**
Add tests to all existed decorators to make sure the parameter list they accept match to the operator.

### Committer

- [x] I acknowledge that I am a maintainer/committer of the Apache Airflow project.

Contributor guide

Open the contributing guide

Research direction

Start with the referenced providers/cncf/kubernetes/operators/pod.py lines and inspect the existing decorators and their corresponding operators across the repository. Add tests covering each existing decorator's accepted parameters against its operator, then run the affected provider tests; done means the parameter lists are verified to match.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
testing
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Active
Clarity
Mostly clear
Newbie friendliness
62/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.