Skip to content

Remove pyoidc and pysaml2 from core - #442

Open
rohe wants to merge 11 commits into
masterfrom
rm_saml2_from_core_2
Open

Remove pyoidc and pysaml2 from core#442
rohe wants to merge 11 commits into
masterfrom
rm_saml2_from_core_2

Conversation

@rohe

@roherohe commented Aug 18, 2023

Copy link
Copy Markdown
Contributor

All Submissions:

  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?
  • Have you added an explanation of what problem you are trying to solve with this PR?
  • Have you added information on what your changes do and why you chose this as your solution?
  • Have you written new tests for your changes?
  • Does your submission pass tests?
  • This project follows PEP8 style guide. Have you run your code against the 'flake8' linter?

This PR makes pytest skip tests that are dependent on uninstalled packages. To do this I use an check (importorskip) that comes with pytest.

Also added a missing test for the idpyoidc based OIDC backend.

roheand others added 11 commits June 8, 2023 09:40
Replacing jwkest with cryptojwt.
Making tests that are dependent on pyoidc not be executed if pyoidc is not installed.
Switched from AES-CBC crypto to AES-GCM.
Replaced dependency on pycryptodomex to cryptojwt.
Refactored idpy_oidc.py to get a create_client function used in testing.
Made pytest skip test modules if a necessary package was not present.
Signed-off-by: Ivan Kanakarakis <ivan.kanak@gmail.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@rohe@c00kiemon5ter