grpc / grpc/grpc-java

Remove internal-only methods from testing package

オープン
#3,105 コメント 1 件 リアクション 0 件 担当者 0 名 GitHub で見る
code health
主要言語
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

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

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