ambika-garg commented on code in PR #43494:
URL: https://github.com/apache/airflow/pull/43494#discussion_r1835269723


##########
providers/tests/microsoft/azure/triggers/test_powerbi.py:
##########
@@ -39,19 +39,22 @@
 
 
 @pytest.fixture
-def powerbi_trigger():
-    trigger = PowerBITrigger(
-        conn_id=POWERBI_CONN_ID,
-        proxies=None,
-        api_version=API_VERSION,
-        dataset_id=DATASET_ID,
-        group_id=GROUP_ID,
-        check_interval=CHECK_INTERVAL,
-        wait_for_termination=True,
-        timeout=TIMEOUT,
-    )
+def powerbi_trigger() -> PowerBITrigger:

Review Comment:
   @dabla , I looked into it, the problem is the parameters aren't getting 
passed to pytest 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