Skip to content

test(synonym-sets): cover item access through the items property - #126

Merged
tharropoulos merged 1 commit into
typesense:masterfrom
tharropoulos:test/synonym-set-items
Nov 9, 2026
Merged

tharropoulos merged 1 commit into
typesense:masterfrom
tharropoulos:test/synonym-set-items

Conversation

@tharropoulos

Copy link
Copy Markdown
Collaborator
  • switch SynonymSetItemsTest from getItems() to ->items, the form the 30.2 docs use
  • drop id from the upsert body since the item path already sets it
  • assert that retrieving an item after delete throws ObjectNotFound
  • add testGetItemsReturnsTheItemsProperty to keep getItems() and ->items pointing at the same object

@tharropoulos
tharropoulos merged commit 3feb6e1 into typesense:master Nov 9, 2026
1 check passed
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.

1 participant