shahar1 commented on PR #69212: URL: https://github.com/apache/airflow/pull/69212#issuecomment-4928733959
> Sorry to come here late. May I ask what is the purpose of introducing another `providers-e2e-tests` directory? Since the existing `airflow-e2e-tests` already includes the providers, for examples: The ES and OS provider for remote logging feature, the Kafka provider for external event driven feature. > > https://github.com/apache/airflow/blob/dc5ddd26202f972f634a6ee9ce25f04f17669ad7/airflow-e2e-tests/tests/airflow_e2e_tests/ > > Would it be better to consolidate there to reuse all the setup instead of re-introducing a new test suite? Good catch - didn't notice that all e2e tests are already there. As a first step it might be indeed better to consolidate them. However, long term-wise - maybe they should reside in `providers/<provider>/tests/e2e/`? -- 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]
