stackabletech / stackabletech/opa-operator

Add tests for user_group_distinguished_names()

Abierto
#640 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

type/internal-debt
Lenguaje dominante
Rust
Estrellas
21
Forks
5
Merge medio
12 h 44 min
PR fusionados (30 d)
11

Descripción

In https://github.com/stackabletech/opa-operator/pull/622#top 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.

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Línea de trabajo

Comienza con el código de implementación y conversión añadido en pull request 622, especialmente con la ruta user_group_distinguished_names(). Añade pruebas unitarias o, si es necesario, pruebas de integración que cubran las representaciones de los ID de usuario y de grupo de Active Directory; se considera terminado cuando el comportamiento de conversión está cubierto por las pruebas sin bloquear el release.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
rust
Área
backend, testing
Tipo de issue
Refactorización
Dificultad
3/5
Tiempo estimado
1-2 días
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
48/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.