Fix missing asserts or assignments

Signed-off-by: Aarni Koskela <akx@iki.fi>
docker · Aug 15, 2023 · 0566f12 · 0566f12
1 parent cc76c9c
commit 0566f12
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/tests/integration/api_swarm_test.py b/tests/integration/api_swarm_test.py
@@ -127,11 +127,11 @@ def test_leave_swarm(self):
         assert self.init_swarm()
         with pytest.raises(docker.errors.APIError) as exc_info:
             self.client.leave_swarm()
-        exc_info.value.response.status_code == 500
+        assert exc_info.value.response.status_code == 500
         assert self.client.leave_swarm(force=True)
         with pytest.raises(docker.errors.APIError) as exc_info:
             self.client.inspect_swarm()
-        exc_info.value.response.status_code == 406
+        assert exc_info.value.response.status_code == 406
         assert self.client.leave_swarm(force=True)
 
     @requires_api_version('1.24')

diff --git a/tests/integration/models_containers_test.py b/tests/integration/models_containers_test.py
@@ -221,7 +221,7 @@ def test_list_sparse(self):
         assert container.status == 'running'
         assert container.image == client.images.get('alpine')
         with pytest.raises(docker.errors.DockerException):
-            container.labels
+            _ = container.labels
 
         container.kill()
         container.remove()

diff --git a/tests/unit/api_test.py b/tests/unit/api_test.py
@@ -581,7 +581,7 @@ def test_read_from_socket_no_stream_tty_demux(self):
 
     def test_read_from_socket_no_stream_no_tty(self):
         res = self.request(stream=False, tty=False, demux=False)
-        res == self.stdout_data + self.stderr_data
+        assert res == self.stdout_data + self.stderr_data
 
     def test_read_from_socket_no_stream_no_tty_demux(self):
         res = self.request(stream=False, tty=False, demux=True)