add collection integration tests

[emrgnt-cmplxty]

Important

Add integration tests for collection management in SDK, update API for access checks, and adjust response models.

• Integration Tests:
  • Added tests for collection management in harness_sdk.py, including creation, update, listing, and document management within collections.
  • Added tests for user permissions and error handling in collection operations.
• API Changes:
  • Updated management_router.py to include user access checks for document chunks and collection operations.
  • Modified list_collections_app() to restrict access to superusers.
• Miscellaneous:
  • Removed debug print statement from auth.py.
  • Adjusted WrappedAddUserResponse in responses.py to ResultsWrapper[None].

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

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to 5cc8079 in 29 seconds

More details

• Looked at 1047 lines of code in 6 files
• Skipped 0 files when reviewing.
• Skipped posting 7 drafted comments based on config settings.

1. py/tests/integration/harness_sdk.py:562

• Draft comment:
  Consider renaming the test function to test_user_overview to maintain consistency with the naming convention used in the rest of the file.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The test function names are inconsistent with the naming convention used in the rest of the file. They should follow the pattern of using underscores to separate words.

2. py/tests/integration/harness_sdk.py:586

• Draft comment:
  Consider renaming the test function to test_user_logout to maintain consistency with the naming convention used in the rest of the file.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The test function names are inconsistent with the naming convention used in the rest of the file. They should follow the pattern of using underscores to separate words.

3. py/tests/integration/harness_sdk.py:622

• Draft comment:
  Consider renaming the test function to test_user_creates_collection to maintain consistency with the naming convention used in the rest of the file.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The test function names are inconsistent with the naming convention used in the rest of the file. They should follow the pattern of using underscores to separate words.

4. py/tests/integration/harness_sdk.py:654

• Draft comment:
  Consider renaming the test function to test_user_updates_collection to maintain consistency with the naming convention used in the rest of the file.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The test function names are inconsistent with the naming convention used in the rest of the file. They should follow the pattern of using underscores to separate words.

5. py/tests/integration/harness_sdk.py:693

• Draft comment:
  Consider renaming the test function to test_user_lists_collections to maintain consistency with the naming convention used in the rest of the file.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The test function names are inconsistent with the naming convention used in the rest of the file. They should follow the pattern of using underscores to separate words.

6. py/tests/integration/harness_sdk.py:768

• Draft comment:
  Consider renaming the test function to test_user_collection_document_management to maintain consistency with the naming convention used in the rest of the file.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The test function names are inconsistent with the naming convention used in the rest of the file. They should follow the pattern of using underscores to separate words.

7. py/tests/integration/harness_sdk.py:840

• Draft comment:
  Consider renaming the test function to test_user_removes_document_from_collection to maintain consistency with the naming convention used in the rest of the file.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The test function names are inconsistent with the naming convention used in the rest of the file. They should follow the pattern of using underscores to separate words.

Workflow ID: wflow_7pDkFPBtbojWrq2K

---

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