turbaszek commented on a change in pull request #10353: URL: https://github.com/apache/airflow/pull/10353#discussion_r471973026
########## File path: tests/providers/jenkins/operators/test_jenkins_job_trigger.py ########## @@ -20,15 +20,21 @@ import jenkins import mock +from parameterized import parameterized from airflow.exceptions import AirflowException from airflow.providers.jenkins.hooks.jenkins import JenkinsHook from airflow.providers.jenkins.operators.jenkins_job_trigger import JenkinsJobTriggerOperator [email protected](mock is None, 'mock package not present') Review comment: Let's just do `from unittest import mock` and remove the skipIf ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
