Skip to content

Commit c1e3a6a

Browse files
cbcoutinhocbcoutinho
committed
chore: ruff format
1 parent 5ee9435 commit c1e3a6a

File tree

3 files changed

+54
-54
lines changed

3 files changed

+54
-54
lines changed

‎tests/integration/test_attachments.py‎

Lines changed: 21 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -173,13 +173,13 @@ async def test_attachments_cleanup_on_note_delete(
173173
logger.error(
174174
f"Attachment directory still exists! PROPFIND returned {status}"
175175
)
176-
assert (
177-
False
178-
), f"Expected attachment directory to be gone, but PROPFIND returned {status}!"
176+
assert False, (
177+
f"Expected attachment directory to be gone, but PROPFIND returned {status}!"
178+
)
179179
except HTTPStatusError as e:
180-
assert (
181-
e.response.status_code == 404
182-
), f"Expected PROPFIND to fail with 404, got {e.response.status_code}"
180+
assert e.response.status_code == 404, (
181+
f"Expected PROPFIND to fail with 404, got {e.response.status_code}"
182+
)
183183
logger.info(
184184
"Verified attachment directory does not exist via PROPFIND (404 received)"
185185
)
@@ -283,13 +283,13 @@ async def test_attachments_category_change_handling(nc_client: NextcloudClient):
283283
logger.error(
284284
f"Old attachment directory still exists! PROPFIND returned {status}"
285285
)
286-
assert (
287-
False
288-
), f"Expected old directory to be gone, but PROPFIND returned {status} - directory still exists!"
286+
assert False, (
287+
f"Expected old directory to be gone, but PROPFIND returned {status} - directory still exists!"
288+
)
289289
except HTTPStatusError as e:
290-
assert (
291-
e.response.status_code == 404
292-
), f"Expected PROPFIND to fail with 404, got {e.response.status_code}"
290+
assert e.response.status_code == 404, (
291+
f"Expected PROPFIND to fail with 404, got {e.response.status_code}"
292+
)
293293
logger.info(
294294
"Verified old attachment directory does not exist via PROPFIND (404 received)"
295295
)
@@ -315,9 +315,9 @@ async def test_attachments_category_change_handling(nc_client: NextcloudClient):
315315
logger.error(
316316
f"New attachment directory not found! PROPFIND failed with {e.response.status_code}"
317317
)
318-
assert (
319-
False
320-
), f"Expected new attachment directory to exist, but PROPFIND failed with {e.response.status_code}"
318+
assert False, (
319+
f"Expected new attachment directory to exist, but PROPFIND failed with {e.response.status_code}"
320+
)
321321

322322
finally:
323323
# 6. Cleanup: Delete the note (client should use the *final* category for cleanup path)
@@ -368,9 +368,9 @@ async def test_attachments_category_change_handling(nc_client: NextcloudClient):
368368
]: # Successful PROPFIND means directory exists
369369
assert False, "New category attachment directory still exists!"
370370
except HTTPStatusError as e:
371-
assert (
372-
e.response.status_code == 404
373-
), f"Expected PROPFIND to fail with 404, got {e.response.status_code}"
371+
assert e.response.status_code == 404, (
372+
f"Expected PROPFIND to fail with 404, got {e.response.status_code}"
373+
)
374374
logger.info(
375375
"Verified new category attachment directory is gone via PROPFIND"
376376
)
@@ -389,9 +389,9 @@ async def test_attachments_category_change_handling(nc_client: NextcloudClient):
389389
]: # Successful PROPFIND means directory exists
390390
assert False, "Old category attachment directory still exists!"
391391
except HTTPStatusError as e:
392-
assert (
393-
e.response.status_code == 404
394-
), f"Expected PROPFIND to fail with 404, got {e.response.status_code}"
392+
assert e.response.status_code == 404, (
393+
f"Expected PROPFIND to fail with 404, got {e.response.status_code}"
394+
)
395395
logger.info(
396396
"Verified old category attachment directory is gone via PROPFIND"
397397
)

‎tests/integration/test_embedded_images.py‎

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -99,9 +99,9 @@ async def test_note_with_embedded_image(
9999
logger.error(
100100
f"Attachment directory not found! PROPFIND failed with {e.response.status_code}"
101101
)
102-
assert (
103-
False
104-
), f"Expected attachment directory to exist, but PROPFIND failed with {e.response.status_code}"
102+
assert False, (
103+
f"Expected attachment directory to exist, but PROPFIND failed with {e.response.status_code}"
104+
)
105105

106106
# 2. Update the note content to include the embedded image references
107107
updated_content = f"""{note_data["content"]}
@@ -171,13 +171,13 @@ async def test_note_with_embedded_image(
171171
logger.error(
172172
f"Attachment directory still exists! PROPFIND returned {status}"
173173
)
174-
assert (
175-
False
176-
), f"Expected attachment directory to be gone, but PROPFIND returned {status}!"
174+
assert False, (
175+
f"Expected attachment directory to be gone, but PROPFIND returned {status}!"
176+
)
177177
except HTTPStatusError as e:
178-
assert (
179-
e.response.status_code == 404
180-
), f"Expected PROPFIND to fail with 404, got {e.response.status_code}"
178+
assert e.response.status_code == 404, (
179+
f"Expected PROPFIND to fail with 404, got {e.response.status_code}"
180+
)
181181
logger.info(
182182
"Verified attachment directory does not exist via PROPFIND (404 received)"
183183
)

‎tests/integration/test_webdav_cleanup.py‎

Lines changed: 24 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -108,17 +108,17 @@ async def test_category_change_cleans_up_old_attachments_directory(
108108
logger.error(
109109
"ISSUE DETECTED: Was able to retrieve attachment from old category path!"
110110
)
111-
assert (
112-
False
113-
), "Old category attachment directory still exists and accessible!"
111+
assert False, (
112+
"Old category attachment directory still exists and accessible!"
113+
)
114114
except HTTPStatusError as e:
115115
# This is the expected outcome - old directory should be gone
116116
logger.info(
117117
f"Correctly got error accessing old category path: {e.response.status_code}"
118118
)
119-
assert (
120-
e.response.status_code == 404
121-
), f"Expected 404, got {e.response.status_code}"
119+
assert e.response.status_code == 404, (
120+
f"Expected 404, got {e.response.status_code}"
121+
)
122122
logger.info(
123123
"Verified old category attachment directory is not accessible (good!)"
124124
)
@@ -144,18 +144,18 @@ async def test_category_change_cleans_up_old_attachments_directory(
144144
logger.error(
145145
f"Old attachment directory still exists! PROPFIND returned {status}"
146146
)
147-
assert (
148-
False
149-
), f"Expected old attachment directory to be gone, but it still exists (PROPFIND returned {status})!"
147+
assert False, (
148+
f"Expected old attachment directory to be gone, but it still exists (PROPFIND returned {status})!"
149+
)
150150
# If we got another status code (like 404), it's also good - the directory doesn't exist
151151
logger.info(
152152
f"Verified old attachment directory does not exist (PROPFIND returned {status})"
153153
)
154154
except HTTPStatusError as e:
155155
# 404 is expected - directory should not exist
156-
assert (
157-
e.response.status_code == 404
158-
), f"Expected PROPFIND to fail with 404, got {e.response.status_code}"
156+
assert e.response.status_code == 404, (
157+
f"Expected PROPFIND to fail with 404, got {e.response.status_code}"
158+
)
159159
logger.info(
160160
"Verified old attachment directory does not exist via PROPFIND (404 received)"
161161
)
@@ -210,17 +210,17 @@ async def test_category_change_cleans_up_old_attachments_directory(
210210
logger.error(
211211
f"New category attachment directory still exists! PROPFIND returned {status}"
212212
)
213-
assert (
214-
False
215-
), f"Expected new category attachment directory to be gone, but it still exists (PROPFIND returned {status})!"
213+
assert False, (
214+
f"Expected new category attachment directory to be gone, but it still exists (PROPFIND returned {status})!"
215+
)
216216
# If we got another status code (like 404), it's also good - the directory doesn't exist
217217
logger.info(
218218
f"Verified new category attachment directory does not exist (PROPFIND returned {status})"
219219
)
220220
except HTTPStatusError as e:
221-
assert (
222-
e.response.status_code == 404
223-
), f"Expected PROPFIND to fail with 404, got {e.response.status_code}"
221+
assert e.response.status_code == 404, (
222+
f"Expected PROPFIND to fail with 404, got {e.response.status_code}"
223+
)
224224
logger.info(
225225
"Verified new category attachment directory is gone via PROPFIND"
226226
)
@@ -241,17 +241,17 @@ async def test_category_change_cleans_up_old_attachments_directory(
241241
logger.error(
242242
f"Old category attachment directory still exists! PROPFIND returned {status}"
243243
)
244-
assert (
245-
False
246-
), f"Expected old category attachment directory to be gone, but it still exists (PROPFIND returned {status})!"
244+
assert False, (
245+
f"Expected old category attachment directory to be gone, but it still exists (PROPFIND returned {status})!"
246+
)
247247
# If we got another status code (like 404), it's also good - the directory doesn't exist
248248
logger.info(
249249
f"Verified old category attachment directory does not exist (PROPFIND returned {status})"
250250
)
251251
except HTTPStatusError as e:
252-
assert (
253-
e.response.status_code == 404
254-
), f"Expected PROPFIND to fail with 404, got {e.response.status_code}"
252+
assert e.response.status_code == 404, (
253+
f"Expected PROPFIND to fail with 404, got {e.response.status_code}"
254+
)
255255
logger.info(
256256
"Verified old category attachment directory is gone via PROPFIND"
257257
)

0 commit comments

Comments
 (0)