search

chroma-core / chroma

the AI-native open-source embedding database
https://www.trychroma.com/
Apache License 2.0
13.36k stars 1.14k forks source link

[CLN] JavaScript Client Tests #2415

Open atroyn opened 4 days ago

atroyn commented 4 days ago

Our JS / TS client tests are currently pretty messy. There doesn't seem to be a clear way to run them locally (running bin/ts-integration-test results in failures which don't appear on the GitHub workers / in CI), and the functionality of the tests is distributed across many files.

Additionally, it does not seem that the JS tests generate a new JS client (see https://github.com/chroma-core/chroma/pull/2412), and they probably should for correct coverage.

It would be good to unify theses tests, make sure they are replicable locally and on CI.

atroyn commented 4 days ago

@AlabasterAxe we might want to take a look at this as part of the current milestone, since we will be modifying JS probably a lot.