21207: Set up MockedPut._state with a deepcopy
authorBrett Smith <brett.smith@curii.com>
Fri, 3 May 2024 13:55:15 +0000 (09:55 -0400)
committerBrett Smith <brett.smith@curii.com>
Fri, 3 May 2024 20:47:23 +0000 (16:47 -0400)
Otherwise its internal state could get into EMPTY_STATE and contaminate
other tests.

Arvados-DCO-1.1-Signed-off-by: Brett Smith <brett.smith@curii.com>

sdk/python/tests/test_arv_put.py

index e3f7c9d446589d3f3a5064518e5c0b00525a606d..e8a3e65bdcd99a7c0a8a9ac6a19af5ebc2655aae 100644 (file)
@@ -6,6 +6,7 @@
 
 import apiclient
 import ciso8601
+import copy
 import datetime
 import json
 import logging
@@ -568,7 +569,7 @@ class ArvPutUploadJobTest(run_test_server.TestCaseWithServers,
 class CachedManifestValidationTest(ArvadosBaseTestCase):
     class MockedPut(arv_put.ArvPutUploadJob):
         def __init__(self, cached_manifest=None):
-            self._state = arv_put.ArvPutUploadJob.EMPTY_STATE
+            self._state = copy.deepcopy(arv_put.ArvPutUploadJob.EMPTY_STATE)
             self._state['manifest'] = cached_manifest
             self._api_client = mock.MagicMock()
             self.logger = mock.MagicMock()