Merge branch '2025-testtips'

Author: egges

2025/testtips/pyproject.toml

@@ -0,0 +1,16 @@
+[project]
+name = "testtips"
+version = "0.0.1"
+authors = [{name = "ArjanCodes"}]
+license = {text = "MIT"}
+requires-python = ">=3.12"
+dependencies = [
+    "httpx>=0.28.1",
+    "hypothesis>=6.135.26",
+    "pytest>=8.4.1",
+    "python-dotenv>=1.1.1",
+]
+
+[tool.pytest.ini_options]
+pythonpath = "."
+

2025/testtips/tests/test_tips.py

@@ -0,0 +1,73 @@
+# tests/test_weather.py
+
+import sys
+
+import pytest
+from weather import WeatherService
+
+# from hypothesis import given
+# from hypothesis.strategies import floats
+
+
+# ✅ Parametrization
+@pytest.mark.parametrize(
+    "city,expected_temp", [("London", 15), ("Berlin", 20), ("Paris", 17)]
+)
+def test_get_temperature_multiple_cities(monkeypatch, city, expected_temp):
+    def fake_get(url, params):
+        class FakeResponse:
+            def raise_for_status(self):
+                pass
+
+            def json(self):
+                return {"current": {"temp_c": expected_temp}}
+
+        return FakeResponse()
+
+    monkeypatch.setattr("weather.httpx.get", fake_get)
+
+    service = WeatherService(api_key="fake-key")
+    temp = service.get_temperature(city)
+
+    assert temp == expected_temp
+
+
+# ✅ pytest.raises
+def test_get_temperature_raises_http_error(monkeypatch):
+    def fake_get(url, params):
+        class FakeResponse:
+            def raise_for_status(self):
+                raise Exception("API error")
+
+        return FakeResponse()
+
+    monkeypatch.setattr("weather.httpx.get", fake_get)
+
+    service = WeatherService(api_key="fake-key")
+
+    with pytest.raises(Exception):
+        service.get_temperature("Tokyo")
+
+
+# ✅ pytest.mark.skip
+@pytest.mark.skip(reason="Skipping this test for demonstration purposes.")
+def test_skipped_example():
+    assert False
+
+
+# ✅ pytest.mark.skipif
+@pytest.mark.skipif(sys.platform == "win32", reason="Does not run on Windows")
+def test_only_runs_on_non_windows():
+    assert True
+
+
+# ✅ pytest.mark.xfail
+@pytest.mark.xfail(reason="Known bug: API sometimes returns wrong temperature")
+def test_expected_failure_example():
+    assert 2 + 2 == 5
+
+
+# ✅ Hypothesis property-based test
+# @given(floats(min_value=-50, max_value=50))
+# def test_temperature_range_property(temp):
+#     assert -50 <= temp <= 50

2025/testtips/tests/test_weather.py

@@ -0,0 +1,78 @@
+import httpx
+import pytest
+from weather import WeatherService
+
+
+# Tip 1: Single assert per test
+def test_get_temperature_returns_expected_value(monkeypatch):
+    def fake_get(url, params):
+        class FakeResponse:
+            def raise_for_status(self):
+                pass
+
+            def json(self):
+                return {"current": {"temp_c": 22}}
+
+        return FakeResponse()
+
+    monkeypatch.setattr("weather.httpx.get", fake_get)
+
+    service = WeatherService(api_key="fake-key")
+    temp = service.get_temperature("Amsterdam")
+
+    assert temp == 22
+
+
+# Tip 2: Clear and descriptive names
+def test_get_temperature_for_different_city(monkeypatch):
+    def fake_get(url, params):
+        class FakeResponse:
+            def raise_for_status(self):
+                pass
+
+            def json(self):
+                return {"current": {"temp_c": 18}}
+
+        return FakeResponse()
+
+    monkeypatch.setattr("weather.httpx.get", fake_get)
+
+    service = WeatherService(api_key="fake-key")
+    temp = service.get_temperature("Berlin")
+
+    assert temp == 18
+
+
+def test_get_temperature_handles_api_error(monkeypatch):
+    def fake_get(url, params):
+        class FakeResponse:
+            def raise_for_status(self):
+                raise httpx.HTTPError("API error")
+
+        return FakeResponse()
+
+    monkeypatch.setattr("weather.httpx.get", fake_get)
+
+    service = WeatherService(api_key="fake-key")
+
+    with pytest.raises(httpx.HTTPError):
+        service.get_temperature("Paris")
+
+
+def test_get_temperature_returns_float(monkeypatch):
+    def fake_get(url, params):
+        class FakeResponse:
+            def raise_for_status(self):
+                pass
+
+            def json(self):
+                return {"current": {"temp_c": 19.5}}
+
+        return FakeResponse()
+
+    monkeypatch.setattr("weather.httpx.get", fake_get)
+
+    service = WeatherService(api_key="fake-key")
+    temp = service.get_temperature("Rome")
+
+    assert isinstance(temp, float)

2025/testtips/tests/test_weather_fixture.py

@@ -0,0 +1,50 @@
+import pytest
+from weather import WeatherService
+
+
+# This fixture creates a WeatherService instance you can reuse
+@pytest.fixture
+def weather_service():
+    return WeatherService(api_key="fake-key")
+
+
+def test_get_temperature_returns_expected_value(weather_service, monkeypatch):
+    """
+    Test that get_temperature returns the correct temperature.
+    """
+
+    def fake_get(url, params):
+        class FakeResponse:
+            def raise_for_status(self):
+                pass
+
+            def json(self):
+                return {"current": {"temp_c": 20}}
+
+        return FakeResponse()
+
+    monkeypatch.setattr("weather.httpx.get", fake_get)
+
+    temp = weather_service.get_temperature("Amsterdam")
+    assert temp == 20
+
+
+def test_get_temperature_returns_float(weather_service, monkeypatch):
+    """
+    Test that get_temperature returns a float value.
+    """
+
+    def fake_get(url, params):
+        class FakeResponse:
+            def raise_for_status(self):
+                pass
+
+            def json(self):
+                return {"current": {"temp_c": 18.5}}
+
+        return FakeResponse()
+
+    monkeypatch.setattr("weather.httpx.get", fake_get)
+
+    temp = weather_service.get_temperature("Berlin")
+    assert isinstance(temp, float)

2025/testtips/tests/test_weather_mock_mp.py

@@ -0,0 +1,41 @@
+from weather import WeatherService
+from unittest.mock import patch, MagicMock
+
+# Section 4: Mocking with patch + MagicMock
+def test_get_temperature_with_mocking():
+    """
+    Example of mocking httpx.get with a MagicMock.
+    """
+    # Create a MagicMock response object
+    mock_response = MagicMock()
+    mock_response.raise_for_status.return_value = None
+    mock_response.json.return_value = {"current": {"temp_c": 25}}
+
+    # Patch httpx.get so it returns our mock_response
+    with patch("weather.httpx.get", return_value=mock_response) as mock_get:
+        service = WeatherService(api_key="fake-key")
+        temp = service.get_temperature("Paris")
+
+        assert temp == 25
+        mock_get.assert_called_once()
+
+
+# Section 5: Monkey patching with monkeypatch fixture
+def test_get_temperature_with_monkeypatch(monkeypatch):
+    """
+    Example of monkeypatching httpx.get with a manual stub.
+    """
+
+    def fake_get(url, params):
+        class FakeResponse:
+            def raise_for_status(self): pass
+            def json(self): return {"current": {"temp_c": 19}}
+        return FakeResponse()
+
+    # Monkeypatch httpx.get to use fake_get
+    monkeypatch.setattr("weather.httpx.get", fake_get)
+
+    service = WeatherService(api_key="fake-key")
+    temp = service.get_temperature("Berlin")
+
+    assert temp == 19

2025/testtips/tests/test_weather_refactor.py

@@ -0,0 +1,12 @@
+from weather_refactor import WeatherService
+
+def test_get_temperature_with_stub_client():
+    class StubClient:
+        def get(self, url, params):
+            class Response:
+                def raise_for_status(self): pass
+                def json(self): return {"current": {"temp_c": 18}}
+            return Response()
+
+    service = WeatherService(client=StubClient(), api_key="fake_key")
+    assert service.get_temperature("Oslo") == 18