Add tests for user_group_distinguished_names()

未关闭
#640 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

还没有人认领这个 Issue。

评估

难度
3/5
预计耗时
1-2 天
新手友好度
48/100
Issue 类型
重构
描述清晰度
基本清楚
活跃度
停滞
技术栈
rust
领域
backend, testing

调研方向

从 pull request 622 中添加的实现和转换代码开始，尤其是 user_group_distinguished_names() 路径。添加涵盖 Active Directory 用户和组 ID 表示形式的单元测试，或在必要时添加集成测试；完成的标准是，在不阻塞 release 的情况下对转换行为进行了测试。

由索引模型根据 Issue 内容生成。

描述

type/internal-debt

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.
主要语言
Rust
星标
21
派生
5
平均合并
12 小时 44 分钟
30 天内合并 PR
11

贡献指南

这个仓库没有索引到贡献指南

从这里开始

  1. 先读完整个 Issue，再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库，在一个分支上完成修改。
  4. 提交 Pull Request，并在描述里引用这个 Issue 编号。

stackabletech/opa-operator 的其他 Issue

查看 stackabletech/opa-operator 的全部 Issue

相似的 Issue

更多 Rust Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。