X-Git-Url: https://git.openstreetmap.org./nominatim.git/blobdiff_plain/8413075249e1bb2832df4edd0f66d61f77fb9f99..9025b0ec78bd770e74a1bed85c8d2e43ce5f571a:/test/python/test_tokenizer_icu_name_processor.py?ds=sidebyside diff --git a/test/python/test_tokenizer_icu_name_processor.py b/test/python/test_tokenizer_icu_name_processor.py index 9c09bccc..cc103116 100644 --- a/test/python/test_tokenizer_icu_name_processor.py +++ b/test/python/test_tokenizer_icu_name_processor.py @@ -12,21 +12,23 @@ from nominatim.errors import UsageError @pytest.fixture def cfgfile(tmp_path, suffix='.yaml'): - def _create_config(suffixes, abbr): + def _create_config(*variants, **kwargs): content = dedent("""\ normalization: - ":: NFD ()" + - "'🜳' > ' '" - "[[:Nonspacing Mark:] [:Cf:]] >" - ":: lower ()" - "[[:Punctuation:][:Space:]]+ > ' '" - ":: NFC ()" transliteration: - ":: Latin ()" + - "'🜵' > ' '" """) - content += "compound_suffixes:\n" - content += '\n'.join((" - " + s for s in suffixes)) + '\n' - content += "abbreviations:\n" - content += '\n'.join((" - " + s for s in abbr)) + '\n' + content += "variants:\n - words:\n" + content += '\n'.join((" - " + s for s in variants)) + '\n' + for k, v in kwargs: + content += " {}: {}\n".format(k, v) fpath = tmp_path / ('test_config' + suffix) fpath.write_text(dedent(content)) return fpath @@ -34,27 +36,69 @@ def cfgfile(tmp_path, suffix='.yaml'): return _create_config -def test_simple_variants(cfgfile): - fpath = cfgfile(['strasse', 'straße', 'weg'], - ['strasse,straße => str', - 'prospekt => pr']) +def get_normalized_variants(proc, name): + return proc.get_variants_ascii(proc.get_normalized(name)) + + +def test_variants_empty(cfgfile): + fpath = cfgfile('saint -> 🜵', 'street -> st') rules = ICUNameProcessorRules(loader=ICURuleLoader(fpath)) proc = ICUNameProcessor(rules) - assert set(proc.get_normalized_variants("Bauwegstraße")) \ - == {'bauweg straße', 'bauweg str'} - assert proc.get_normalized_variants("Bauwegstr") == ['bauweg str'] - assert proc.get_normalized_variants("holzweg") == ['holz weg'] - assert proc.get_normalized_variants("hallo") == ['hallo'] + assert get_normalized_variants(proc, '🜵') == [] + assert get_normalized_variants(proc, '🜳') == [] + assert get_normalized_variants(proc, 'saint') == ['saint'] + + +VARIANT_TESTS = [ +(('~strasse,~straße -> str', '~weg => weg'), "hallo", {'hallo'}), +(('weg => wg',), "holzweg", {'holzweg'}), +(('weg -> wg',), "holzweg", {'holzweg'}), +(('~weg => weg',), "holzweg", {'holz weg', 'holzweg'}), +(('~weg -> weg',), "holzweg", {'holz weg', 'holzweg'}), +(('~weg => w',), "holzweg", {'holz w', 'holzw'}), +(('~weg -> w',), "holzweg", {'holz weg', 'holzweg', 'holz w', 'holzw'}), +(('~weg => weg',), "Meier Weg", {'meier weg', 'meierweg'}), +(('~weg -> weg',), "Meier Weg", {'meier weg', 'meierweg'}), +(('~weg => w',), "Meier Weg", {'meier w', 'meierw'}), +(('~weg -> w',), "Meier Weg", {'meier weg', 'meierweg', 'meier w', 'meierw'}), +(('weg => wg',), "Meier Weg", {'meier wg'}), +(('weg -> wg',), "Meier Weg", {'meier weg', 'meier wg'}), +(('~strasse,~straße -> str', '~weg => weg'), "Bauwegstraße", + {'bauweg straße', 'bauweg str', 'bauwegstraße', 'bauwegstr'}), +(('am => a', 'bach => b'), "am bach", {'a b'}), +(('am => a', '~bach => b'), "am bach", {'a b'}), +(('am -> a', '~bach -> b'), "am bach", {'am bach', 'a bach', 'am b', 'a b'}), +(('am -> a', '~bach -> b'), "ambach", {'ambach', 'am bach', 'amb', 'am b'}), +(('saint -> s,st', 'street -> st'), "Saint Johns Street", + {'saint johns street', 's johns street', 'st johns street', + 'saint johns st', 's johns st', 'st johns st'}), +(('river$ -> r',), "River Bend Road", {'river bend road'}), +(('river$ -> r',), "Bent River", {'bent river', 'bent r'}), +(('^north => n',), "North 2nd Street", {'n 2nd street'}), +(('^north => n',), "Airport North", {'airport north'}), +(('am -> a',), "am am am am am am am am", {'am am am am am am am am'}), +(('am => a',), "am am am am am am am am", {'a a a a a a a a'}) +] + +@pytest.mark.parametrize("rules,name,variants", VARIANT_TESTS) +def test_variants(cfgfile, rules, name, variants): + fpath = cfgfile(*rules) + proc = ICUNameProcessor(ICUNameProcessorRules(loader=ICURuleLoader(fpath))) + + result = get_normalized_variants(proc, name) + + assert len(result) == len(set(result)) + assert set(get_normalized_variants(proc, name)) == variants -def test_multiple_replacements(cfgfile): - fpath = cfgfile([], ['saint => s,st', 'street => st']) +def test_search_normalized(cfgfile): + fpath = cfgfile('~street => s,st', 'master => mstr') rules = ICUNameProcessorRules(loader=ICURuleLoader(fpath)) proc = ICUNameProcessor(rules) - assert set(proc.get_normalized_variants("Saint Johns Street")) == \ - {'saint johns street', 's johns street', 'st johns street', - 'saint johns st', 's johns st', 'st johns st'} + assert proc.get_search_normalized('Master Street') == 'master street' + assert proc.get_search_normalized('Earnes St') == 'earnes st' + assert proc.get_search_normalized('Nostreet') == 'nostreet'