codex-team / codex-team/notes.api

Refactor tests for team rights

未关闭
#207 0 条评论 0 个 reaction 已指派 1 人 已被 @GoldenJaden 认领 在 GitHub 查看
Tests
主要语言
TypeScript
星标
5
派生
2
PR 合并指标
30 天内没有已合并 PR

描述

# Problem
With the come of **team rights policies**, there are a lot of almost indentical tests that should be run for some routes.

# Solution
Parametrize tests that test team rights - make use of the `test.each` method to stuff similar test to one pack.

You can peek into the `note.settings.test.ts` to understand how it should be done.

Also, it would be nice to save this `test.each` as separate util for better convenience:
something like
```
test.roles = test.each([{ role: MemberRole.Read,
isPublic: false,
isAuthorized: true,
statusCode: 200 },
{ ... },
{ ... },
{ ... }
]);
```
and use it as
```
test.roles('Returns note with access rights by public id ....', ({ role, isPublic, isAuthorized, statusCode }) => {

})
```

# Tasks

- [x] Refactor tests regarding team rights
- [ ] Make the method as a separate util
- [ ] Find out how to make the better
naming for tests in `test.each` (from `jest` tool). Right now we have same name for all tests in one scope, and it's not very convenient.
- [ ] Update test docs

贡献指南

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

评估

这个 Issue 还没有评估数据。

把新 issue 发到你的邮箱

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