matrix-org / matrix-org/complement

Password change tests depend on oneshot UIA

オープン
#897 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

主要言語
Go
スター
99
フォーク
72
平均マージ
4日 1時間
マージ済み PR(30日)
8

説明

Related: #893

https://github.com/matrix-org/complement/blob/1a2baac5dbf45ead46409793fe625b3c253eb4f6/tests/csapi/account_change_password_test.go#L110-L122

The password change helper requires being capable of skipping UIA, which is not spec compliant and causes flaky failures in all tests that use it.

The test also depends on being able to do oneshot UIA as it does not use the helper: https://github.com/matrix-org/complement/blob/1a2baac5dbf45ead46409793fe625b3c253eb4f6/tests/csapi/account_change_password_test.go#L82-L95

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

調査の方向性

まず tests/csapi/account_change_password_test.go の 82-95 行と 110-122 行を読み、次に関連する issue #893 と、そこで参照されているパスワード変更ヘルパーを確認します。テストが現在どのように UIA をスキップまたは実行しているかを確認し、パスワード変更のカバレッジが oneshot 依存なしで仕様準拠の UIA 処理を使用するようにします。完了の条件は、対象のテストがこれらの挙動に依存しなくなり、間欠的に失敗しなくなることです。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
go
領域
testing-qa
issue の種類
バグ
難易度
3/5
見積もり時間
1〜2日
活発さ
静か
明瞭さ
おおむね明確
初心者へのやさしさ
62/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。