Fix CLI Tests

[NolanTrem]

🚀	This description was created by Ellipsis for commit 807988f

Summary:

This PR fixes CLI tests, renames a flag, and enhances test coverage, improving robustness and handling of None values.

Key points:

• Fixes CLI tests and adds comprehensive test coverage.
• Renames flag do-hybrid-search to use-hybrid-search in py/cli/commands/retrieval.py and other files.
• Handles None values in JsonParamType.convert.
• Updates py/core/examples/hello_r2r.py for consistent output formatting.
• Creates new test files for CLI commands.
• Adds tests for CLI group and command options.
• Adds tests for Docker utility functions.
• Adds tests for JSON parameter type.
• Adds tests for timer utility.
• Removes old CLI test file.
• Updates tests for knowledge graph extraction and clustering.

---

Generated with ❤️ by ellipsis.dev

[ellipsis-dev]

❌ Changes requested. Reviewed everything up to f84da73 in 21 seconds

More details

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

Workflow ID: wflow_jRTgkS9DsGb7FQhQ

---

Want Ellipsis to fix these issues? Tag @ellipsis-dev in a comment. You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

[ellipsis-dev]

The function test_documents_overview_with_document_id is defined twice. Consider renaming or removing the duplicate function to avoid redundancy.

[ellipsis-dev]

Reset cli.callback to original_callback instead of None to maintain consistency for other tests.

[ellipsis-dev]

👍 Looks good to me! Incremental review on e639c8a in 49 seconds

More details

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

1. go/cli/rag_operations.go:51

• Draft comment:
  Ensure that the flag name change from do-hybrid-search to use-hybrid-search is consistently applied across all files and references to avoid runtime errors.
• Reason this comment was not posted:
  Decided after close inspection that this draft comment was likely wrong and/or not actionable:
  The comment violates the rule of not asking the PR author to verify changes across all files, as it is speculative and not actionable within the context of the current diff. The change is consistent within the file, and the comment does not point out a specific issue within the diff itself.
  I might be missing the possibility that the change could affect other parts of the codebase not shown in the diff, but the comment is still speculative and not actionable within the current context.
  The comment is speculative and not actionable within the current diff, so it should be removed regardless of potential issues outside the diff.
  Remove the comment as it is speculative and not actionable within the context of the current diff.

2. go/sdk/pkg/sdk/retrieval.go:42

• Draft comment:
  Ensure that the flag name change from do_hybrid_search to use_hybrid_search is consistently applied across all files and references to avoid runtime errors.
• Reason this comment was not posted:
  Marked as duplicate.

3. js/sdk/README.md:98

• Draft comment:
  Ensure that the flag name change from do_hybrid_search to use_hybrid_search is consistently applied across all files and references to avoid runtime errors.
• Reason this comment was not posted:
  Marked as duplicate.

4. js/sdk/src/models.tsx:40

• Draft comment:
  Ensure that the flag name change from do_hybrid_search to use_hybrid_search is consistently applied across all files and references to avoid runtime errors.
• Reason this comment was not posted:
  Marked as duplicate.

5. py/cli/commands/retrieval.py:26

• Draft comment:
  Ensure that the flag name change from do_hybrid_search to use_hybrid_search is consistently applied across all files and references to avoid runtime errors.
• Reason this comment was not posted:
  Marked as duplicate.

6. py/core/examples/hello_r2r.py:12

• Draft comment:
  Ensure that the flag name change from do_hybrid_search to use_hybrid_search is consistently applied across all files and references to avoid runtime errors.
• Reason this comment was not posted:
  Marked as duplicate.

7. py/core/main/services/ingestion_service.py:107

• Draft comment:
  Ensure that the flag name change from do_hybrid_search to use_hybrid_search is consistently applied across all files and references to avoid runtime errors.
• Reason this comment was not posted:
  Marked as duplicate.

Workflow ID: wflow_Ku6NDBRZkqupZHu1

---

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

[ellipsis-dev]

❌ Changes requested. Incremental review on 807988f in 22 seconds

More details

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

1. py/tests/cli/commands/test_restructure.py:50

• Draft comment:
  The function name test_enrich_graph is not descriptive enough. Consider renaming it to something more specific, like test_enrich_graph_execution or test_enrich_graph_time_taken.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The test function name test_enrich_graph is misleading because it doesn't specify what aspect of the graph enrichment is being tested. It should be more descriptive.

Workflow ID: wflow_ZvDIaoqzJjWzVLlk

---

Want Ellipsis to fix these issues? Tag @ellipsis-dev in a comment. You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

[ellipsis-dev]

The function name test_documents_overview_without_document_id is misleading as it actually tests the scenario with a document ID. Consider renaming it to test_documents_overview_with_document_id.