DockerImageOperations.BuildAsync - Throwing meaningful exception if docker image name is not lower case

[michal-korniak]

Currently this scenario isn't properly handled which leads to the exception with low level message that doesn't give any clues what went wrong.

I wanted to add unit tests to this, but it will require more time than I can afford, The BuildAsync method invokes methods from static classes, so it will be necessary to create additional abstractions to make it mockable.

[HofmeisterAn]

Can we add that as a guard methode to:

testcontainers-dotnet/src/Testcontainers/Images/DockerImage.cs

Line 42 in 39671c0

public DockerImage(

? Then it's easier to test too. What do think?

[michal-korniak]

Much easier :) I moved the logic to the class you mentioned and I added some tests to check if it work properly.

[HofmeisterAn]

easier to test too. What do th

Cool, thanks. I'll do the review and merge later that day 👏.

[HofmeisterAn]

I fixed the unit test, thanks (dcd1858).