leakec commented on PR #53541:
URL: https://github.com/apache/airflow/pull/53541#issuecomment-3114781291
> @leakec LGTM, can you please fix the static checks and compat tests?
@gopidesupavan Sure thing. I've added a few more commits, 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 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.
--
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]
