Abhiaps commented on PR #29375: URL: https://github.com/apache/airflow/pull/29375#issuecomment-1421018357
> > In one file they are using self.subTest which was giving me problem while migrating, > > As far as I remember: > > 1. In always subtest is useless and could be replaced by regular assert. And this test is broken and always success > 2. In operators subtest also broken (do not cleanup stage between subtest run). In general all subtest could be rewrited by parametrised tests (`@pytest.mark.parametrize`) > > > So i converted only a part of tests/always. > > No problem, there is no target to do everything in one go 😊Okay i will try -- 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]
