dabla commented on PR #46338: URL: https://github.com/apache/airflow/pull/46338#issuecomment-2629347670
> > > Looks like code loading resource file in tests needs to be fixed. > > > > > > Yes indeed but the conftest was shared across all Microsoft providers so think!it will have to be split now > > Yeah. There are two options: > > * if common code is small -> copy it. Tests do not have to be DRY > * If the code is substantial enough to justify keeping it DRY - we have `tests_common` project which is added as a plugin to all providers and common code is there. Should be fixed now, I've copied the logic in conftest of provider, maybe later we can consolidate (make helper tests methods more DRY), will see when azure provider is done, then it will be easier I think. -- 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-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
