Convert remaining smoke tests

[zeitlinger]

No description provided.

[otelbot-java-instrumentation]

🔧 The result from spotlessApply was committed to the PR branch.

[otelbot-java-instrumentation]

🔧 The result from spotlessApply was committed to the PR branch.

[zeitlinger]

@laurit please take a look

[laurit]

@laurit please take a look

I'm not convinced that adding everything to the SmokeTestExtension is a good idea. Ideally that extension could also be used by others, making it specific to our smoke tests limits its usefulness. Also instead of the custom AppServer annotations that we used in groovy tests it might be more straightforward to use parameterized tests. Need to think more about this.

[zeitlinger]

I'm not convinced that adding everything to the SmokeTestExtension is a good idea.

the version with the superclass got too confusing - but maybe you have an idea

Also instead of the custom AppServer annotations that we used in groovy tests it might be more straightforward to use parameterized tests. Need to think more about this.

Also thought about this one: I think it would be possible, but you'd have to start the app server once for each test rather than for all tests together.

[zeitlinger]

I'm not convinced that adding everything to the SmokeTestExtension is a good idea

we can also keep the extension as it is now (57 lines including commets) and rename the extension in this PR to something else.

[zeitlinger]

closed in favor of #14795