3 Commits

Author SHA1 Message Date
Sebastiaan van Stijn
4970333621 integration: remove // import comments
These comments were added to enforce using the correct import path for
our packages ("github.com/docker/docker", not "github.com/moby/moby").
However, when working in go module mode (not GOPATH / vendor), they have
no effect, so their impact is limited.

Remove these imports in preparation of migrating our code to become an
actual go module.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
2025-05-30 15:59:14 +02:00
Sebastiaan van Stijn
7bd1b64058 daemon/logger/loggertest: remove workaround for OSC string terminator parsing
This workaround was added in 2ec3e14c0f
to work around a [bug in go-ansiterm][1], which was fixed in [o-ansiterm#35][1].

[1]: https://github.com/Azure/go-ansiterm/issues/34
[2]: https://github.com/Azure/go-ansiterm/pull/35

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
2025-01-02 11:00:57 +01:00
Paweł Gronowski
2ec3e14c0f test: Add tests for logging
1. Add integration tests for the ContainerLogs API call
Each test handle a distinct case of ContainerLogs output.
- Muxed stream, when container is started without tty
- Single stream, when container is started with tty

2. Add unit test for LogReader suite that tests concurrent logging
It checks that there are no race conditions when logging concurrently
from multiple goroutines.

Co-authored-by: Cory Snider <csnider@mirantis.com>
Signed-off-by: Cory Snider <csnider@mirantis.com>
Signed-off-by: Paweł Gronowski <pawel.gronowski@docker.com>
2022-06-10 09:26:17 +02:00