feluelle commented on pull request #11015:
URL: https://github.com/apache/airflow/pull/11015#issuecomment-715136031


   > I had in mind one test per method and use parameterization for the 
implicit and explicit factory cases.
   
   I like this. 👍 
   
   > It Is also possible to go far and have one or two hyper parameterized 
functions but I think it will make the tests less readable.
   
   Readability is far more important than code duplication in my opinion. So I 
am in favor for one test per method.


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


Reply via email to