import pytest

from namespace.notifications import get_notifier
from namespace.notifications import notifier


def test_double_definition():
    """Check protection against notifier duplication."""
    _ = get_notifier("slack")
    with pytest.raises(AttributeError):

        class DuplicatedNotifier(notifier.Notifier, name="slack"):  # NOQA
            """Duplicated notifier name."""

get_notifier

#!/usr/bin/env python3
# -*- coding: utf-8 -*-
"""Load notification plugins."""

import pkgutil
import importlib


def get_notifier(name):
    """Load the correct notifier.

    Imports all modules in `proton.common.notifiers` and checks the loaded
    grabbers in Notifier.

    Parameters
    ----------
    name : str
        Name of the Notifier to load.

    Returns
    -------
    Notifier
        None if not found.

    """
    from namespace.notifications import notifiers as __notifiers
    from namespace.notifications.notifier import Notifier

    if name not in Notifier.NOTIFIERS:
        for _, module_name, _ispkg in pkgutil.iter_modules(
            __notifiers.__path__, __notifiers.__name__ + "."
        ):
            importlib.import_module(module_name)
    if name in Notifier.NOTIFIERS:
        return Notifier.NOTIFIERS[name]
    return None

notifier.py

class Notifier:
    """Generic notifier.

    Attributes
    ----------
    REQUIRED_KEYS : set
        See below.
    SERVICE_KEYS : set
        See below.
    NOTIFIERS : dict
        Autodiscovered subclasse, i.e., available notifiers.

    Parameters
    ----------
    environment : str
        Environment in which the app is running (local, qa, production...)
    common_config : dict
        Common config as specified in proton.common/config_data/notifiers.
    notifier_config : dict
        Notifier-specific config. Required entries in the dict are defined in
        REQUIRED_KEYS.
    service_config : dict
        Config of the notification service, with required entries defined in SERVICE_KEYS.
    """

    NOTIFIERS = {}
    REQUIRED_KEYS = set()
    SERVICE_KEYS = set()

    def __init_subclass__(cls, name, **kwargs):
        """Add grabber class to loaded grabbers list."""
        super().__init_subclass__(**kwargs)
        cls.name = name
        if name:
            if name in Notifier.NOTIFIERS:
                raise AttributeError(f"Notifier defined twice -> {name}")
            Notifier.NOTIFIERS[name] = cls

namespace/notifications/notifiers/*.py

>       from namespace.notifications import notifiers as __notifiers
E       ImportError: cannot import name 'notifiers' from 'namespace.notifications' (lib/notifications/namespace/notifications/__init__.py)

#!/usr/bin/env python3
# -*- coding: utf-8 -*-
"""Test the get_notifier functionality."""

from namespace.notifications import get_notifier
from namespace.notifications.notifiers.slack import SlackNotifier


def test_get_slack_notifier():
    """Load Slack notifier."""
    notifier = get_notifier("slack")
    assert notifier == SlackNotifier

notifiers

repl

lib/notifications
├── BUILD
├── README.md
├── namespace
│   └── notifications
│       ├── __init__.py
│       ├── center.py
│       ├── config_data
│       │   └── notifiers.yaml
│       ├── levels.py
│       ├── notifier.py
│       └── notifiers
│           ├── email.py
│           └── slack.py
├── requirements.txt
└── tests
    ├── test_get_notifier.py
    ├── test_levels.py
    ├── test_notification_center.py
    ├── test_notifications.py
    ├── test_notifier.py
    ├── test_notifiers_email.py
    └── test_notifiers_slack.py

$ ./pants dependencies lib/notifications:unit
16:02:30.30 [WARN] Unmatched glob from lib/notifications:unit's `sources` field: "lib/notifications/tests/unit/*.py"

Do the file(s) exist? If so, check if the file(s) are in your `.gitignore` or the global `pants_ignore` option, which may result in Pants not being able to see the file(s) even though they exist on disk. Refer to <https://www.pantsbuild.org/v2.6/docs/troubleshooting#pants-cannot-find-a-file-in-your-project>.
//:pytest
lib/notifications/namespace/notifications/__init__.py:../../notifications
lib/notifications/namespace/notifications/center.py:../../notifications
lib/notifications/namespace/notifications/levels.py:../../notifications
lib/notifications/namespace/notifications/notifier.py:../../notifications
lib/notifications/namespace/notifications/notifiers/email.py:../../../notifications
lib/notifications/namespace/notifications/notifiers/slack.py:../../../notifications
lib/notifications/tests/test_get_notifier.py:../unit
lib/notifications/tests/test_levels.py:../unit
lib/notifications/tests/test_notification_center.py:../unit
lib/notifications/tests/test_notifications.py:../unit
lib/notifications/tests/test_notifier.py:../unit
lib/notifications/tests/test_notifiers_email.py:../unit
lib/notifications/tests/test_notifiers_slack.py:../unit

@astonishing-london-2419 I think the recursive glob in the macro is shooting you in the foot. You’ll need to have a target just for the notifiers and then add an explicit dependency on that target.

notifications

notifications

notifiers

python_library(
    name="notifications", 
    sources=["namespace/notifications/**/*.py"], 
    dependencies=[
      ":config",
      # This target depends on itself because dependencies on the
      # `notifiers` package are not inferred, and must be included manually.
      ":notifications",
    ],
)