-
Notifications
You must be signed in to change notification settings - Fork 52
Open
Labels
Typesense backendRelates to the Typesense backend / search engine.Relates to the Typesense backend / search engine.help wantedReady to be picked up by anyone in the communityReady to be picked up by anyone in the community
Description
The new Typesense backend was not originally planned to merge, and we were requested to develop it without tests for now.
It has been included without unit tests for now as the budget is tight, there is limited value in unit tests compared to integration tests, Typesense integration tests would be a bigger project, all the common edx-search functionality is already unit-tested, and Typesense support is still somewhat experimental.
However, we can and should add both unit and integration tests for the Typesense backend.
It may make sense to do this after or together with #245 (re-architecting this repo).
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
Typesense backendRelates to the Typesense backend / search engine.Relates to the Typesense backend / search engine.help wantedReady to be picked up by anyone in the communityReady to be picked up by anyone in the community