weathergrabber 0.0.9a0__tar.gz → 0.0.10__tar.gz

{weathergrabber-0.0.9a0/weathergrabber.egg-info → weathergrabber-0.0.10}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: weathergrabber
-Version: 0.0.9a0
+Version: 0.0.10
 Summary: A grabber for weather.com data with various output formats.
 Author-email: Carlos Anselmo Mendes Junior <cjuniorfox@gmail.com>
 License: MIT

{weathergrabber-0.0.9a0 → weathergrabber-0.0.10}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "weathergrabber"
-version = "0.0.9a"
+version = "0.0.10"
 description = "A grabber for weather.com data with various output formats."
 readme = "README.md"
 requires-python = ">=3.12"

{weathergrabber-0.0.9a0 → weathergrabber-0.0.10}/tests/test_core.py

@@ -45,3 +45,34 @@ def test_main_sets_log_level(mock_app):
         cache_statistics=False,
     )
     assert logging.getLogger().level == logging.DEBUG
+
+
+@pytest.mark.parametrize("log_level, expect_trace", [
+    ("debug", True),
+    ("info", False),
+])
+@patch('weathergrabber.core.WeatherGrabberApplication')
+def test_main_logs_stack_trace_conditionally(mock_app, caplog, log_level, expect_trace):
+    mock_app.side_effect = RuntimeError("boom")
+    from weathergrabber.core import main
+
+    caplog.set_level(logging.DEBUG)
+
+    with pytest.raises(SystemExit) as excinfo:
+        main(
+            log_level=log_level,
+            location_name='Berlin',
+            location_id='789',
+            lang='de-DE',
+            output='console',
+            keep_open=False,
+            icons='emoji',
+            force_cache=False,
+            cache_statistics=False,
+        )
+
+    assert excinfo.value.code == 1
+
+    error_records = [r for r in caplog.records if r.levelno >= logging.ERROR]
+    assert error_records, "Expected at least one error log record"
+    assert any(rec.exc_info for rec in error_records) == expect_trace

{weathergrabber-0.0.9a0 → weathergrabber-0.0.10}/weathergrabber/__init__.py

@@ -4,7 +4,7 @@ from .core import main
 from .cli import main_cli
 
 __all__ = ["main", "main_cli"]
-__version__ = "0.0.9a"
+__version__ = "0.0.10"
 
 def get_version():
     return __version__

{weathergrabber-0.0.9a0 → weathergrabber-0.0.10}/weathergrabber/application/services/search_location_service.py

@@ -13,13 +13,9 @@ class SearchLocationService:
         if not location_name:
             self.logger.debug("No location name provided. Bypassing search.")
             return None
+        data = self.api.search(location_name, lang)
         
         try:
-            data = self.api.search(location_name, lang)
-            if not data:
-                self.logger.error(f"No data found for location: {location_name}")
-                raise ValueError(f"Location '{location_name}' not found.")
-            
             dal = data["dal"]["getSunV3LocationSearchUrlConfig"]
 
             # Pick the first (arbitrary) key, then get the value
@@ -29,7 +25,7 @@ class SearchLocationService:
             self.logger.debug(f"Found location ID: {location_id} for location name: {location_name}")
             
             return location_id
-        except Exception as e:
-            self.logger.error(f"Error searching for location '{location_name}': {e}")
+        except KeyError as e:
+            self.logger.debug(f"Error searching for location '{location_name}': {e}")
             raise ValueError(f"Could not find location '{location_name}'.")
 

{weathergrabber-0.0.9a0 → weathergrabber-0.0.10}/weathergrabber/application/usecases/weather_forecast_uc.py

@@ -68,10 +68,10 @@ class WeatherForecastUC:
         
         try:
             location_id = self._resolve_location_id(params)
+            weather_data = self.read_weather_service.execute(params.language, location_id)
         except ConnectionError as e:
+            self.logger.debug("A connection error occurred while fetching weather data. Trying to retrieve from cache.")
             return self.retrieve_forecast_from_cache_service.execute(params)
-        
-        weather_data = self.read_weather_service.execute(params.language, location_id)
 
         current_conditions = self.extract_current_conditions_service.execute(weather_data)
         today_details = self.extract_today_details_service.execute(weather_data)

{weathergrabber-0.0.9a0 → weathergrabber-0.0.10}/weathergrabber/cli.py

@@ -16,9 +16,9 @@ def main_cli():
     parser.add_argument("--version", "-v", action='version', version=f'Weathergrabber {weathergrabber.get_version()}', help="Show version and exit")
     parser.add_argument(
         "--log",
-        default="critical",
+        default="ERROR",
         choices=["debug", "info", "warning", "error", "critical", "DEBUG", "INFO", "WARNING", "ERROR", "CRITICAL"],
-        help="Set the logging level (default: CRITICAL)"
+        help="Set the logging level (default: ERROR)"
     )
     args = parser.parse_args()
 

{weathergrabber-0.0.9a0 → weathergrabber-0.0.10}/weathergrabber/core.py

@@ -32,4 +32,12 @@ def main(log_level: str, location_name: str, location_id: str, lang: str, output
         cache_statistics=cache_statistics,
     )
 
-    app = WeatherGrabberApplication(params)
+    try:
+        app = WeatherGrabberApplication(params)
+    except Exception as e:
+        logger = logging.getLogger()
+        if logger.level == logging.DEBUG:
+            logger.error(f"{e}", exc_info=True)
+        else:
+            logger.error(f"{e}")
+        raise SystemExit(1)

{weathergrabber-0.0.9a0 → weathergrabber-0.0.10}/weathergrabber/domain/entities/weather_icon_enum.py

@@ -47,6 +47,7 @@ class WeatherIconEnum(Enum):
     # Suggestions from Copilot
     BLIZZARD = ('blizzard', '\u2744', '🌨️')
     DUST = ('dust', '\uf063', '🌪️')
+    BLOWING_DUST_SANDSTORM = ('blowing-dust-sandstorm', '\uf063', '🌪️')
     FLURRIES = ('flurries', '\u2744', '🌨️')
     FREEZING_DRIZZLE = ('freezing-drizzle', '\uf0e9', '🌧️')
     FREEZING_DRIZZLE_RAIN = ('freezing-drizzle-rain', '\uf0e9', '🌧️')

{weathergrabber-0.0.9a0 → weathergrabber-0.0.10/weathergrabber.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: weathergrabber
-Version: 0.0.9a0
+Version: 0.0.10
 Summary: A grabber for weather.com data with various output formats.
 Author-email: Carlos Anselmo Mendes Junior <cjuniorfox@gmail.com>
 License: MIT