-
-
Notifications
You must be signed in to change notification settings - Fork 112
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: Fix unit tests in CI on pg_bm25
and pg_search
#588
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Ignored Deployment
|
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## dev #588 +/- ##
==========================================
- Coverage 81.74% 81.44% -0.30%
==========================================
Files 28 30 +2
Lines 3664 3703 +39
==========================================
+ Hits 2995 3016 +21
- Misses 669 687 +18
|
pg_bm25
and pg_search
Looks good! It sounds like we'll rely exclusively on Are we running |
We're just running it on the extensions currently, but I believe that includes the test in shared? |
I do not think we should use |
Ticket(s) Closed
What
There are two issues with our tests:
pg_bm25
, the Rust cache pulls broken extensions and they try to update, causing a recursive loop. This should fix it.pg_search
, some of the new functions and unit tests did not usepg_test
, but used regulartest
. This fixes it.Why
Broken tests
How
^
Tests
See CI