ENG-8251 | Fix failing move/copy task tests

Author: sh-andriy

tests/tasks/test_copy.py

@@ -44,7 +44,7 @@ def make_provider(name=None, **kwargs):
     return src_provider, dest_provider
 
 
-@pytest.mark.skip('TODO: test fails sometimes')
+# @pytest.mark.skip('TODO: test fails sometimes')
 @pytest.mark.celery(result_backend=None)
 def test_copy_calls_copy(providers, bundles, callback):
     src, dest = providers
@@ -62,7 +62,7 @@ def test_is_task():
     assert not asyncio.iscoroutine(copy.copy)
     assert asyncio.iscoroutinefunction(copy.copy.adelay)
 
-@pytest.mark.skip('TODO: test fails sometimes')
+# @pytest.mark.skip('TODO: test fails sometimes')
 @pytest.mark.celery(result_backend=None)
 def test_imputes_exceptions(providers, bundles, callback):
     src, dest = providers
@@ -82,7 +82,7 @@ def test_imputes_exceptions(providers, bundles, callback):
     assert method == 'PUT'
     assert data['errors'] == ["Exception('This is a string')"]
 
-@pytest.mark.skip('TODO: test fails sometimes')
+# @pytest.mark.skip('TODO: test fails sometimes')
 @pytest.mark.celery(result_backend=None)
 def test_return_values(providers, bundles, callback, src_path, dest_path, mock_time, FAKE_TIME):
     src, dest = providers
@@ -136,7 +136,7 @@ def test_return_values(providers, bundles, callback, src_path, dest_path, mock_t
         'sizeInt': metadata.size_as_int,
     }
 
-@pytest.mark.skip('TODO: test fails sometimes')
+# @pytest.mark.skip('TODO: test fails sometimes')
 @pytest.mark.celery(result_backend=None)
 def test_starttime_override(providers, bundles, callback, mock_time, FAKE_TIME):
     src, dest = providers

tests/tasks/test_move.py

@@ -44,7 +44,7 @@ def make_provider(name=None, **kwargs):
     return src_provider, dest_provider
 
 
-@pytest.mark.skip('TODO: test fails sometimes')
+# @pytest.mark.skip('TODO: test fails sometimes')
 @pytest.mark.celery(result_backend=None)
 def test_move_calls_move(providers, bundles, callback):
     src, dest = providers
@@ -64,7 +64,7 @@ def test_is_task():
     assert asyncio.iscoroutinefunction(move.move.adelay)
 
 
-@pytest.mark.skip('TODO: test fails sometimes')
+# @pytest.mark.skip('TODO: test fails sometimes')
 @pytest.mark.celery(result_backend=None)
 def test_imputes_exceptions(providers, bundles, callback):
     src, dest = providers
@@ -85,7 +85,7 @@ def test_imputes_exceptions(providers, bundles, callback):
     assert url == 'dest_callback'
 
 
-@pytest.mark.skip('TODO: test fails sometimes')
+# @pytest.mark.skip('TODO: test fails sometimes')
 @pytest.mark.celery(result_backend=None)
 def test_return_values(providers, bundles, callback, src_path, dest_path, mock_time, FAKE_TIME):
     src, dest = providers
@@ -140,7 +140,7 @@ def test_return_values(providers, bundles, callback, src_path, dest_path, mock_t
     }
 
 
-@pytest.mark.skip('TODO: test fails sometimes')
+# @pytest.mark.skip('TODO: test fails sometimes')
 @pytest.mark.celery(result_backend=None)
 def test_starttime_override(providers, bundles, callback, mock_time, FAKE_TIME):
     src, dest = providers