apolloconfig / apolloconfig/apollo

upgrade junit to 5 or not?

オープン
#4,195 コメント 1 件 リアクション 0 件 担当者 0 名 GitHub で見る
area/test discussion
主要言語
Java
スター
29.8k
フォーク
10.2k
平均マージ
4日 7時間
マージ済み PR(30日)
4

説明

**Is your feature request related to a problem? Please describe.**

Does need to upgrade junit 4 to junit 5?

See #4193 if we upgrade the project's test to junit 5, some test feature like `@Nested` annotation can be used.

**Describe the solution you'd like**
* change the maven dependencies
* change the import in java code
* change the ci

**Additional context**

awaitility use junit 4, see
https://github.com/apolloconfig/apollo/blob/master/pom.xml#L282-L285
```java

org.awaitility
awaitility
test

```
We need to investigate that awaitility can use junit 5 or not.

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

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

評価

この issue はまだ評価されていません。

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

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