dirkrkotzeml commented on PR #45551: URL: https://github.com/apache/airflow/pull/45551#issuecomment-2589877618
> Could you add a unit test to cover that? @ferruzzi @vincbeck We have updated the prior unit tests and added two of our own, the provider tests all pass. Just a small note the previous mock for time_ns was using an incorrect format that the time_ns() function would not return. We updated the unit tests to use a representative integer for the tests. In our updated function we limit the returned timestamp to 10 characters to capture only the date time up to the seconds as this is all that is needed. -- 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]
