coopergillan commented on a change in pull request #10353: URL: https://github.com/apache/airflow/pull/10353#discussion_r471561676
########## 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: Coming to think of it, I think this line doesn't make sense. Wouldn't we need to add this up in the imports? ```python try: import mock except ImportError: mock = None ``` ---------------------------------------------------------------- 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]
