assertj / assertj/assertj-generator

hasXXX assertions generate invalid code when XXX is an Iterable of an interface type.

オープン
#133 コメント 1 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
Java
スター
72
フォーク
47
平均マージ
1日 16時間
マージ済み PR(30日)
2

説明

Given a class:
```java
public class Blah {
private List sequences;
public List getSequences() { return sequences; }
}
```

the AbstractBlahAssert is generated with code such as:

```java
public S hasSequences(interface java.lang.CharSequence... sequences) { ... }
```

(note the "interface" keyword in there.)

This is the result of org.assertj.assertions.generator.util.ClassUtil#getTypeDeclaration correctly handling situations where the contained type is a class:

```java
String name = typeVariable.getName(); // capture#1-of ? extends class java.lang.String
name = removeAll(name, "capture#\\d+-of\\s+"); // extends class java.lang.String
name = removeAll(name, " class"); // extends java.lang.String
typeDeclaration.append(name); // List
```

But not when it's an interface:

```java
String name = typeVariable.getName(); // capture#1-of ? extends interface java.lang.CharSequence
name = removeAll(name, "capture#\\d+-of\\s+"); // extends interface java.lang.CharSequence
name = removeAll(name, " class"); // extends interface java.lang.CharSequence
typeDeclaration.append(name); // List
```

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

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

調査の方向性

org.assertj.assertions.generator.util.ClassUtil#getTypeDeclaration から開始し、キャプチャされたクラス境界とインターフェース境界の扱いを比較します。List 用に生成された AbstractBlahAssert を再現します。hasSequences が "interface" キーワードなしで有効な Java を使用すれば完了です。

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

評価

技術スタック
java
領域
tooling
issue の種類
バグ
難易度
2/5
見積もり時間
1〜3時間
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
42/100

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

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