Skip to content

Commit c0fc89b

Browse files
committed
missing env
1 parent bd0dc6a commit c0fc89b

File tree

2 files changed

+13
-8
lines changed

2 files changed

+13
-8
lines changed

services/storage/docker/boot.sh

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -58,5 +58,6 @@ if [ "${SC_BOOT_MODE}" = "debug" ]; then
5858
else
5959
exec uvicorn simcore_service_storage.main:the_app \
6060
--host 0.0.0.0 \
61+
--port ${STORAGE_PORT} \
6162
--log-level "${SERVER_LOG_LEVEL}"
6263
fi

services/web/server/tests/unit/with_dbs/03/test_storage_handlers.py

Lines changed: 12 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -178,7 +178,6 @@ async def test_openapi_regression_test(
178178

179179

180180
def test_url_storage_resolver_helpers(faker: Faker, app_environment: EnvVarsDict):
181-
182181
app = web.Application()
183182
setup_settings(app)
184183

@@ -188,14 +187,17 @@ def test_url_storage_resolver_helpers(faker: Faker, app_environment: EnvVarsDict
188187
assert urllib.parse.quote("/", safe="") == "%2F"
189188
assert urllib.parse.quote("%2F", safe="") == "%252F"
190189

191-
file_id = urllib.parse.quote(f"{faker.uuid4()}/{faker.uuid4()}/file.py", safe="")
192-
assert "%2F" in file_id
193-
assert "%252F" not in file_id
190+
file_id = f"{faker.uuid4()}/{faker.uuid4()}/file.py"
191+
encoded_file_id = urllib.parse.quote(file_id, safe="")
192+
assert "%2F" in encoded_file_id
193+
assert "%252F" not in encoded_file_id
194194

195-
url = URL(f"/v0/storage/locations/0/files/{file_id}:complete", encoded=True)
196-
assert url.raw_parts[-1] == f"{file_id}:complete"
195+
encoded_url = URL(
196+
f"/v0/storage/locations/0/files/{encoded_file_id}:complete", encoded=True
197+
)
198+
assert encoded_url.raw_parts[-1] == f"{encoded_file_id}:complete"
197199

198-
web_request = make_mocked_request("GET", str(url), app=app)
200+
web_request = make_mocked_request("GET", str(encoded_url), app=app)
199201
web_request[RQT_USERID_KEY] = faker.pyint()
200202

201203
# web -> storage
@@ -211,7 +213,9 @@ def test_url_storage_resolver_helpers(faker: Faker, app_environment: EnvVarsDict
211213

212214
# storage -> web
213215
web_url: AnyUrl = _from_storage_url(
214-
web_request, TypeAdapter(AnyUrl).validate_python(f"{storage_url}")
216+
web_request,
217+
TypeAdapter(AnyUrl).validate_python(f"{storage_url}"),
218+
url_encode=None,
215219
)
216220

217221
assert storage_url.host != web_url.host

0 commit comments

Comments
 (0)