eclipse-pde / eclipse.pde

Eclipse Public License 2.0
28 stars 79 forks source link

Plug-in JUnit Launch does not detec Junit5 tests #650

Open Pontesegger opened 1 year ago

Pontesegger commented 1 year ago

Beginning from 2022-09 I am not able to execute JUnit Plug-in Tests. Works fine with 2022-06 IDE. (Update: issue still there in 2023-06)

My Setup:

When installing an IDE 2022-06 I have a context menu Run As -> JUnit Plug-in Test. Further tests execute fine and results are displayed in the JUnit view.

Now I switched to an IDE 2022-09, keeping the rest of my setup above.

Finding 1) the context menu entry Run As -> JUnit Plug-in Test is gone. To make it reappear I have to add following dependencies to my fragment: org.junit.platform.commons;bundle-version="1.8.1", org.junit.platform.engine;bundle-version="1.8.1", org.junit.jupiter.engine;bundle-version="5.8.1"

Finding 2) With the context menu back I can launch Unit tests, but after execution the JUnit view shows 0 tests. Trying to run with coverage reveals that the test code is actually not executed.

Followed it down to RemoteTestRunner.java#516 The count returned is always 0, so test discovery somehow does not work.

merks commented 1 year ago

It will probably be good to provide a simple test case that reproduces the problem so that folks can focus on fixing the problem rather than on how exactly to reproduce it.

Pontesegger commented 1 year ago

One option is to use the EASE setup from Oomph, then try with any existing unit test

Pontesegger commented 1 year ago

How to reproduce:

example.zip

HannesWell commented 1 month ago
  • Target definition based on 2022-06

Have you tried using a more recent target-platform? IIRC the JUnit-5 bundles got a new symbolic-name at some time in the past, might be around the time when the problem started. Because in general JUnit-5 tests work.

HannesWell commented 1 month ago

Respectively this might be to https://github.com/eclipse-pde/eclipse.pde/issues/1006 or https://github.com/eclipse-pde/eclipse.pde/issues/278.