Skip to content

Commit 122703f

Browse files
authored
♻️ Maintenance/remove pytest docker storage (ITISFoundation#3011)
1 parent 10618a4 commit 122703f

File tree

17 files changed

+203
-509
lines changed

17 files changed

+203
-509
lines changed

packages/dask-task-models-library/src/dask_task_models_library/container_tasks/io.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,7 @@ class Config:
6969
schema_extra = {
7070
"examples": [
7171
{
72-
"url": "s3://some_file_url",
72+
"url": "https://some_file_url",
7373
},
7474
{"url": "s3://some_file_url", "file_mapping": "some_file_name.txt"},
7575
]
@@ -124,7 +124,7 @@ class Config(DictModel.Config):
124124
"string_output": {"required": False},
125125
"file_output": {
126126
"required": True,
127-
"url": "s3://some_file_url",
127+
"url": "https://some_file_url",
128128
"mapping": "the_output_filename",
129129
},
130130
"optional_file_output": {

packages/pytest-simcore/src/pytest_simcore/helpers/utils_assert.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,7 @@ async def assert_status(
5656
async def assert_error(
5757
response: ClientResponse,
5858
expected_cls: Type[HTTPException],
59-
expected_msg: str = None,
59+
expected_msg: Optional[str] = None,
6060
):
6161
data, error = unwrap_envelope(await response.json())
6262
return do_assert_error(data, error, expected_cls, expected_msg)
@@ -66,7 +66,7 @@ def do_assert_error(
6666
data,
6767
error,
6868
expected_cls: Type[HTTPException],
69-
expected_msg: str = None,
69+
expected_msg: Optional[str] = None,
7070
expected_error_code: Optional[str] = None,
7171
):
7272
assert not data, pformat(data)

packages/pytest-simcore/src/pytest_simcore/postgres_service.py

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -164,6 +164,13 @@ def postgres_engine(postgres_dsn: Dict[str, str]) -> Iterator[sa.engine.Engine]:
164164
engine.dispose()
165165

166166

167+
@pytest.fixture(scope="module")
168+
def postgres_dsn_url(postgres_dsn: Dict[str, str]) -> str:
169+
return "postgresql://{user}:{password}@{host}:{port}/{database}".format(
170+
**postgres_dsn
171+
)
172+
173+
167174
@pytest.fixture(scope="module")
168175
def postgres_db(
169176
postgres_dsn: Dict[str, str],

services/storage/requirements/_test.in

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@
99
pytest
1010
pytest-aiohttp # incompatible with pytest-asyncio. See https://github.com/pytest-dev/pytest-asyncio/issues/76
1111
pytest-cov
12-
pytest-docker
1312
pytest-icdiff
1413
pytest-instafail
1514
pytest-mock

services/storage/requirements/_test.txt

Lines changed: 3 additions & 60 deletions
Original file line numberDiff line numberDiff line change
@@ -23,20 +23,11 @@ attrs==20.3.0
2323
# via
2424
# -c requirements/_base.txt
2525
# aiohttp
26-
# jsonschema
2726
# pytest
28-
# pytest-docker
29-
bcrypt==3.2.0
30-
# via paramiko
3127
certifi==2021.10.8
3228
# via
3329
# -c requirements/_base.txt
3430
# requests
35-
cffi==1.15.0
36-
# via
37-
# bcrypt
38-
# cryptography
39-
# pynacl
4031
charset-normalizer==2.0.12
4132
# via
4233
# -c requirements/_base.txt
@@ -52,26 +43,11 @@ coverage==6.3.2
5243
# pytest-cov
5344
coveralls==3.3.1
5445
# via -r requirements/_test.in
55-
cryptography==36.0.2
56-
# via
57-
# -c requirements/../../../requirements/constraints.txt
58-
# paramiko
5946
dill==0.3.4
6047
# via pylint
61-
distro==1.7.0
62-
# via docker-compose
6348
docker==5.0.3
64-
# via docker-compose
65-
docker-compose==1.29.1
66-
# via
67-
# -c requirements/../../../requirements/constraints.txt
68-
# pytest-docker
69-
dockerpty==0.4.1
70-
# via docker-compose
7149
docopt==0.6.2
72-
# via
73-
# coveralls
74-
# docker-compose
50+
# via coveralls
7551
faker==13.3.3
7652
# via -r requirements/_test.in
7753
frozenlist==1.3.0
@@ -90,10 +66,6 @@ iniconfig==1.1.1
9066
# via pytest
9167
isort==5.10.1
9268
# via pylint
93-
jsonschema==3.2.0
94-
# via
95-
# -c requirements/_base.txt
96-
# docker-compose
9769
lazy-object-proxy==1.4.3
9870
# via
9971
# -c requirements/_base.txt
@@ -113,8 +85,6 @@ packaging==21.3
11385
# pytest-sugar
11486
pandas==1.4.1
11587
# via -r requirements/_test.in
116-
paramiko==2.10.3
117-
# via docker
11888
platformdirs==2.5.1
11989
# via pylint
12090
pluggy==1.0.0
@@ -125,25 +95,16 @@ ptvsd==4.3.2
12595
# via -r requirements/_test.in
12696
py==1.11.0
12797
# via pytest
128-
pycparser==2.21
129-
# via cffi
13098
pylint==2.13.2
13199
# via -r requirements/_test.in
132-
pynacl==1.5.0
133-
# via paramiko
134100
pyparsing==3.0.7
135101
# via packaging
136-
pyrsistent==0.18.1
137-
# via
138-
# -c requirements/_base.txt
139-
# jsonschema
140102
pytest==7.1.1
141103
# via
142104
# -r requirements/_test.in
143105
# pytest-aiohttp
144106
# pytest-asyncio
145107
# pytest-cov
146-
# pytest-docker
147108
# pytest-icdiff
148109
# pytest-instafail
149110
# pytest-mock
@@ -154,8 +115,6 @@ pytest-asyncio==0.18.3
154115
# via pytest-aiohttp
155116
pytest-cov==3.0.0
156117
# via -r requirements/_test.in
157-
pytest-docker==0.11.0
158-
# via -r requirements/_test.in
159118
pytest-icdiff==0.5
160119
# via -r requirements/_test.in
161120
pytest-instafail==0.4.2
@@ -172,35 +131,21 @@ python-dateutil==2.8.2
172131
# faker
173132
# pandas
174133
python-dotenv==0.20.0
175-
# via
176-
# -r requirements/_test.in
177-
# docker-compose
134+
# via -r requirements/_test.in
178135
pytz==2022.1
179136
# via pandas
180-
pyyaml==5.4.1
181-
# via
182-
# -c requirements/../../../requirements/constraints.txt
183-
# -c requirements/_base.txt
184-
# docker-compose
185137
requests==2.27.1
186138
# via
187139
# codecov
188140
# coveralls
189141
# docker
190-
# docker-compose
191142
six==1.16.0
192143
# via
193144
# -c requirements/_base.txt
194-
# bcrypt
195-
# dockerpty
196-
# jsonschema
197-
# paramiko
198145
# python-dateutil
199146
# websocket-client
200147
termcolor==1.1.0
201148
# via pytest-sugar
202-
texttable==1.6.4
203-
# via docker-compose
204149
tomli==2.0.1
205150
# via
206151
# coverage
@@ -217,9 +162,7 @@ urllib3==1.26.9
217162
# -c requirements/_base.txt
218163
# requests
219164
websocket-client==0.59.0
220-
# via
221-
# docker
222-
# docker-compose
165+
# via docker
223166
wrapt==1.14.0
224167
# via
225168
# -c requirements/_base.txt

services/storage/requirements/_tools.txt

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,6 @@ pyyaml==5.4.1
4646
# via
4747
# -c requirements/../../../requirements/constraints.txt
4848
# -c requirements/_base.txt
49-
# -c requirements/_test.txt
5049
# pre-commit
5150
# watchdog
5251
six==1.16.0

0 commit comments

Comments
 (0)