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

feat: Improve tar stream logging (copy files to container) #1050

Closed HofmeisterAn closed 8 months ago

HofmeisterAn commented 8 months ago

What does this PR do?

The PR enhances the log messages related to copying files into containers, adding extra information about the specific content being copied.

Why is it important?

Previous log messages lacked information and were not helpful for debugging issues. Additionally, the container builder configuration stores now the IResourceMapping implementations in a list instead of a dictionary. The dictionary prevented copying multiple source files into the same target directory (In very specific configurations, copying multiple source files was an issue, but generally, it wasn't a problem).

Related issues

-

netlify[bot] commented 8 months ago

Deploy Preview for testcontainers-dotnet ready!

Name Link
Latest commit 0c5ce32293fc0421089ed1517eee728a3d6e1e59
Latest deploy log https://app.netlify.com/sites/testcontainers-dotnet/deploys/65527db06f16b20008dd4ea4
Deploy Preview https://deploy-preview-1050--testcontainers-dotnet.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.