blazingmq-sdk-java icon indicating copy to clipboard operation
blazingmq-sdk-java copied to clipboard

Use temporary folders for ITs with native broker

Open 678098 opened this issue 2 years ago • 0 comments

Is there an existing issue for this?

  • [X] I have searched the existing issues

Current Behavior

For native broker currently we have some ENV vars being set in the code, but these variables do not do anything right now, because we use static config. Here:

https://github.com/bloomberg/blazingmq-sdk-java/blob/7a89c7199b68f1d10853e539d7c9bb3283eb9a75/bmq-sdk/src/test/java/com/bloomberg/bmq/it/util/BmqBrokerTestServer.java#L145-L155

This means that currently we can use params listed in static config only, so storage path, logs path, broker port are the same for each launched broker. The last means that native IT brokers could only be launched on a default 30114 port, and no parallel ITs execution possible.

We create a TMP folder which should contain storage, logs etc, but we are not able to use it with a static config:

https://github.com/bloomberg/blazingmq-sdk-java/blob/7a89c7199b68f1d10853e539d7c9bb3283eb9a75/bmq-sdk/src/test/java/com/bloomberg/bmq/it/util/BmqBrokerTestServer.java#L139

Expected Behavior

Native brokers in ITs should be able to use TMP folders and non-default parameters.

Steps To Reproduce

Launch ITs with -Dit.brokerDir=/path/to/native/broker

BlazingMQ Java SDK Version

0.0.10

Anything else?

No response

678098 avatar Sep 22 '23 12:09 678098