arlopurcell commented on issue #5600: [AIRFLOW-4981][AIRFLOW-4788] Output 
pendulum DateTime for fixed cron schedules
URL: https://github.com/apache/airflow/pull/5600#issuecomment-513252163
 
 
   There seem to be a bunch of issues in the test where sometimes the dates are 
still datetimes instead of pendulum DateTimes. I think it's because 
refresh_from_db() isn't quite working as expected in test cases. so after 
construction of some model, (which includes a call to refresh_from_db()), the 
field should converted, but it's not. I think it's only a problem in testing, 
because in real usage, the original object is thrown out and it only matters 
what comes back from the database later.
   
   I'm not really familiar enough with the codebase to debug through all the 
test failures. I'm not sure what to do

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to