search

eclipse / microprofile-lra

microprofile-lra
Apache License 2.0
101 stars 30 forks source link

Fix TckRecoveryTests deployment URL handling #292

Closed xstefank closed 4 years ago

xstefank commented 4 years ago

With deployment URL handling reworked in #291 the deployment URL is taken now as an Arquillian resource not from the MP config. The issue in TckRecoveryTests is that the deployments are not managed by arquillian and thus the deployment URL is not injected. This fix passes the deployment URL as a header to invocations in the individual phases that are run in the manually started containers (deployments not managed by arquillian).

xstefank commented 4 years ago

@ochaloup also review please.

xstefank commented 4 years ago

fixed only the @mmusgrov suggestion.

xstefank commented 4 years ago

@rdebusscher Rudy, I would like to merge this later today. Can you take a look, please?