# -*- coding: utf-8 -*-
# © Copyright EnterpriseDB UK Limited 2013-2023
#
# Client Utilities for Barman, Backup and Recovery Manager for PostgreSQL
#
# Barman is free software: you can redistribute it and/or modify
# it under the terms of the GNU General Public License as published by
# the Free Software Foundation, either version 3 of the License, or
# (at your option) any later version.
#
# Barman is distributed in the hope that it will be useful,
# but WITHOUT ANY WARRANTY; without even the implied warranty of
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
# GNU General Public License for more details.
#
# You should have received a copy of the GNU General Public License
# along with Barman.  If not, see <http://www.gnu.org/licenses/>.

import logging
import mock
import pytest

from azure.core.exceptions import ResourceNotFoundError
from botocore.exceptions import ClientError
from google.api_core.exceptions import NotFound

from barman.cloud import CloudProviderError

from barman.cloud_providers import (
    CloudProviderUnsupported,
    get_snapshot_interface,
    get_snapshot_interface_from_backup_info,
    get_snapshot_interface_from_server_config,
)
from barman.cloud_providers.aws_s3 import AwsCloudSnapshotInterface, AwsVolumeMetadata
from barman.cloud_providers.azure_blob_storage import (
    AzureCloudSnapshotInterface,
    AzureVolumeMetadata,
)
from barman.cloud_providers.google_cloud_storage import (
    GcpCloudSnapshotInterface,
    GcpVolumeMetadata,
)
from barman.exceptions import (
    BarmanException,
    CommandException,
    ConfigurationException,
    SnapshotBackupException,
    SnapshotInstanceNotFoundException,
)


class TestGetSnapshotInterface(object):
    """
    Verify get_snapshot_interface creates the required CloudSnapshotInterface
    """

    @pytest.mark.parametrize(
        ("snapshot_provider", "interface_cls"),
        [
            ("aws", AwsCloudSnapshotInterface),
            ("azure", AzureCloudSnapshotInterface),
            ("gcp", GcpCloudSnapshotInterface),
            ("unsupportedcloud", None),
        ],
    )
    @mock.patch("barman.cloud_providers._get_azure_credential")
    @mock.patch("barman.cloud_providers.azure_blob_storage.import_azure_mgmt_compute")
    @mock.patch(
        "barman.cloud_providers.google_cloud_storage.import_google_cloud_compute"
    )
    @mock.patch("barman.cloud_providers.aws_s3.boto3")
    def test_from_config_cloud_provider(
        self,
        _mock_boto3,
        _mock_google_cloud_compute,
        _mock_azure_mgmt_compute,
        _mock_get_azure_credential,
        snapshot_provider,
        interface_cls,
    ):
        """Verify supported and unsupported cloud providers with server config."""
        # GIVEN a server config with the specified snapshot provider
        mock_config = mock.Mock(snapshot_provider=snapshot_provider)

        # WHEN get_snapshot_interface_from_server_config is called
        if interface_cls:
            # THEN supported providers return the expected interface
            assert isinstance(
                get_snapshot_interface_from_server_config(mock_config), interface_cls
            )
        else:
            # AND unsupported providers raise the expected exception
            with pytest.raises(CloudProviderUnsupported) as exc:
                get_snapshot_interface_from_server_config(mock_config)
            assert "Unsupported snapshot provider: {}".format(snapshot_provider) == str(
                exc.value
            )

    def test_from_config_gcp_no_project(self):
        """
        Verify an exception is raised for gcp snapshots with no project in server config.
        """
        # GIVEN a server config with the gcp snapshot provider and neither the
        # gcp_project option nor the deprecated snapshot_gcp_project option
        mock_config = mock.Mock(
            snapshot_provider="gcp", gcp_project=None, snapshot_gcp_project=None
        )
        # WHEN get snapshot_interface_from_server_config is called
        with pytest.raises(ConfigurationException) as exc:
            get_snapshot_interface_from_server_config(mock_config)
        # THEN the expected exception is raised
        assert "gcp_project option must be set when snapshot_provider is gcp" in str(
            exc.value
        )

    def test_from_config_azure_no_subscription_id(self):
        """
        Verify an exception is raised for azure snapshots with no azure_subscription_id
        in server config.
        """
        # GIVEN a server config with the azure snapshot provider and the
        # azure_subscription_id
        mock_config = mock.Mock(snapshot_provider="azure", azure_subscription_id=None)
        # WHEN get snapshot_interface_from_server_config is called
        with pytest.raises(ConfigurationException) as exc:
            get_snapshot_interface_from_server_config(mock_config)
        # THEN the expected exception is raised
        assert (
            "azure_subscription_id option must be set when snapshot_provider is azure"
            in str(exc.value)
        )

    @mock.patch("barman.cloud_providers.aws_s3.boto3")
    def test_from_config_aws(self, mock_boto3):
        """
        Verify aws-specific Barman config options are passed to snapshot interface.
        """
        # GIVEN a server config with the aws snapshot provider and the specified
        # parameters
        mock_config = mock.Mock(
            snapshot_provider="aws",
            aws_region="us-east-2",
            aws_profile="default",
            aws_await_snapshots_timeout=7200,
        )
        # WHEN get_snapshot_interface_from_server_config is called
        snapshot_interface = get_snapshot_interface_from_server_config(mock_config)
        # THEN the config values are passed to the snapshot interface
        assert isinstance(snapshot_interface, AwsCloudSnapshotInterface)
        assert snapshot_interface.region == "us-east-2"
        assert snapshot_interface.await_snapshots_timeout == 7200
        mock_boto3.Session.assert_called_once_with(profile_name="default")

    @pytest.mark.parametrize(
        ("snapshot_provider", "interface_cls"),
        [
            ("aws", AwsCloudSnapshotInterface),
            ("azure", AzureCloudSnapshotInterface),
            ("gcp", GcpCloudSnapshotInterface),
            ("unsupportedcloud", None),
        ],
    )
    @mock.patch("barman.cloud_providers.aws_s3.boto3")
    @mock.patch("barman.cloud_providers._get_azure_credential")
    @mock.patch("barman.cloud_providers.azure_blob_storage.import_azure_mgmt_compute")
    @mock.patch(
        "barman.cloud_providers.google_cloud_storage.import_google_cloud_compute"
    )
    def test_from_backup_info_cloud_provider(
        self,
        _mock_google_cloud_compute,
        _mock_azure_mgmt_compute,
        _mock_get_azure_credential,
        _mock_boto3,
        snapshot_provider,
        interface_cls,
    ):
        """Verify supported and unsupported cloud providers with backup_info."""
        # GIVEN a backup_info with snapshots_info containing the specified snapshot
        # provider
        mock_backup_info = mock.Mock(
            snapshots_info=mock.Mock(provider=snapshot_provider)
        )
        # AND a mock server config
        mock_config = mock.Mock()

        # WHEN get_snapshot_interface_from_server_config is called
        if interface_cls:
            # THEN supported providers return the expected interface
            assert isinstance(
                get_snapshot_interface_from_backup_info(mock_backup_info, mock_config),
                interface_cls,
            )
        else:
            # AND unsupported providers raise the expected exception
            with pytest.raises(CloudProviderUnsupported) as exc:
                get_snapshot_interface_from_backup_info(mock_backup_info)
            assert "Unsupported snapshot provider in backup info: {}".format(
                snapshot_provider
            ) == str(exc.value)

    def test_from_backup_info_gcp_no_project(self):
        """
        Verify an exception is raised for gcp snapshots with no project in backup_info.
        """
        # GIVEN a server config with the gcp snapshot provider and no gcp_project
        mock_backup_info = mock.Mock(
            snapshots_info=mock.Mock(provider="gcp", project=None)
        )
        # WHEN get snapshot_interface_from_backup_info is called
        with pytest.raises(BarmanException) as exc:
            get_snapshot_interface_from_backup_info(mock_backup_info)
        # THEN the expected exception is raised
        assert "backup_info has snapshot provider 'gcp' but project is not set" in str(
            exc.value
        )

    def test_from_backup_info_azure_no_subscription_id(self):
        """
        Verify an exception is raised for azure snapshots with no azure_subscription_id
        in backup info.
        """
        # GIVEN a server config with the azure snapshot provider and no
        # azure_subscription_id
        mock_config = mock.Mock(snapshot_provider="azure", azure_subscription_id=None)
        # AND a backup info with no azure_subscription_id
        mock_backup_info = mock.Mock(
            snapshots_info=mock.Mock(provider="azure", subscription_id=None)
        )
        # WHEN get snapshot_interface_from_backup_info is called
        with pytest.raises(ConfigurationException) as exc:
            get_snapshot_interface_from_backup_info(mock_backup_info, mock_config)
        # THEN the expected exception is raised
        assert (
            "backup_info has snapshot provider 'azure' but subscription_id is not set"
            in str(exc.value)
        )

    @pytest.mark.parametrize(
        ("config_region", "backup_info_region", "expected_region"),
        (
            # If neither config nor backup_info have a region we expect None
            (None, None, None),
            # If the config has a region but backup_info does not then we expect the
            # region in the config
            ("config-region", None, "config-region"),
            # If the backup_info has a region but the config does not then we expect
            # the region in the backup_info
            (None, "backup-info-region", "backup-info-region"),
            # If both config and backup_info have a region we expect the region in the
            # config
            ("config-region", "backup-info-region", "config-region"),
        ),
    )
    @mock.patch("barman.cloud_providers.aws_s3.boto3")
    def test_from_backup_info_aws_region(
        self, mock_boto3, config_region, backup_info_region, expected_region
    ):
        """
        Verify that the region is taken from the backup_info but can be overridden by
        the config.
        """
        # GIVEN a server config with the aws snapshot provider and the specified region
        mock_config = mock.Mock(snapshot_provider="aws", aws_region=config_region)
        # AND a backup info with the specified region
        mock_backup_info = mock.Mock(
            snapshots_info=mock.Mock(provider="aws", region=backup_info_region)
        )
        # AND the session has no default region name
        mock_boto3.Session.return_value.region_name = None

        # WHEN get snapshot_interface_from_backup_info is called
        snapshot_interface = get_snapshot_interface_from_backup_info(
            mock_backup_info, mock_config
        )
        # THEN the snapshot interface has the expected region
        assert snapshot_interface.region == expected_region

    @pytest.mark.parametrize(
        ("cloud_provider", "interface_cls"),
        [
            ("aws-s3", AwsCloudSnapshotInterface),
            (
                "azure-blob-storage",
                AzureCloudSnapshotInterface,
            ),
            ("google-cloud-storage", GcpCloudSnapshotInterface),
            ("unsupportedcloud", None),
        ],
    )
    @mock.patch("barman.cloud_providers._get_azure_credential")
    @mock.patch("barman.cloud_providers.azure_blob_storage.import_azure_mgmt_compute")
    @mock.patch(
        "barman.cloud_providers.google_cloud_storage.import_google_cloud_compute"
    )
    @mock.patch("barman.cloud_providers.aws_s3.boto3")
    def test_from_args_cloud_provider(
        self,
        _mock_boto3,
        _mock_google_cloud_compute,
        _mock_azure_mgmt_compute,
        _mock_get_azure_credential,
        cloud_provider,
        interface_cls,
    ):
        """Verify supported and unsupported cloud providers with config args."""
        # GIVEN a cloud config with the specified snapshot provider
        mock_config = mock.Mock(cloud_provider=cloud_provider)

        # WHEN get_snapshot_interface_from_server_config is called
        if interface_cls:
            # THEN supported providers return the expected interface
            assert isinstance(get_snapshot_interface(mock_config), interface_cls)
        else:
            # AND unsupported providers raise the expected exception
            with pytest.raises(CloudProviderUnsupported) as exc:
                get_snapshot_interface(mock_config)
            assert "No snapshot provider for cloud provider: {}".format(
                cloud_provider
            ) == str(exc.value)

    def test_from_args_gcp_no_project(self):
        """
        Verify an exception is raised for gcp snapshots with no project in args.
        """
        # GIVEN a cloud config with the specified snapshot provider where the
        # gcp_project and deprecated snapshot_gcp_project arguments are missing
        mock_config = mock.Mock(
            cloud_provider="google-cloud-storage",
            gcp_project=None,
            snapshot_gcp_project=None,
        )

        # WHEN get snapshot_interface_from_backup_info is called
        with pytest.raises(ConfigurationException) as exc:
            get_snapshot_interface(mock_config)
        # AND the exception has the expected message
        assert (
            "--gcp-project option must be set for snapshot backups when "
            "cloud provider is google-cloud-storage"
        ) == str(exc.value)

    def test_from_args_azure_no_subscription(self):
        """
        Verify an exception is raised for azure snapshots with no azure_subscription_id
        in args.
        """
        # GIVEN a cloud config with the azure snapshot provider where the
        # azure_subscription_id argument is missing
        mock_config = mock.Mock(
            cloud_provider="azure-blob-storage",
            azure_subscription_id=None,
        )

        # WHEN get snapshot_interface_from_backup_info is called
        with pytest.raises(ConfigurationException) as exc:
            get_snapshot_interface(mock_config)
        # AND the exception has the expected message
        assert (
            "--azure-subscription-id option must be set for snapshot backups when "
            "cloud provider is azure-blob-storage"
        ) == str(exc.value)

    @mock.patch("barman.cloud_providers.aws_s3.boto3")
    def test_from_args_aws(self, mock_boto3):
        """
        Verify aws-specific barman-cloud args are passed to the snapshot interface.
        """
        # GIVEN a cloud config with the aws snapshot provider and the specified
        # parameters
        mock_config = mock.Mock(
            cloud_provider="aws-s3",
            aws_region="us-east-2",
            aws_profile="default",
            aws_await_snapshots_timeout=7200,
        )
        # WHEN get_snapshot_interface is called
        snapshot_interface = get_snapshot_interface(mock_config)
        # THEN the config values are passed to the snapshot interface
        assert isinstance(snapshot_interface, AwsCloudSnapshotInterface)
        assert snapshot_interface.region == "us-east-2"
        assert snapshot_interface.await_snapshots_timeout == 7200
        mock_boto3.Session.assert_called_once_with(profile_name="default")


class TestGcpCloudSnapshotInterface(object):
    """
    Verify behaviour of the GcpCloudSnapshotInterface class.
    """

    backup_id = "20380119T031407"
    gcp_disks = (
        {
            "name": "test_disk_0",
            "device_name": "dev0",
            "physical_block_size": 1024,
            "size_gb": 1,
            "mount_options": "rw,noatime",
            "mount_point": "/opt/disk0",
        },
        {
            "name": "test_disk_1",
            "device_name": "dev1",
            "physical_block_size": 2048,
            "size_gb": 10,
            "mount_options": "rw",
            "mount_point": "/opt/disk1",
        },
        {
            "name": "test_disk_2",
            "device_name": "dev2",
            "physical_block_size": 4096,
            "size_gb": 100,
            "mount_options": "rw,relatime",
            "mount_point": "/opt/disk2",
        },
    )
    gcp_zone = "us-east1-b"
    gcp_project = "test_project"
    gcp_instance_name = "test_instance"
    server_name = "test_server"

    def _get_disk_link(self, project, zone, disk_name):
        return "projects/{}/zones/{}/disks/{}".format(project, zone, disk_name)

    def _get_snapshot_name(self, disk_name):
        return "{}-{}".format(disk_name, self.backup_id.lower())

    def _get_device_path(self, device):
        return "/dev/disk/by-id/google-{}".format(device)

    @mock.patch(
        "barman.cloud_providers.google_cloud_storage.import_google_cloud_compute"
    )
    def test_init_with_null_project(self, _mock_google_cloud_compute):
        """
        Verify creating GcpCloudSnapshotInterface fails if gcp_project is not set.
        """
        # GIVEN a null project
        gcp_project = None

        # WHEN a GcpCloudSnapshotInterface is created
        # THEN a TypeError is raised
        with pytest.raises(TypeError) as exc:
            GcpCloudSnapshotInterface(gcp_project)

        # AND the expected message is included
        assert str(exc.value) == "project cannot be None"

    @pytest.fixture
    def mock_google_cloud_compute(self):
        with mock.patch(
            "barman.cloud_providers.google_cloud_storage.import_google_cloud_compute"
        ) as mock_import_google_cloud_compute:
            yield mock_import_google_cloud_compute.return_value

    def test_init(self, mock_google_cloud_compute):
        """
        Verify creating GcpCloudSnapshotInterface creates the necessary GCP clients.
        """
        # GIVEN a non-null project
        gcp_project = self.gcp_project
        # WHEN a GcpCloudSnapshotInterface is created
        snapshot_interface = GcpCloudSnapshotInterface(gcp_project)
        # THEN a SnapshotsClient was created
        assert snapshot_interface.client == (
            mock_google_cloud_compute.SnapshotsClient.return_value
        )
        # AND a DisksClient was created
        assert snapshot_interface.disks_client == (
            mock_google_cloud_compute.DisksClient.return_value
        )
        # AND an InstancesClient was created
        assert snapshot_interface.instances_client == (
            mock_google_cloud_compute.InstancesClient.return_value
        )

    def test_take_snapshot(self, mock_google_cloud_compute, caplog):
        """
        Verify that _take_snapshot calls the GCP library and waits for the result.
        """
        # GIVEN a new GcpCloudSnapshotInterface
        snapshot_interface = GcpCloudSnapshotInterface(self.gcp_project)
        # AND a backup_info for a given server name and backup ID
        backup_info = mock.Mock(backup_id=self.backup_id, server_name=self.server_name)
        # AND a mock SnapshotsClient which returns a successful response
        mock_snapshots_client = mock_google_cloud_compute.SnapshotsClient.return_value
        mock_resp = mock_snapshots_client.insert.return_value
        mock_resp.result.return_value = True
        mock_resp.error_code = None
        mock_resp.warnings = None
        # AND log level is INFO
        caplog.set_level(logging.INFO)

        # WHEN _take_snapshot is called
        snapshot_name = snapshot_interface._take_snapshot(
            backup_info,
            self.gcp_zone,
            self.gcp_disks[0]["name"],
        )

        # THEN insert is called on the SnapshotsClient with the expected args
        expected_disk_name = self.gcp_disks[0]["name"]
        expected_full_disk_name = self._get_disk_link(
            self.gcp_project,
            self.gcp_zone,
            self.gcp_disks[0]["name"],
        )
        expected_snapshot_name = self._get_snapshot_name(expected_disk_name)
        mock_snapshots_client.insert.assert_called_once_with(
            {
                "project": self.gcp_project,
                "snapshot_resource": {
                    "name": expected_snapshot_name,
                    "source_disk": expected_full_disk_name,
                },
            }
        )
        # AND result() was called on the response to await completion of the snapshot
        mock_resp.result.assert_called_once()
        # AND the name of the snapshot was returned
        assert snapshot_name == expected_snapshot_name
        # AND the expected log output occurred
        expected_log_content = (
            "Taking snapshot '{}' of disk '{}'".format(
                expected_snapshot_name, expected_disk_name
            ),
            "Waiting for snapshot '{}' completion".format(expected_snapshot_name),
            "Snapshot '{}' completed".format(expected_snapshot_name),
        )
        for expected_log, log_line in zip(
            expected_log_content, caplog.text.split("\n")
        ):
            assert expected_log in log_line

    def test_take_snapshot_warnings(self, mock_google_cloud_compute, caplog):
        """
        Verify that warnings are logged if present in the snapshots response.
        """
        # GIVEN a new GcpCloudSnapshotInterface
        snapshot_interface = GcpCloudSnapshotInterface(self.gcp_project)
        # AND a backup_info for a given server name and backup ID
        backup_info = mock.Mock(backup_id=self.backup_id, server_name=self.server_name)
        # AND a mock SnapshotsClient which returns a successful response
        mock_snapshots_client = mock_google_cloud_compute.SnapshotsClient.return_value
        mock_resp = mock_snapshots_client.insert.return_value
        mock_resp.result.return_value = True
        mock_resp.error_code = None
        # AND the response has warnings
        mock_resp.warnings = [mock.Mock(code="123", message="warning message")]

        # WHEN _take_snapshot is called
        snapshot_name = snapshot_interface._take_snapshot(
            backup_info,
            self.gcp_zone,
            self.gcp_disks[0]["name"],
        )

        # THEN the warning is included in the log output
        assert (
            "Warnings encountered during snapshot {}: 123:warning message".format(
                snapshot_name
            )
            in caplog.text
        )

    def test_take_snapshot_failed(self, mock_google_cloud_compute):
        """
        Verify that _take_snapshot raises an exception on failure.
        """
        # GIVEN a new GcpCloudSnapshotInterface
        snapshot_interface = GcpCloudSnapshotInterface(self.gcp_project)
        # AND a backup_info for a given server name and backup ID
        backup_info = mock.Mock(backup_id=self.backup_id, server_name=self.server_name)
        # AND a mock SnapshotsClient which returns a failed response
        mock_snapshots_client = mock_google_cloud_compute.SnapshotsClient.return_value
        mock_resp = mock_snapshots_client.insert.return_value
        mock_resp.result.return_value = True
        mock_resp.error_code = "503"
        mock_resp.error_message = "test error message"

        # WHEN _take_snapshot is called
        # THEN a CloudProviderError is raised
        with pytest.raises(CloudProviderError) as exc:
            snapshot_interface._take_snapshot(
                backup_info,
                self.gcp_zone,
                self.gcp_disks[0]["name"],
            )

        # AND the exception message contains the snapshot name, error code and error
        # message
        expected_snapshot_name = self._get_snapshot_name(self.gcp_disks[0]["name"])
        expected_message = "Snapshot '{}' failed with error code {}: {}".format(
            expected_snapshot_name,
            mock_resp.error_code,
            mock_resp.error_message,
        )
        assert str(exc.value) == expected_message

    def _get_mock_instances_client(self, gcp_project, gcp_zone, gcp_instance, disks):
        """
        Helper which create a mock instances client for the given project/zone/instance
        with the specified disks attached as the specified device.
        """

        def get_fun(instance, zone, project):
            if instance == gcp_instance and zone == gcp_zone and project == gcp_project:
                mock_attached_disks = [
                    mock.Mock(
                        device_name=disk["device_name"],
                        source=self._get_disk_link(project, zone, disk["name"]),
                    )
                    for disk in disks
                ]
                return mock.Mock(disks=mock_attached_disks)
            else:
                raise NotFound("instance not found")

        return mock.Mock(get=get_fun)

    def _get_mock_disks_client(self, gcp_project, gcp_zone, disks):
        """
        Helper which creates a mock disks client for the given project/zone with the
        specified disks available with the specified physical_block_size and size_gb.
        """
        disk_metadata = dict(
            (
                disk["name"],
                mock.Mock(
                    physical_block_size_bytes=disk["physical_block_size"],
                    self_link="projects/{}/zones/{}/disks/{}".format(
                        gcp_project, gcp_zone, disk["name"]
                    ),
                    size_gb=disk["size_gb"],
                    source_snapshot="source_snapshot" in disk
                    and disk["source_snapshot"]
                    or None,
                ),
            )
            for disk in disks
        )

        def get_fun(disk, zone, project):
            if zone == self.gcp_zone and project == self.gcp_project:
                try:
                    return disk_metadata[disk]
                except KeyError:
                    raise NotFound("disk not found")

        return mock.Mock(get=get_fun)

    def _get_mock_snapshots_client(self):
        """
        Helper which returns a mock snapshots client that always succeeds.
        """
        snapshots_client = mock.Mock()
        snapshots_client.insert.return_value = mock.Mock(error_code=None, warnings=None)
        snapshots_client.delete.return_value = mock.Mock(error_code=None, warnings=None)
        return snapshots_client

    def _get_mock_volumes(self, disks):
        return dict(
            (
                disk["name"],
                mock.Mock(
                    mount_point=disk["mount_point"], mount_options=disk["mount_options"]
                ),
            )
            for disk in disks
        )

    @pytest.mark.parametrize("number_of_disks", (1, 2, 3))
    def test_take_snapshot_backup(
        self,
        number_of_disks,
        mock_google_cloud_compute,
    ):
        """
        Verify that take_snapshot_backup takes the required snapshots and updates the
        backup_info when prerequisites are met.
        """
        # GIVEN a set of disks, represented as VolumeMetadata
        disks = self.gcp_disks[:number_of_disks]
        # AND a backup_info for a given server name and backup ID
        backup_info = mock.Mock(backup_id=self.backup_id, server_name=self.server_name)
        # AND a mock InstancesClient which returns an instance with the required disks
        # attached
        mock_instances_client = self._get_mock_instances_client(
            self.gcp_project, self.gcp_zone, self.gcp_instance_name, disks
        )
        mock_google_cloud_compute.InstancesClient.return_value = mock_instances_client
        # AND a mock DisksClient which returns the required disks
        mock_disks_client = self._get_mock_disks_client(
            self.gcp_project, self.gcp_zone, disks
        )
        mock_google_cloud_compute.DisksClient.return_value = mock_disks_client
        # AND a mock SnapshotsClient which returns successful responses
        mock_google_cloud_compute.SnapshotsClient.return_value = (
            self._get_mock_snapshots_client()
        )
        # AND a new GcpCloudSnapshotInterface
        snapshot_interface = GcpCloudSnapshotInterface(self.gcp_project, self.gcp_zone)

        # WHEN take_snapshot_backup is called for multiple disks
        snapshot_interface.take_snapshot_backup(
            backup_info, self.gcp_instance_name, self._get_mock_volumes(disks)
        )

        # THEN the backup_info is updated with the expected snapshot metadata
        snapshots_info = backup_info.snapshots_info
        assert snapshots_info.project == self.gcp_project
        assert snapshots_info.provider == "gcp"
        assert len(snapshots_info.snapshots) == len(disks)
        for disk in disks:
            snapshot_name = self._get_snapshot_name(disk["name"])
            snapshot = next(
                snapshot
                for snapshot in snapshots_info.snapshots
                if snapshot.snapshot_name == snapshot_name
            )
            assert snapshot.identifier == snapshot_name
            assert snapshot.snapshot_name == snapshot_name
            assert snapshot.snapshot_project == self.gcp_project
            assert snapshot.device_name == disk["device_name"]
            assert snapshot.mount_options == disk["mount_options"]
            assert snapshot.mount_point == disk["mount_point"]

    def test_take_snapshot_backup_instance_not_found(self, mock_google_cloud_compute):
        """
        Verify that a SnapshotBackupException is raised if the instance cannot be
        found.
        """
        # GIVEN a new GcpCloudSnapshotInterface
        snapshot_interface = GcpCloudSnapshotInterface(self.gcp_project, self.gcp_zone)
        # AND a mock InstancesClient which cannot find the instance
        mock_instances_client = mock_google_cloud_compute.InstancesClient.return_value
        mock_instances_client.get.side_effect = NotFound("instance not found")

        # WHEN take_snapshot_backup is called
        # THEN a SnapshotBackupException is raised
        with pytest.raises(SnapshotBackupException) as exc:
            snapshot_interface.take_snapshot_backup(
                mock.Mock(),
                self.gcp_instance_name,
                self._get_mock_volumes(self.gcp_disks),
            )

        # AND the exception contains the expected message
        assert str(
            exc.value
        ) == "Cannot find instance with name {} in zone {} for project {}".format(
            self.gcp_instance_name, self.gcp_zone, self.gcp_project
        )

    def test_get_attached_volumes_disk_not_found(
        self,
        mock_google_cloud_compute,
    ):
        """
        Verify that a SnapshotBackupException is raised if a disk cannot be found.
        """
        # GIVEN a set of disks
        disks = self.gcp_disks
        # AND a mock InstancesClient which returns an instance with a subset of the
        # required disks attached
        mock_instances_client = self._get_mock_instances_client(
            self.gcp_project, self.gcp_zone, self.gcp_instance_name, disks[:-1]
        )
        mock_google_cloud_compute.InstancesClient.return_value = mock_instances_client
        # AND a mock DisksClient which returns only that same subset of disks
        mock_disks_client = self._get_mock_disks_client(
            self.gcp_project, self.gcp_zone, disks[:-1]
        )
        mock_google_cloud_compute.DisksClient.return_value = mock_disks_client
        # AND a mock SnapshotsClient which returns successful responses
        mock_google_cloud_compute.SnapshotsClient.return_value = (
            self._get_mock_snapshots_client()
        )
        # AND a new GcpCloudSnapshotInterface
        snapshot_interface = GcpCloudSnapshotInterface(self.gcp_project, self.gcp_zone)

        # WHEN get_attached_volumes is called
        # THEN a SnapshotBackupException is raised
        with pytest.raises(SnapshotBackupException) as exc:
            snapshot_interface.get_attached_volumes(
                self.gcp_instance_name, [disk["name"] for disk in disks]
            )

        # AND the exception contains the expected message
        assert str(
            exc.value
        ) == "Cannot find disk with name {} in zone {} for project {}".format(
            disks[-1]["name"], self.gcp_zone, self.gcp_project
        )

        # WHEN get_attached_volumes is called with fail_on_missing=False
        # THEN no exception is raised
        attached_volumes = snapshot_interface.get_attached_volumes(
            self.gcp_instance_name,
            [disk["name"] for disk in disks],
            fail_on_missing=False,
        )
        # AND the attached volumes contains only those disks which were present
        expected_volumes = [d["name"] for d in disks[:-1]]
        assert set(attached_volumes.keys()) == set(expected_volumes)

    def test_get_attached_volumes_disk_not_attached(
        self,
        mock_google_cloud_compute,
    ):
        """
        Verify that a SnapshotBackupException is raised if a disk is not attached.
        """
        # GIVEN a set of disks
        disks = self.gcp_disks
        # AND a mock InstancesClient which returns an instance with a subset of the
        # required disks attached
        mock_instances_client = self._get_mock_instances_client(
            self.gcp_project, self.gcp_zone, self.gcp_instance_name, disks[:-1]
        )
        mock_google_cloud_compute.InstancesClient.return_value = mock_instances_client
        # AND a mock DisksClient which returns all required disks
        mock_disks_client = self._get_mock_disks_client(
            self.gcp_project, self.gcp_zone, disks
        )
        mock_google_cloud_compute.DisksClient.return_value = mock_disks_client
        # AND a mock SnapshotsClient which returns successful responses
        mock_google_cloud_compute.SnapshotsClient.return_value = (
            self._get_mock_snapshots_client()
        )
        # AND a new GcpCloudSnapshotInterface
        snapshot_interface = GcpCloudSnapshotInterface(self.gcp_project, self.gcp_zone)

        # WHEN get_attached_volumes is called
        # THEN a SnapshotBackupException is raised
        with pytest.raises(SnapshotBackupException) as exc:
            snapshot_interface.get_attached_volumes(
                self.gcp_instance_name, [disk["name"] for disk in disks]
            )

        # AND the exception contains the expected message
        assert str(exc.value) == "Disks not attached to instance {}: {}".format(
            self.gcp_instance_name, disks[-1]["name"]
        )

        # WHEN get_attached_volumes is called with fail_on_missing=False
        # THEN no exception is raised
        attached_volumes = snapshot_interface.get_attached_volumes(
            self.gcp_instance_name,
            [disk["name"] for disk in disks],
            fail_on_missing=False,
        )
        # AND the attached volumes contains only those disks which were present
        expected_volumes = [d["name"] for d in disks[:-1]]
        assert set(attached_volumes.keys()) == set(expected_volumes)

    def test_get_attached_volumes_disk_attached_multiple_times(
        self,
        mock_google_cloud_compute,
    ):
        """
        Verify that a SnapshotBackupException is raised if a disk appears to be
        attached more than once.
        """
        # GIVEN a set of disks
        disks = self.gcp_disks
        # AND a mock InstancesClient which returns an instance where one named disk is
        # attached twice
        mock_instances_client = self._get_mock_instances_client(
            self.gcp_project, self.gcp_zone, self.gcp_instance_name, disks + disks[-1:]
        )
        mock_google_cloud_compute.InstancesClient.return_value = mock_instances_client
        # AND a mock DisksClient which returns all required disks
        mock_disks_client = self._get_mock_disks_client(
            self.gcp_project, self.gcp_zone, disks
        )
        mock_google_cloud_compute.DisksClient.return_value = mock_disks_client
        # AND a mock SnapshotsClient which returns successful responses
        mock_google_cloud_compute.SnapshotsClient.return_value = (
            self._get_mock_snapshots_client()
        )
        # AND a new GcpCloudSnapshotInterface
        snapshot_interface = GcpCloudSnapshotInterface(self.gcp_project, self.gcp_zone)

        # WHEN get_attached_volumes is called
        # THEN a SnapshotBackupException is raised
        with pytest.raises(AssertionError):
            snapshot_interface.get_attached_volumes(
                self.gcp_instance_name,
                [disk["name"] for disk in disks],
            )

    def test_delete_snapshot(self, mock_google_cloud_compute, caplog):
        """Verify that a snapshot can be deleted successfully."""
        # GIVEN the snapshots client deletes successfully
        mock_google_cloud_compute.SnapshotsClient.return_value = (
            self._get_mock_snapshots_client()
        )
        # AND a new GcpCloudSnapshotInterface
        snapshot_interface = GcpCloudSnapshotInterface(self.gcp_project)
        # AND log level is info
        caplog.set_level(logging.INFO)

        # WHEN a snapshot is deleted
        snapshot_name = self._get_snapshot_name(self.gcp_disks[0])
        snapshot_interface._delete_snapshot(snapshot_name)

        # THEN delete was called on the SnapshotsClient for that project/snapshot
        mock_snapshots_client = mock_google_cloud_compute.SnapshotsClient.return_value
        mock_snapshots_client.delete.assert_called_once_with(
            {"project": self.gcp_project, "snapshot": snapshot_name}
        )
        # AND result was called on the response
        resp = mock_snapshots_client.delete.return_value
        resp.result.assert_called_once()
        # AND a success message was logged
        assert "Snapshot {} deleted".format(snapshot_name) in caplog.text

    def test_delete_snapshot_not_found(self, mock_google_cloud_compute, caplog):
        """
        Verify that a snapshot deletion which fails with NotFound is considered
        successful.
        """
        # GIVEN a new GcpCloudSnapshotInterface
        snapshot_interface = GcpCloudSnapshotInterface(self.gcp_project)
        # AND a snapshots client which will fail with a NotFound error
        mock_snapshots_client = mock_google_cloud_compute.SnapshotsClient.return_value
        mock_snapshots_client.delete.side_effect = NotFound("snapshot not found")

        # WHEN a snapshot is deleted
        snapshot_name = self._get_snapshot_name(self.gcp_disks[0])
        snapshot_interface._delete_snapshot(snapshot_name)

        # THEN delete was called on the SnapshotsClient for that project/snapshot
        mock_snapshots_client = mock_google_cloud_compute.SnapshotsClient.return_value
        mock_snapshots_client.delete.assert_called_once_with(
            {"project": self.gcp_project, "snapshot": snapshot_name}
        )
        # AND result was not called on the response
        resp = mock_snapshots_client.delete.return_value
        resp.result.assert_not_called()

    def test_delete_snapshot_warnings(self, mock_google_cloud_compute, caplog):
        """Verify that warnings are logged if present in the snapshots response."""
        # GIVEN a snapshots client which will delete a snapshot
        mock_snapshots_client = self._get_mock_snapshots_client()
        mock_google_cloud_compute.SnapshotsClient.return_value = mock_snapshots_client
        # AND the response has warnings
        mock_snapshots_client.delete.return_value.warnings = [
            mock.Mock(code="123", message="warning message")
        ]
        # AND a new GcpCloudSnapshotInterface
        snapshot_interface = GcpCloudSnapshotInterface(self.gcp_project)

        # WHEN a snapshot is deleted
        snapshot_name = self._get_snapshot_name(self.gcp_disks[0])
        snapshot_interface._delete_snapshot(snapshot_name)

        # THEN the warning is included in the log output
        assert (
            "Warnings encountered during deletion of {}: 123:warning message".format(
                snapshot_name
            )
            in caplog.text
        )

    def test_delete_snapshot_failed(self, mock_google_cloud_compute, caplog):
        """Verify that a snapshot can be deleted successfully."""
        # GIVEN a new GcpCloudSnapshotInterface
        snapshot_interface = GcpCloudSnapshotInterface(self.gcp_project)
        # AND a snapshots client which will fail to delete a snapshot
        mock_snapshots_client = mock_google_cloud_compute.SnapshotsClient.return_value
        mock_resp = mock_snapshots_client.delete.return_value
        mock_resp.result.return_value = True
        mock_resp.error_code = "503"
        mock_resp.error_message = "test error message"

        # WHEN a snapshot is deleted
        # THEN a CloudProviderError is raised
        snapshot_name = self._get_snapshot_name(self.gcp_disks[0])
        with pytest.raises(CloudProviderError) as exc:
            snapshot_interface._delete_snapshot(snapshot_name)

        # AND the exception message contains the snapshot name, error code and error
        # message
        expected_message = (
            "Deletion of snapshot {} failed with error code {}: {}".format(
                snapshot_name,
                mock_resp.error_code,
                mock_resp.error_message,
            )
        )
        assert str(exc.value) == expected_message

    @pytest.mark.parametrize(
        "snapshots_list",
        (
            [],
            [mock.Mock(identifier="snapshot0")],
            [mock.Mock(identifier="snapshot0"), mock.Mock(identifier="snapshot1")],
        ),
    )
    def test_delete_snapshot_backup(
        self, snapshots_list, mock_google_cloud_compute, caplog
    ):
        """Verfiy that all snapshots for a backup are deleted."""
        # GIVEN a backup_info specifying zero or more snapshots
        backup_info = mock.Mock(
            backup_id=self.backup_id, snapshots_info=mock.Mock(snapshots=snapshots_list)
        )
        # AND log level is info
        caplog.set_level(logging.INFO)
        # AND the snapshots client deletes successfully
        mock_google_cloud_compute.SnapshotsClient.return_value = (
            self._get_mock_snapshots_client()
        )
        # AND a new GcpCloudSnapshotInterface
        snapshot_interface = GcpCloudSnapshotInterface(self.gcp_project, zone=None)

        # WHEN delete_snapshot_backup is called
        snapshot_interface.delete_snapshot_backup(backup_info)

        # THEN delete was called on the SnapshotsClient for each snapshot
        mock_snapshots_client = mock_google_cloud_compute.SnapshotsClient.return_value
        assert mock_snapshots_client.delete.call_count == len(snapshots_list)
        for snapshot in snapshots_list:
            assert (
                ({"project": self.gcp_project, "snapshot": snapshot.identifier},),
                {},
            ) in mock_snapshots_client.delete.call_args_list
            # AND the expected log message was logged for each snapshot
            assert (
                "Deleting snapshot '{}' for backup {}".format(
                    snapshot.identifier, self.backup_id
                )
                in caplog.text
            )

    @pytest.mark.parametrize(
        (
            "mock_disks",
            "mock_disk_metadata",
            "expected_disk_names",
            "expected_device_names",
            "expected_source_snapshots",
        ),
        (
            ([], [], [], [], []),
            (
                [
                    mock.Mock(
                        source="projects/test_project/zones/us-east1-b/disks/disk0",
                        device_name="dev0",
                    )
                ],
                [
                    mock.Mock(
                        source_snapshot=None,
                    ),
                ],
                ["disk0"],
                ["dev0"],
                [None],
            ),
            (
                [
                    mock.Mock(
                        source="projects/test_project/zones/us-east1-b/disks/disk0",
                        device_name="dev0",
                    )
                ],
                [
                    mock.Mock(
                        source_snapshot="snap0",
                    ),
                ],
                ["disk0"],
                ["dev0"],
                ["snap0"],
            ),
            (
                [
                    mock.Mock(
                        source="projects/test_project/zones/us-east1-b/disks/disk0",
                        device_name="dev0",
                    ),
                    mock.Mock(
                        source="projects/test_project/zones/us-east1-b/disks/disk1",
                        device_name="dev1",
                    ),
                ],
                [
                    mock.Mock(
                        source_snapshot="snap0",
                    ),
                    mock.Mock(
                        source_snapshot="snap1",
                    ),
                ],
                ["disk0", "disk1"],
                ["dev0", "dev1"],
                ["snap0", "snap1"],
            ),
        ),
    )
    def test_get_attached_volumes(
        self,
        mock_disks,
        mock_disk_metadata,
        expected_disk_names,
        expected_device_names,
        expected_source_snapshots,
        mock_google_cloud_compute,
    ):
        """Verify that attached volumes are returned as a dict keyed by disk name."""
        # GIVEN a new GcpCloudSnapshotInterface
        snapshot_interface = GcpCloudSnapshotInterface(self.gcp_project, self.gcp_zone)
        # AND a mock InstancesClient which returns metadata listing the devices
        mock_instances_client = mock_google_cloud_compute.InstancesClient.return_value
        mock_instance_metadata = mock.Mock(disks=mock_disks)
        mock_instances_client.get.return_value = mock_instance_metadata
        # AND a mock DisksClient which returns the specified source snapshots
        mock_disks_client = mock_google_cloud_compute.DisksClient.return_value
        mock_disks_client.get.side_effect = mock_disk_metadata

        # WHEN get_attached_volumes is called
        attached_volumes = snapshot_interface.get_attached_volumes(
            self.gcp_instance_name
        )

        # THEN a dict of devices returned by the instance metadata is returned, keyed
        # by disk name
        assert len(attached_volumes) == len(expected_disk_names)
        for expected_disk_name, expected_device_name, expected_source_snapshot in zip(
            expected_disk_names, expected_device_names, expected_source_snapshots
        ):
            assert expected_disk_name in attached_volumes
            # AND the device name matches that returned by the instance metadata
            assert attached_volumes[
                expected_disk_name
            ]._device_path == self._get_device_path(expected_device_name)
            # AND the source snapshot matches that returned by the disk metadata
            assert (
                attached_volumes[expected_disk_name].source_snapshot
                == expected_source_snapshot
            )

    def test_get_attached_volumes_for_disks(self, mock_google_cloud_compute):
        """
        Verifies that only the requested disks are returned when the disks parameter is
        passed to get_attached_volumes.
        """
        # GIVEN a new GcpCloudSnapshotInterface
        snapshot_interface = GcpCloudSnapshotInterface(self.gcp_project, self.gcp_zone)
        # AND a mock InstancesClient which returns metadata listing two disks
        mock_instances_client = mock_google_cloud_compute.InstancesClient.return_value
        mock_instance_metadata = mock.Mock(
            disks=[
                mock.Mock(
                    source="projects/test_project/zones/us-east1-b/disks/disk0",
                    device_name="dev0",
                ),
                mock.Mock(
                    source="projects/test_project/zones/us-east1-b/disks/disk1",
                    device_name="dev1",
                ),
            ]
        )
        mock_instances_client.get.return_value = mock_instance_metadata
        # AND a mock DisksClient which returns some arbitrary metadata
        mock_disks_client = mock_google_cloud_compute.DisksClient.return_value
        mock_disks_client.get.return_value = mock.Mock(source_snapshot=None)

        # WHEN get_attached_volumes is called requesting only disk1
        attached_volumes = snapshot_interface.get_attached_volumes(
            self.gcp_instance_name, disks=["disk1"]
        )

        # THEN only "disk1" is included in the resulting dict
        assert len(attached_volumes) == 1
        assert "disk1" in attached_volumes

    @pytest.mark.parametrize(
        "mock_disks",
        (
            [mock.Mock(source="", device_name="dev0")],
            [mock.Mock(source="/", device_name="dev0")],
            [mock.Mock(source="foo/", device_name="dev0")],
        ),
    )
    def test_get_attached_volumes_bad_disk_name(
        self,
        mock_disks,
        mock_google_cloud_compute,
    ):
        """Verify that unparseable disk names are handled."""
        # GIVEN a new GcpCloudSnapshotInterface
        snapshot_interface = GcpCloudSnapshotInterface(self.gcp_project, self.gcp_zone)
        # AND a mock InstancesClient which returns metadata listing the devices
        mock_instances_client = mock_google_cloud_compute.InstancesClient.return_value
        mock_instance_metadata = mock.Mock(disks=mock_disks)
        mock_instances_client.get.return_value = mock_instance_metadata

        # WHEN get_attached_volumes is called
        # THEN a SnapshotBackupException is raised
        with pytest.raises(SnapshotBackupException) as exc:
            snapshot_interface.get_attached_volumes(self.gcp_instance_name)
        # AND the expected message is included
        assert str(
            exc.value
        ) == "Could not parse disk name for source {} attached to instance {}".format(
            mock_disks[0].source, self.gcp_instance_name
        )

    @pytest.mark.parametrize(
        "mock_disks",
        (
            [
                mock.Mock(
                    source="projects/test_project/zones/us-east1-b/disks/disk0",
                    device_name="dev0",
                ),
                mock.Mock(
                    source="projects/test_project/zones/us-east1-b/disks/disk0",
                    device_name="dev1",
                ),
            ],
            [
                mock.Mock(
                    source="projects/test_project/zones/us-east1-b/disks/disk1",
                    device_name="dev2",
                ),
                mock.Mock(
                    source="projects/test_project/zones/us-east1-b/disks/disk0",
                    device_name="dev0",
                ),
                mock.Mock(
                    source="projects/test_project/zones/us-east1-b/disks/disk0",
                    device_name="dev1",
                ),
            ],
        ),
    )
    def test_get_attached_volumes_multiple_names(
        self,
        mock_disks,
        mock_google_cloud_compute,
    ):
        """
        Verify that an exception is raised if a disk appears to be attached more than
        once.
        """
        # GIVEN a new GcpCloudSnapshotInterface
        snapshot_interface = GcpCloudSnapshotInterface(self.gcp_project, self.gcp_zone)
        # AND a mock InstancesClient which returns metadata listing the devices
        mock_instances_client = mock_google_cloud_compute.InstancesClient.return_value
        mock_instance_metadata = mock.Mock(disks=mock_disks)
        mock_instances_client.get.return_value = mock_instance_metadata
        # AND a mock DisksClient which returns minimal information
        mock_disks_client = mock_google_cloud_compute.DisksClient.return_value
        mock_disks_client.get.return_value = mock.Mock(source_snapshot=None)

        # WHEN get_attached_volumes is called
        # THEN an AssertionError is raised
        with pytest.raises(AssertionError):
            snapshot_interface.get_attached_volumes(self.gcp_instance_name)

    def test_get_attached_volumes_instance_not_found(self, mock_google_cloud_compute):
        """
        Verify that a SnapshotBackupException is raised if the instance cannot be
        found.
        """
        # GIVEN a new GcpCloudSnapshotInterface
        snapshot_interface = GcpCloudSnapshotInterface(self.gcp_project, self.gcp_zone)
        # AND a mock InstancesClient which cannot find the instance
        mock_instances_client = mock_google_cloud_compute.InstancesClient.return_value
        mock_instances_client.get.side_effect = NotFound("instance not found")

        # WHEN get_attached_volumes is called
        # THEN a SnapshotBackupException is raised
        with pytest.raises(SnapshotBackupException) as exc:
            snapshot_interface.get_attached_volumes(self.gcp_instance_name)

        # AND the exception contains the expected message
        assert str(
            exc.value
        ) == "Cannot find instance with name {} in zone {} for project {}".format(
            self.gcp_instance_name, self.gcp_zone, self.gcp_project
        )

    def test_instance_exists(self, mock_google_cloud_compute):
        """Verify successfully retrieving the instance results in a True response."""
        # GIVEN a new GcpCloudSnapshotInterface
        snapshot_interface = GcpCloudSnapshotInterface(self.gcp_project)

        # WHEN instance_exists is called for an instance which exists
        result = snapshot_interface.instance_exists(self.gcp_instance_name)

        # THEN it returns True
        assert result is True

    def test_instance_exists_not_found(self, mock_google_cloud_compute):
        """Verify a NotFound error results in a False response."""
        # GIVEN a new GcpCloudSnapshotInterface
        snapshot_interface = GcpCloudSnapshotInterface(self.gcp_project)
        # AND a mock InstancesClient which cannot find the instance
        mock_instances_client = mock_google_cloud_compute.InstancesClient.return_value
        mock_instances_client.get.side_effect = NotFound("instance not found")

        # WHEN instance_exists is called
        result = snapshot_interface.instance_exists(self.gcp_instance_name)

        # THEN it returns False
        assert result is False


class TestGcpVolumeMetadata(object):
    """Verify behaviour of GcpVolumeMetadata."""

    @pytest.mark.parametrize(
        (
            "attachment_metadata",
            "disk_metadata",
            "expected_device_path",
            "expected_source_snapshot",
        ),
        (
            (None, None, None, None),
            (
                mock.Mock(device_name="pgdata"),
                None,
                "/dev/disk/by-id/google-pgdata",
                None,
            ),
            (None, mock.Mock(source_snapshot="prefix/snap0"), None, "snap0"),
            (
                mock.Mock(device_name="pgdata"),
                mock.Mock(source_snapshot="prefix/snap0"),
                "/dev/disk/by-id/google-pgdata",
                "snap0",
            ),
        ),
    )
    def test_init(
        self,
        attachment_metadata,
        disk_metadata,
        expected_device_path,
        expected_source_snapshot,
    ):
        """Verify GcpVolumeMetadata is created from supplied metadata."""
        # WHEN volume metadata is created from the specified attachment_metadata and
        # disk_metadata
        volume = GcpVolumeMetadata(attachment_metadata, disk_metadata)

        # THEN the metadata has the expected source snapshot
        assert volume.source_snapshot == expected_source_snapshot
        # AND the internal _device_path has the expected value
        assert volume._device_path == expected_device_path

    def test_resolve_mounted_volume(self):
        """Verify resolve_mounted_volume sets mount info from findmnt output."""
        # GIVEN a GcpVolumeMetadata for device `pgdata`
        attachment_metadata = mock.Mock(device_name="pgdata")
        volume = GcpVolumeMetadata(attachment_metadata)
        # AND the specified findmnt response
        mock_cmd = mock.Mock()
        mock_cmd.findmnt.return_value = ("/opt/disk0", "rw,noatime")

        # WHEN resolve_mounted_volume is called
        volume.resolve_mounted_volume(mock_cmd)

        # THEN findmnt was called with the expected arguments
        mock_cmd.findmnt.assert_called_once_with("/dev/disk/by-id/google-pgdata")

        # AND the expected mount point and options are set on the volume metadata
        assert volume.mount_point == "/opt/disk0"
        assert volume.mount_options == "rw,noatime"

    @pytest.mark.parametrize(
        ("findmnt_fun", "device_name", "expected_exception_msg"),
        (
            (
                lambda x: (None, None),
                "pgdata",
                "Could not find device /dev/disk/by-id/google-pgdata at any mount point",
            ),
            (
                CommandException("error doing findmnt"),
                "pgdata",
                "Error finding mount point for device /dev/disk/by-id/google-pgdata: error doing findmnt",
            ),
            (
                lambda x: (None, None),
                None,
                "Cannot resolve mounted volume: Device path unknown",
            ),
        ),
    )
    def test_resolve_mounted_volume_failure(
        self, findmnt_fun, device_name, expected_exception_msg
    ):
        """Verify the failure modes of resolve_mounted_volume."""
        # GIVEN a GcpVolumeMetadata for device `pgdata`
        attachment_metadata = mock.Mock(device_name=device_name)
        volume = GcpVolumeMetadata(attachment_metadata)
        # AND the specified findmnt response
        mock_cmd = mock.Mock()
        mock_cmd.findmnt.side_effect = findmnt_fun

        # WHEN resolve_mounted_volume is called
        # THEN the expected exception occurs
        with pytest.raises(SnapshotBackupException) as exc:
            volume.resolve_mounted_volume(mock_cmd)

        # AND the exception has the expected error message
        assert str(exc.value) == expected_exception_msg


class TestAzureCloudSnapshotInterface(object):
    """
    Verify behaviour of the AzureCloudSnapshotInterface class.
    """

    azure_disks = (
        {
            "id": "disk_0",
            "location": "uksouth",
            "lun": 10,
            "managed_disk_id": "disk_id_0",
            "mount_options": "rw,noatime",
            "mount_point": "/opt/disk0",
            "name": "test_disk_0",
        },
        {
            "id": "disk_1",
            "location": "uksouth",
            "lun": 11,
            "managed_disk_id": "disk_id_1",
            "mount_options": "rw",
            "mount_point": "/opt/disk1",
            "name": "test_disk_1",
        },
        {
            "id": "disk_2",
            "location": "uksouth",
            "lun": 12,
            "managed_disk_id": "disk_id_2",
            "mount_options": "rw,relatime",
            "mount_point": "/opt/disk2",
            "name": "test_disk_2",
        },
    )
    azure_instance_name = "azure_vm"
    azure_resource_group = "test_resource_group"
    azure_subscription_id = "test_subscription_id"
    backup_id = "20380119T031407"
    server_name = "test_server"

    def _get_snapshot_name(self, disk_name):
        """Helper which forges the expected snapshot name for the given disk name."""
        return "{}-{}".format(disk_name, self.backup_id.lower())

    def _get_mock_snapshot_operations(self):
        """
        Helper which creates a mock SnapshotOperations client that always succeeds.
        """
        mock_resp = mock.Mock()
        mock_resp.status.return_value = "Succeeded"
        mock_resp.result.return_value.provisioning_state = "Succeeded"
        mock_snapshot_operations = mock.Mock()
        mock_snapshot_operations.begin_create_or_update.return_value = mock_resp
        mock_snapshot_operations.begin_delete.return_value = mock_resp
        return mock_snapshot_operations

    def _get_mock_instances_client(self, resource_group_name, instance_name, disks):
        """
        Helper which create a mock instances client for the given
        resource_group/instance with the specified disks attached as the specified
        device.
        """

        def get_fun(resource_group, instance):
            if instance == instance_name and resource_group == resource_group_name:
                mock_data_disks = []
                for disk in disks:
                    data_disk = mock.Mock(
                        lun=disk["lun"],
                        managed_disk=mock.Mock(id=disk["managed_disk_id"]),
                    )
                    data_disk.name = disk["name"]
                    mock_data_disks.append(data_disk)
                mock_storage_profile = mock.Mock(data_disks=mock_data_disks)
                return mock.Mock(storage_profile=mock_storage_profile)
            else:
                raise ResourceNotFoundError("instance not found")

        return mock.Mock(get=get_fun)

    def _get_mock_disks_client(self, resource_group_name, disks):
        """
        Helper which creates a mock disks client for the given resource group with the
        specified disks available.
        """
        disk_metadata = dict(
            (
                disk["name"],
                mock.Mock(location=disk["location"]),
            )
            for disk in disks
        )

        def get_fun(resource_group, disk):
            if resource_group == resource_group_name:
                try:
                    return disk_metadata[disk]
                except KeyError:
                    raise ResourceNotFoundError("disk not found")

        return mock.Mock(get=get_fun)

    def _get_mock_volumes(self, disks):
        """Helper which returns mock VolumeMetadata objects for the named disks."""
        return dict(
            (
                disk["name"],
                mock.Mock(
                    mount_point=disk["mount_point"],
                    mount_options=disk["mount_options"],
                    location=disk["location"],
                ),
            )
            for disk in disks
        )

    @pytest.fixture(autouse=True)
    def mock_azure_mgmt_compute(self):
        with mock.patch(
            "barman.cloud_providers.azure_blob_storage.import_azure_mgmt_compute"
        ) as mock_import_azure_mgmt_compute:
            self._mock_azure_mgmt_compute = mock_import_azure_mgmt_compute.return_value
            yield mock_import_azure_mgmt_compute.return_value

    @pytest.fixture(autouse=True)
    def mock_azure_identity(self):
        with mock.patch(
            "barman.cloud_providers.azure_blob_storage.import_azure_identity"
        ) as mock_import_azure_identity:
            self._mock_azure_identity = mock_import_azure_identity.return_value
            yield mock_import_azure_identity.return_value

    def test_init_with_null_subscription_id(self):
        """
        Verify creating AzureCloudSnapshotInterface fails if subscription_id is not set.
        """
        # GIVEN a null subscription ID
        azure_subscription_id = None

        # WHEN an AzureCloudSnapshotInterface is created
        # THEN a TypeError is raised
        with pytest.raises(TypeError) as exc:
            AzureCloudSnapshotInterface(azure_subscription_id)

        # AND the expected message is included
        assert str(exc.value) == "subscription_id cannot be None"

    def test_init(self):
        """
        Verify creating AzureCloudSnapshotInterface creates the necessary Azure client.
        """
        # GIVEN a non-null Azure subscription
        subscription_id = self.azure_subscription_id
        # WHEN an AzureCloudSnapshotInterface is created
        snapshot_interface = AzureCloudSnapshotInterface(subscription_id)
        # THEN an azure.mgmt.compute.ComputeManagementClient is created
        assert (
            snapshot_interface.client
            == self._mock_azure_mgmt_compute.ComputeManagementClient.return_value
        )
        # AND the DefaultAzureCredential was used
        self._mock_azure_mgmt_compute.ComputeManagementClient.assert_called_once_with(
            self._mock_azure_identity.DefaultAzureCredential.return_value,
            subscription_id,
        )

    def test_init_with_credential(self):
        """
        Verify creation of AzureCloudSnapshotInterface with provided credential.
        """
        # GIVEN a non-null Azure subscription
        subscription_id = self.azure_subscription_id
        # AND a user-specified credential
        credential = mock.Mock()
        # WHEN an AzureCloudSnapshotInterface is created
        AzureCloudSnapshotInterface(subscription_id, credential=credential)
        # THEN the user-specified credential was used to create the
        # ComputeManagementClient
        self._mock_azure_mgmt_compute.ComputeManagementClient.assert_called_once_with(
            credential.return_value, subscription_id
        )

    def test_take_snapshot(self, caplog):
        """
        Verify that _take_snapshot calls the Azure library and waits for the result.
        """
        # GIVEN a new AzureCloudSnapshotInterface
        snapshot_interface = AzureCloudSnapshotInterface(self.azure_subscription_id)
        # AND a backup_info for a given server name and backup ID
        backup_info = mock.Mock(backup_id=self.backup_id, server_name=self.server_name)
        # AND a mock SnapshotsOperations which returns a successful response to
        # begin_create_or_update
        mock_snapshot_operations = self._get_mock_snapshot_operations()
        mock_resp = mock_snapshot_operations.begin_create_or_update.return_value
        self._mock_azure_mgmt_compute.ComputeManagementClient.return_value.snapshots = (
            mock_snapshot_operations
        )
        # AND log level is INFO
        caplog.set_level(logging.INFO)

        # WHEN _take_snapshot is called
        snapshot_name = snapshot_interface._take_snapshot(
            backup_info,
            self.azure_resource_group,
            self.azure_disks[0]["location"],
            self.azure_disks[0]["name"],
            self.azure_disks[0]["id"],
        )

        # THEN begin_create_or_update is called on the SnapshotsOperations with the
        # expected args
        expected_disk_id = self.azure_disks[0]["id"]
        expected_disk_name = self.azure_disks[0]["name"]
        expected_location = self.azure_disks[0]["location"]
        expected_snapshot_name = self._get_snapshot_name(expected_disk_name)
        mock_snapshot_operations.begin_create_or_update.assert_called_once_with(
            self.azure_resource_group,
            expected_snapshot_name,
            {
                "location": expected_location,
                "incremental": True,
                "creation_data": {
                    "create_option": "Copy",
                    "source_uri": expected_disk_id,
                },
            },
        )
        # AND wait() was called on the response to await completion of the snapshot
        mock_resp.wait.assert_called_once()
        # AND the name of the snapshot was returned
        assert snapshot_name == expected_snapshot_name
        # AND the expected log output occurred
        expected_log_content = (
            "Taking snapshot '{}' of disk '{}'".format(
                expected_snapshot_name, expected_disk_name
            ),
            "Waiting for snapshot '{}' completion".format(expected_snapshot_name),
            "Snapshot '{}' completed".format(expected_snapshot_name),
        )
        for expected_log, log_line in zip(
            expected_log_content, caplog.text.split("\n")
        ):
            assert expected_log in log_line

    @pytest.mark.parametrize(
        ("mock_status", "mock_provisioning_state"),
        (("Succeeded", "Failed"), ("123", "Succeeded"), ("123", "Failed")),
    )
    def test_take_snapshot_failed(self, mock_status, mock_provisioning_state):
        """
        Verify that _take_snapshot raises an exception on failure.
        """
        # GIVEN a new AzureCloudSnapshotInterface
        snapshot_interface = AzureCloudSnapshotInterface(self.azure_subscription_id)
        # AND a backup_info for a given server name and backup ID
        backup_info = mock.Mock(backup_id=self.backup_id, server_name=self.server_name)
        # AND a mock SnapshotsOperations which returns a failed response to
        # begin_create_or_update
        mock_snapshot_operations = (
            self._mock_azure_mgmt_compute.ComputeManagementClient.return_value.snapshots
        )
        mock_resp = mock_snapshot_operations.begin_create_or_update.return_value
        mock_resp.status.return_value = mock_status
        mock_resp.result.return_value.provisioning_state = mock_provisioning_state
        mock_resp.result.return_value.__str__.return_value = "result_string"

        # WHEN _take_snapshot is called
        # THEN a CloudProviderError is raised
        with pytest.raises(CloudProviderError) as exc:
            snapshot_interface._take_snapshot(
                backup_info,
                self.azure_resource_group,
                self.azure_disks[0]["location"],
                self.azure_disks[0]["name"],
                self.azure_disks[0]["id"],
            )
        # AND the exception message contains the snapshot name, error code and error
        # message
        expected_snapshot_name = self._get_snapshot_name(self.azure_disks[0]["name"])
        expected_message = "Snapshot '{}' failed with error code {}: {}".format(
            expected_snapshot_name, mock_status, "result_string"
        )
        assert str(exc.value) == expected_message

    @pytest.mark.parametrize("number_of_disks", (1, 2, 3))
    def test_take_snapshot_backup(self, number_of_disks):
        """
        Verify that take_snapshot_backup takes the required snapshots and updates the
        backup_info when prerequisites are met.
        """
        # GIVEN a set of disks, represented as VolumeMetadata
        disks = self.azure_disks[:number_of_disks]
        assert len(disks) == number_of_disks
        # AND a backup_info for a given server name and backup ID
        backup_info = mock.Mock(backup_id=self.backup_id, server_name=self.server_name)
        # AND a mock VirtualMachinesOperations which returns an instance with the
        # required disks attached
        mock_compute_client = (
            self._mock_azure_mgmt_compute.ComputeManagementClient.return_value
        )
        mock_compute_client.virtual_machines = self._get_mock_instances_client(
            self.azure_resource_group, self.azure_instance_name, disks
        )
        # AND a mock SnapshotsOperation which returns a successful response to
        # begin_create_or_update
        mock_compute_client.snapshots = self._get_mock_snapshot_operations()
        # AND a new AzureCloudSnapshotInterface
        snapshot_interface = AzureCloudSnapshotInterface(
            self.azure_subscription_id, self.azure_resource_group
        )

        # WHEN take_snapshot_backup is called for multiple disks
        snapshot_interface.take_snapshot_backup(
            backup_info, self.azure_instance_name, self._get_mock_volumes(disks)
        )

        # THEN the backup_info is updated with the expected snapshot metadata
        snapshots_info = backup_info.snapshots_info
        assert snapshots_info.subscription_id == self.azure_subscription_id
        assert snapshots_info.resource_group == self.azure_resource_group
        assert snapshots_info.provider == "azure"
        assert len(snapshots_info.snapshots) == len(disks)
        for disk in disks:
            snapshot_name = self._get_snapshot_name(disk["name"])
            snapshot = next(
                snapshot
                for snapshot in snapshots_info.snapshots
                if snapshot.snapshot_name == snapshot_name
            )
            assert snapshot.identifier == snapshot_name
            assert snapshot.snapshot_name == snapshot_name
            assert snapshot.location == disk["location"]
            assert snapshot.lun == disk["lun"]
            assert snapshot.mount_options == disk["mount_options"]
            assert snapshot.mount_point == disk["mount_point"]

    def test_take_snapshot_backup_instance_not_found(self):
        """
        Verify that a SnapshotBackupException is raised if the instance cannot be
        found.
        """
        # GIVEN a new AzureCloudSnapshotInterface
        snapshot_interface = AzureCloudSnapshotInterface(
            self.azure_subscription_id, self.azure_resource_group
        )
        # AND a mock VirtualMachinesOperations which cannot find the instance
        mock_compute_client = (
            self._mock_azure_mgmt_compute.ComputeManagementClient.return_value
        )
        mock_compute_client.virtual_machines.get.side_effect = ResourceNotFoundError(
            "instance not found"
        )

        # WHEN take_snapshot_backup is called
        # THEN a SnapshotBackupException is raised
        with pytest.raises(SnapshotBackupException) as exc:
            snapshot_interface.take_snapshot_backup(
                mock.Mock(),
                self.azure_instance_name,
                self._get_mock_volumes(self.azure_disks),
            )

        # AND the exception contains the expected message
        assert str(exc.value) == (
            "Cannot find instance with name {} in resource group {} "
            "in subscription {}"
        ).format(
            self.azure_instance_name,
            self.azure_resource_group,
            self.azure_subscription_id,
        )

    def test_take_snapshot_backup_disks_not_attached(self):
        """
        Verify that a SnapshotBackupException is raised if the expected disks are not
        attached.
        """
        # GIVEN a new AzureCloudSnapshotInterface
        snapshot_interface = AzureCloudSnapshotInterface(
            self.azure_subscription_id, self.azure_resource_group
        )
        # AND a mock VirtualMachinesOperations which returns an instance with no disks
        # attached
        mock_compute_client = (
            self._mock_azure_mgmt_compute.ComputeManagementClient.return_value
        )
        mock_compute_client.virtual_machines = self._get_mock_instances_client(
            self.azure_resource_group, self.azure_instance_name, {}
        )

        # WHEN take_snapshot_backup is called
        # THEN a SnapshotBackupException is raised
        with pytest.raises(SnapshotBackupException) as exc:
            snapshot_interface.take_snapshot_backup(
                mock.Mock(),
                self.azure_instance_name,
                self._get_mock_volumes(self.azure_disks),
            )

        # AND the exception contains the expected message
        assert str(exc.value) == (
            "Disk {} not attached to instance {}".format(
                self.azure_disks[0]["name"], self.azure_instance_name
            )
        )

    def test_delete_snapshot(self, caplog):
        """Verify that a snapshot can be deleted successfully."""
        # GIVEN a successful response from the delete snapshot request
        mock_compute_client = (
            self._mock_azure_mgmt_compute.ComputeManagementClient.return_value
        )
        mock_compute_client.snapshots = self._get_mock_snapshot_operations()
        # AND a new AzureCloudSnapshotInterface
        snapshot_interface = AzureCloudSnapshotInterface(
            self.azure_subscription_id, self.azure_resource_group
        )
        # AND log level is info
        caplog.set_level(logging.INFO)

        # WHEN a snapshot is deleted
        snapshot_name = "test_snapshot"
        resource_group = "test_resource_group"
        snapshot_interface._delete_snapshot(snapshot_name, resource_group)

        # THEN delete was called on the client with the expected arguments
        mock_compute_client.snapshots.begin_delete.assert_called_once_with(
            resource_group, snapshot_name
        )
        # AND wait was called on the response
        mock_compute_client.snapshots.begin_delete.return_value.wait.assert_called_once()
        # AND a success message was logged
        assert "Snapshot {} deleted".format(snapshot_name) in caplog.text

    def test_delete_snapshot_not_found(self):
        """
        Verify that a ResourceNotFound error is propagated rather than being considered
        a successful deletion. This is because a ResourceNotFoundError is raised if the
        resource group cannot be found - this is an error condition.
        """
        # GIVEN a delete snapshot request which will raise a ResourceNotFoundError
        mock_compute_client = (
            self._mock_azure_mgmt_compute.ComputeManagementClient.return_value
        )
        mock_compute_client.snapshots = self._get_mock_snapshot_operations()
        mock_compute_client.snapshots.begin_delete.side_effect = ResourceNotFoundError
        # AND a new AzureCloudSnapshotInterface
        snapshot_interface = AzureCloudSnapshotInterface(
            self.azure_subscription_id, self.azure_resource_group
        )

        # WHEN a snapshot is deleted
        # THEN a ResourceNotFoundError is raised
        snapshot_name = "test_snapshot"
        resource_group = "test_resource_group"
        with pytest.raises(ResourceNotFoundError):
            snapshot_interface._delete_snapshot(snapshot_name, resource_group)

    def test_delete_snapshot_failed(self):
        """Verify that an unsuccessful response results in a CloudProviderError."""
        # GIVEN a delete snapshot request which will return an unsuccessful response
        mock_compute_client = (
            self._mock_azure_mgmt_compute.ComputeManagementClient.return_value
        )
        mock_compute_client.snapshots = self._get_mock_snapshot_operations()
        mock_resp = mock_compute_client.snapshots.begin_delete.return_value
        mock_resp.status.return_value = "failed"
        mock_resp.result.return_value = "failure message"
        # AND a new AzureCloudSnapshotInterface
        snapshot_interface = AzureCloudSnapshotInterface(
            self.azure_subscription_id, self.azure_resource_group
        )

        # WHEN a snapshot is deleted and a failure response is received
        # THEN a CloudProviderError is raised
        snapshot_name = "snapshot_name"
        with pytest.raises(CloudProviderError) as exc:
            snapshot_interface._delete_snapshot(snapshot_name, "resource group")
        # AND the exception has the expected message
        expected_message = (
            "Deletion of snapshot {} failed with error code {}: {}".format(
                snapshot_name, "failed", "failure message"
            )
        )
        assert expected_message in str(exc.value)

    @pytest.mark.parametrize(
        "snapshots_list",
        (
            [],
            [mock.Mock(identifier="snapshot0")],
            [mock.Mock(identifier="snapshot0"), mock.Mock(identifier="snapshot1")],
        ),
    )
    def test_delete_snapshot_backup(
        self,
        snapshots_list,
        caplog,
    ):
        """Verify that all snapshots for a backup are deleted."""
        # GIVEN a backup_info specifying zero or more snapshots in a given resource group
        resource_group = "resource group"
        backup_info = mock.Mock(
            backup_id=self.backup_id,
            snapshots_info=mock.Mock(
                resource_group=resource_group, snapshots=snapshots_list
            ),
        )
        # AND log level is info
        caplog.set_level(logging.INFO)
        # AND the snapshot delete requests are successful
        mock_compute_client = (
            self._mock_azure_mgmt_compute.ComputeManagementClient.return_value
        )
        mock_compute_client.snapshots = self._get_mock_snapshot_operations()
        # AND a new AzureCloudSnapshotInterface
        snapshot_interface = AzureCloudSnapshotInterface(self.azure_subscription_id)

        # WHEN delete_snapshot_backup is called
        snapshot_interface.delete_snapshot_backup(backup_info)

        # THEN begin_delete was called for each snapshot
        mock_snapshots_operation = mock_compute_client.snapshots
        assert mock_snapshots_operation.begin_delete.call_count == len(snapshots_list)
        for snapshot in snapshots_list:
            assert (
                (
                    resource_group,
                    snapshot.identifier,
                ),
            ) in mock_snapshots_operation.begin_delete.call_args_list
            # AND the expected log message was logged for each snapshot
            assert (
                "Deleting snapshot '{}' for backup {}".format(
                    snapshot.identifier, self.backup_id
                )
                in caplog.text
            )

    @pytest.mark.parametrize(
        (
            "disks_metadata",
            "expected_disk_names",
            "expected_luns",
            "expected_locations",
        ),
        (
            ([], [], [], []),
            (
                [
                    {
                        "location": "uksouth",
                        "lun": "10",
                        "managed_disk_id": "disk_id_0",
                        "name": "disk0",
                    }
                ],
                ["disk0"],
                ["10"],
                ["uksouth"],
            ),
            (
                [
                    {
                        "location": "uksouth",
                        "lun": "10",
                        "managed_disk_id": "disk_id_0",
                        "name": "disk0",
                    },
                    {
                        "location": "ukwest",
                        "lun": "11",
                        "managed_disk_id": "disk_id_1",
                        "name": "disk1",
                    },
                ],
                ["disk0", "disk1"],
                ["10", "11"],
                ["uksouth", "ukwest"],
            ),
        ),
    )
    def test_get_attached_volumes(
        self,
        disks_metadata,
        expected_disk_names,
        expected_luns,
        expected_locations,
    ):
        """Verify that attached volumes are returned as a dict keyed by disk name."""
        # GIVEN a new AzureCloudSnapshotInterface
        snapshot_interface = AzureCloudSnapshotInterface(
            self.azure_subscription_id, self.azure_resource_group
        )
        # AND a mock VirtualMachinesOperations which returns an instance with the
        # required disks attached
        mock_compute_client = (
            self._mock_azure_mgmt_compute.ComputeManagementClient.return_value
        )
        mock_compute_client.virtual_machines = self._get_mock_instances_client(
            self.azure_resource_group, self.azure_instance_name, disks_metadata
        )
        # AND a mock DisksOperation which returns the specified metadata
        mock_compute_client.disks = self._get_mock_disks_client(
            self.azure_resource_group, disks_metadata
        )

        # WHEN get_attached_volumes is called
        attached_volumes = snapshot_interface.get_attached_volumes(
            self.azure_instance_name
        )

        # THEN a dict of VolumeMetadata is returned, keyed by disk name
        assert len(attached_volumes) == len(expected_disk_names)
        for expected_disk_name, expected_lun, expected_location in zip(
            expected_disk_names, expected_luns, expected_locations
        ):
            assert expected_disk_name in attached_volumes
            # AND the lun matches that returned by the instance metadata
            assert attached_volumes[expected_disk_name]._lun == expected_lun
            # AND the location matches that returned by the disk metadata
            assert attached_volumes[expected_disk_name].location == expected_location

    def test_get_attached_volumes_for_disks(self):
        """
        Verifies that only the requested disks are returned when the disks parameter is
        passed to get_attached_volumes.
        """
        # GIVEN a new AzureCloudSnapshotInterface
        snapshot_interface = AzureCloudSnapshotInterface(
            self.azure_subscription_id, self.azure_resource_group
        )
        # AND a mock VirtualMachinesOperations which returns an instance with the
        # required disks attached
        mock_compute_client = (
            self._mock_azure_mgmt_compute.ComputeManagementClient.return_value
        )
        disks_metadata = [
            {
                "location": "uksouth",
                "lun": "10",
                "managed_disk_id": "disk_id_0",
                "name": "disk0",
            },
            {
                "location": "ukwest",
                "lun": "11",
                "managed_disk_id": "disk_id_1",
                "name": "disk1",
            },
        ]
        mock_compute_client.virtual_machines = self._get_mock_instances_client(
            self.azure_resource_group, self.azure_instance_name, disks_metadata
        )
        # AND a mock DisksOperation which returns the specified metadata
        mock_compute_client.disks = self._get_mock_disks_client(
            self.azure_resource_group, disks_metadata
        )

        # WHEN get_attached_volumes is called
        attached_volumes = snapshot_interface.get_attached_volumes(
            self.azure_instance_name, disks=["disk1"]
        )

        # THEN only "disk1" is included in the resulting dict
        assert len(attached_volumes) == 1
        assert "disk1" in attached_volumes

    def test_get_attached_volumes_disk_not_found(self):
        """
        Verify that a SnapshotBackupException is raised if a disk cannot be found.
        """
        # GIVEN a set of disks
        disks = self.azure_disks
        # AND a mock VirtualMachinesOperations which returns an instance with a
        # subset of the required disks attached
        mock_compute_client = (
            self._mock_azure_mgmt_compute.ComputeManagementClient.return_value
        )
        mock_compute_client.virtual_machines = self._get_mock_instances_client(
            self.azure_resource_group, self.azure_instance_name, disks[:-1]
        )
        # AND a mock DisksOperation which returns the specified metadata for the
        # same subset of disks
        mock_compute_client.disks = self._get_mock_disks_client(
            self.azure_resource_group, disks[:-1]
        )
        # AND a new AzureCloudSnapshotInterface
        snapshot_interface = AzureCloudSnapshotInterface(
            self.azure_subscription_id, self.azure_resource_group
        )

        # WHEN get_attached_volumes is called
        # THEN a SnapshotBackupException is raised
        with pytest.raises(SnapshotBackupException) as exc:
            snapshot_interface.get_attached_volumes(
                self.azure_instance_name, [disk["name"] for disk in disks]
            )

        # AND the exception contains the expected message
        assert str(exc.value) == (
            "Cannot find disk with name {} in resource group {} in subscription {}"
        ).format(
            disks[-1]["name"], self.azure_resource_group, self.azure_subscription_id
        )

        # WHEN get_attached_volumes is called with fail_on_missing=False
        # THEN no exception is raised
        attached_volumes = snapshot_interface.get_attached_volumes(
            self.azure_instance_name,
            [disk["name"] for disk in disks],
            fail_on_missing=False,
        )
        # AND the attached volumes contains only those disks which were present
        expected_volumes = [d["name"] for d in disks[:-1]]
        assert set(attached_volumes.keys()) == set(expected_volumes)

    def test_get_attached_volumes_disk_not_attached(self):
        """
        Verify that a SnapshotBackupException is raised if a disk is not attached
        to the instance.
        """
        # GIVEN a set of disks
        disks = self.azure_disks
        # AND a mock VirtualMachinesOperations which returns an instance with a
        # subset of the required disks attached
        mock_compute_client = (
            self._mock_azure_mgmt_compute.ComputeManagementClient.return_value
        )
        mock_compute_client.virtual_machines = self._get_mock_instances_client(
            self.azure_resource_group, self.azure_instance_name, disks[:-1]
        )
        # AND a mock DisksOperation which returns the specified metadata for all
        # disks
        mock_compute_client.disks = self._get_mock_disks_client(
            self.azure_resource_group, disks
        )
        # AND a new AzureCloudSnapshotInterface
        snapshot_interface = AzureCloudSnapshotInterface(
            self.azure_subscription_id, self.azure_resource_group
        )

        # WHEN get_attached_volumes is called
        # THEN a SnapshotBackupException is raised
        with pytest.raises(SnapshotBackupException) as exc:
            snapshot_interface.get_attached_volumes(
                self.azure_instance_name, [disk["name"] for disk in disks]
            )

        # AND the exception contains the expected message
        assert str(exc.value) == "Disks not attached to instance {}: {}".format(
            self.azure_instance_name, disks[-1]["name"]
        )

        # WHEN get_attached_volumes is called with fail_on_missing=False
        # THEN no exception is raised
        attached_volumes = snapshot_interface.get_attached_volumes(
            self.azure_instance_name,
            [disk["name"] for disk in disks],
            fail_on_missing=False,
        )
        # AND the attached volumes contains only those disks which were present
        expected_volumes = [d["name"] for d in disks[:-1]]
        assert set(attached_volumes.keys()) == set(expected_volumes)

    @pytest.mark.parametrize(
        "mock_disks",
        (
            [
                {
                    "lun": "10",
                    "managed_disk_id": "disk_id_0",
                    "name": "disk0",
                },
                {
                    "lun": "11",
                    "managed_disk_id": "disk_id_0",
                    "name": "disk0",
                },
            ],
            [
                {
                    "lun": "10",
                    "managed_disk_id": "disk_id_0",
                    "name": "disk0",
                },
                {
                    "lun": "11",
                    "managed_disk_id": "disk_id_1",
                    "name": "disk1",
                },
                {
                    "lun": "11",
                    "managed_disk_id": "disk_id_1",
                    "name": "disk1",
                },
            ],
        ),
    )
    def test_get_attached_volumes_duplicate_names(self, mock_disks):
        """
        Verify that an exception is raised if a disk appears to be attached more than
        once.
        """
        # GIVEN a mock VirtualMachinesOperations which returns an instance with the
        # specified disks attached
        mock_compute_client = (
            self._mock_azure_mgmt_compute.ComputeManagementClient.return_value
        )
        mock_compute_client.virtual_machines = self._get_mock_instances_client(
            self.azure_resource_group, self.azure_instance_name, mock_disks
        )
        # AND a new AzureCloudSnapshotInterface
        snapshot_interface = AzureCloudSnapshotInterface(
            self.azure_subscription_id, self.azure_resource_group
        )

        # WHEN get_attached_volumes is called
        # THEN an AssertionError is raised
        with pytest.raises(AssertionError):
            snapshot_interface.get_attached_volumes(self.azure_instance_name)

    def test_get_attached_volumes_instance_not_found(self):
        """
        Verify that a SnapshotBackupException is raised if the instance cannot be
        found.
        """
        # GIVEN a mock VirtualMachinesOperations which cannot find the instance
        mock_compute_client = (
            self._mock_azure_mgmt_compute.ComputeManagementClient.return_value
        )
        mock_compute_client.virtual_machines.get.side_effect = ResourceNotFoundError(
            "instance_not_found"
        )
        # AND a new AzureCloudSnapshotInterface
        snapshot_interface = AzureCloudSnapshotInterface(
            self.azure_subscription_id, self.azure_resource_group
        )

        # WHEN get_attached_volumes is called
        # THEN a SnapshotBackupException is raised
        with pytest.raises(SnapshotBackupException) as exc:
            snapshot_interface.get_attached_volumes(self.azure_instance_name)

        # AND the exception contains the expected message
        assert str(exc.value) == (
            "Cannot find instance with name {} in resource group {} "
            "in subscription {}"
        ).format(
            self.azure_instance_name,
            self.azure_resource_group,
            self.azure_subscription_id,
        )

    def test_instance_exists(self):
        """Verify successfully retrieving the instance results in a True response."""
        # GIVEN a new AzureCloudSnapshotInterface
        snapshot_interface = AzureCloudSnapshotInterface(
            self.azure_subscription_id, self.azure_resource_group
        )
        # WHEN instance exists is called for an instance which exists
        result = snapshot_interface.instance_exists(self.azure_instance_name)

        # THEN it returns True
        assert result is True

    def test_instance_exists_not_found(self):
        """Verify a NotFound error results in a False response."""
        # GIVEN a new AzureCloudSnapshotInterface
        snapshot_interface = AzureCloudSnapshotInterface(
            self.azure_subscription_id, self.azure_resource_group
        )
        # AND a mock VirtualMachinesOperations which cannot find the instance
        mock_compute_client = (
            self._mock_azure_mgmt_compute.ComputeManagementClient.return_value
        )
        mock_compute_client.virtual_machines.get.side_effect = ResourceNotFoundError(
            "instance_not_found"
        )

        # WHEN instance exists is called
        result = snapshot_interface.instance_exists(self.azure_instance_name)

        # THEN it returns False
        assert result is False


class TestAzureVolumeMetadata(object):
    """Verify behaviour of AzureVolumeMetadata."""

    @pytest.mark.parametrize(
        [
            "attachment_metadata",
            "disk_metadata",
            "expected_lun",
            "expected_location",
        ],
        (
            # If no attachment metadata or disk metadata is passed then we expect init
            # to succeed but the lun and location to be None.
            (None, None, None, None),
            # If the lun is set in the attachment metadata then we expect it to be
            # set in the VolumeMetadata instance.
            (
                mock.Mock(lun="10"),
                None,
                "10",
                None,
            ),
            # If the location is set in the attachment metadata then we expect it to
            # be set in the VolumeMetadata instance.
            (None, mock.Mock(location="uksouth"), None, "uksouth"),
            # If lun and location are set in the attachment metadata then we expect
            # them to be set in the VolumeMetadata instance.
            (
                mock.Mock(lun="10"),
                mock.Mock(location="uksouth"),
                "10",
                "uksouth",
            ),
        ),
    )
    def test_init(
        self,
        attachment_metadata,
        disk_metadata,
        expected_lun,
        expected_location,
    ):
        """Verify AzureVolumeMetadata is created from supplied metadata"""
        # WHEN volume metadata is created from the specified attachment_metadata and
        # disk_metadata
        volume = AzureVolumeMetadata(attachment_metadata, disk_metadata)

        # THEN the metadata has the expected location
        assert volume.location == expected_location
        # AND the internal _lun has the expected value
        assert volume._lun == expected_lun

    @pytest.mark.parametrize(
        ("create_option", "source_resource_id", "expected_source_snapshot"),
        (
            # If there is no create_option or source_resource_id then we expect
            # source_snapshot to be None
            (None, None, None),
            # If there is no create_option and source_resource_id is set to
            # a snapshot name we expect source_snapshot to be None
            (
                None,
                "/subscriptions/id/resourceGroups/group/providers/Microsoft.Compute/snapshots/snapshot_name",
                None,
            ),
            # If the create_option is not Copy we expect source_snapshot to be None
            (
                "NotCopy",
                "/subscriptions/id/resourceGroups/group/providers/Microsoft.Compute/snapshots/snapshot_name",
                None,
            ),
            # If the create_option is Copy but the source_resource_id is not a snapshot
            # then we expect source_snapshot to be None
            (
                "Copy",
                "/subscriptions/id/resourceGroups/group/providers/Microsoft.Compute/disks/disk_name",
                None,
            ),
            # If create_option is Copy and source_resource_id is a snapshot ID then we
            # expect source_snapshot to be set to the snapshot name
            (
                "Copy",
                "/subscriptions/id/resourceGroups/group/providers/Microsoft.Compute/snapshots/snapshot_name",
                "snapshot_name",
            ),
        ),
    )
    def test_init_source_snapshot(
        self, create_option, source_resource_id, expected_source_snapshot
    ):
        """
        Verify that the source snapshot is set correctly when available in the disk
        metadata.
        """
        # GIVEN disk metadata with the specified create_option and source_resource_id
        disk_metadata = mock.Mock(
            creation_data=mock.Mock(
                create_option=create_option,
                source_resource_id=source_resource_id,
            )
        )

        # WHEN volume metadata is created from the disk metadata
        volume = AzureVolumeMetadata(None, disk_metadata)

        # THEN the metadata has the expected source_snapshot value
        assert volume.source_snapshot == expected_source_snapshot

    @pytest.mark.parametrize(
        "source_resource_id",
        (
            # Not a valid fully qualified resource ID
            "providers/Microsoft.Compute/snapshots/foo",
            # Missing snapshot name
            "/subscriptions/id/resourceGroups/group/providers/Microsoft.Compute/snapshots/",
            # Missing subscription ID
            "/subscriptions//resourceGroups/group/providers/Microsoft.Compute/snapshots/foo",
            # Missing resource group
            "/subscriptions/id/resourceGroups//providers/Microsoft.Compute/snapshots/foo",
        ),
    )
    def test_source_snapshot_parsing_error(self, source_resource_id):
        """
        Verify that an exception is raised when the source_resource_id cannot be
        parsed.
        """
        # GIVEN disk metadata with create_option "Copy" and the specified
        # source_resource_id
        disk_name = "disk0"
        disk_metadata = mock.Mock(
            creation_data=mock.Mock(
                create_option="Copy",
                source_resource_id=source_resource_id,
            )
        )
        disk_metadata.name = disk_name
        # AND some arbitrary attachment metadata
        attachment_metadata = mock.Mock()

        # WHEN volume metadata is created from the disk metadata
        # THEN a SnapshotBackupException is raised
        with pytest.raises(SnapshotBackupException) as exc:
            AzureVolumeMetadata(attachment_metadata, disk_metadata)

        # AND the exception has the expected message
        expected_msg = (
            "Could not determine source snapshot for disk {} with source resource ID "
            "{}"
        ).format(disk_name, source_resource_id)
        assert expected_msg in str(exc.value)

    def test_resolve_mounted_volume(self):
        """Verify resolve_mounted_volume sets mount info from findmnt output."""
        # GIVEN a AzureVolumeMetadata for lun `10`
        attachment_metadata = mock.Mock(lun="10")
        volume = AzureVolumeMetadata(attachment_metadata)
        # AND the specified findmnt response
        mock_cmd = mock.Mock()
        mock_cmd.findmnt.return_value = ("/opt/disk0", "rw,noatime")

        # WHEN resolve_mounted_volume is called
        volume.resolve_mounted_volume(mock_cmd)

        # THEN findmnt was called with the expected arguments
        mock_cmd.findmnt.assert_called_once_with("/dev/disk/azure/scsi1/lun10")

        # AND the expected mount point and options are set on the volume metadata
        assert volume.mount_point == "/opt/disk0"
        assert volume.mount_options == "rw,noatime"

    @pytest.mark.parametrize(
        ("findmnt_fun", "lun", "expected_exception_msg"),
        (
            (
                lambda x: (None, None),
                "10",
                "Could not find volume with lun 10 at any mount point",
            ),
            (
                CommandException("error doing findmnt"),
                "10",
                "Error finding mount point for volume with lun 10: error doing findmnt",
            ),
            (
                lambda x: (None, None),
                None,
                "Cannot resolve mounted volume: LUN unknown",
            ),
        ),
    )
    def test_resolve_mounted_volume_failure(
        self, findmnt_fun, lun, expected_exception_msg
    ):
        """Verify the failure modes of resolve_mounted_volume."""
        # GIVEN a AzureVolumeMetadata for lun `10`
        attachment_metadata = mock.Mock(lun=lun)
        volume = AzureVolumeMetadata(attachment_metadata)
        # AND the specified findmnt response
        mock_cmd = mock.Mock()
        mock_cmd.findmnt.side_effect = findmnt_fun

        # WHEN resolve_mounted_volume is called
        # THEN the expected exception occurs
        with pytest.raises(SnapshotBackupException) as exc:
            volume.resolve_mounted_volume(mock_cmd)

        # AND the exception has the expected error message
        assert str(exc.value) == expected_exception_msg


class TestAwsCloudSnapshotInterface(object):
    """
    Verify behaviour of the AwsCloudSnapshotInterface class.
    """

    aws_account_id = "0123456789"
    # aws_disks defines several storage volumes as dicts, which will be used by mocks
    # in order to create boto3 responses and other data structures required by tests.
    aws_disks = [
        {
            "device": "/dev/xvdf",
            "id": "vol-0",
            "mount_point": "/opt/disk0",
            "mount_options": "rw,noatime",
            "name": "test_disk_0",
        },
        {
            "device": "/dev/xvdg",
            "id": "vol-1",
            "mount_point": "/opt/disk1",
            "mount_options": "rw",
            "name": "test_disk_1",
        },
        {
            "device": "/dev/xvdh",
            "id": "vol-2",
            "mount_point": "/opt/disk2",
            "mount_options": "rw,relatime",
            "name": "test_disk_2",
        },
    ]
    aws_instance_id = "i-0123456789abcdef01"
    aws_region = "eu-west-1"
    backup_id = "20380119T031407"
    server_name = "test_server"

    def _get_mock_volumes(self, disks):
        """Helper which returns mock AwsVolumeMetadata objects for the given disks."""
        return dict(
            (
                disk["name"],
                mock.Mock(
                    mount_point=disk["mount_point"],
                    mount_options=disk["mount_options"],
                    id=disk["id"],
                ),
            )
            for disk in disks
        )

    def _get_mock_create_snapshot(self, disks):
        """Helper which returns mock create_snapshots responses for the given disks."""
        responses = iter(
            [
                {
                    "OwnerId": self.aws_account_id,
                    "SnapshotId": "snap-{}".format(disk["id"].split("-")[-1]),
                    "State": "pending",
                }
                for disk in disks
            ]
        )

        def mock_fun(*args, **kwargs):
            return next(responses)

        return mock_fun

    def _get_mock_describe_instances_resp(self, disks, virtualization_type="hvm"):
        """Helper which returns a mock describe_instances response."""
        block_device_mappings = [
            {"DeviceName": disk["device"], "Ebs": {"VolumeId": disk["id"]}}
            for disk in disks
        ]
        return {
            "Reservations": [
                {
                    "Instances": [
                        {
                            "BlockDeviceMappings": block_device_mappings,
                            "InstanceId": self.aws_instance_id,
                            "RootDeviceName": "/dev/xvda",
                            "VirtualizationType": virtualization_type,
                        }
                    ]
                }
            ]
        }

    def _get_mock_describe_volumes_resp(self, disks):
        """Helper which returns a mock describe_volumes response."""
        return {
            "Volumes": [
                {
                    "Attachments": [
                        {
                            "Device": disk["device"],
                            "InstanceId": self.aws_instance_id,
                            "VolumeId": disk["id"],
                        }
                    ],
                    "Tags": [{"Key": "Name", "Value": disk["name"]}],
                    "VolumeId": disk["id"],
                }
                for disk in disks
            ]
        }

    def _get_snapshot_id(self, disk):
        """Helper which forges the expected snapshot id for the given disk id."""
        return disk["id"].replace("vol", "snap")

    def _get_snapshot_name(self, disk):
        """Helper which forges the expected snapshot name for the given disk name."""
        return "{}-{}".format(disk["name"], self.backup_id.lower())

    @pytest.fixture()
    def mock_ec2_client(self, mock_boto3):
        yield mock_boto3.Session.return_value.client.return_value

    @pytest.fixture(autouse=True)
    def mock_boto3(self):
        with mock.patch("barman.cloud_providers.aws_s3.boto3") as mock_boto3:
            self._mock_boto3 = mock_boto3
            yield mock_boto3

    @pytest.mark.parametrize(
        ("init_args", "expected_session_args", "expected_region"),
        (
            # GIVEN no arguments, session should be created with no profile
            (
                (),
                {"profile_name": None},
                None,
            ),
            # GIVEN a profile name, session should be created with that profile
            (
                ("test_profile",),
                {"profile_name": "test_profile"},
                None,
            ),
            # GIVEN a region in the args, we expect that region to be used
            (
                ("test_profile", "eu-west-1"),
                {"profile_name": "test_profile"},
                "eu-west-1",
            ),
        ),
    )
    def test_init(self, init_args, expected_session_args, expected_region):
        """
        Verify creating AwsCloudSnapshotInterface creates the necessary EC2 client.
        """
        # WHEN an AwsCloudSnapshotInterface is created with the specified arguments
        snapshot_interface = AwsCloudSnapshotInterface(*init_args)
        # THEN a boto3.Session is created with the expected arguments
        self._mock_boto3.Session.assert_called_once_with(**expected_session_args)
        mock_session = self._mock_boto3.Session.return_value
        assert snapshot_interface.session == mock_session
        # AND an ec2 client was created
        snapshot_interface.ec2_client = mock_session.client.return_value
        # AND if we expected a region, it was used when creating the ec2 client
        if expected_region is not None:
            mock_session.client.assert_called_once_with(
                "ec2", region_name=expected_region
            )
            # AND the region is set on the snapshot interface
            assert snapshot_interface.region == expected_region
        # OR if we did not, then the session default region was used
        else:
            mock_session.client.assert_called_once_with(
                "ec2", region_name=mock_session.region_name
            )
            # AND the default region is set on the snapshot interface
            assert snapshot_interface.region == mock_session.region_name

    def test_create_snapshot(self, caplog):
        """
        Verify that _create_snapshot calls boto3 and returns the expected values.
        """
        # GIVEN a new AwsCloudInterface
        snapshot_interface = AwsCloudSnapshotInterface()
        # AND a backup_info for a given server name and backup ID
        backup_info = mock.Mock(backup_id=self.backup_id, server_name=self.server_name)
        # AND a mock create_snapshot function which returns a successful response
        mock_ec2_client = self._mock_boto3.Session.return_value.client.return_value
        mock_resp = mock_ec2_client.create_snapshot.return_value
        mock_resp["State"] = "pending"
        # AND log level is INFO
        caplog.set_level(logging.INFO)

        # WHEN _create_snapshot is called
        volume_name = "my-pgdata-volume"
        volume_id = "vol-0123456789abcdef01"
        snapshot_name, snapshot_resp = snapshot_interface._create_snapshot(
            backup_info,
            volume_name,
            volume_id,
        )

        # THEN create_snapshot is called on the EC2 client with the expected args
        mock_ec2_client.create_snapshot.assert_called_once()
        mock_ec2_client.create_snapshot.assert_called_once_with(
            TagSpecifications=[
                {
                    "ResourceType": "snapshot",
                    "Tags": [{"Key": "Name", "Value": snapshot_name}],
                }
            ],
            VolumeId=volume_id,
        )
        # AND snapshot_name has the expected value
        expected_snapshot_name = "my-pgdata-volume-{}".format(self.backup_id.lower())
        assert snapshot_name == expected_snapshot_name
        # AND the create_snapshot response was returned
        assert snapshot_resp == mock_resp
        # AND the expected log message occurred
        assert (
            "Taking snapshot '{}' of disk '{}' ({})".format(
                expected_snapshot_name, volume_name, volume_id
            )
            in caplog.text
        )

    def test_create_snapshot_failed(self):
        """
        Verify that _create_snapshot calls boto3 and returns the expected values.
        """
        # GIVEN a new AwsCloudInterface
        snapshot_interface = AwsCloudSnapshotInterface()
        # AND a backup_info for a given server name and backup ID
        backup_info = mock.Mock(backup_id=self.backup_id, server_name=self.server_name)
        # AND a mock create_snapshot function which returns a snapshot in an error
        # state
        mock_resp = {}
        mock_ec2_client = self._mock_boto3.Session.return_value.client.return_value
        mock_ec2_client.create_snapshot.return_value = mock_resp
        mock_resp["State"] = "error"

        # WHEN _create_snapshot is called
        # THEN a CloudProviderError is raised
        volume_name = "my-pgdata-volume"
        volume_id = "vol-0123456789abcdef01"
        with pytest.raises(CloudProviderError) as exc:
            snapshot_interface._create_snapshot(
                backup_info,
                volume_name,
                volume_id,
            )
        # AND the exception message contains the snapshot name
        expected_snapshot_name = "my-pgdata-volume-{}".format(self.backup_id.lower())
        assert "Snapshot '{}' failed".format(expected_snapshot_name) in str(exc.value)

    @pytest.mark.parametrize("number_of_disks", (1, 2, 3))
    def test_take_snapshot_backup(self, number_of_disks, mock_ec2_client):
        """
        Verify that take_snapshot_backup waits for completion of all snapshots and
        updates the backup_info when complete.
        """
        # GIVEN a set of disks, represented as VolumeMetadata
        disks = self.aws_disks[:number_of_disks]
        assert len(disks) == number_of_disks
        volumes = self._get_mock_volumes(disks)
        # AND a backup_info for a given server name and backup ID
        backup_info = mock.Mock(backup_id=self.backup_id, server_name=self.server_name)
        # AND a mock EC2 client which returns an instance with the required disks
        # attached
        mock_ec2_client.describe_instances.return_value = (
            self._get_mock_describe_instances_resp(disks)
        )
        # AND the mock EC2 client returns successful create_snapashot responses
        mock_ec2_client.create_snapshot.side_effect = self._get_mock_create_snapshot(
            disks
        )
        # AND a new AwsCloudSnapshotInterface
        snapshot_interface = AwsCloudSnapshotInterface(region=self.aws_region)

        # WHEN take_snapshot_backup is called
        snapshot_interface.take_snapshot_backup(
            backup_info, self.aws_instance_id, volumes
        )

        # THEN we waited for completion of all snapshots
        expected_snapshot_ids = [self._get_snapshot_id(disk) for disk in disks]
        mock_ec2_client.get_waiter.return_value.wait.assert_called_once_with(
            Filters=[{"Name": "snapshot-id", "Values": expected_snapshot_ids}],
            WaiterConfig={"Delay": 15, "MaxAttempts": 240},
        )

        # AND the backup_info is updated with the expected snapshot metadata
        snapshots_info = backup_info.snapshots_info
        assert snapshots_info.account_id == self.aws_account_id
        assert snapshots_info.region == self.aws_region
        assert snapshots_info.provider == "aws"
        assert len(snapshots_info.snapshots) == len(disks)
        for disk in disks:
            snapshot_id = self._get_snapshot_id(disk)
            snapshot = next(
                snapshot
                for snapshot in snapshots_info.snapshots
                if snapshot.identifier == snapshot_id
            )
            assert snapshot.identifier == snapshot_id
            assert snapshot.snapshot_name == self._get_snapshot_name(disk)
            assert snapshot.device_name == disk["device"]
            assert snapshot.mount_options == disk["mount_options"]
            assert snapshot.mount_point == disk["mount_point"]

    def test_take_snapshot_backup_instance_not_found(self, mock_ec2_client):
        """
        Verify that a SnapshotBackupException is raised if the instance cannot be
        found.
        """
        # GIVEN a new AwsCloudSnapshotInterface
        snapshot_interface = AwsCloudSnapshotInterface(region=self.aws_region)
        # AND a mock ec2_client which cannot find the instance
        mock_ec2_client.describe_instances.return_value = {"Reservations": []}

        # WHEN take_snapshot_backup is called
        # THEN a SnapshotBackupException is raised
        with pytest.raises(SnapshotBackupException) as exc:
            snapshot_interface.take_snapshot_backup(
                mock.Mock(),
                self.aws_instance_id,
                self._get_mock_volumes(self.aws_disks),
            )

        # AND the exception contains the expected message
        assert str(exc.value) == "Cannot find instance {}".format(self.aws_instance_id)

    def test_take_snapshot_backup_disks_not_attached(self, mock_ec2_client):
        """
        Verify that a SnapshotBackupException is raised if the expected disks are not
        attached.
        """
        # GIVEN a new AwsCloudSnapshotInterface
        snapshot_interface = AwsCloudSnapshotInterface(region=self.aws_region)
        # AND a mock ec2_client which returns an instance with no disks attached
        mock_ec2_client.describe_instances.return_value = (
            self._get_mock_describe_instances_resp([])
        )

        # WHEN take_snapshot_backup is called
        # THEN a SnapshotBackupException is raised
        with pytest.raises(SnapshotBackupException) as exc:
            snapshot_interface.take_snapshot_backup(
                mock.Mock(),
                self.aws_instance_id,
                self._get_mock_volumes(self.aws_disks),
            )

        # AND the exception contains the expected message
        assert str(exc.value) == (
            "Disk {} not attached to instance {}".format(
                self.aws_disks[0]["name"], self.aws_instance_id
            )
        )

    @pytest.mark.parametrize(
        ("await_snapshots_timeout", "expected_wait_config"),
        (
            # No timeout specified, default values should be used
            (None, {"Delay": 15, "MaxAttempts": 240}),
            # Timeout of zero specified, only one attempt should be used
            (0, {"Delay": 15, "MaxAttempts": 1}),
            # Timeout less than delay, only one attempt should be used
            (10, {"Delay": 15, "MaxAttempts": 1}),
            # Timeout greater than delay, but less than 2*delay, two attempts should be used
            (20, {"Delay": 15, "MaxAttempts": 2}),
            # Large timeout value should result in many attempts
            (7200, {"Delay": 15, "MaxAttempts": 480}),
        ),
    )
    def test_take_snapshot_backup_wait(
        self, await_snapshots_timeout, expected_wait_config, mock_ec2_client
    ):
        """
        Verify that take_snapshot_backup waits for completion of all snapshots and
        updates the backup_info when complete.
        """
        # GIVEN a set of disks, represented as VolumeMetadata
        number_of_disks = 2
        disks = self.aws_disks[:number_of_disks]
        assert len(disks) == number_of_disks
        volumes = self._get_mock_volumes(disks)
        # AND a backup_info for a given server name and backup ID
        backup_info = mock.Mock(backup_id=self.backup_id, server_name=self.server_name)
        # AND a mock EC2 client which returns an instance with the required disks
        # attached
        mock_ec2_client.describe_instances.return_value = (
            self._get_mock_describe_instances_resp(disks)
        )
        # AND the mock EC2 client returns successful create_snapashot responses
        mock_ec2_client.create_snapshot.side_effect = self._get_mock_create_snapshot(
            disks
        )
        # AND a new AwsCloudSnapshotInterface
        kwargs = {"region": self.aws_region}
        if await_snapshots_timeout is not None:
            kwargs["await_snapshots_timeout"] = await_snapshots_timeout
        snapshot_interface = AwsCloudSnapshotInterface(**kwargs)

        # WHEN take_snapshot_backup is called
        snapshot_interface.take_snapshot_backup(
            backup_info, self.aws_instance_id, volumes
        )

        # THEN we waited for completion of all snapshots with the expected WaiterConfig
        expected_snapshot_ids = [self._get_snapshot_id(disk) for disk in disks]
        mock_ec2_client.get_waiter.return_value.wait.assert_called_once_with(
            Filters=[{"Name": "snapshot-id", "Values": expected_snapshot_ids}],
            WaiterConfig=expected_wait_config,
        )

    aws_live_states = ["pending", "running", "shutting-down", "stopping", "stopped"]

    def test_get_instance_metadata_by_id(self, mock_ec2_client):
        """
        Verify that instance metadata is returned when queried by ID.
        """
        # GIVEN a mock snapshots interface
        snapshots_interface = AwsCloudSnapshotInterface(region=self.aws_region)
        # AND an EC2 client which responds successfully
        mock_instance_metadata = mock.Mock()
        mock_ec2_client.describe_instances.return_value = {
            "Reservations": [{"Instances": [mock_instance_metadata]}]
        }

        # WHEN _get_instance_metadata is called with an instance ID
        instance_metadata = snapshots_interface._get_instance_metadata(
            self.aws_instance_id
        )

        # THEN describe_instances was called once with the instance ID and filter
        mock_ec2_client.describe_instances.assert_called_once_with(
            InstanceIds=[self.aws_instance_id],
            Filters=[{"Name": "instance-state-name", "Values": self.aws_live_states}],
        )

        # AND the mock instance metadata was returned
        assert instance_metadata == mock_instance_metadata

    def test_get_instance_metadata_by_id_like_name(self, mock_ec2_client):
        """
        Verify that if an ID is malformed, Barman attempts to look it up by name.
        """
        # GIVEN a mock snapshots interface
        snapshots_interface = AwsCloudSnapshotInterface(region=self.aws_region)
        # AND an EC2 client which responds first with a InvalidInstanceID.Malformed
        # and then with a successful response
        mock_instance_metadata = mock.Mock()
        mock_ec2_client.describe_instances.side_effect = [
            ClientError({"Error": {"Code": "InvalidInstanceID.Malformed"}}, ""),
            {"Reservations": [{"Instances": [mock_instance_metadata]}]},
        ]

        # WHEN _get_instance_metadata is called with an instance name which looks
        # superficially like an ID
        instance_name = "i-not-an-id"
        instance_metadata = snapshots_interface._get_instance_metadata(instance_name)

        # THEN describe_instances was called once with the instance ID and filter
        assert mock_ec2_client.describe_instances.call_args_list[0][1] == {
            "InstanceIds": [instance_name],
            "Filters": [
                {"Name": "instance-state-name", "Values": self.aws_live_states}
            ],
        }
        # AND again with a tag filter
        assert mock_ec2_client.describe_instances.call_args_list[1][1] == {
            "Filters": [
                {"Name": "tag:Name", "Values": [instance_name]},
                {"Name": "instance-state-name", "Values": self.aws_live_states},
            ]
        }

        # AND the mock instance metadata was returned
        assert instance_metadata == mock_instance_metadata

    def test_get_instance_metadata_by_name(self, mock_ec2_client):
        """
        Verify that names which do not look like IDs are only looked up by name.
        """
        # GIVEN a mock snapshots interface
        snapshots_interface = AwsCloudSnapshotInterface(region=self.aws_region)
        # AND an EC2 client which responds successfully
        mock_instance_metadata = mock.Mock()
        mock_ec2_client.describe_instances.return_value = {
            "Reservations": [{"Instances": [mock_instance_metadata]}]
        }

        # WHEN _get_instance_metadata is called with an instance name
        instance_name = "the name of an instance"
        instance_metadata = snapshots_interface._get_instance_metadata(instance_name)

        # THEN describe_instances was called once with the a tag filter
        mock_ec2_client.describe_instances.assert_called_once_with(
            Filters=[
                {"Name": "tag:Name", "Values": [instance_name]},
                {"Name": "instance-state-name", "Values": self.aws_live_states},
            ]
        )

        # AND the mock instance metadata was returned
        assert instance_metadata == mock_instance_metadata

    @pytest.mark.parametrize(
        ("describe_instances_resp", "expected_exception", "expected_msg"),
        (
            # If no reservations are returned we expect a
            # SnapshotInstanceNotFoundException
            (
                {"Reservations": []},
                SnapshotInstanceNotFoundException,
                "Cannot find instance",
            ),
            # If no instances are returned we expect a
            # SnapshotInstanceNotFoundException
            (
                {"Reservations": [{"Instances": []}]},
                SnapshotInstanceNotFoundException,
                "Cannot find instance",
            ),
            # If multiple reservations are returned we expect a
            # CloudProviderError
            (
                {"Reservations": [{"Instances": []}, {"Instances": []}]},
                CloudProviderError,
                "Cannot find a unique EC2 reservation containing instance",
            ),
            # If multiple instances are returned we expect a
            # CloudProviderError
            (
                {"Reservations": [{"Instances": [{}, {}]}]},
                CloudProviderError,
                "Cannot find a unique EC2 instance matching",
            ),
            # If multiple reservations and instances are returned we expect a
            # CloudProviderError
            (
                {"Reservations": [{"Instances": [{}, {}]}, {"Instances": []}]},
                CloudProviderError,
                "Cannot find a unique EC2 reservation containing instance",
            ),
        ),
    )
    def test_get_instance_metadata_errors(
        self, describe_instances_resp, expected_exception, expected_msg, mock_ec2_client
    ):
        """
        Verify the expected exceptions are raised when there are either too few or
        too many matching reservations and instances.
        """
        # GIVEN a mock snapshots interface
        snapshots_interface = AwsCloudSnapshotInterface(region=self.aws_region)

        # AND an EC2 client which responds with the specified response
        mock_ec2_client.describe_instances.return_value = describe_instances_resp

        # WHEN _get_instance_metadata is called
        # THEN the expected exception is raised
        with pytest.raises(expected_exception) as exc:
            snapshots_interface._get_instance_metadata("some instance name")

        # AND the exception has the expected value
        assert expected_msg in str(exc.value)

    def test_get_attached_volumes(self, mock_ec2_client):
        """
        Verify that attached volumes are returned as a dict keyed by the expected
        identifier.
        """
        # GIVEN a mock snapshots interface
        snapshot_interface = AwsCloudSnapshotInterface(region=self.aws_region)
        # AND a mock EC2 client which returns an instance with the required disks
        # attached
        virtualization_type = "hvm"
        mock_ec2_client.describe_instances.return_value = (
            self._get_mock_describe_instances_resp(
                self.aws_disks, virtualization_type=virtualization_type
            )
        )
        # AND the mock EC2 client returns describe_volume_responses for these disks
        mock_ec2_client.describe_volumes.return_value = (
            self._get_mock_describe_volumes_resp(self.aws_disks)
        )
        # AND the first disk is the root device
        root_disk = self.aws_disks[0]
        attached_disks = self.aws_disks[1:]
        mock_instance_resp = mock_ec2_client.describe_instances.return_value
        mock_instance = mock_instance_resp["Reservations"][0]["Instances"][0]
        mock_instance["RootDeviceName"] = root_disk["device"]

        # WHEN get_attached_volumes is called
        volumes = snapshot_interface.get_attached_volumes(self.aws_instance_id)

        # THEN describe_volumes was called filtering by instance ID
        mock_ec2_client.describe_volumes.assert_called_once_with(
            Filters=[
                {
                    "Name": "attachment.instance-id",
                    "Values": [self.aws_instance_id],
                }
            ]
        )
        # AND the attached disks have been returned, indexed by volume ID
        assert set(volumes.keys()) == set(disk["id"] for disk in attached_disks)
        # AND the instance virtualization type has been saved
        assert all(
            volume._virtualization_type == virtualization_type
            for volume in volumes.values()
        )
        # AND the volume ID has been saved
        assert [volume.id for volume in volumes.values()] == [
            disk["id"] for disk in attached_disks
        ]
        # AND the root volume was not included
        assert root_disk["id"] not in volumes

    def test_get_attached_volumes_with_source_snapshots(self, mock_ec2_client):
        """
        Verify that attached volumes contain snapshot IDs when the AWS response
        includes a snapshot ID for that volume.
        """
        # GIVEN a mock snapshots interface
        snapshot_interface = AwsCloudSnapshotInterface(region=self.aws_region)
        # AND a mock EC2 client which returns an instance with the required disks
        # attached
        mock_ec2_client.describe_instances.return_value = (
            self._get_mock_describe_instances_resp(
                self.aws_disks,
            )
        )
        # AND the mock EC2 client returns describe_volume_responses for these disks
        mock_ec2_client.describe_volumes.return_value = (
            self._get_mock_describe_volumes_resp(self.aws_disks)
        )
        # AND one of those disks has a SnapshotId
        mock_ec2_client.describe_volumes.return_value["Volumes"][0][
            "SnapshotId"
        ] = "snap-0123"

        # WHEN get_attached_volumes is called
        volumes = snapshot_interface.get_attached_volumes(self.aws_instance_id)

        # THEN the source snapshot is set on the volume which had a SnapshotId
        assert volumes[self.aws_disks[0]["id"]].source_snapshot == "snap-0123"

        # AND the source snapshot is not set on the other volumes
        assert all(
            volumes[disk["id"]].source_snapshot is None for disk in self.aws_disks[1:]
        )

    def test_get_attached_volumes_for_disks(self, mock_ec2_client):
        """
        Verify that the requested disks are returned as a dict keyed by the expected
        identifier.
        """
        # GIVEN a mock snapshots interface
        snapshot_interface = AwsCloudSnapshotInterface(region=self.aws_region)
        # AND a mock EC2 client which returns an instance with the required disks
        # attached
        virtualization_type = "hvm"
        mock_ec2_client.describe_instances.return_value = (
            self._get_mock_describe_instances_resp(
                self.aws_disks, virtualization_type=virtualization_type
            )
        )
        # AND the mock EC2 client returns describe_volume_responses for these disks
        mock_ec2_client.describe_volumes.return_value = (
            self._get_mock_describe_volumes_resp(self.aws_disks)
        )

        # WHEN get_attached_volumes is called for disks specified by a mix of name
        # and id
        requested_disks = [self.aws_disks[1]["id"], self.aws_disks[2]["name"]]
        volumes = snapshot_interface.get_attached_volumes(
            self.aws_instance_id, requested_disks
        )

        # THEN describe_volumes was called filtering by instance ID
        mock_ec2_client.describe_volumes.assert_called_once_with(
            Filters=[
                {
                    "Name": "attachment.instance-id",
                    "Values": [self.aws_instance_id],
                }
            ]
        )
        # AND only the requested disks have been returned, indexed by the identifier
        # used to request them
        assert set(volumes.keys()) == set(disk for disk in requested_disks)
        # AND the volume ID has been saved
        assert [volume.id for volume in volumes.values()] == [
            disk["id"] for disk in self.aws_disks[1:]
        ]

    def test_get_attached_volumes_disks_not_found(self, mock_ec2_client):
        """Verify behaviour when a requested disk cannot be found."""
        # GIVEN a mock snapshots interface
        snapshot_interface = AwsCloudSnapshotInterface(region=self.aws_region)
        # AND a mock EC2 client which returns an instance with a subset of disks
        # attached
        virtualization_type = "hvm"
        mock_ec2_client.describe_instances.return_value = (
            self._get_mock_describe_instances_resp(
                self.aws_disks[-1:], virtualization_type=virtualization_type
            )
        )
        # AND the mock EC2 client returns describe_volume_responses for these disks
        mock_ec2_client.describe_volumes.return_value = (
            self._get_mock_describe_volumes_resp(self.aws_disks[-1:])
        )

        # WHEN get_attached_volumes is called requesting all disks using a mix of
        # names and IDs
        # THEN a SnapshotBackupException is raised
        requested_disks = [
            self.aws_disks[0]["id"],
            self.aws_disks[1]["name"],
            self.aws_disks[2]["id"],
        ]
        expected_missing_disks = [
            self.aws_disks[0]["id"],
            self.aws_disks[1]["name"],
        ]
        with pytest.raises(SnapshotBackupException) as exc:
            snapshot_interface.get_attached_volumes(
                self.aws_instance_id, requested_disks
            )
        # AND The exception contains the expected message
        assert str(exc.value) == "Disks not attached to instance {}: {}".format(
            self.aws_instance_id, ", ".join(expected_missing_disks)
        )
        # WHEN get_attached_volumes is called with fail_on_missing=False
        # THEN no exception is raised
        volumes = snapshot_interface.get_attached_volumes(
            self.aws_instance_id,
            requested_disks,
            fail_on_missing=False,
        )
        # AND the attached volumes contains only those disks which were present
        assert set(volumes.keys()) == set(requested_disks[-1:])

    @pytest.mark.parametrize(
        ("mock_disks", "identifier_key", "should_succeed"),
        (
            # Looking up by name should fail when there are multiple matching names
            (
                [
                    {"id": "vol-0", "name": "test_disk", "device": "/dev/sdf"},
                    {"id": "vol-1", "name": "test_disk", "device": "/dev/sdg"},
                ],
                "name",
                False,
            ),
            # Looking up by name should succeed even if volume IDs are duplicates
            (
                [
                    {"id": "vol-0", "name": "test disk", "device": "/dev/sdf"},
                    {"id": "vol-0", "name": "other disk", "device": "/dev/sdg"},
                ],
                "name",
                True,
            ),
            # Looking up by id should fail if there are multiple matching IDs
            (
                [
                    {"id": "vol-0", "name": "test disk", "device": "/dev/sdf"},
                    {"id": "vol-0", "name": "other disk", "device": "/dev/sdg"},
                ],
                "id",
                False,
            ),
            # Looking up by id should succeed even if volume names are duplicates
            (
                [
                    {"id": "vol-0", "name": "test_disk", "device": "/dev/sdf"},
                    {"id": "vol-1", "name": "test_disk", "device": "/dev/sdg"},
                ],
                "id",
                True,
            ),
        ),
    )
    def test_get_attached_volumes_duplicates(
        self, mock_disks, identifier_key, should_succeed, mock_ec2_client
    ):
        """Verify behaviour when a requested disk has multiple matches."""
        # GIVEN a mock snapshots interface
        snapshot_interface = AwsCloudSnapshotInterface(region=self.aws_region)
        # AND a mock EC2 client which returns an instance with the speified disks
        mock_ec2_client.describe_instances.return_value = (
            self._get_mock_describe_instances_resp(mock_disks)
        )
        # AND the mock EC2 client returns describe_volume_responses for these disks
        mock_ec2_client.describe_volumes.return_value = (
            self._get_mock_describe_volumes_resp(mock_disks)
        )

        # WHEN get_attached_volumes is called requesting the disks using the specified
        # identifier
        # AND we expect it to succeed
        # THEN no exception is raised
        disks_to_request = [disk[identifier_key] for disk in mock_disks]
        if should_succeed:
            snapshot_interface.get_attached_volumes(
                self.aws_instance_id, disks_to_request
            )
        # AND if we expect it to fail
        # THEN a SnapshotBackupException is raised
        else:
            with pytest.raises(CloudProviderError) as exc:
                snapshot_interface.get_attached_volumes(
                    self.aws_instance_id, disks_to_request
                )
            # AND the exception message has the expected content
            assert "Duplicate volumes found matching {}: {}".format(
                disks_to_request[0], ", ".join(d["id"] for d in mock_disks)
            ) in str(exc.value)

    def test_get_attached_volumes_instance_not_found(self, mock_ec2_client):
        """Verify behaviour when a requested instance cannot be found."""
        # GIVEN a mock snapshots interface
        snapshot_interface = AwsCloudSnapshotInterface(region=self.aws_region)
        # AND a mock EC2 client which returns no instances
        mock_ec2_client.describe_instances.return_value = {"Reservations": []}

        # WHEN get_attached_volumes is called
        # THEN a SnapshotInstanceNotFoundException is raised
        with pytest.raises(SnapshotInstanceNotFoundException):
            snapshot_interface.get_attached_volumes(self.aws_instance_id)

    def test_instance_exists(self, mock_ec2_client):
        """Verify that instance_exists returns True if an instance exists."""
        # GIVEN a mock snapshots interface
        snapshot_interface = AwsCloudSnapshotInterface(region=self.aws_region)
        # AND a mock EC2 client which returns a matching instance
        mock_ec2_client.describe_instances.return_value = {
            "Reservations": [{"Instances": [{"InstanceId": self.aws_instance_id}]}]
        }
        # WHEN instance_exists is called
        resp = snapshot_interface.instance_exists(self.aws_instance_id)
        # THEN it returns True
        assert resp is True

    def test_instance_exists_not_found(self, mock_ec2_client):
        """Verify that instance_exists returns False if an instance is not found."""
        # GIVEN a mock snapshots interface
        snapshot_interface = AwsCloudSnapshotInterface(region=self.aws_region)
        # AND a mock EC2 client which returns no instances
        mock_ec2_client.describe_instances.return_value = {"Reservations": []}
        # WHEN instance_exists is called
        resp = snapshot_interface.instance_exists(self.aws_instance_id)
        # THEN it returns False
        assert resp is False

    def test_delete_snapshot(self, mock_ec2_client, caplog):
        """Verify that a snapshot can be deleted successfully."""
        # GIVEN a successful response from the delete snapshot request
        mock_ec2_client.delete_snapshot.return_value = {}
        # AND a mock snapshots interface
        snapshot_interface = AwsCloudSnapshotInterface(region=self.aws_region)
        # AND log level is info
        caplog.set_level(logging.INFO)

        # WHEN a snapshot is deleted
        snapshot_id = "snap-0123"
        snapshot_interface._delete_snapshot(snapshot_id)

        # THEN delete was called on the client with the expected arguments
        mock_ec2_client.delete_snapshot.assert_called_once_with(SnapshotId=snapshot_id)
        # AND a success message was logged
        assert "Snapshot {} deleted".format(snapshot_id) in caplog.text

    def test_delete_snapshot_not_found(self, mock_ec2_client, caplog):
        """Verify that a snapshot ID which can't be found is success."""
        # GIVEN a successful response from the delete snapshot request
        mock_ec2_client.delete_snapshot.side_effect = (
            ClientError({"Error": {"Code": "InvalidSnapshot.NotFound"}}, "message"),
        )
        # AND a mock snapshots interface
        snapshot_interface = AwsCloudSnapshotInterface(region=self.aws_region)
        # AND log level is info
        caplog.set_level(logging.INFO)

        # WHEN a snapshot is deleted
        # THEN no exceptions are raised
        snapshot_id = "snap-0123"
        snapshot_interface._delete_snapshot(snapshot_id)

        # THEN delete was called on the client with the expected arguments
        mock_ec2_client.delete_snapshot.assert_called_once_with(SnapshotId=snapshot_id)
        # AND a success message was logged
        assert "Snapshot {} deleted".format(snapshot_id) in caplog.text
        # AND a warning message was logged
        assert "Snapshot {} could not be found".format(snapshot_id) in caplog.text

    def test_delete_snapshot_failed(self, mock_ec2_client, caplog):
        """Verify that a failed deletion results in a CloudProviderError."""
        # GIVEN an unexpected error from the delete snapshot request
        mock_ec2_client.delete_snapshot.side_effect = (
            ClientError({"Error": {"Code": "Something.Bad"}}, "message"),
        )
        # AND a mock snapshots interface
        snapshot_interface = AwsCloudSnapshotInterface(region=self.aws_region)

        # WHEN a snapshot is deleted
        # THEN a CloudProviderError is raised
        snapshot_id = "snap-0123"
        with pytest.raises(CloudProviderError) as exc:
            snapshot_interface._delete_snapshot(snapshot_id)

        # AND the exception has the expected message
        expected_message = "Deletion of snapshot {} failed with error code {}".format(
            snapshot_id, "Something.Bad"
        )
        assert expected_message in str(exc.value)

    @pytest.mark.parametrize(
        "snapshots_list",
        (
            [],
            [mock.Mock(identifier="snap-0123")],
            [mock.Mock(identifier="snap-0123"), mock.Mock(identifier="snap0124")],
        ),
    )
    def test_delete_snapshot_backup(self, snapshots_list, mock_ec2_client, caplog):
        """Verify that all snapshots for a backup are deleted."""
        # GIVEN a backup_info specifying zero or more snapshots
        backup_info = mock.Mock(
            backup_id=self.backup_id,
            snapshots_info=mock.Mock(snapshots=snapshots_list),
        )
        # AND log level is info
        caplog.set_level(logging.INFO)
        # AND the snapshot delete requests are successful
        mock_ec2_client.delete_snapshot.return_value = {}
        # AND a new AwsCloudSnapshotInterface
        snapshot_interface = AwsCloudSnapshotInterface(region=self.aws_region)

        # WHEN delete_snapshot_backup is called
        snapshot_interface.delete_snapshot_backup(backup_info)

        # THEN delete_snapshot was called for each snapshot
        expected_calls = [
            mock.call(SnapshotId=snapshot.identifier) for snapshot in snapshots_list
        ]
        mock_ec2_client.delete_snapshot.assert_has_calls(expected_calls)


class TestAwsVolumeMetadata(object):
    """Verify behaviour of AwsVolumeMetadata."""

    @pytest.mark.parametrize(
        (
            "attachment_metadata",
            "virtualization_type",
            "source_snapshot",
            "expected_virtualization_type",
            "expected_source_snapshot",
            "expected_device_name",
            "expected_id",
        ),
        (
            (None, None, None, None, None, None, None),
            ({}, None, None, None, None, None, None),
            ({}, "hvm", None, "hvm", None, None, None),
            (
                {"Device": "/dev/xvdf", "VolumeId": "vol-0123"},
                "hvm",
                None,
                "hvm",
                None,
                "/dev/xvdf",
                "vol-0123",
            ),
            (
                {"Device": "/dev/xvdf", "VolumeId": "vol-0123"},
                None,
                "snap-0123",
                None,
                "snap-0123",
                "/dev/xvdf",
                "vol-0123",
            ),
            (
                {"Device": "/dev/xvdf", "VolumeId": "vol-0123"},
                "hvm",
                "snap-0123",
                "hvm",
                "snap-0123",
                "/dev/xvdf",
                "vol-0123",
            ),
        ),
    )
    def test_init(
        self,
        attachment_metadata,
        virtualization_type,
        source_snapshot,
        expected_virtualization_type,
        expected_source_snapshot,
        expected_device_name,
        expected_id,
    ):
        """Verify AwsVolumeMetadata is created from the supplied data."""
        # WHEN an AwsVolumeMetadata is created
        volume = AwsVolumeMetadata(
            attachment_metadata, virtualization_type, source_snapshot
        )
        # THEN the resulting objecth as the expected properties
        assert volume._virtualization_type == expected_virtualization_type
        assert volume.source_snapshot == expected_source_snapshot
        assert volume._device_name == expected_device_name
        assert volume.id == expected_id

    @pytest.mark.parametrize(
        (
            "device_name_from_api",
            "device_name_on_instance",
            "virtualization_type",
        ),
        (
            # Devices mapped with the same name should be found for either type of
            # virtualization
            ("/dev/sdf", "/dev/sdf", "hvm"),
            ("/dev/sdf", "/dev/sdf", "paravirtual"),
            # Devices mapped to xvdf should be found with hardware virtualization
            ("/dev/sdf", "/dev/xvdf", "hvm"),
            # Devices mapped to hdf should be found with paravirtualization
            ("/dev/sdf", "/dev/hdf", "paravirtual"),
        ),
    )
    def test_resolve_mounted_volume(
        self, device_name_from_api, device_name_on_instance, virtualization_type
    ):
        # GIVEN AwsVolumeMetadata with the API-reported device name and virtualization
        # type
        attachment_metadata = {"VolumeId": "vol-0123", "Device": device_name_from_api}
        volume = AwsVolumeMetadata(attachment_metadata, virtualization_type)
        # AND a findmnt response which returns mount data for the mapped device name
        mock_cmd = mock.Mock()

        def mock_findmnt(device):
            if device == device_name_on_instance:
                return "mount_point", "mount_options"
            else:
                return None, None

        mock_cmd.findmnt.side_effect = mock_findmnt

        # WHEN resolve_mounted_volume is called
        volume.resolve_mounted_volume(mock_cmd)

        # THEN the expected mount data is set on the volume metadata
        assert volume.mount_point == "mount_point"
        assert volume.mount_options == "mount_options"

    @pytest.mark.parametrize(
        (
            "mock_findmnt",
            "device_name_from_api",
            "expected_exception_msg",
        ),
        (
            (
                lambda x: (None, None),
                "/dev/sdf",
                "Could not find device /dev/sdf at any mount point",
            ),
            (
                CommandException("error doing findmnt"),
                "/dev/sdf",
                "Error finding mount point for device path /dev/sdf: error doing findmnt",
            ),
            (
                lambda x: (None, None),
                None,
                "Cannot resolve mounted volume: device name unknown",
            ),
        ),
    )
    def test_resolve_mounted_volume_failure(
        self, mock_findmnt, device_name_from_api, expected_exception_msg
    ):
        # GIVEN AwsVolumeMetadata with the API-reported device name and virtualization
        # type
        attachment_metadata = {"VolumeId": "vol-0123", "Device": device_name_from_api}
        volume = AwsVolumeMetadata(attachment_metadata)
        # AND a findmnt response which returns mount data for the mapped device name
        mock_cmd = mock.Mock()
        mock_cmd.findmnt.side_effect = mock_findmnt

        # WHEN resolve_mounted_volume is called
        # THEN a SnapshotBackupException is raised
        with pytest.raises(SnapshotBackupException) as exc:
            volume.resolve_mounted_volume(mock_cmd)

        # AND the exception has the expected error message
        assert str(exc.value) == expected_exception_msg
