admin/mcp-nextcloud
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: update tests

Browse Source
This commit is contained in:
Chris Coutinho
2025-07-06 09:40:27 +02:00
parent d47e2bb8f0
commit ca5bbb783a
1 changed files with 10 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files
tests/integration/test_webdav_cleanup.py
+10 -10
View File
@@ -29,7 +29,7 @@ async def test_category_change_cleans_up_old_attachments_directory(
try: try:
# 1. Create note with initial category # 1. Create note with initial category
logger.info(f"Creating note '{note_title}' in category '{initial_category}'") logger.info(f"Creating note '{note_title}' in category '{initial_category}'")
created_note = await nc_client.notes_create_note( created_note = await nc_client.notes.create_note(
title=note_title, content="Initial content", category=initial_category title=note_title, content="Initial content", category=initial_category
) )
note_id = created_note["id"] note_id = created_note["id"]
@@ -41,7 +41,7 @@ async def test_category_change_cleans_up_old_attachments_directory(
logger.info( logger.info(
f"Adding attachment '{attachment_filename}' to note {note_id} (in {initial_category})" f"Adding attachment '{attachment_filename}' to note {note_id} (in {initial_category})"
) )
upload_response = await nc_client.add_note_attachment( upload_response = await nc_client.webdav.add_note_attachment(
note_id=note_id, note_id=note_id,
filename=attachment_filename, filename=attachment_filename,
content=attachment_content, content=attachment_content,
@@ -56,7 +56,7 @@ async def test_category_change_cleans_up_old_attachments_directory(
logger.info( logger.info(
f"Verifying attachment retrieval from initial category '{initial_category}'" f"Verifying attachment retrieval from initial category '{initial_category}'"
) )
retrieved_content1, _ = await nc_client.get_note_attachment( retrieved_content1, _ = await nc_client.webdav.get_note_attachment(
note_id=note_id, filename=attachment_filename, category=initial_category note_id=note_id, filename=attachment_filename, category=initial_category
) )
assert retrieved_content1 == attachment_content assert retrieved_content1 == attachment_content
@@ -72,9 +72,9 @@ async def test_category_change_cleans_up_old_attachments_directory(
logger.info( logger.info(
f"Updating note {note_id} category from '{initial_category}' to '{new_category}'" f"Updating note {note_id} category from '{initial_category}' to '{new_category}'"
) )
current_note_data = await nc_client.notes_get_note(note_id=note_id) current_note_data = await nc_client.notes.get_note(note_id=note_id)
current_etag = current_note_data["etag"] current_etag = current_note_data["etag"]
updated_note = await nc_client.notes_update_note( updated_note = await nc_client.notes.update(
note_id=note_id, note_id=note_id,
etag=current_etag, etag=current_etag,
category=new_category, category=new_category,
@@ -90,7 +90,7 @@ async def test_category_change_cleans_up_old_attachments_directory(
logger.info( logger.info(
f"Verifying attachment retrieval from new category '{new_category}'" f"Verifying attachment retrieval from new category '{new_category}'"
) )
retrieved_content2, _ = await nc_client.get_note_attachment( retrieved_content2, _ = await nc_client.webdav.get_note_attachment(
note_id=note_id, filename=attachment_filename, category=new_category note_id=note_id, filename=attachment_filename, category=new_category
) )
assert retrieved_content2 == attachment_content assert retrieved_content2 == attachment_content
@@ -101,7 +101,7 @@ async def test_category_change_cleans_up_old_attachments_directory(
f"Trying to retrieve attachment from old category '{initial_category}' - should fail" f"Trying to retrieve attachment from old category '{initial_category}' - should fail"
) )
try: try:
await nc_client.get_note_attachment( await nc_client.webdav.get_note_attachment(
note_id=note_id, filename=attachment_filename, category=initial_category note_id=note_id, filename=attachment_filename, category=initial_category
) )
# If we get here, it means the old directory still exists (a problem) # If we get here, it means the old directory still exists (a problem)
@@ -165,14 +165,14 @@ async def test_category_change_cleans_up_old_attachments_directory(
if note_id: if note_id:
logger.info(f"Cleaning up note ID: {note_id}") logger.info(f"Cleaning up note ID: {note_id}")
try: try:
await nc_client.notes_delete_note(note_id=note_id) await nc_client.notes.delete_note(note_id=note_id)
logger.info(f"Note {note_id} deleted.") logger.info(f"Note {note_id} deleted.")
time.sleep(1) time.sleep(1)
# 9. Verify both old and new attachment paths are gone # 9. Verify both old and new attachment paths are gone
logger.info("Verifying all attachment paths are gone") logger.info("Verifying all attachment paths are gone")
with pytest.raises(HTTPStatusError) as excinfo_new: with pytest.raises(HTTPStatusError) as excinfo_new:
await nc_client.get_note_attachment( await nc_client.webdav.get_note_attachment(
note_id=note_id, note_id=note_id,
filename=attachment_filename, filename=attachment_filename,
category=new_category, category=new_category,
@@ -180,7 +180,7 @@ async def test_category_change_cleans_up_old_attachments_directory(
assert excinfo_new.value.response.status_code == 404 assert excinfo_new.value.response.status_code == 404
with pytest.raises(HTTPStatusError) as excinfo_old: with pytest.raises(HTTPStatusError) as excinfo_old:
await nc_client.get_note_attachment( await nc_client.webdav.get_note_attachment(
note_id=note_id, note_id=note_id,
filename=attachment_filename, filename=attachment_filename,
category=initial_category, category=initial_category,