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.

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

Add tests for user_group_distinguished_names() #640

Open
3 tasks
soenkeliebau opened this issue Oct 25, 2024 · 0 comments
Open
3 tasks

Add tests for user_group_distinguished_names() #640

soenkeliebau opened this issue Oct 25, 2024 · 0 comments

Comments

@soenkeliebau
Copy link
Member

In #622 (comment) we added an Active Directory backend to the user info fetcher.

As part of this, there was fairly extensive code to convert between some weird representations which AD chose for userids and groupids.

In order to not block the release with this we decided to punt on adding unit tests (hopefully, worst case: integration tests) for this later.

We should try and get this done fairly soon after 24.11.

Issue checklist

This is a simple checklist of things to bear in mind when creating a new issue.

  • Describe the use-case, as far is possible. For instance, using the pattern "As a XXXX, I would like XXXX to be able to do XXXX" helps to identify the feature as well as the problem it is intended to address.
  • Indicate an approximate level of importance and urgency.
  • Indicate if there is a known work-around until such time as the issue has been implemented.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant