Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Added dummy identity provider to remove Keystone dependancy during testing #162

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

Added dummy identity provider to remove Keystone dependancy during testing #162

wants to merge 2 commits into from

Commits on Nov 13, 2024

  1. Added dummy identity provider to remove Keystone dependency 

    during testing

The identity provider used by `esi_leap` is now abstracted with
a `BaseIDP` ABC, and implemented by two subclasses: A `KeystoneIDP`
class which mostly copied the code from `common/keystone.py`,
and `DummyIDP` which mocks a real IDP

The IDP used by `esi_leap` can be set by overriding the `idp_type` CONF value like so:

CONF.set_override(
       "idp_type", "dummy_idp", group="esi"
 )

As a consequence of abstracting the IDP, `common/keystone.py`
has been removed. All references to `common/keystone.py` and
the functions defined in it has been appropriately changed.

New unit tests have been added for the keystone and dummy IDP clients
    @QuanMPhm
    QuanMPhm committed Nov 13, 2024
    Configuration menu
    Copy the full SHA
    2b31ea8 View commit details
    Browse the repository at this point in the history

  2. Moved some utility functions into LeaseController and `OfferControl… 

    …ler`.
    @QuanMPhm
    QuanMPhm committed Nov 13, 2024
    Configuration menu
    Copy the full SHA
    f8099ba View commit details
    Browse the repository at this point in the history