Skip to content

Conversation

yh-0
Copy link
Contributor

@yh-0 yh-0 commented Jul 14, 2025

https://issues.hibernatingrhinos.com/issue/RDBC-782/Python-client-various-tests-2

Part of the tests that I was able to write. note that some of them don't give expected results.

@poissoncorp
Copy link
Contributor

Please add note which new tests are passing and which need more work

@yh-0 yh-0 changed the title Various tests #2 Various tests Jul 16, 2025
@yh-0
Copy link
Contributor Author

yh-0 commented Jul 16, 2025

Passing:

  • test_can_cache_document_with_includes
  • test_can_wait_for_index_after_patch
  • test_query_create_clauses_for_query_dynamically_with_on_before_query_event
  • test_can_sort_files
  • test_after_query_executed_should_be_executed_only_once
  • test_can_get_revisions_by_date
  • test_can_persist_pattern_for_output_reduce_to_collection_references
  • test_can_use_dynamic_query_order_by_spatial_with_alias
  • test_can_get_distance_from_spatial_query
  • test_can_update_metadata_with_nested_dictionary
  • test_can_update_metadata_with_nested_dictionary_same_session
  • test_delete_document_and_revisions
  • test_request_with_timeout_should_work
  • test_can_load_entities_with_no_tracking
  • test_can_run_spatial_queries_in_memory

Not passing:

  • test_plain_revisions_subscriptions_compare_docs
  • test_session_in_subscriptions_should_not_track_revisions
  • test_will_call_on_before_delete_when_calling_delete_by_id
  • test_wait_for_indexes_after_save_changes_can_exit_when_throw_on_timeout_is_false
  • test_can_use_on_session_disposing_event
  • test_request_timeout_should_work
  • test_search_with_highlight

All of the tests that don't pass have comments with either exceptions they throw, or short explanations of what went wrong.

Other tests that are in the ticket, but not in this pr, don't work because of either non functioning event listeners (addBeforeQueryListener and similar), or they lack exception types.

@poissoncorp
Copy link
Contributor

Let's remove the tests that don't pass yet from this PR. We'll include them in second phase. Mark them as unresolved on YouTrack issue @yh-0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants