fix worldbank fallback by raising a ValueError instead of a RuntimeError

Author: emanuel-schmid

climada/test/test_litpop_integr.py

@@ -53,7 +53,9 @@ def test_netherlands150_pass(self):
         self.assertEqual(ent.gdf.shape[0], 2829)
 
     def test_BLM150_pass(self):
-        """Test from_countries for BLM at 150 arcsec, 2 data points"""
+        """Test from_countries for BLM at 150 arcsec, 2 data points
+        The world bank doesn't provide data for Saint Barthélemy, fall back to natearth
+        """
         ent = lp.LitPop.from_countries("BLM", res_arcsec=150, reference_year=2016)
         self.assertEqual(ent.gdf.shape[0], 2)
 

climada/util/finance.py

@@ -216,7 +216,7 @@ def download_world_bank_indicator(
         # Check if we received an error message
         try:
             if json_data[0]["message"][0]["id"] == "120":
-                raise RuntimeError(
+                raise ValueError(
                     "Error requesting data from the World Bank API. Did you use the "
                     "correct country code and indicator ID?"
                 )