Skip to content

Commit 5bdb06f

Browse files
fix mypy errors
1 parent 6e48502 commit 5bdb06f

File tree

2 files changed

+14
-12
lines changed

2 files changed

+14
-12
lines changed

podman_compose.py

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -2745,7 +2745,7 @@ async def list_running_projects(compose: PodmanCompose, args: argparse.Namespace
27452745
img_containers = [cnt for cnt in compose.containers if "image" in cnt]
27462746
parsed_args = vars(args)
27472747
_format = parsed_args.get("format", "table")
2748-
data = []
2748+
data: list[Any] = []
27492749
if _format == "table":
27502750
data.append(["NAME", "STATUS", "CONFIG_FILES"])
27512751

@@ -2766,13 +2766,13 @@ async def list_running_projects(compose: PodmanCompose, args: argparse.Namespace
27662766
''',
27672767
],
27682768
)
2769-
output = output.decode().split()
2770-
running = bool(json.loads(output[1]))
2771-
status = "{}({})".format(output[0], 1 if running else 0)
2772-
path = "{}/{}".format(output[2], output[3])
2769+
command_output = output.decode().split()
2770+
running = bool(json.loads(command_output[1]))
2771+
status = "{}({})".format(command_output[0], 1 if running else 0)
2772+
path = "{}/{}".format(command_output[2], command_output[3])
27732773

27742774
if _format == "table":
2775-
if isinstance(output, list):
2775+
if isinstance(command_output, list):
27762776
data.append([name, status, path])
27772777

27782778
elif _format == "json":

tests/integration/list/test_podman_compose_list.py

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
import ast
22
import os
33
import unittest
4+
from typing import Any
45

56
from tests.integration.test_utils import RunSubprocessMixin
67
from tests.integration.test_utils import podman_compose_path
@@ -56,10 +57,10 @@ def test_podman_compose_list(self) -> None:
5657
running_containers = []
5758
self.run_subprocess_assert_returncode(command_up)
5859
out, _ = self.run_subprocess_assert_returncode(command_list)
59-
out = out.decode()
60+
str_out = out.decode()
6061

6162
# Test for table view
62-
services = out.strip().split("\n")
63+
services = str_out.strip().split("\n")
6364
headers = [h.strip() for h in services[0].split("\t")]
6465

6566
for service in services[1:]:
@@ -74,16 +75,17 @@ def test_podman_compose_list(self) -> None:
7475
# Test for json view
7576
command_list.extend(["--format", "json"])
7677
out, _ = self.run_subprocess_assert_returncode(command_list)
77-
out = out.decode()
78-
services = ast.literal_eval(out)
78+
str_out = out.decode()
79+
json_services: Any = ast.literal_eval(str_out)
7980

80-
for service in services:
81+
for service in json_services:
8182
self.assertIsInstance(service, dict)
8283
self.assertNotEqual(service.get("Name"), None)
8384
self.assertNotEqual(service.get("Status"), None)
8485
self.assertNotEqual(service.get("ConfigFiles"), None)
8586

86-
self.assertEqual(len(services), 3)
87+
self.assertIsInstance(json_services, list)
88+
self.assertEqual(len(json_services), 3)
8789

8890
# Get container ID to remove it
8991
out, _ = self.run_subprocess_assert_returncode(command_container_id)

0 commit comments

Comments
 (0)