Skip to content

Commit 3f54d45

Browse files
committed
Don't stop container explicitly
It'll be stopped via the `_database_connection` context manager
1 parent 5b4ac1a commit 3f54d45

File tree

2 files changed

+2
-10
lines changed

2 files changed

+2
-10
lines changed

tests/test_database_commands.py

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
from planemo.database.postgres_docker import stop_postgres_docker
21
from .test_utils import (
32
CliTestCase,
43
skip_unless_environ,
@@ -27,7 +26,4 @@ def test_database_commands(self):
2726
@skip_unless_environ("PLANEMO_ENABLE_POSTGRES_TESTS")
2827
@skip_unless_executable("docker")
2928
def test_database_commands_docker(self):
30-
try:
31-
self._database_commands(database_type="postgres_docker")
32-
finally:
33-
stop_postgres_docker()
29+
self._database_commands(database_type="postgres_docker")

tests/test_profile_commands.py

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
from planemo.database.postgres_docker import stop_postgres_docker
21
from .test_utils import (
32
CliTestCase,
43
skip_unless_environ,
@@ -25,7 +24,4 @@ def test_profile_commands(self):
2524

2625
@skip_unless_executable("docker")
2726
def test_profile_commands_docker(self):
28-
try:
29-
self._profile_commands(database_type="postgres_docker")
30-
finally:
31-
stop_postgres_docker()
27+
self._profile_commands(database_type="postgres_docker")

0 commit comments

Comments
 (0)