Fix local tests (#855)

CHANGES/849.bugfix

@@ -0,0 +1 @@
+Fixes unittests that don't run locally due to deprecations in later versions of Docker. Tested with 26.00, v1.45.

CONTRIBUTORS.txt

@@ -7,6 +7,7 @@ Byeongjun Park
 Cecil Tonglet
 Christian Barra
 Danny Song
+eevelweezel
 Gaopeiliang
 Greg Guthe
 Hiroki Kiyohara

tests/conftest.py

@@ -52,7 +52,7 @@ async def random_name():
     docker = Docker()
     images = await docker.images.list()
     for img in images:
-        if img["RepoTags"] is None:
+        if not img["RepoTags"]:
             continue
         try:
             if img["RepoTags"][0].startswith("aiodocker-"):

tests/test_containers.py

@@ -170,12 +170,17 @@ async def test_resize(shell_container):
 )
 @pytest.mark.asyncio
 async def test_commit(docker, image_name, shell_container):
+    """
+    "Container" key was removed in v1.45.
+    "ContainerConfig" is not present, although this information is now present in "Config"
+    These changes have been verified against v1.45.
+    """
     ret = await shell_container.commit()
     img_id = ret["Id"]
     img = await docker.images.inspect(img_id)
-    assert img["Container"].startswith(shell_container.id)
-    assert "Image" in img["ContainerConfig"]
-    assert image_name == img["ContainerConfig"]["Image"]
+
+    assert "Image" in img["Config"]
+    assert image_name == img["Config"]["Image"]
     python_img = await docker.images.inspect(image_name)
     python_id = python_img["Id"]
     assert "Parent" in img
@@ -191,7 +196,6 @@ async def test_commit_with_changes(docker, image_name, shell_container):
     ret = await shell_container.commit(changes=["EXPOSE 8000", 'CMD ["py"]'])
     img_id = ret["Id"]
     img = await docker.images.inspect(img_id)
-    assert img["Container"].startswith(shell_container.id)
     assert "8000/tcp" in img["Config"]["ExposedPorts"]
     assert img["Config"]["Cmd"] == ["py"]
     await docker.images.delete(img_id)

tests/test_networks.py

@@ -13,12 +13,17 @@ async def test_list_networks(docker):
 
 @pytest.mark.asyncio
 async def test_list_networks_with_filter(docker):
-    await docker.networks.create({
+    network = await docker.networks.create({
         "Name": "test-net-filter",
         "Labels": {"some": "label"},
     })
-    networks = await docker.networks.list(filters={"label": "some=label"})
-    assert len(networks) == 1
+    try:
+        networks = await docker.networks.list(filters={"label": "some=label"})
+        assert len(networks) == 1
+    finally:
+        if network:
+            deleted = await network.delete()
+            assert deleted is True
 
 
 @pytest.mark.asyncio