google / google/s2-geometry-library-java
S2CellUnionTests Line 258-260 should be removed.
オープン
auto-migrated
Priority-Medium
Type-Defect
- 主要言語
- Java
- スター
- 587
- フォーク
- 231
- PR マージ指標
- 30日以内にマージされた PR はありません
説明
```
Line 258-260 as below should be removed:
if (!u.contains(xId)) {
u.getIntersection(xCells, yId);
}
It is extraneous and guarantees the subsequent assertTrue(u.contains(xId)); on
line 261 will always work even though this is the line that is being ultimately
tested.
```
Original issue reported on code.google.com by `uvico...@gmail.com` on 9 Jul 2013 at 1:39
コントリビューションガイド
調査の方向性
S2CellUnionTests を開いて 258-261 行を確認し、assertion を現在どのように到達可能にしているかに注目してください。余分なブロックがなくてもテストが意図した動作を引き続き検証していることを確認し、その後、関連する S2CellUnion テストを実行して assertion が引き続き意味を持つことを検証してください。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- java
- 領域
- testing
- issue の種類
- リファクタリング
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 活発さ
- 停滞
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 35/100