arquillian / arquillian-organization

Arquillian Project Umbrella - used to gather all issues at one place by using ZenHub
http://arquillian.org
3 stars 0 forks source link

Boosters should use standalone dependency if they are running tests in standalone mode #33

Closed dipak-pawar closed 6 years ago

dipak-pawar commented 6 years ago

I have observed that boosters [1], [2] are using arquillian-junit-container dependency even they want to use standalone mode.

Tasks

[1] - https://github.com/snowdrop/spring-boot-http-booster/blob/master/pom.xml#L56 [2] - https://github.com/openshiftio-vertx-boosters/vertx-http-booster/blob/master/pom.xml#L84

bartoszmajsak commented 6 years ago

@dipak-pawar are these all boosters with such dependencies? I think we are missing Wildfly Swarm if not more.

Another thing - as this inherits Cube version from parent pom - shouldn't we first update this one and ask for the release?

Can we get the list of all boosters first @hemanik? Probably something we could script/figure out from https://github.com/fabric8-launcher/launcher-booster-catalog/tree/master

I believe the update procedure to use -starter should be automated.

bartoszmajsak commented 6 years ago

This might be handy too https://github.com/fabric8-launcher/launcher-documentation/wiki

dipak-pawar commented 6 years ago

are these all boosters with such dependencies? I think we are missing Wildfly Swarm if not more.

I found above two when I was playing with them. Couldn't find time to check all others. But we should check all others and update it with single dependency.

Another thing - as this inherits Cube version from parent pom - shouldn't we first update this one and ask for the release?

You are right. You can see second task Update boosters parent to use new version of released cube with arquillian/arquillian-cube#993 merged which is for that only.

I believe the update procedure to use -starter should be automated.

:+1:

bartoszmajsak commented 6 years ago

Let's prepare automation part of it and do actual PRs/contributions to upstream when we have Cube/Che experience improved.

hemanik commented 6 years ago

The following needs to be done for the automation part and to do actual PRs/contributions to upstream:

  1. Spring Boot boosters are at https://github.com/snowdrop/
  2. Vert.x boosters are at https://github.com/openshiftio-vertx-boosters/
  3. Swarm boosters are at https://github.com/wildfly-swarm-openshiftio-boosters/
  1. Replace the two arquillian dependencies for standalone with the single starter dependency. (Dropped from the scope of the automation script since it is a one-time thing.)
  2. Update the Arquillian Cube version used if old.
hemanik commented 6 years ago

Initial Spike for the Automation Script - https://github.com/hemanik/version-bump-automation.

hemanik commented 6 years ago

As discussed offline, updating the starter-dependencies can be best done manually since it's a one-time thing.

With regards to this issue then, the following needs to be addressed:

The PRs for the last two tasks on hold for the release of booster-parent with the updated cube version.