gopidesupavan commented on PR #53541: URL: https://github.com/apache/airflow/pull/53541#issuecomment-3119939837
> > @leakec LGTM, can you please fix the static checks and compat tests? > > @gopidesupavan Sure thing. I've added a few more commits to patch the issues, and now it passes all the tests. I do still need some help with the unit test though. The one I added passes, but the problem is it doesn't fail before the fix I implemented, i.e., it passes with both versions, so it doesn't really replicate the problem I saw when running this DAG in a normal airflow instance: > > ```python > from airflow import DAG > from datetime import datetime > from airflow.providers.standard.operators.bash import BashOperator > from airflow.providers.standard.operators.latest_only import LatestOnlyOperator > > with DAG( > "test_job", > description="Job to test latest only operator.", > schedule="* * * * *", > start_date=datetime(2024, 12, 11), > catchup=False, > ): > > # latest only operator > task1 = LatestOnlyOperator(task_id="latest_only") > task2 = BashOperator(bash_command="echo test", task_id="task2") > > task1 >> task2 > ``` > > I think there is likely a mistake I'm making when setting up or running tasks in the unit test, but I'm not familiar enough with the airflow testing system to know where. I'll continue reading up on it some more in my free time, but if you see anything obviously amiss, please let me know. the test is fine i just triggered same one , it looks fine, it failed in my case. <img width="1730" height="89" alt="image" src="https://github.com/user-attachments/assets/3f7004e0-9fac-4d8b-b255-fbdd9be9f629"; /> -- 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]
