1712n / dn-institute

Distributed Networks Institute
http://dn.institute/
The Unlicense
29 stars 69 forks source link

Integration tests for Similarity Search API. #450

Open kol3x opened 5 months ago

kol3x commented 5 months ago

Closes #430

Hello! I am glad to take part in your challenge program. I started with the easiest one, but I plan to try harder ones later.

My process was:

I ended up making two new tests out of an authentication test that was already there, to cover all cases. Plus I added tests for validation and functionality.

One more thing to consider is that we are currently using vitest 1.2, but the docs mention that they only support 1.5. image I haven't upgraded the package, as I wanted to consult first.

kol3x commented 5 months ago

@Lavriz Let me know your thoughts on this

Lavriz commented 4 months ago

@Lavriz Let me know your thoughts on this

hey! sorry for the delay, you're absolutely right on upgrading the version! thanks for noticing it

kol3x commented 4 months ago

@Lavriz Let me know your thoughts on this

hey! sorry for the delay, you're absolutely right on upgrading the version! thanks for noticing it

I included the vitest upgrade, let me know if there's anything else needed

kol3x commented 4 months ago

Putting this on draft for now, as Evgeny pointed out that additional tests for vectorization and vector DB querying should be added. I've researched Cloudflare docs on how to properly test auxiliary workers, but no success yet.

kol3x commented 4 months ago
kol3x commented 4 months ago

Figured out how workers mocks work, now tests actually pass... image