|
| 1 | +# ================================================================= |
| 2 | +# |
| 3 | +# Authors: Edward Lewis <eddlewis85@gmail.com> |
| 4 | +# |
| 5 | +# Copyright (c) 2026 Edward Lewis |
| 6 | +# |
| 7 | +# Permission is hereby granted, free of charge, to any person |
| 8 | +# obtaining a copy of this software and associated documentation |
| 9 | +# files (the "Software"), to deal in the Software without |
| 10 | +# restriction, including without limitation the rights to use, |
| 11 | +# copy, modify, merge, publish, distribute, sublicense, and/or sell |
| 12 | +# copies of the Software, and to permit persons to whom the |
| 13 | +# Software is furnished to do so, subject to the following |
| 14 | +# conditions: |
| 15 | +# |
| 16 | +# The above copyright notice and this permission notice shall be |
| 17 | +# included in all copies or substantial portions of the Software. |
| 18 | +# |
| 19 | +# THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, |
| 20 | +# EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES |
| 21 | +# OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND |
| 22 | +# NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT |
| 23 | +# HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, |
| 24 | +# WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING |
| 25 | +# FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR |
| 26 | +# OTHER DEALINGS IN THE SOFTWARE. |
| 27 | +# |
| 28 | +# ================================================================= |
| 29 | +# Test that get_engine() separates SQLAlchemy connection-pool tuning |
| 30 | +# options from DBAPI connect_args. This is the contract introduced by |
| 31 | +# the configurable-pool change; it needs no live database. |
| 32 | +# ================================================================= |
| 33 | + |
| 34 | +from unittest import mock |
| 35 | + |
| 36 | +from pygeoapi.provider import sql |
| 37 | + |
| 38 | + |
| 39 | +@mock.patch.object(sql, 'create_engine') |
| 40 | +def test_get_engine_separates_pool_options_from_connect_args(mock_create): |
| 41 | + sql.get_engine.cache_clear() |
| 42 | + sql.get_engine( |
| 43 | + 'postgresql+psycopg2', 'h', 5432, 'd', 'u', 'p', None, |
| 44 | + pool_size=2, pool_recycle=300, connect_timeout=10, |
| 45 | + ) |
| 46 | + |
| 47 | + _, kwargs = mock_create.call_args |
| 48 | + # pool keys are applied to the engine (QueuePool), with overrides |
| 49 | + # honoured and unset pool keys falling back to the documented defaults |
| 50 | + assert kwargs['pool_size'] == 2 |
| 51 | + assert kwargs['pool_recycle'] == 300 |
| 52 | + assert kwargs['max_overflow'] == 10 |
| 53 | + assert kwargs['pool_timeout'] == 30 |
| 54 | + assert kwargs['pool_pre_ping'] is True |
| 55 | + # genuine DBAPI args are forwarded via connect_args; pool keys are not |
| 56 | + assert kwargs['connect_args'] == {'connect_timeout': 10} |
0 commit comments