NServiceBus.RabbitMQ icon indicating copy to clipboard operation
NServiceBus.RabbitMQ copied to clipboard

Improve testing multiple transport configurations

Open bording opened this issue 9 years ago • 4 comments

After working on the native delays implementation, it's become more apparent that we need some better way to manage running tests with different permutations of configurations settings.

We previously could be trying to run the tests for both routing topologies, but with the native delays, we also have whether the timeout manager is enabled or not.

bording avatar Feb 16 '17 20:02 bording

Do we need to necessarily solve this as part of the native delays solution?

adamralph avatar Feb 17 '17 12:02 adamralph

Do we need to necessarily solve this as part of the native delays solution?

The reason I raised this as a separate issue is that I don't want to solve this as part of native delays. It doesn't make sense to hold up on delivering native delay to deal with it.

bording avatar Feb 20 '17 17:02 bording

OK, just to clarify: in native delays, we are planning to create an explicit set of tests which run the other permutation, correct? Rather than actually applying permutations to the same set of tests?

adamralph avatar Feb 20 '17 17:02 adamralph

@adamralph Currently, there aren't plans to do anything special in the native delays PR.

Aside from not wanting to special case the delays tests, the other reason for this is with the recent change in the PR where we're removing the timeout manager behavior instead of working around it,, there aren't really separate code paths to worry about any more.

bording avatar Feb 20 '17 17:02 bording