RavinduWeerakoon commented on PR #59943:
URL: https://github.com/apache/airflow/pull/59943#issuecomment-3728432381

   > > Hi @vatsrahul1001, thanks for the comments. I’ve resolved some of the 
issues.
   > > I still have some doubts about creating the test scenario for the 
filter. For these different aspects (such as trigger rules) to be available, do 
we need to define a DAG at runtime with the required behavior and then test how 
the filters work
   > 
   > For this PR, I suggest Keep filter tests that work with 
example_bash_operator (operator, search), remove or skip filters that don't 
have meaningful data (mapped, multiple trigger rules). You can implement more 
comprehensive tests in a follow-up PR with appropriate DAGs
   
   Ok, I will remove the meaningless filter tests then


-- 
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]

Reply via email to