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]

Reply via email to