michalslowikowski00 commented on a change in pull request #6306: [AIRFLOW-5632] Rename ComputeEngine operators URL: https://github.com/apache/airflow/pull/6306#discussion_r334427298
########## File path: tests/test_core_to_contrib.py ########## @@ -16,787 +16,814 @@ # KIND, either express or implied. See the License for the # specific language governing permissions and limitations # under the License. + import importlib -from unittest import TestCase +from typing import Any +from unittest import TestCase, mock from parameterized import parameterized +NEW_HOOK_PATHS = [ + ( + "airflow.gcp.hooks.compute.ComputeEngineHook", + "airflow.contrib.hooks.gcp_compute_hook.GceHook", + ), +] +OLD_HOOKS_PATHS = [ Review comment: OLD_HOOKS_PATHS is variable which store paths that are not testable right now, NEW_HOOK_PATH is a place where I put paths that can be tested. With the end of the task OLD_HOOKS_PATHS will be removed permanently and NEW_HOOK_PATH will be replace with HOOK_PATH (just a var name for now), which will store all testable paths. ---------------------------------------------------------------- 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] With regards, Apache Git Services
