3706: Really make the API server return the wrong data, instead of just saying so.
authorTom Clegg <tom@curoverse.com>
Fri, 31 Oct 2014 21:21:31 +0000 (17:21 -0400)
committerTom Clegg <tom@curoverse.com>
Fri, 31 Oct 2014 21:21:31 +0000 (17:21 -0400)
sdk/python/tests/test_collections.py

index 00b2ac0f2970563d8d356aba05e5f28846f0c240..df8ba388d8ea4721dcc286621ed8f0927cac0de7 100644 (file)
@@ -668,6 +668,9 @@ class CollectionTestMixin(object):
     DEFAULT_DATA_HASH = DEFAULT_COLLECTION['portable_data_hash']
     DEFAULT_MANIFEST = DEFAULT_COLLECTION['manifest_text']
     DEFAULT_UUID = DEFAULT_COLLECTION['uuid']
+    ALT_COLLECTION = API_COLLECTIONS['bar_file']
+    ALT_DATA_HASH = ALT_COLLECTION['portable_data_hash']
+    ALT_MANIFEST = ALT_COLLECTION['manifest_text']
 
     def _mock_api_call(self, mock_method, code, body):
         mock_method = mock_method().execute
@@ -747,11 +750,11 @@ class CollectionReaderTestCase(unittest.TestCase, CollectionTestMixin):
         # To verify that CollectionReader tries Keep first here, we
         # mock API server to return the wrong data.
         client = self.api_client_mock(200)
-        with tutil.mock_responses(self.DEFAULT_MANIFEST, 200):
+        with tutil.mock_responses(self.ALT_MANIFEST, 200):
             self.assertEqual(
-                self.DEFAULT_MANIFEST,
+                self.ALT_MANIFEST,
                 arvados.CollectionReader(
-                    self.DEFAULT_DATA_HASH + '+Affffffffffffffffffffffffffffffffffffffff@fedcba98',
+                    self.ALT_DATA_HASH + '+Affffffffffffffffffffffffffffffffffffffff@fedcba98',
                     api_client=client).manifest_text())
 
     def test_init_num_retries_propagated(self):