kyungjunleeme commented on PR #53039:
URL: https://github.com/apache/airflow/pull/53039#issuecomment-3064373375
-
providers.databricks.tests.unit.databricks.operators.test_databricks.TestDatabricksCreateJobsOperator.test_exec_update_job_permission
```python
@mock.patch("airflow.providers.databricks.operators.databricks.DatabricksHook")
def test_exec_update_job_permission(self, db_mock_class):
```
In this case, since the entire DatabricksHook is mocked, the internal
_do_api_call() method is never actually invoked. Therefore, you cannot verify
whether the path is "api/2.0/..." or "2.0/...".
I think that
This is a good example of one of the risks of relying too heavily on mocking
in tests. Since the actual implementation isn't exercised, critical issues like
incorrect API paths can easily slip through unnoticed.
--
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]
