Skip to content

Commit

Permalink
Removed redundant overridings
Browse files Browse the repository at this point in the history
  • Loading branch information
IbrayevRamil committed Jan 28, 2025
1 parent 3a4c4a7 commit e0c0dad
Showing 1 changed file with 2 additions and 21 deletions.
23 changes: 2 additions & 21 deletions src/Testcontainers.Kafka/ApacheKafkaBuilder.cs
Original file line number Diff line number Diff line change
Expand Up @@ -25,14 +25,7 @@ private ApacheKafkaBuilder(KafkaConfiguration resourceConfiguration) : base(reso
{
DockerResourceConfiguration = resourceConfiguration;
}

/// <inheritdoc />
public override KafkaContainer Build()
{
Validate();
return new KafkaContainer(DockerResourceConfiguration);
}


/// <inheritdoc />
protected override KafkaConfiguration DockerResourceConfiguration { get; }

Expand All @@ -56,19 +49,7 @@ protected override string GetStartupScript(KafkaContainer container)
exec /etc/kafka/docker/run
""";
}

/// <inheritdoc />
protected override ApacheKafkaBuilder Clone(IResourceConfiguration<CreateContainerParameters> resourceConfiguration)
{
return Merge(DockerResourceConfiguration, new KafkaConfiguration(resourceConfiguration));
}

/// <inheritdoc />
protected override ApacheKafkaBuilder Clone(IContainerConfiguration resourceConfiguration)
{
return Merge(DockerResourceConfiguration, new KafkaConfiguration(resourceConfiguration));
}


/// <inheritdoc />
protected override ApacheKafkaBuilder Merge(KafkaConfiguration oldValue, KafkaConfiguration newValue)
{
Expand Down

0 comments on commit e0c0dad

Please sign in to comment.