|
| 1 | +import sys |
| 2 | +from types import SimpleNamespace |
| 3 | +from unittest.mock import MagicMock, patch |
| 4 | + |
| 5 | +import pytest |
| 6 | + |
| 7 | +pytestmark = pytest.mark.skipif( |
| 8 | + sys.platform == "win32", reason="gunicorn is not used on Windows" |
| 9 | +) |
| 10 | + |
| 11 | + |
| 12 | +def _start_server(feature_server_config, gc_freeze: bool): |
| 13 | + """Run start_server with the gunicorn app mocked; return the call order.""" |
| 14 | + from feast.feature_server import start_server |
| 15 | + |
| 16 | + calls = MagicMock() |
| 17 | + store = MagicMock() |
| 18 | + store.config = SimpleNamespace( |
| 19 | + feature_server=feature_server_config, auth_config=MagicMock() |
| 20 | + ) |
| 21 | + application = MagicMock() |
| 22 | + application.run.side_effect = lambda: calls.run() |
| 23 | + with ( |
| 24 | + patch("feast.feature_server.feast_metrics"), |
| 25 | + patch("feast.feature_server.str_to_auth_manager_type"), |
| 26 | + patch("feast.feature_server.init_security_manager"), |
| 27 | + patch("feast.feature_server.init_auth_manager"), |
| 28 | + patch("feast.feature_server.FeastServeApplication", return_value=application), |
| 29 | + patch("feast.feature_server.gc.collect", side_effect=calls.collect), |
| 30 | + patch("feast.feature_server.gc.freeze", side_effect=calls.freeze), |
| 31 | + ): |
| 32 | + start_server( |
| 33 | + store=store, |
| 34 | + host="127.0.0.1", |
| 35 | + port=6566, |
| 36 | + no_access_log=True, |
| 37 | + workers=1, |
| 38 | + worker_connections=1000, |
| 39 | + max_requests=1000, |
| 40 | + max_requests_jitter=50, |
| 41 | + keep_alive_timeout=30, |
| 42 | + registry_ttl_sec=60, |
| 43 | + tls_key_path="", |
| 44 | + tls_cert_path="", |
| 45 | + metrics=False, |
| 46 | + gc_freeze=gc_freeze, |
| 47 | + ) |
| 48 | + return [name for name, _, _ in calls.mock_calls] |
| 49 | + |
| 50 | + |
| 51 | +def test_cli_flag_freezes_before_workers_start(): |
| 52 | + assert _start_server(None, gc_freeze=True) == ["collect", "freeze", "run"] |
| 53 | + |
| 54 | + |
| 55 | +def test_config_enables_freeze(): |
| 56 | + config = SimpleNamespace(metrics=None, gc_freeze=True) |
| 57 | + assert _start_server(config, gc_freeze=False) == ["collect", "freeze", "run"] |
| 58 | + |
| 59 | + |
| 60 | +def test_no_freeze_by_default(): |
| 61 | + config = SimpleNamespace(metrics=None, gc_freeze=False) |
| 62 | + assert _start_server(config, gc_freeze=False) == ["run"] |
| 63 | + assert _start_server(None, gc_freeze=False) == ["run"] |
0 commit comments