Skip to content

dbeaver/pro#8766 Boom#84

Open
labanovichttps wants to merge 7 commits into
develfrom
dbeaver/pro#8766-migrate-to-junit5
Open

dbeaver/pro#8766 Boom#84
labanovichttps wants to merge 7 commits into
develfrom
dbeaver/pro#8766-migrate-to-junit5

Conversation

@labanovichttps
Copy link
Copy Markdown
Contributor

No description provided.

…estExtension

Without extending DBVRTest, @ExtendWith(OSGITestExtension.class) was never
applied to the suite class, so the CLI application was never started before
tests ran, causing "Application is not running" for all 27 tests.
In JUnit 4, @BeforeClass on DBVRTestSuite called initApplication() which started
the CLI application before tests ran. The JUnit 5 migration stripped @BeforeClass
but added nothing to replace it — OSGITestExtension is IDEA-only and is a no-op
in CI. Add @BeforeAll setUpApplication() to DBVRTest so initApplication() is
called before each test class runs.
# Conflicts:
#	test/org.dbvr.test.platform/META-INF/MANIFEST.MF
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant