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
piontec · Sep 27, 2018 · f9070a5 · f9070a5
1 parent 69b3e89
commit f9070a5
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 2 deletions.
diff --git a/.gitignore b/.gitignore
@@ -1,3 +1,4 @@
 __pycache__/
 .idea/
 .coverage
+*.pyc
diff --git a/dockerenforcer/docker_helper.py b/dockerenforcer/docker_helper.py
@@ -34,7 +34,7 @@ def __init__(self, cid: str, params: Dict[str, Any], metrics: Dict[str, Any], po
         self.owner: str = owner
 
     def __str__(self, *args, **kwargs) -> str:
-        return self.params.get('Name', self.cid)
+        return self.params.get('name', self.cid)
 
 
 class DockerHelper:

diff --git a/test/test_docker_helpers.py b/test/test_docker_helpers.py
@@ -14,7 +14,7 @@ def test_create_without_name(self):
         self.assertEqual("123", str(container))
 
     def test_create_with_name(self):
-        container = Container("123", {"Name": "container1"}, {"cpu": 7}, 0, CheckSource.Periodic)
+        container = Container("123", {"name": "container1"}, {"cpu": 7}, 0, CheckSource.Periodic)
         self.assertEqual("container1", str(container))