Merge pull request #1132 from Hirurg103/master

Now result.country doesn't fail if result is not in a country

lib/geocoder/results/yandex.rb

@@ -25,11 +25,19 @@ module Geocoder::Result
     end
 
     def country
-      address_details['CountryName']
+      if address_details
+        address_details['CountryName']
+      else
+        ""
+      end
     end
 
     def country_code
-      address_details['CountryNameCode']
+      if address_details
+        address_details['CountryNameCode']
+      else
+        ""
+      end
     end
 
     def state

test/fixtures/yandex_black_sea

+{
+  "response": {
+    "GeoObjectCollection": {
+      "metaDataProperty": {
+        "GeocoderResponseMetaData": {
+          "request": "32,43",
+          "found": "2",
+          "results": "10",
+          "Point": {
+            "pos": "32.000000 43.000000"
+          }
+        }
+      },
+      "featureMember": [
+        {
+          "GeoObject": {
+            "metaDataProperty": {
+              "GeocoderMetaData": {
+                "kind": "hydro",
+                "text": "Black Sea",
+                "precision": "other",
+                "AddressDetails": {
+                  "Locality": {
+                    "Premise": {
+                      "PremiseName": "Black Sea"
+                    }
+                  }
+                }
+              }
+            },
+            "name": "Black Sea",
+            "boundedBy": {
+              "Envelope": {
+                "lowerCorner": "27.442409 40.908662",
+                "upperCorner": "41.777787 46.627275"
+              }
+            },
+            "Point": {
+              "pos": "34.188281 43.229215"
+            }
+          }
+        },
+        {
+          "GeoObject": {
+            "metaDataProperty": {
+              "GeocoderMetaData": {
+                "kind": "hydro",
+                "text": "Black Sea",
+                "precision": "other",
+                "AddressDetails": {
+                  "Locality": {
+                    "Premise": {
+                      "PremiseName": "Black Sea"
+                    }
+                  }
+                }
+              }
+            },
+            "name": "Black Sea",
+            "boundedBy": {
+              "Envelope": {
+                "lowerCorner": "27.628351 40.908893",
+                "upperCorner": "41.777796 46.627999"
+              }
+            },
+            "Point": {
+              "pos": "33.383633 44.235582"
+            }
+          }
+        }
+      ]
+    }
+  }
+}

test/unit/lookups/yandex_test.rb

@@ -14,4 +14,9 @@ class YandexTest < GeocoderTestCase
       result.viewport
   end
 
+  def test_yandex_empty_results
+    result = Geocoder.search('black sea').first
+    assert_equal "", result.country_code
+    assert_equal "", result.country
+  end
 end