Chris Coutinho
115 lines
4.7 KiB
Python
115 lines
4.7 KiB
Python
import pytest
|
|
import logging
|
|
import time
|
|
import uuid # Keep uuid if needed for generating unique data within tests
|
|
from httpx import HTTPStatusError
|
|
|
|
from nextcloud_mcp_server.client import NextcloudClient
|
|
|
|
# Note: nc_client fixture is now session-scoped in conftest.py
|
|
|
|
logger = logging.getLogger(__name__)
|
|
|
|
# Mark all tests in this module as integration tests
|
|
pytestmark = pytest.mark.integration
|
|
|
|
def test_notes_api_create_and_read(nc_client: NextcloudClient, temporary_note: dict):
|
|
"""
|
|
Tests creating a note via the API (using fixture) and then reading it back.
|
|
"""
|
|
created_note_data = temporary_note # Get data from fixture
|
|
note_id = created_note_data["id"]
|
|
|
|
logger.info(f"Reading note created by fixture, ID: {note_id}")
|
|
read_note = nc_client.notes_get_note(note_id=note_id)
|
|
|
|
assert read_note["id"] == note_id
|
|
assert read_note["title"] == created_note_data["title"]
|
|
assert read_note["content"] == created_note_data["content"]
|
|
assert read_note["category"] == created_note_data["category"]
|
|
logger.info(f"Successfully read and verified note ID: {note_id}")
|
|
|
|
def test_notes_api_update(nc_client: NextcloudClient, temporary_note: dict):
|
|
"""
|
|
Tests updating a note created by the fixture.
|
|
"""
|
|
created_note_data = temporary_note
|
|
note_id = created_note_data["id"]
|
|
original_etag = created_note_data["etag"]
|
|
original_category = created_note_data["category"]
|
|
|
|
update_title = f"Updated Title {uuid.uuid4().hex[:8]}"
|
|
update_content = f"Updated Content {uuid.uuid4().hex[:8]}"
|
|
|
|
logger.info(f"Attempting to update note ID: {note_id} with etag: {original_etag}")
|
|
updated_note = nc_client.notes_update_note(
|
|
note_id=note_id,
|
|
etag=original_etag,
|
|
title=update_title,
|
|
content=update_content,
|
|
# category=original_category # Explicitly pass category if required by update
|
|
)
|
|
logger.info(f"Note updated: {updated_note}")
|
|
|
|
assert updated_note["id"] == note_id
|
|
assert updated_note["title"] == update_title
|
|
assert updated_note["content"] == update_content
|
|
assert updated_note["category"] == original_category # Verify category didn't change
|
|
assert "etag" in updated_note
|
|
assert updated_note["etag"] != original_etag # Etag must change
|
|
|
|
# Optional: Verify update by reading again
|
|
time.sleep(1) # Allow potential propagation delay
|
|
read_updated_note = nc_client.notes_get_note(note_id=note_id)
|
|
assert read_updated_note["title"] == update_title
|
|
assert read_updated_note["content"] == update_content
|
|
logger.info(f"Successfully updated and verified note ID: {note_id}")
|
|
|
|
def test_notes_api_update_conflict(nc_client: NextcloudClient, temporary_note: dict):
|
|
"""
|
|
Tests that attempting to update with an old etag fails with 412.
|
|
"""
|
|
created_note_data = temporary_note
|
|
note_id = created_note_data["id"]
|
|
original_etag = created_note_data["etag"]
|
|
|
|
# Perform a first update to change the etag
|
|
first_update_title = f"First Update {uuid.uuid4().hex[:8]}"
|
|
logger.info(f"Performing first update on note ID: {note_id} to change etag.")
|
|
first_updated_note = nc_client.notes_update_note(
|
|
note_id=note_id,
|
|
etag=original_etag,
|
|
title=first_update_title,
|
|
content="First update content",
|
|
# category=created_note_data["category"] # Pass category if required
|
|
)
|
|
new_etag = first_updated_note["etag"]
|
|
assert new_etag != original_etag
|
|
logger.info(f"Note ID: {note_id} updated, new etag: {new_etag}")
|
|
time.sleep(1)
|
|
|
|
# Now attempt update with the *original* etag
|
|
logger.info(f"Attempting second update on note ID: {note_id} with OLD etag: {original_etag}")
|
|
with pytest.raises(HTTPStatusError) as excinfo:
|
|
nc_client.notes_update_note(
|
|
note_id=note_id,
|
|
etag=original_etag, # Use the stale etag
|
|
title="This update should fail due to conflict",
|
|
# category=created_note_data["category"] # Pass category if required
|
|
)
|
|
assert excinfo.value.response.status_code == 412 # Precondition Failed
|
|
logger.info("Update with old etag correctly failed with 412 Precondition Failed.")
|
|
|
|
def test_notes_api_delete_nonexistent(nc_client: NextcloudClient):
|
|
"""
|
|
Tests deleting a note that doesn't exist fails with 404.
|
|
"""
|
|
non_existent_id = 999999999 # Use an ID highly unlikely to exist
|
|
logger.info(f"\nAttempting to delete non-existent note ID: {non_existent_id}")
|
|
with pytest.raises(HTTPStatusError) as excinfo:
|
|
nc_client.notes_delete_note(note_id=non_existent_id)
|
|
assert excinfo.value.response.status_code == 404
|
|
logger.info(f"Deleting non-existent note ID: {non_existent_id} correctly failed with 404.")
|
|
|
|
# --- Attachment tests moved to test_attachments.py ---
|