The release time of JUnit junpiter 5.4.0 is uncertain, so I used 5.3.2. Because of some test cases, I copied the source code of JUnit junpiter 5.4.0-M1 Do I need to modify the LICENSE file?
Ian Luo <ian....@gmail.com> 于2019年1月9日周三 上午10:53写道: > Thanks for taking care of this, Yunkun. > > -Ian. > > On Wed, Jan 9, 2019 at 10:16 AM YunKun Huang <hyun...@apache.org> wrote: > > > The changes looks good to me. > > But somehow the test coverage drop to zero, it could be some > compatibility > > issue with jacoco or codecov. > > I will try to investigate this later today > > > > > > On 2019/01/07 14:41:48, Ian Luo <ian....@gmail.com> wrote: > > > Understood. Junit 5 should be better than 4. > > > > > > Thanks, > > > -Ian. > > > > > > On Mon, Jan 7, 2019 at 1:38 PM Xin Wang <xin.victorw...@gmail.com> > > wrote: > > > > > > > > I noticed that you switch all junit API to jupiter engine which > > > > looks pretty new to the folks who has not touched junit5. I am > > wondering if > > > > the old API works or not in junit5. > > > > > > > > Yes, I switch all junit API to jupiter engine , for example, the base > > Api: > > > > *org.junt.Test* is changed to org.junit.jupiter.api.Test*, * > > > > *org.junt.Assert* is changed to * > org.junit.jupiter.api.Assertions . > > > > *It's > > > > because , compared with junit4, junit5 has greatly refactoring the > API > > > > path, and the basic API path is migrated from org.junt to > > > > org.junit.jupiter.api. > > > > The adaptation strategy of junit5 to junit4 is to put the jars of > > junit4 > > > > and junit5 in the project at the same time. In this way, there will > be > > > > JUnit4 and junit5 test cases in the project at the same time. I think > > this > > > > will lead to confusion, not clean code. So I switch the api to junit5 > > > > directly. > > > > > > > > > > > > > > > > > > > > Ian Luo <ian....@gmail.com> 于2019年1月7日周一 上午11:27写道: > > > > > > > > > Xin, > > > > > > > > > > Your pull request looks pretty straightforward to me. I have only > one > > > > minor > > > > > question: I noticed that you switch all junit API to jupiter engine > > which > > > > > looks pretty new to the folks who has not touched junit5. I am > > wondering > > > > if > > > > > the old API works or not in junit5. > > > > > > > > > > Thanks, > > > > > -Ian. > > > > > > > > > > On Sat, Jan 5, 2019 at 11:48 PM Xin Wang <xin.victorw...@gmail.com > > > > > > wrote: > > > > > > > > > > > Hi all, > > > > > > I want to upgrade the junit dependency of Dubbo, contains : > > > > > > > > > > > > * Functions* > > > > > > 1). Upgrade junit to junit 5 > > > > > > 2). Upgrade the test cases > > > > > > > > > > > > I create a issue[1] and a pr [2] , please discuss about this , > > > > > > > > > > > > *Why do this ?* > > > > > > 1). Some new features:test extensions 、Future of JVM > > testing、easier > > > > > > parameterized tests > > > > > > 2).Java 8 lambda support > > > > > > 3). Optimized api > > > > > > 4). Basically compatible with junit4 > > > > > > Please read the referenced article :《7 reasons why you should > > start > > > > > using > > > > > > JUnit 5 today》[3] > > > > > > > > > > > > *How to write junit5 test case ?* > > > > > > Please see 《JUnit 5 User Guide》[4] > > > > > > > > > > > > What do you think about this ? > > > > > > > > > > > > [1] https://github.com/apache/incubator-dubbo/issues/3148 > > > > > > [2] https://github.com/apache/incubator-dubbo/pull/3149 > > > > > > [3] > > > > > > > > > > > > > > > > > > > > > > > > http://dolszewski.com/testing/7-reasons-why-you-should-start-using-junit-5-today/ > > > > > > [4] https://junit.org/junit5/docs/current/user-guide/ > > > > > > > > > > > > > > > > > > > > >