feat(core): DockerCompose.stop now stops only services that it starts (does not stop the other services)

[aksel]

The command would otherwise stop/down all services, not just the services the instance itself started.

Useful for e.g. one fixture per service, and you want different scopes for the services.

[aksel]

Apologies if my PR breaks any conventions or contribution guidelines. I was a bit confused by the docs, as they seemed to be targeted towards contributing features, which I don't think this is.

Let me know if I should close, resubmit, or change anything

Thanks! 😄

[aksel]

My specific use case is this:

importpytestfromtestcontainers.composeimportDockerCompose@pytest.fixture(scope="session")asyncdefaws() ->DockerCompose:
withDockerCompose(
".",
compose_file_name="docker-compose.yaml",
pull=True,
build=True,
services=["aws"],
# Wait for localstack to be healthywait=True,
) ascompose:
yieldcompose@pytest.fixture(scope="module")asyncdefdb(aws) ->DockerCompose:
withDockerCompose(
".",
compose_file_name="docker-compose.yaml",
pull=True,
build=True,
# Only start the DBservices=["db"],
# Wait for the DB to be healthywait=True,
) ascompose:
yieldcompose

Starting up aws takes a while, maybe around 20 seconds, but it can be session scoped and only done once. Until now, both have been session scoped. But now I want to get a clean DB in between modules (or packages, more likely). Nicer isolation and all that. But I couldn't figure out a way to stop onlydb, while keeping aws running. At first glance, seemed like an oversight, so I thought I might as well submit a PR with the two-line change.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Please upload report for BASE (main@27f2a6b). Learn more about missing BASE report.

Additional details and impacted files

@@ Coverage Diff @@## main #620 +/- ##
=======================================
Coverage ? 80.29% =======================================
Files ? 11 Lines ? 604 Branches ? 86 =======================================
Hits ? 485 Misses ? 92 Partials ? 27 

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.