snowykami/meilisearch
1
0
Fork 0
mirror of https://github.com/meilisearch/meilisearch.git synced 2024-11-26 12:05:05 +08:00
Code Issues Actions 5 Packages Projects Releases Wiki Activity

Merge #4685

Browse Source 
4685: Fix ci tests r=dureuill a=ManyTheFish

# Pull Request
Make the all following CI succeed:
https://github.com/meilisearch/meilisearch/actions/runs/9477183091

## Related issue
Fixes #4629

## What does this PR do?
- Change the test behavior for `swedish-recomposition` feature flag
- Remove the `-v` parameter from grep

Co-authored-by: ManyTheFish <many@meilisearch.com>
Co-authored-by: Many the fish <many@meilisearch.com>
This commit is contained in:
meili-bors[bot] 2024-06-12 07:58:33 +00:00 committed by GitHub
commit e0eff08095
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
3 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
.github/workflows/test-suite.yml vendored
View File

@ -116,7 +116,7 @@ jobs:
override: true override: true
- name: Run cargo tree without default features and check lindera is not present - name: Run cargo tree without default features and check lindera is not present
run: | run: |
if cargo tree -f '{p} {f}' -e normal --no-default-features | grep -vqz lindera; then if cargo tree -f '{p} {f}' -e normal --no-default-features | grep -qz lindera; then
echo "lindera has been found in the sources and it shouldn't" echo "lindera has been found in the sources and it shouldn't"
exit 1 exit 1
fi fi

2
meilisearch/tests/search/mod.rs
View File

@ -301,7 +301,7 @@ async fn negative_special_cases_search() {
index.add_documents(documents, None).await; index.add_documents(documents, None).await;
index.wait_task(0).await; index.wait_task(0).await;
index.update_settings(json!({"synonyms": { "escape": ["glass"] }})).await; index.update_settings(json!({"synonyms": { "escape": ["gläss"] }})).await;
index.wait_task(1).await; index.wait_task(1).await;
// There is a synonym for escape -> glass but we don't want "escape", only the derivates: glass // There is a synonym for escape -> glass but we don't want "escape", only the derivates: glass

5
milli/src/search/new/tests/stop_words.rs
View File

@ -13,7 +13,7 @@ use std::collections::BTreeSet;
use std::iter::FromIterator; use std::iter::FromIterator;
use crate::index::tests::TempIndex; use crate::index::tests::TempIndex;
use crate::{db_snap, Search, SearchResult, TermsMatchingStrategy}; use crate::{Search, SearchResult, TermsMatchingStrategy};
fn create_index() -> TempIndex { fn create_index() -> TempIndex {
let index = TempIndex::new(); let index = TempIndex::new();
@ -66,9 +66,10 @@ fn create_index() -> TempIndex {
} }
#[test] #[test]
#[cfg(not(feature = "swedish-recomposition"))]
fn test_stop_words_not_indexed() { fn test_stop_words_not_indexed() {
let index = create_index(); let index = create_index();
db_snap!(index, word_docids, @"6288f9d7db3703b02c57025eb4a69264"); crate::db_snap!(index, word_docids, @"6288f9d7db3703b02c57025eb4a69264");
} }
#[test] #[test]