Skip to content

Commit

Permalink
chore: remove redundant reference to config builder
Browse files Browse the repository at this point in the history
  • Loading branch information
rafek1241 committed Jan 18, 2025
1 parent 0ba4f84 commit 79d4b40
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 13 deletions.
6 changes: 1 addition & 5 deletions src/Testcontainers.EventHubs/EventHubsBuilder.cs
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ public EventHubsBuilder WithConfigurationBuilder(ConfigurationBuilder configurat
{
var configBytes = Encoding.UTF8.GetBytes(configurationBuilder.Build());

return Merge(DockerResourceConfiguration, new EventHubsConfiguration(configurationBuilder: configurationBuilder))
return Merge(DockerResourceConfiguration, new EventHubsConfiguration())
.WithResourceMapping(configBytes, "Eventhubs_Emulator/ConfigFiles/Config.json");
}

Expand Down Expand Up @@ -101,10 +101,6 @@ protected override void Validate()
{
base.Validate();

_ = Guard.Argument(DockerResourceConfiguration.ConfigurationBuilder,
nameof(DockerResourceConfiguration.ConfigurationBuilder))
.NotNull();

_ = Guard.Argument(DockerResourceConfiguration.AzuriteBlobEndpoint,
nameof(DockerResourceConfiguration.AzuriteBlobEndpoint))
.NotNull()
Expand Down
8 changes: 0 additions & 8 deletions src/Testcontainers.EventHubs/EventHubsConfiguration.cs
Original file line number Diff line number Diff line change
Expand Up @@ -11,11 +11,9 @@ public sealed class EventHubsConfiguration : ContainerConfiguration
/// <param name="azuriteBlobEndpoint">The Azurite blob endpoint.</param>
/// <param name="azuriteTableEndpoint">The Azurite table endpoint.</param>
public EventHubsConfiguration(
ConfigurationBuilder configurationBuilder = null,
string azuriteBlobEndpoint = null,
string azuriteTableEndpoint = null)
{
ConfigurationBuilder = configurationBuilder;
AzuriteBlobEndpoint = azuriteBlobEndpoint;
AzuriteTableEndpoint = azuriteTableEndpoint;
}
Expand Down Expand Up @@ -58,16 +56,10 @@ public EventHubsConfiguration(EventHubsConfiguration resourceConfiguration)
public EventHubsConfiguration(EventHubsConfiguration oldValue, EventHubsConfiguration newValue)
: base(oldValue, newValue)
{
ConfigurationBuilder = BuildConfiguration.Combine(oldValue.ConfigurationBuilder, newValue.ConfigurationBuilder);
AzuriteBlobEndpoint = BuildConfiguration.Combine(oldValue.AzuriteBlobEndpoint, newValue.AzuriteBlobEndpoint);
AzuriteTableEndpoint = BuildConfiguration.Combine(oldValue.AzuriteTableEndpoint, newValue.AzuriteTableEndpoint);
}

/// <summary>
/// Gets the configuration builder
/// </summary>
public ConfigurationBuilder ConfigurationBuilder { get; }

/// <summary>
/// Gets the Azurite blob endpoint
/// </summary>
Expand Down

0 comments on commit 79d4b40

Please sign in to comment.