X-Git-Url: https://git.arvados.org/arvados.git/blobdiff_plain/16f704326f44fd1e5e5e60b936c9b5895d6a6ff8..HEAD:/services/dockercleaner/tests/test_cleaner.py diff --git a/services/dockercleaner/tests/test_cleaner.py b/services/dockercleaner/tests/test_cleaner.py index 7580b0128a..cd03538fcd 100644 --- a/services/dockercleaner/tests/test_cleaner.py +++ b/services/dockercleaner/tests/test_cleaner.py @@ -13,7 +13,7 @@ import time import unittest import docker -import mock +from unittest import mock from arvados_docker import cleaner @@ -394,7 +394,7 @@ class RunTestCase(unittest.TestCase): self.assertEqual(event_kwargs[0]['until'], event_kwargs[1]['since']) -@mock.patch('docker.Client', name='docker_client') +@mock.patch('docker.APIClient', name='docker_client') @mock.patch('arvados_docker.cleaner.run', name='cleaner_run') class MainTestCase(unittest.TestCase): @@ -404,11 +404,9 @@ class MainTestCase(unittest.TestCase): cf.flush() cleaner.main(['--config', cf.name]) self.assertEqual(1, docker_client.call_count) - # 1.14 is the first version that's well defined, going back to - # Docker 1.2, and still supported up to at least Docker 1.9. - # See - # . - self.assertEqual('1.14', + # We are standardized on Docker API version 1.35. + # See DockerAPIVersion in lib/crunchrun/docker.go. + self.assertEqual('1.35', docker_client.call_args[1].get('version')) self.assertEqual(1, run_mock.call_count) self.assertIs(run_mock.call_args[0][1], docker_client())