Remove internal-only methods from testing package
- 主要言語
- Java
- スター
- 12.1k
- フォーク
- 4k
- 平均マージ
- 2日 17時間
- マージ済み PR(30日)
- 37
説明
As mentioned in comments in #3099, there are some issues and limitation to the current unit test tools:
- It's hard to test a user-defined generic interceptor.
- People are abusing `NoopClientCall`, using it as an alternative of `mock(ClientCall.class)`, which is as bad as a mock.
- `TestMethodDescriptors.noopMarshaller` does not gain much benefit. It violates the interface's contract, it's equivalent to a `mock(Marshaller.class)`, and it does not work in InProcess transport or interop test.
コントリビューションガイド
調査の方向性
testing パッケージでの NoopClientCall と TestMethodDescriptors.noopMarshaller の使用箇所、および #3099 の議論を確認してください。この issue の対象となる内部専用ヘルパーと、それらの影響を受ける caller を特定してください。完了とは、それらのヘルパーが列挙された testing パターンを推奨しなくなり、testing ツールを壊さないことを意味します。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- grpc, java
- 領域
- testing
- issue の種類
- リファクタリング
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 活発さ
- 停滞
- 明瞭さ
- 説明が足りない
- 初心者へのやさしさ
- 35/100