potiuk commented on issue #11657:
URL: https://github.com/apache/airflow/issues/11657#issuecomment-714496293
I think I agree with all of you @ashb @turbaszek and @mik-laj :). Seems we
all agree actually :)
+1 asserts are better (but let's not make it consistent just yet)
+1 on using both fixtures and mocks - and @mik-laj's case is a good example.
I also prefer an explicit method where I do not have to something "common" to
modules/session. And simple setup/tearDown is so much embedded now in my unit
test thinking that in simple cases I prefer to use them. But this might change
in the future as I get used to it.
----------------------------------------------------------------
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]
