feluelle commented on a change in pull request #6811: [AIRFLOW-6245] Add custom waiters for AWS batch jobs URL: https://github.com/apache/airflow/pull/6811#discussion_r363912077
########## File path: tests/providers/amazon/aws/hooks/test_batch_waiters.py ########## @@ -0,0 +1,486 @@ +# -*- coding: utf-8 -*- +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor license agreements. See the NOTICE file +# distributed with this work for additional information +# regarding copyright ownership. The ASF licenses this file +# to you under the Apache License, Version 2.0 (the +# "License"); you may not use this file except in compliance +# with the License. You may obtain a copy of the License at +# +# http://www.apache.org/licenses/LICENSE-2.0 +# +# Unless required by applicable law or agreed to in writing, +# software distributed under the License is distributed on an +# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY +# KIND, either express or implied. See the License for the +# specific language governing permissions and limitations +# under the License. + +# pylint: disable=do-not-use-asserts, missing-docstring, redefined-outer-name + + +""" +Test AwsBatchWaiters + +This test suite uses a large suite of moto mocks for the +AWS batch infrastructure. These infrastructure mocks are +derived from the moto test suite for testing the batch client. + +.. seealso:: + + - https://github.com/spulec/moto/pull/1197/files + - https://github.com/spulec/moto/blob/master/tests/test_batch/test_batch.py +""" + +import inspect +import unittest +from typing import NamedTuple, Optional + +import boto3 +import botocore.client +import botocore.exceptions +import botocore.waiter +import mock +import pytest +from moto import mock_batch, mock_ec2, mock_ecs, mock_iam, mock_logs + +from airflow import AirflowException +from airflow.providers.amazon.aws.hooks.batch_waiters import AwsBatchWaiters + +# Use dummy AWS credentials +AWS_REGION = "eu-west-1" +AWS_ACCESS_KEY_ID = "airflow_dummy_key" +AWS_SECRET_ACCESS_KEY = "airflow_dummy_secret" + + [email protected](scope="module") +def aws_region(): + return AWS_REGION + + [email protected](scope="module") +def job_queue_name(): + return "moto_test_job_queue" + + [email protected](scope="module") +def job_definition_name(): + return "moto_test_job_definition" + + +# +# AWS Clients +# + + +class AwsClients(NamedTuple): + batch: "botocore.client.Batch" + ec2: "botocore.client.EC2" + ecs: "botocore.client.ECS" + iam: "botocore.client.IAM" + log: "botocore.client.CloudWatchLogs" + + [email protected]_fixture(scope="module") +def batch_client(aws_region): + with mock_batch(): + yield boto3.client("batch", region_name=aws_region) + + [email protected]_fixture(scope="module") +def ec2_client(aws_region): + with mock_ec2(): + yield boto3.client("ec2", region_name=aws_region) + + [email protected]_fixture(scope="module") +def ecs_client(aws_region): + with mock_ecs(): + yield boto3.client("ecs", region_name=aws_region) + + [email protected]_fixture(scope="module") +def iam_client(aws_region): + with mock_iam(): + yield boto3.client("iam", region_name=aws_region) + + [email protected]_fixture(scope="module") +def logs_client(aws_region): + with mock_logs(): + yield boto3.client("logs", region_name=aws_region) + + [email protected](scope="module") +def aws_clients(batch_client, ec2_client, ecs_client, iam_client, logs_client): + return AwsClients( + batch=batch_client, ec2=ec2_client, ecs=ecs_client, iam=iam_client, log=logs_client + ) + + +# +# Batch Infrastructure +# + + +class Infrastructure: + aws_region: str + aws_clients: AwsClients + vpc_id: Optional[str] = None + subnet_id: Optional[str] = None + security_group_id: Optional[str] = None + iam_arn: Optional[str] = None + compute_env_name: Optional[str] = None + compute_env_arn: Optional[str] = None + job_queue_name: Optional[str] = None + job_queue_arn: Optional[str] = None + job_definition_name: Optional[str] = None + job_definition_arn: Optional[str] = None + + +def batch_infrastructure( + aws_clients: AwsClients, aws_region: str, job_queue_name: str, job_definition_name: str +) -> Infrastructure: + """ + This function is not a fixture so that tests can pass the AWS clients to it and then + continue to use the infrastructure created by it while the client fixtures are in-tact for + the duration of a test. + + :param aws_clients: + :type aws_clients: AwsClients + + :param aws_region: + :type aws_region: str + + :param job_queue_name: + :type job_queue_name: str + + :param job_definition_name: + :type job_definition_name: str Review comment: You are right. Let's remove them then. 👍 ---------------------------------------------------------------- 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
