14965: Fixes token expiry time mock, byte data, and open mock
authorEric Biagiotti <ebiagiotti@veritasgenetics.com>
Tue, 9 Jul 2019 19:47:25 +0000 (15:47 -0400)
committerEric Biagiotti <ebiagiotti@veritasgenetics.com>
Tue, 9 Jul 2019 19:47:25 +0000 (15:47 -0400)
Arvados-DCO-1.1-Signed-off-by:  <ebiagiotti@veritasgenetics.com>

services/fuse/tests/test_token_expiry.py

index 8b89e0cc1662effd5984d6a9bbf4367c11ae6c56..040db2e096252f4f538800e8175f98ed48fcd5cb 100644 (file)
@@ -33,10 +33,10 @@ class TokenExpiryTest(IntegrationTest):
     def fake_open(self, operations, *args, **kwargs):
         self.time_now += 86400*13
         logger.debug('opening file at time=%f', self.time_now)
-        return self.orig_open(operations, *args, **kwargs)
+        return TokenExpiryTest.orig_open(operations, *args, **kwargs)
 
     @mock.patch.object(arvados_fuse.Operations, 'open', autospec=True)
-    @mock.patch('time.time')
+    @mock.patch.object(time, 'time', return_value=0)
     @mock.patch('arvados.keep.KeepClient.get')
     @IntegrationTest.mount(argv=['--mount-by-id', 'zzz'])
     def test_refresh_old_manifest(self, mocked_get, mocked_time, mocked_open):
@@ -45,7 +45,7 @@ class TokenExpiryTest(IntegrationTest):
         # blobSignatureTtl seconds elapse between open() and
         # read(). See https://dev.arvados.org/issues/10008
 
-        mocked_get.return_value = 'fake data'
+        mocked_get.return_value = b'fake data'
         mocked_time.side_effect = self.fake_time
         mocked_open.side_effect = self.fake_open