Order of WireMockContainerBuilder WithX calls caused duplicate Networks in Configuration

[epDugas]

Describe the bug

The order of the .WithX calls to WireMockContainerBuilder can cause duplicate Networks in Configuration.

Expected behavior:

Expected the order of WireMockContainerBuilder.WithX calls to not duplicate Networks in the Configuration.

Test to reproduce

• 1 Call WireMockContainerBuilder.WithNetwork(x);
• 2 Call WireMockContainerBuilder.WithMappings(x) and WireMockContainerBuilder.WithWatchStaticMappings(true);
• 3 Observe the configuration has two Networks, when only one was added.

Other related info

This was observed when updating WireMock.net. Some previous working code failed, an exception was thrown because of the duplicate Network entries in:
https://github.com/testcontainers/testcontainers-dotnet/blob/develop/src/Testcontainers/Clients/ContainerConfigurationConverter.cs at line 30.

Networks = new ToNetworks(configuration).Convert(configuration.Networks)?.ToDictionary(item => item.Key, item => item.Value);

Changing the order of the .WithX calls resolved the issue.

[StefH]

@epDugas
Can you please provide a full working example (console app or unit test) which shows this issue?

[epDugas]

See attached for console app demonstrating issue.
dotnet_console_test.zip

[StefH]

See attached for console app demonstrating issue. dotnet_console_test.zip

Are you using Linux or Windows containers?

[StefH]

@epDugas
Thanks for creating this issue. Some more entries where duplicated it seems.

It should be fixed now, can you use preview version 1.6.10-ci-19504
(https://github.com/WireMock-Net/WireMock.Net/wiki/MyGet-preview-versions)

[epDugas]

Thanks @StefH !

Are you using Linux or Windows containers?

Linux

[StefH]

#1222

[StefH]

@epDugas
If you have time, you can test version 1.6.10-ci-19519.

[epDugas]

@epDugas If you have time, you can test version 1.6.10-ci-19519.

Verified issue resolved. Thanks @StefH !