Allow to get logs from compose (#93)

* Allow to get logs from compose

* fixed escaping in compose documentation

* use run instead of Popen to fetch logs from compose

* fix to make log fetching <3.7 compatible

Author: romank0

testcontainers/compose.py

@@ -30,6 +30,9 @@ class DockerCompose(object):
                 desired_capabilities=CHROME,
             )
             driver.get("http://automation-remarks.com")
+            stdout, stderr = compose.get_logs()
+            if stderr:
+                print("Errors\\n:{}".format(stderr))
 
 
     .. code-block:: yaml
@@ -80,6 +83,16 @@ def stop(self):
             subprocess.call(["docker-compose", "-f", self.compose_file_name, "down", "-v"],
                             cwd=self.filepath)
 
+    def get_logs(self):
+        with blindspin.spinner():
+            result = subprocess.run(
+                ["docker-compose", "-f", self.compose_file_name, "logs"],
+                cwd=self.filepath,
+                stdout=subprocess.PIPE,
+                stderr=subprocess.PIPE,
+            )
+            return result.stdout, result.stderr
+
     def get_service_port(self, service_name, port):
         return self._get_service_info(service_name, port)[1]
 

tests/test_docker_compose.py

@@ -31,3 +31,11 @@ def test_compose_wait_for_container_ready():
     with DockerCompose("tests") as compose:
         docker = DockerClient()
         compose.wait_for("http://%s:4444/wd/hub" % docker.host())
+
+
+def test_can_get_logs():
+    with DockerCompose("tests") as compose:
+        docker = DockerClient()
+        compose.wait_for("http://%s:4444/wd/hub" % docker.host())
+        stdout, stderr = compose.get_logs()
+        assert stdout, 'There should be something on stdout'