OCA / partner-contact

98%

9.0: 95%

LAST BUILD ON BRANCH 12.0-ocabot-merge-pr-757-by-pedrobaeza-bump-patch

branch: 12.0-ocabot-merge-pr-757-by-pedrobaeza-bump-patch

CHANGE BRANCH

x

Reset

• 12.0-ocabot-merge-pr-757-by-pedrobaeza-bump-patch
• 10.0
• 10.0-ocabot-merge-pr-498-by-hbrunn-bump-no
• 10.0-ocabot-merge-pr-508-by-hbrunn-bump-no
• 10.0-ocabot-merge-pr-519-by-NL66278-bump-patch
• 10.0-ocabot-merge-pr-528-by-NL66278-bump-minor
• 10.0-ocabot-merge-pr-751-by-rafaelbn-bump-no
• 10.0-ocabot-merge-pr-770-by-pedrobaeza-bump-patch
• 10.0-ocabot-merge-pr-782-by-pedrobaeza-bump-patch
• 10.0-ocabot-merge-pr-795-by-pedrobaeza-bump-patch
• 10.0-partner_multi_relation-relation_all-write_multiple
• 11.0
• 11.0-mix-content
• 11.0-ocabot-merge-pr-524-by-sebastienbeau-bump-no
• 11.0-ocabot-merge-pr-771-by-hbrunn-bump-patch
• 11.0-ocabot-merge-pr-777-by-pedrobaeza-bump-patch
• 11.0-ocabot-merge-pr-779-by-NL66278-bump-major
• 12.0
• 12.0-ocabot-merge-pr-742-by-pedrobaeza-bump-no
• 12.0-ocabot-merge-pr-755-by-max3903-bump-no
• 12.0-ocabot-merge-pr-763-by-NL66278-bump-no
• 12.0-ocabot-merge-pr-768-by-Yajo-bump-no
• 12.0-ocabot-merge-pr-769-by-pedrobaeza-bump-no
• 12.0-ocabot-merge-pr-772-by-rafaelbn-bump-no
• 12.0-ocabot-merge-pr-773-by-pedrobaeza-bump-minor
• 12.0-ocabot-merge-pr-775-by-rousseldenis-bump-no
• 12.0-ocabot-merge-pr-778-by-pedrobaeza-bump-patch
• 12.0-ocabot-merge-pr-807-by-pedrobaeza-bump-no
• 12.0-ocabot-merge-pr-820-by-pedrobaeza-bump-no
• 12.0-ocabot-merge-pr-836-by-NL66278-bump-minor
• 12.0-ocabot-merge-pr-837-by-NL66278-bump-patch
• 12.0-ocabot-merge-pr-839-by-NL66278-bump-minor
• 13.0
• 13.0-ocabot-merge-pr-785-by-pedrobaeza-bump-no
• 13.0-ocabot-merge-pr-785-by-sbidoul-bump-no
• 13.0-ocabot-merge-pr-786-by-pedrobaeza-bump-no
• 13.0-ocabot-merge-pr-788-by-NL66278-bump-no
• 13.0-ocabot-merge-pr-792-by-NL66278-bump-no
• 13.0-ocabot-merge-pr-793-by-pedrobaeza-bump-no
• 13.0-ocabot-merge-pr-801-by-NL66278-bump-no
• 13.0-ocabot-merge-pr-809-by-rousseldenis-bump-patch
• 13.0-ocabot-merge-pr-811-by-NL66278-bump-no
• 13.0-ocabot-merge-pr-814-by-pedrobaeza-bump-no
• 13.0-ocabot-merge-pr-816-by-sebastienbeau-bump-no
• 13.0-ocabot-merge-pr-818-by-pedrobaeza-bump-no
• 13.0-ocabot-merge-pr-821-by-pedrobaeza-bump-no
• 13.0-ocabot-merge-pr-830-by-NL66278-bump-patch
• 7.0
• 7.0-street3
• 8.0
• 8.0-base_location_geonames_import-imp
• 8.0-base_location_geonames_import-readme
• 9.0
• 9.0-partner_contact_nutrition_goal
• fix/partner_contact_street/8.0/unintentional_street_write
• sbidoul-patch-1
• sbidoul-patch-2

Build # 4090

Build Type

push

travis-ci

Committed by OCA-git-bot

Commit Message

[FIX] base_location_geonames_import: Handle duplicated city names

If there are several cities with the same name in different states, previous code doesn't
handle correctly this situation.

We amend this storing in the cached dictionary both city name and state.

Includes a test for checking this specific condition, got from real data in US.

Fixes #749

Run Details

11 of 11 new or added lines in 1 file covered. (100.0%)

1658 of 1696 relevant lines covered (97.76%)

1.96 hits per line