test: docker-gen network segregation

* fix tests
* remove obsolete compose version

test/test_dockergen/test_dockergen_network_segregation.base.yml

@@ -1,38 +1,41 @@
-version: '2'
-
 services:
-  nginx:
+  nginx-proxy-nginx:
     image: nginx
     container_name: nginx
     volumes:
-      - "/etc/nginx/conf.d"
+      - nginx_conf:/etc/nginx/conf.d:ro
-    labels:
+    ports:
-      - "com.github.nginx-proxy.nginx-proxy.nginx"
+      - "80:80"
+      - "443:443"
     networks:
       - proxy
+    labels:
+      - "com.github.nginx-proxy.nginx-proxy.nginx"
 
-  dockergen:
+  nginx-proxy-dockergen:
     image: nginxproxy/docker-gen
     command: -notify-sighup nginx -watch /etc/docker-gen/templates/nginx.tmpl /etc/nginx/conf.d/default.conf
-    volumes_from:
-      - nginx
     volumes:
       - /var/run/docker.sock:/tmp/docker.sock:ro
       - ../../nginx.tmpl:/etc/docker-gen/templates/nginx.tmpl
+      - nginx_conf:/etc/nginx/conf.d
     networks:
       - internal
 
   web:
     image: web
-    container_name: whoami
+    container_name: whoami2
     expose:
       - "80"
     environment:
       WEB_PORTS: "80"
-      VIRTUAL_HOST: "whoami.nginx.container.docker"
+      VIRTUAL_HOST: whoami2.nginx.container.docker
     networks:
       - proxy
 
 networks:
   proxy:
   internal:
+
+volumes:
+  nginx_conf:

test/test_dockergen/test_dockergen_network_segregation.py

@@ -1,11 +1,11 @@
 import docker
 import pytest
-from distutils.version import LooseVersion
+from packaging.version import Version
 
 
 raw_version = docker.from_env().version()["Version"]
 pytestmark = pytest.mark.skipif(
-    LooseVersion(raw_version) < LooseVersion("1.13"),
+    Version(raw_version) < Version("1.13"),
     reason="Docker compose syntax v3 requires docker engine v1.13 or later (got {raw_version})"
 )
 
@@ -16,9 +16,9 @@ def test_unknown_virtual_host_is_503(docker_compose, nginxproxy):
 
 
 def test_forwards_to_whoami(docker_compose, nginxproxy):
-    r = nginxproxy.get("http://whoami.nginx.container.docker/")
+    r = nginxproxy.get("http://whoami2.nginx.container.docker/")
     assert r.status_code == 200
-    whoami_container = docker_compose.containers.get("whoami")
+    whoami_container = docker_compose.containers.get("whoami2")
     assert r.text == f"I'm {whoami_container.id[:12]}\n"
 
 

test/test_dockergen/test_dockergen_network_segregation_v2.py

@@ -1,10 +0,0 @@
-def test_unknown_virtual_host_is_503(docker_compose, nginxproxy):
-    r = nginxproxy.get("http://unknown.nginx.container.docker/")
-    assert r.status_code == 503
-
-
-def test_forwards_to_whoami(docker_compose, nginxproxy):
-    r = nginxproxy.get("http://whoami.nginx.container.docker/")
-    assert r.status_code == 200
-    whoami_container = docker_compose.containers.get("whoami")
-    assert r.text == f"I'm {whoami_container.id[:12]}\n"

test/test_dockergen/test_dockergen_network_segregation_v3.yml

@@ -1,40 +0,0 @@
-version: '3'
-
-services:
-  nginx:
-    image: nginx
-    container_name: nginx
-    volumes:
-      - "nginx_conf:/etc/nginx/conf.d"
-    labels:
-      - "com.github.nginx-proxy.nginx-proxy.nginx"
-    networks:
-      - proxy
-
-  dockergen:
-    image: nginxproxy/docker-gen
-    command: -notify-sighup nginx -watch /etc/docker-gen/templates/nginx.tmpl /etc/nginx/conf.d/default.conf
-    volumes:
-      - "/var/run/docker.sock:/tmp/docker.sock:ro"
-      - "../../nginx.tmpl:/etc/docker-gen/templates/nginx.tmpl"
-      - "nginx_conf:/etc/nginx/conf.d"
-    networks:
-      - internal
-
-  web:
-    image: web
-    container_name: whoami
-    expose:
-      - "80"
-    environment:
-      WEB_PORTS: "80"
-      VIRTUAL_HOST: "whoami.nginx.container.docker"
-    networks:
-      - proxy
-
-networks:
-  proxy:
-  internal:
-
-volumes:
-  nginx_conf: {}