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