bugraoz93 commented on code in PR #64154:
URL: https://github.com/apache/airflow/pull/64154#discussion_r2989840618
##########
scripts/ci/testing/run_unit_tests.sh:
##########
@@ -102,7 +102,12 @@ function providers_tests() {
exit 1
fi
set -e
- if [[ ${RESULT} != "0" ]]; then
+ # If pytest returns exit code 1 (no tests collected) for DB-only runs,
treat it as success
+ # to avoid failing CI when there are simply no DB tests defined for the
group.
+ if [[ ${RESULT} == "1" && "${TEST_SCOPE}" == "DB" ]]; then
Review Comment:
If no test were collected is actual error, wouldn't this eliminate to
identify the issue?
Maybe could better to exclude no collection in one higher on breeze where we
have file group definitions and better control over the what is going to be
tested
--
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]