Add cleanup to documents in tests

[NolanTrem]

Important

Adds cleanup_documents fixture to test_documents.py for automatic document cleanup in tests, replacing manual cleanup code.

• Fixture Addition:
  • Adds cleanup_documents fixture in test_documents.py to track and delete documents created during tests.
• Test Modifications:
  • Updates test_create_document_with_file, test_create_document_with_raw_text, test_create_document_with_chunks, and test_create_document_different_modes to use cleanup_documents for automatic cleanup.
  • Modifies test_list_document_chunks, test_search_documents_extended, test_access_document_not_owned, test_list_documents_with_pagination, test_delete_by_complex_filter, test_search_documents_no_match, test_delete_by_workflow_metadata, test_delete_by_classification_metadata, and test_delete_by_version_metadata to use cleanup_documents.
• Behavior:
  • Ensures all documents created in tests are deleted, preventing leftover test data.

^{This description was created by for dc94a6b. It will automatically update as commits are pushed.}

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to dc94a6b in 1 minute and 40 seconds

More details

• Looked at 541 lines of code in 1 files
• Skipped 0 files when reviewing.
• Skipped posting 4 drafted comments based on config settings.

1. py/tests/integration/test_documents.py:78

• Draft comment:
  Consider using the cleanup_documents fixture to ensure documents are deleted even if the test fails.
• Reason this comment was not posted:
  Comment looked like it was already resolved.

2. py/tests/integration/test_documents.py:228

• Draft comment:
  Consider using the cleanup_documents fixture to ensure documents are deleted even if the test fails.
• Reason this comment was not posted:
  Marked as duplicate.

3. py/tests/integration/test_documents.py:282

• Draft comment:
  Consider using the cleanup_documents fixture to ensure documents are deleted even if the test fails.
• Reason this comment was not posted:
  Marked as duplicate.

4. py/tests/integration/test_documents.py:515

• Draft comment:
  Consider using the cleanup_documents fixture to ensure documents are deleted even if the test fails.
• Reason this comment was not posted:
  Marked as duplicate.

Workflow ID: wflow_BhyFqLyKUjsCbWGU

---

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.