Merge remote-tracking branch 'upstream/master'

[nominatim.git] / test / python / test_cli_replication.py

diff --git a/test/python/test_cli_replication.py b/test/python/test_cli_replication.py
index b95e6ede1787dd7619bee512e95a5694c1e30b02..dcaeaf25fb8fff283f3228ac0fe2d12d56a99ad5 100644 (file)
--- a/test/python/test_cli_replication.py
+++ b/test/python/test_cli_replication.py
@@ -3,7 +3,6 @@ Tests for replication command of command-line interface wrapper.
 """
 import datetime as dt
 import time
 """
 import datetime as dt
 import time

-from pathlib import Path

 
 import pytest
 
 
 import pytest
 


@@ -14,18 +13,6 @@ from nominatim.db import status
 
 from mocks import MockParamCapture
 
 
 from mocks import MockParamCapture
 

-SRC_DIR = (Path(__file__) / '..' / '..' / '..').resolve()
-
-def call_nominatim(*args):
-    return nominatim.cli.nominatim(module_dir='build/module',
-                                   osm2pgsql_path='build/osm2pgsql/osm2pgsql',
-                                   phplib_dir=str(SRC_DIR / 'lib-php'),
-                                   data_dir=str(SRC_DIR / 'data'),
-                                   phpcgi_path='/usr/bin/php-cgi',
-                                   sqllib_dir=str(SRC_DIR / 'lib-sql'),
-                                   config_dir=str(SRC_DIR / 'settings'),
-                                   cli_args=['replication'] + list(args))
-

 @pytest.fixture
 def tokenizer_mock(monkeypatch):
     class DummyTokenizer:
 @pytest.fixture
 def tokenizer_mock(monkeypatch):
     class DummyTokenizer:


@@ -40,23 +27,14 @@ def tokenizer_mock(monkeypatch):
             self.finalize_import_called = True
 
     tok = DummyTokenizer()
             self.finalize_import_called = True
 
     tok = DummyTokenizer()

-    monkeypatch.setattr(nominatim.tokenizer.factory, 'get_tokenizer_for_db' ,
+    monkeypatch.setattr(nominatim.tokenizer.factory, 'get_tokenizer_for_db',

                         lambda *args: tok)
                         lambda *args: tok)

-    monkeypatch.setattr(nominatim.tokenizer.factory, 'create_tokenizer' ,
+    monkeypatch.setattr(nominatim.tokenizer.factory, 'create_tokenizer',

                         lambda *args: tok)
 
     return tok
 
 
                         lambda *args: tok)
 
     return tok
 
 

-@pytest.fixture
-def index_mock(monkeypatch, tokenizer_mock):
-    mock = MockParamCapture()
-    monkeypatch.setattr(nominatim.indexer.indexer.Indexer, 'index_boundaries', mock)
-    monkeypatch.setattr(nominatim.indexer.indexer.Indexer, 'index_by_rank', mock)
-
-    return mock
-
-

 @pytest.fixture
 def mock_func_factory(monkeypatch):
     def get_mock(module, func):
 @pytest.fixture
 def mock_func_factory(monkeypatch):
     def get_mock(module, func):


@@ -70,80 +48,95 @@ def mock_func_factory(monkeypatch):
 @pytest.fixture
 def init_status(temp_db_conn, status_table):
     status.set_status(temp_db_conn, date=dt.datetime.now(dt.timezone.utc), seq=1)
 @pytest.fixture
 def init_status(temp_db_conn, status_table):
     status.set_status(temp_db_conn, date=dt.datetime.now(dt.timezone.utc), seq=1)

-    return 1
+
+
+@pytest.fixture
+def index_mock(monkeypatch, tokenizer_mock, init_status):
+    mock = MockParamCapture()
+    monkeypatch.setattr(nominatim.indexer.indexer.Indexer, 'index_boundaries', mock)
+    monkeypatch.setattr(nominatim.indexer.indexer.Indexer, 'index_by_rank', mock)
+
+    return mock

 
 
 @pytest.fixture
 def update_mock(mock_func_factory, init_status, tokenizer_mock):
     return mock_func_factory(nominatim.tools.replication, 'update')
 
 
 
 @pytest.fixture
 def update_mock(mock_func_factory, init_status, tokenizer_mock):
     return mock_func_factory(nominatim.tools.replication, 'update')
 

-@pytest.mark.parametrize("params,func", [
-                         (('--init', '--no-update-functions'), 'init_replication'),
-                         (('--check-for-updates',), 'check_for_updates')
-                         ])
-def test_replication_command(mock_func_factory, temp_db, params, func):
-    func_mock = mock_func_factory(nominatim.tools.replication, func)

 
 

-    assert 0 == call_nominatim(*params)
-    assert func_mock.called == 1
+class TestCliReplication:
+
+    @pytest.fixture(autouse=True)
+    def setup_cli_call(self, cli_call, temp_db):
+        self.call_nominatim = lambda *args: cli_call('replication', *args)
+
+    @pytest.mark.parametrize("params,func", [
+                             (('--init', '--no-update-functions'), 'init_replication'),
+                             (('--check-for-updates',), 'check_for_updates')
+                             ])
+    def test_replication_command(self, mock_func_factory, params, func):
+        func_mock = mock_func_factory(nominatim.tools.replication, func)
+
+        assert self.call_nominatim(*params) == 0
+        assert func_mock.called == 1

 
 
 
 

-def test_replication_update_bad_interval(monkeypatch, temp_db):
-    monkeypatch.setenv('NOMINATIM_REPLICATION_UPDATE_INTERVAL', 'xx')
+    def test_replication_update_bad_interval(self, monkeypatch):
+        monkeypatch.setenv('NOMINATIM_REPLICATION_UPDATE_INTERVAL', 'xx')

 
 

-    assert call_nominatim() == 1
+        assert self.call_nominatim() == 1

 
 
 
 

-def test_replication_update_bad_interval_for_geofabrik(monkeypatch, temp_db):
-    monkeypatch.setenv('NOMINATIM_REPLICATION_URL',
-                       'https://download.geofabrik.de/europe/ireland-and-northern-ireland-updates')
+    def test_replication_update_bad_interval_for_geofabrik(self, monkeypatch):
+        monkeypatch.setenv('NOMINATIM_REPLICATION_URL',
+                           'https://download.geofabrik.de/europe/italy-updates')

 
 

-    assert call_nominatim() == 1
+        assert self.call_nominatim() == 1

 
 
 
 

-def test_replication_update_once_no_index(update_mock):
-    assert 0 == call_nominatim('--once', '--no-index')
+    def test_replication_update_once_no_index(self, update_mock):
+        assert self.call_nominatim('--once', '--no-index') == 0

 
 

-    assert str(update_mock.last_args[1]['osm2pgsql']) == 'build/osm2pgsql/osm2pgsql'
+        assert str(update_mock.last_args[1]['osm2pgsql']) == 'OSM2PGSQL NOT AVAILABLE'

 
 
 
 

-def test_replication_update_custom_osm2pgsql(monkeypatch, update_mock):
-    monkeypatch.setenv('NOMINATIM_OSM2PGSQL_BINARY', '/secret/osm2pgsql')
-    assert 0 == call_nominatim('--once', '--no-index')
+    def test_replication_update_custom_osm2pgsql(self, monkeypatch, update_mock):
+        monkeypatch.setenv('NOMINATIM_OSM2PGSQL_BINARY', '/secret/osm2pgsql')
+        assert self.call_nominatim('--once', '--no-index') == 0

 
 

-    assert str(update_mock.last_args[1]['osm2pgsql']) == '/secret/osm2pgsql'
+        assert str(update_mock.last_args[1]['osm2pgsql']) == '/secret/osm2pgsql'

 
 
 
 

-def test_replication_update_custom_threads(update_mock):
-    assert 0 == call_nominatim('--once', '--no-index', '--threads', '4')
+    def test_replication_update_custom_threads(self, update_mock):
+        assert self.call_nominatim('--once', '--no-index', '--threads', '4') == 0

 
 

-    assert update_mock.last_args[1]['threads'] == 4
+        assert update_mock.last_args[1]['threads'] == 4

 
 
 
 

-def test_replication_update_continuous(monkeypatch, init_status, index_mock):
-    states = [nominatim.tools.replication.UpdateState.UP_TO_DATE,
-              nominatim.tools.replication.UpdateState.UP_TO_DATE]
-    monkeypatch.setattr(nominatim.tools.replication, 'update',
-                        lambda *args, **kwargs: states.pop())
+    def test_replication_update_continuous(self, monkeypatch, index_mock):
+        states = [nominatim.tools.replication.UpdateState.UP_TO_DATE,
+                  nominatim.tools.replication.UpdateState.UP_TO_DATE]
+        monkeypatch.setattr(nominatim.tools.replication, 'update',
+                            lambda *args, **kwargs: states.pop())

 
 

-    with pytest.raises(IndexError):
-        call_nominatim()
+        with pytest.raises(IndexError):
+            self.call_nominatim()

 
 

-    assert index_mock.called == 4
+        assert index_mock.called == 4

 
 
 
 

-def test_replication_update_continuous_no_change(monkeypatch, init_status, index_mock):
-    states = [nominatim.tools.replication.UpdateState.NO_CHANGES,
-              nominatim.tools.replication.UpdateState.UP_TO_DATE]
-    monkeypatch.setattr(nominatim.tools.replication, 'update',
-                        lambda *args, **kwargs: states.pop())
+    def test_replication_update_continuous_no_change(self, monkeypatch, index_mock):
+        states = [nominatim.tools.replication.UpdateState.NO_CHANGES,
+                  nominatim.tools.replication.UpdateState.UP_TO_DATE]
+        monkeypatch.setattr(nominatim.tools.replication, 'update',
+                            lambda *args, **kwargs: states.pop())

 
 

-    sleep_mock = MockParamCapture()
-    monkeypatch.setattr(time, 'sleep', sleep_mock)
+        sleep_mock = MockParamCapture()
+        monkeypatch.setattr(time, 'sleep', sleep_mock)

 
 

-    with pytest.raises(IndexError):
-        call_nominatim()
+        with pytest.raises(IndexError):
+            self.call_nominatim()

 
 

-    assert index_mock.called == 2
-    assert sleep_mock.called == 1
-    assert sleep_mock.last_args[0] == 60
+        assert index_mock.called == 2
+        assert sleep_mock.called == 1
+        assert sleep_mock.last_args[0] == 60