dstandish commented on a change in pull request #21261:
URL: https://github.com/apache/airflow/pull/21261#discussion_r798959601



##########
File path: tests/providers/elasticsearch/log/test_es_task_handler.py
##########
@@ -38,6 +39,19 @@
 from .elasticmock import elasticmock
 
 
+def get_ti(dag_id, task_id, execution_date, create_task_instance):
+    ti = create_task_instance(
+        dag_id=dag_id,
+        task_id=task_id,
+        execution_date=execution_date,
+        dagrun_state=DagRunState.RUNNING,
+        state=TaskInstanceState.RUNNING,
+    )
+    ti.try_number = 1
+    ti.raw = False
+    return ti

Review comment:
       i'm confused @uranusjr 
   i'm not seeing how this fixture helps me.
   there is already a  fixture here like this now.  i just pulled out a portion 
of it (and still use it in the existing fixture) but i just want to be able to 
specify a different execution date in my specific test than the one used by the 
fixture.  




-- 
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]


Reply via email to