Desdroid commented on PR #56881:
URL: https://github.com/apache/airflow/pull/56881#issuecomment-3432415980

   @ashb I wasn't quite sure how to best test the method flow, but I added a 
test. If you have a better idea how to do it, let me know. 
   Getting the ordering right was a bit more tricky than i thought, as we e.g. 
need the built-ins before the primitives. Otherwise `np.float64` would be 
treated as a primitive since `isinstance(np.float64(0.123), float) == true`. 
   
   However now some test seems to fail and I'm not sure I understand what's 
going wrong there. If I get the CI logs right it's this dag that is failing 
https://github.com/apache/airflow/blob/main/providers/standard/src/airflow/providers/standard/example_dags/example_sensor_decorator.py
   Can you help me pinpointing the issue? I'm rather new to the entire Airflow 
test suite unfortunately.
      


-- 
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