search

testcontainers / testcontainers-dotnet

A library to support tests with throwaway instances of Docker containers for all compatible .NET Standard versions.
https://dotnet.testcontainers.org
MIT License
3.67k stars 254 forks source link

[Enhancement]: Renaming a file (very minor) #1066

Closed mgroves closed 7 months ago

mgroves commented 7 months ago

Problem

In the tests/Testcontainers.Couchbase.Tests folder, there is a file named CouchDbContainerTest.cs. The code itself is fine, there's a class called CouchbaseContainerTest. However, I think that class names and file names should match. Further, this might cause confusion (of which there is already a considerable amount) that CouchDb and Couchbase are the same thing.

Solution

File rename from CouchDbContainerTest.cs -> CouchbaseContainerTest.cs

Benefit

Reduction of potential confusion

Alternatives

I can't think of an alternative, other than leaving it the way it is.

Would you like to help contributing this enhancement?

Yes

HofmeisterAn commented 7 months ago

Thanks for the PR.