Skip to content
This repository was archived by the owner on Jun 27, 2023. It is now read-only.

Commit f9070a5

Browse files
kizziepiontec
authored andcommitted
Fix/display name correctly in warning output (#52)
* Correct variable name so container name is returned when asked for instead of always defaulting to ID * Add .pyc files to ignore list
1 parent 69b3e89 commit f9070a5

File tree

3 files changed

+3
-2
lines changed

3 files changed

+3
-2
lines changed

.gitignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
11
__pycache__/
22
.idea/
33
.coverage
4+
*.pyc

dockerenforcer/docker_helper.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ def __init__(self, cid: str, params: Dict[str, Any], metrics: Dict[str, Any], po
3434
self.owner: str = owner
3535

3636
def __str__(self, *args, **kwargs) -> str:
37-
return self.params.get('Name', self.cid)
37+
return self.params.get('name', self.cid)
3838

3939

4040
class DockerHelper:

test/test_docker_helpers.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ def test_create_without_name(self):
1414
self.assertEqual("123", str(container))
1515

1616
def test_create_with_name(self):
17-
container = Container("123", {"Name": "container1"}, {"cpu": 7}, 0, CheckSource.Periodic)
17+
container = Container("123", {"name": "container1"}, {"cpu": 7}, 0, CheckSource.Periodic)
1818
self.assertEqual("container1", str(container))
1919

2020

0 commit comments

Comments
 (0)