search

jic-dtool / dtool-s3

S3 backend for dtool
MIT License
1 stars 3 forks source link

Add test for user-specified prefix for object path functionality #4

Closed tjelvar-olsson closed 3 years ago

tjelvar-olsson commented 3 years ago

Add test to check behaviour of: https://github.com/jic-dtool/dtool-s3/pull/3

It should be relatively easy to do using the tmp_env_var decorator: https://github.com/jic-dtool/dtool-s3/blob/master/tests/__init__.py#L22

and DTOOL_S3_DATASET_PREFIX environment variable

@pastewka

tjelvar-olsson commented 3 years ago

Added skeleton code for test in https://github.com/jic-dtool/dtool-s3/commit/8c2768603b8313df81a6e290bf49ca8d703aab2c

pastewka commented 3 years ago

Yes that sounds good. The ECS has a similar environment variable.

tjelvar-olsson commented 3 years ago

Implemented in 21f2e8f