[GitHub] [airflow] ashb opened a new pull request #17791: Move Model collation args tests to correct folder

[GitBox <gi...@apache.org>]

ashb opened a new pull request #17791:
URL: https://github.com/apache/airflow/pull/17791


   The tests for this got added to test_base.py in #17729, which is the right file name, but inside tests/sensors/, which isn't right :)
   
   Created a new tests/models/test_base.py for this.
   
   And cos I'm on a "removing lines of code" kick I've parameterized it too so it a "data driven" test.
   


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [airflow] potiuk commented on pull request #17791: Move Model collation args tests to correct folder

[GitBox <gi...@apache.org>]

potiuk commented on pull request #17791:
URL: https://github.com/apache/airflow/pull/17791#issuecomment-904154511


   Nice catch!


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [airflow] potiuk merged pull request #17791: Move Model collation args tests to correct folder

[GitBox <gi...@apache.org>]

potiuk merged pull request #17791:
URL: https://github.com/apache/airflow/pull/17791


   


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org