apolloconfig / apolloconfig/apollo

upgrade junit to 5 or not?

Offen
#4,195 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
area/test discussion
Vorherrschende Sprache
Java
Sterne
29.8k
Forks
10.2k
Ø Merge
4 T. 7 Std.
Gemergte PRs (30 T.)
4

Beschreibung

**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.

Beitragsleitfaden

Beitragsleitfaden öffnen

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.