x.sunamoto.k
|
3f7a9ed11a
|
Merged PR 75: API実装（ユーザー一覧取得）
## 概要
[Task1592: API実装（ユーザー一覧取得）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1592)
- ユーザ一覧取得のAPIを実装
- アクセストークンにより権限を確認する
- src/common/jwt/jwt.ts verifyAuthority（[Task1593: API実装（ユーザー登録）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1593)で作成）を呼び出すため追って再修正します。（レビュー対象外です）
- src/features/users/users.controller.ts getUsersから
src/features/users/users.service.ts getUsersへ
- DBから同一アカウントのユーザ一覧を取得する
- findSameAccountUsersを新規作成
- Azure AD B2Cからユーザーを取得してマージ
- src/gateways/adb2c/adb2c.service.ts getUserを新規作成
- マージはfor文でまわしています（力技）
- マージした結果を返却
- 影響範囲
- usersテーブルの変更が入るときにマージ部分の手直しが要ります。（TODOを添えています）
## レビューポイント
- 新規に作成したfindSameAccountUsersの妥当性
- 新規に作成したgetUserの妥当性
→Azureからの返り値はsrc/common/token/types.tsに定義済。
（Azure AD B2Cから取得できた項目で再定義）
## UIの変更
- 特になし
## 動作確認状況
- ローカルでビルド、テストを実行した後に動作を確認済。
## 補足
- ご不便をおかけしました。よろしくお願いします。
|
2023-05-12 01:27:19 +00:00 |
|
oura.a
|
e9af39bd47
|
Merged PR 71: API実装（ユーザー登録）
## 概要
[Task1593: API実装（ユーザー登録）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1593)
https://dev.azure.com/ODMSCloud/ODMS%20Cloud/_git/ODMS%20Cloud
users.controllerにアクセストークン取得処理を追加
users.serviceにユーザ追加処理を追加
user.entityにauto_renew、license_alert、notificationを追加
users.repository.serviceにユーザ追加・AuthorId検索処理を追加
## レビューポイント
処理の記載場所が適切かどうか
期待通りの処理になっているかどうか
テストコードの記載方法が正しいかどうか
## UIの変更
なし
## 動作確認状況
ローカルでのビルド・実行を確認
## 補足
テスト実装について不安要素があります。
・テストの粒度はこれでよいのでしょうか？
・テスト実行に４０分かかってしまうのですが実装方法を間違えている箇所がありそうでしょうか？
|
2023-05-11 09:05:54 +00:00 |
|
maruyama.t
|
d297301212
|
Merged PR 82: API実装（メール認証）
## 概要
[Task1594: API実装（メール認証）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1594)
- メール認証APIを作成
- src/api/common/にpasswordを追加（ランダムパスワード発行ロジック）
- src/fuatures/gateway/adb2c.service.tsにユーザのパスワードを変更するメソッドchangePasswordを追加
- user.service.spec.tsにメール認証と仮パスワード発行のテストケースを追加
- 影響範囲
（user.service.spec.tsで行っていた既存のテストケース）
## レビューポイント
- commonにpasswordを追加したが、配置として適切かどうか
- user.service.tsのエラー発生時のロジックが十分であるか
## 動作確認状況
- ローカルで確認
|
2023-05-11 07:45:30 +00:00 |
|
makabe.t
|
bd4aaa8ae1
|
Merged PR 63: API実装（メール認証）
## 概要
[Task1497: API実装（メール認証）](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/1497)
- メール認証APIとテストを実装しました。
- IDトークンの型を現状と合わせて修正しました
- `family_name`と`given_name`を削除しました。
- auth.serviceのテストも併せて修正しました。
- テストケースのIDトークンを環境変数の鍵で生成するように修正しました。
## レビューポイント
- DBのユーザを検証済みにする処理について、トランザクション内で取得と更新をしていますがトランザクションの使い方として問題ないでしょうか？
- 本APIで使用するカスタムエラーを`common/error/types`に暫定的においていますがどこに配置するのが適切でしょうか？
- `common/error/types`に配置する、もしくは`common`配下にカスタムエラー用のフォルダを作成してその下に配置するのが良いかと考えています。
- テストのモックでエラーを発生させる際に、テストケース内でエラーを設定していますがモックファイル内でエラー用のモックを設定するべきでしょうか？
## UIの変更
- なし
## 動作確認状況
- ローカルで確認
- テストが通ることを確認
|
2023-04-10 04:44:16 +00:00 |
|