Skip to content

Commit

Permalink
fix(synonyms): "custom_street" synonyms should be applied to name.* a…
Browse files Browse the repository at this point in the history
…nd phrase.* indices
  • Loading branch information
missinglink committed Sep 3, 2020
1 parent fd4dd97 commit f14061f
Show file tree
Hide file tree
Showing 3 changed files with 18 additions and 0 deletions.
6 changes: 6 additions & 0 deletions settings.js
Original file line number Diff line number Diff line change
Expand Up @@ -51,6 +51,8 @@ function generate(){
"lowercase",
"trim",
"synonyms/custom_name/multiword",
"synonyms/custom_street/multiword",
"synonyms/custom_admin/multiword",
"name_synonyms_multiplexer",
"icu_folding",
"remove_ordinals",
Expand Down Expand Up @@ -84,6 +86,8 @@ function generate(){
"trim",
"remove_duplicate_spaces",
"synonyms/custom_name/multiword",
"synonyms/custom_street/multiword",
"synonyms/custom_admin/multiword",
"name_synonyms_multiplexer",
"icu_folding",
"remove_ordinals",
Expand Down Expand Up @@ -156,6 +160,8 @@ function generate(){
"preserve_original": false,
"filters": [
"synonyms/custom_name",
"synonyms/custom_street",
"synonyms/custom_admin",
"synonyms/personal_titles",
"synonyms/place_names",
"synonyms/streets",
Expand Down
6 changes: 6 additions & 0 deletions test/fixtures/expected.json
Original file line number Diff line number Diff line change
Expand Up @@ -50,6 +50,8 @@
"lowercase",
"trim",
"synonyms/custom_name/multiword",
"synonyms/custom_street/multiword",
"synonyms/custom_admin/multiword",
"name_synonyms_multiplexer",
"icu_folding",
"remove_ordinals",
Expand Down Expand Up @@ -89,6 +91,8 @@
"trim",
"remove_duplicate_spaces",
"synonyms/custom_name/multiword",
"synonyms/custom_street/multiword",
"synonyms/custom_admin/multiword",
"name_synonyms_multiplexer",
"icu_folding",
"remove_ordinals",
Expand Down Expand Up @@ -172,6 +176,8 @@
"preserve_original": false,
"filters": [
"synonyms/custom_name",
"synonyms/custom_street",
"synonyms/custom_admin",
"synonyms/personal_titles",
"synonyms/place_names",
"synonyms/streets",
Expand Down
6 changes: 6 additions & 0 deletions test/settings.js
Original file line number Diff line number Diff line change
Expand Up @@ -105,6 +105,8 @@ module.exports.tests.nameSynonymsMultiplexerFilter = function (test, common) {
t.equal(filter.type, 'multiplexer');
t.deepEqual(filter.filters, [
'synonyms/custom_name',
'synonyms/custom_street',
'synonyms/custom_admin',
'synonyms/personal_titles',
'synonyms/place_names',
'synonyms/streets',
Expand Down Expand Up @@ -132,6 +134,8 @@ module.exports.tests.peliasIndexOneEdgeGramAnalyzer = function(test, common) {
"lowercase",
"trim",
"synonyms/custom_name/multiword",
"synonyms/custom_street/multiword",
"synonyms/custom_admin/multiword",
"name_synonyms_multiplexer",
"icu_folding",
"remove_ordinals",
Expand Down Expand Up @@ -189,6 +193,8 @@ module.exports.tests.peliasPhraseAnalyzer = function(test, common) {
"trim",
"remove_duplicate_spaces",
"synonyms/custom_name/multiword",
"synonyms/custom_street/multiword",
"synonyms/custom_admin/multiword",
"name_synonyms_multiplexer",
"icu_folding",
"remove_ordinals",
Expand Down

0 comments on commit f14061f

Please sign in to comment.