[jira] [Commented] (AIRFLOW-3551) Improve BashOperator Test Coverage

2018-12-23 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-3551?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16727999#comment-16727999 ] ASF GitHub Bot commented on AIRFLOW-3551: - feluelle commented on pull request #4367:

[jira] [Commented] (AIRFLOW-3551) Improve BashOperator Test Coverage

2018-12-23 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-3551?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16727998#comment-16727998 ] ASF GitHub Bot commented on AIRFLOW-3551: - feluelle commented on pull request #4366:

[jira] [Commented] (AIRFLOW-3551) Improve BashOperator Test Coverage

2018-12-23 Thread Felix Uellendall (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-3551?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16727991#comment-16727991 ] Felix Uellendall commented on AIRFLOW-3551: --- You propably mean the `pre_execute` function

[jira] [Commented] (AIRFLOW-3551) Improve BashOperator Test Coverage

2018-12-23 Thread Felix Uellendall (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-3551?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16727987#comment-16727987 ] Felix Uellendall commented on AIRFLOW-3551: --- I think it is not really testable because it is

[jira] [Commented] (AIRFLOW-3551) Improve BashOperator Test Coverage

2018-12-21 Thread Felix Uellendall (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-3551?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16726900#comment-16726900 ] Felix Uellendall commented on AIRFLOW-3551: --- I can't find it in the BaseOperator's

[jira] [Commented] (AIRFLOW-3551) Improve BashOperator Test Coverage

2018-12-21 Thread Ash Berlin-Taylor (JIRA)
[ https://issues.apache.org/jira/browse/AIRFLOW-3551?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16726770#comment-16726770 ] Ash Berlin-Taylor commented on AIRFLOW-3551: pre_exec is not specific to the BashOperator,