Remove outdated tests for overwrite/skip cache and Prometheus exporter fallback

Author: Borda

tests/sql_tests/test_sql_core.py

@@ -62,6 +62,8 @@ def f(x, y):
 
 @pytest.mark.sql
 def test_sql_core_keywords():
+    """Keyword arguments produce a cache hit the same as positional arguments."""
+
     @cachier(backend="sql", sql_engine=SQL_CONN_STR)
     def f(x, y):
         return random() + x + y
@@ -70,14 +72,7 @@ def f(x, y):
     v1 = f(1, y=2)
     v2 = f(1, y=2)
     assert v1 == v2
-    v3 = f(1, y=2, cachier__skip_cache=True)
-    assert v3 != v1
-    v4 = f(1, y=2)
-    assert v4 == v1
-    v5 = f(1, y=2, cachier__overwrite_cache=True)
-    assert v5 != v1
-    v6 = f(1, y=2)
-    assert v6 == v5
+    f.clear_cache()
 
 
 @pytest.mark.sql
@@ -100,24 +95,6 @@ def f(x, y):
     assert v3 != v1
 
 
-@pytest.mark.sql
-def test_sql_overwrite_and_skip_cache():
-    @cachier(backend="sql", sql_engine=SQL_CONN_STR)
-    def f(x):
-        return random() + x
-
-    f.clear_cache()
-    v1 = f(1)
-    v2 = f(1)
-    assert v1 == v2
-    v3 = f(1, cachier__skip_cache=True)
-    assert v3 != v1
-    v4 = f(1, cachier__overwrite_cache=True)
-    assert v4 != v1
-    v5 = f(1)
-    assert v5 == v4
-
-
 @pytest.mark.sql
 def test_sql_concurrency():
     @cachier(backend="sql", sql_engine=SQL_CONN_STR)

tests/test_exporters.py

@@ -121,64 +121,6 @@ def test_metrics_exporter_interface():
     assert isinstance(exporter, MetricsExporter)
 
 
-@pytest.mark.memory
-def test_prometheus_exporter_with_prometheus_client_fallback():
-    """Test PrometheusExporter with use_prometheus_client=True falls back gracefully."""
-
-    # When prometheus_client is not available, it should fall back to text mode
-    @cachier(backend="memory", enable_metrics=True)
-    def test_func(x):
-        return x * 2
-
-    test_func.clear_cache()
-
-    # Create exporter with use_prometheus_client=True (will use text mode as fallback)
-    exporter = PrometheusExporter(port=9095, use_prometheus_client=True)
-    exporter.register_function(test_func)
-
-    # Generate some metrics
-    test_func(5)
-    test_func(5)
-
-    # Verify function is registered
-    assert test_func in exporter._registered_functions.values()
-
-    # Verify text metrics can be generated (fallback mode)
-    metrics_text = exporter._generate_text_metrics()
-    assert "cachier_cache_hits_total" in metrics_text
-
-    test_func.clear_cache()
-
-
-@pytest.mark.memory
-def test_prometheus_exporter_collector_metrics():
-    """Test that custom collector generates correct metrics."""
-    from cachier import cachier
-    from cachier.exporters import PrometheusExporter
-
-    @cachier(backend="memory", enable_metrics=True)
-    def test_func(x):
-        return x * 2
-
-    test_func.clear_cache()
-
-    # Use text mode to verify metrics are accessible
-    exporter = PrometheusExporter(port=9096, use_prometheus_client=False)
-    exporter.register_function(test_func)
-
-    # Generate metrics
-    test_func(5)
-    test_func(5)  # hit
-    test_func(10)  # miss
-
-    # Get stats to verify
-    stats = test_func.metrics.get_stats()
-    assert stats.hits == 1
-    assert stats.misses == 2
-
-    test_func.clear_cache()
-
-
 @pytest.mark.memory
 def test_prometheus_exporter_double_instantiation():
     """Test that two PrometheusExporter instances both work independently."""