diff --git a/test_runner/regress/test_download_extensions.py b/test_runner/regress/test_download_extensions.py index 8ee0c85490..2941b72b09 100644 --- a/test_runner/regress/test_download_extensions.py +++ b/test_runner/regress/test_download_extensions.py @@ -106,6 +106,8 @@ def test_file_download(neon_env_builder: NeonEnvBuilder, remote_storage_kind: Re 111 """ ) + # TODO: maybe if we are using REAL_S3 storage, we should not upload files + # or at least, maybe we should delete them afterwards env.remote_storage_client.upload_fileobj( test_lib_file, env.ext_remote_storage.bucket_name, @@ -164,6 +166,12 @@ def test_file_download(neon_env_builder: NeonEnvBuilder, remote_storage_kind: Re # - try to load non-existing library # cleanup downloaded extensions + # TODO: clean up downloaded libraries too + # TODO: make sure this runs even if the test fails + # this is important because if the files aren't cleaned up then the test can + # pass even without successfully downloading the files if a previous run (or + # run with different type of remote storage) of the test did download the + # files for file in cleanup_files: try: log.info(f"Deleting {file}")