Remove all usages of days_ago from test suite

[eladkal]

Body

days_ago is deprecated #21653

We have several references to the function in test suite / example dags.

Task:
there should be no imports of airflow.utils.dates import days_ago in the code.
In most cases we can simple replace with datetime(2022, 1, 1)

Committer

• I acknowledge that I am a maintainer/committer of the Apache Airflow project.

[hubert-pietron]

Hi, can i take it?

[eladkal]

@hubert-pietron sure, assigned to you.

[Dev232001]

Hi, can you assign this to me?

[potiuk]

Just check with @hubert-pietron - if it's not something already started.

[hubert-pietron]

Let me know @Dev232001 how you want to split, or you can take whole and i will look for another issue to work on :)

[Dev232001]

Hi @hubert-pietron , thanks for letting me join this issue here. I am relatively new to open source contributions and was looking for a beginner level task. This looked like an easy problem so I asked for this. If you can just give me an update on what you have done till now that will be the best. I think I will be able to work on the rest of the issue from thereon :)

[Dev232001]

https://github.com/apache/airflow/tree/main/airflow/example_dags
@potiuk is this the correct folder in which the task is to be done?

[eladkal]

it can be in example dags or in tests.
You can find all relevant files by searching from airflow.utils.dates import days_ago in the code base.

[Dev232001]

Okay. Appreciate the help. Thanks

…

On Wed, Feb 23, 2022, 10:36 PM eladkal ***@***.***> wrote: it can be in example dags or in tests. You can find all references by searching airflow.utils.dates import days_ago in the code base. — Reply to this email directly, view it on GitHub <#21683 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AXEWNKGB3OWNKGQ3MGINIJ3U4UHZHANCNFSM5O2LUIXQ> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>. You are receiving this because you were mentioned.Message ID: ***@***.***>

[hubert-pietron]

I just looked how many occurrences there is in code, so You can take it all :)

[Dev232001]

I have completed the given task and created a pull request for the same. Kindly review it.
Besides this, there are quite a few syntax related errors in tests folder like in
from fake_datetime import FakeDatetime - test_impersonation_custom.py - dags_corrupted - tests

tests - decorators - test_python_virtualenv.py - errors found

tests - operators - test_python.py - import funcsigs - error

[Dev232001]

This is my first contribution and its asking for a approval review
#21796

[eladkal]

fixed in #23237
and duplicated of #23233
I don't know why I opened two issues for this :/