test: avoid use of tempfile module

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

diff --git a/test/python/conftest.py b/test/python/conftest.py
index ecd40d7cf8b616c0af126d5c411c030527d30c77..c25449486546fa318e918b4000efbc8f9dec81e1 100644 (file)
--- a/test/python/conftest.py
+++ b/test/python/conftest.py
@@ -3,7 +3,6 @@ import sys
 from pathlib import Path
 
 import psycopg2
 from pathlib import Path
 
 import psycopg2

-import psycopg2.extras

 import pytest
 
 SRC_DIR = Path(__file__) / '..' / '..' / '..'
 import pytest
 
 SRC_DIR = Path(__file__) / '..' / '..' / '..'


@@ -13,28 +12,15 @@ sys.path.insert(0, str(SRC_DIR.resolve()))
 
 from nominatim.config import Configuration
 from nominatim.db import connection
 
 from nominatim.config import Configuration
 from nominatim.db import connection

+from nominatim.db.sql_preprocessor import SQLPreprocessor
+from nominatim.db import properties
+import nominatim.tokenizer.factory
+import nominatim.cli

 
 

-class _TestingCursor(psycopg2.extras.DictCursor):
-    """ Extension to the DictCursor class that provides execution
-        short-cuts that simplify writing assertions.
-    """
-
-    def scalar(self, sql, params=None):
-        """ Execute a query with a single return value and return this value.
-            Raises an assertion when not exactly one row is returned.
-        """
-        self.execute(sql, params)
-        assert self.rowcount == 1
-        return self.fetchone()[0]
-
-    def row_set(self, sql, params=None):
-        """ Execute a query and return the result as a set of tuples.
-        """
-        self.execute(sql, params)
-        if self.rowcount == 1:
-            return set(tuple(self.fetchone()))
+import dummy_tokenizer
+import mocks
+from cursor import CursorForTesting

 
 

-        return set((tuple(row) for row in self))

 
 @pytest.fixture
 def temp_db(monkeypatch):
 
 @pytest.fixture
 def temp_db(monkeypatch):


@@ -51,7 +37,7 @@ def temp_db(monkeypatch):
 
     conn.close()
 
 
     conn.close()
 

-    monkeypatch.setenv('NOMINATIM_DATABASE_DSN' , 'dbname=' + name)
+    monkeypatch.setenv('NOMINATIM_DATABASE_DSN', 'dbname=' + name)

 
     yield name
 
 
     yield name
 


@@ -63,6 +49,12 @@ def temp_db(monkeypatch):
 
     conn.close()
 
 
     conn.close()
 

+
+@pytest.fixture
+def dsn(temp_db):
+    return 'dbname=' + temp_db
+
+

 @pytest.fixture
 def temp_db_with_extensions(temp_db):
     conn = psycopg2.connect(database=temp_db)
 @pytest.fixture
 def temp_db_with_extensions(temp_db):
     conn = psycopg2.connect(database=temp_db)


@@ -77,9 +69,8 @@ def temp_db_with_extensions(temp_db):
 def temp_db_conn(temp_db):
     """ Connection to the test database.
     """
 def temp_db_conn(temp_db):
     """ Connection to the test database.
     """

-    conn = connection.connect('dbname=' + temp_db)
-    yield conn
-    conn.close()
+    with connection.connect('dbname=' + temp_db) as conn:
+        yield conn

 
 
 @pytest.fixture
 
 
 @pytest.fixture


@@ -89,14 +80,58 @@ def temp_db_cursor(temp_db):
     """
     conn = psycopg2.connect('dbname=' + temp_db)
     conn.set_isolation_level(0)
     """
     conn = psycopg2.connect('dbname=' + temp_db)
     conn.set_isolation_level(0)

-    with conn.cursor(cursor_factory=_TestingCursor) as cur:
+    with conn.cursor(cursor_factory=CursorForTesting) as cur:

         yield cur
     conn.close()
 
 
         yield cur
     conn.close()
 
 

+@pytest.fixture
+def table_factory(temp_db_cursor):
+    """ A fixture that creates new SQL tables, potentially filled with
+        content.
+    """
+    def mk_table(name, definition='id INT', content=None):
+        temp_db_cursor.execute('CREATE TABLE {} ({})'.format(name, definition))
+        if content is not None:
+            temp_db_cursor.execute_values("INSERT INTO {} VALUES %s".format(name), content)
+
+    return mk_table
+
+

 @pytest.fixture
 def def_config():
 @pytest.fixture
 def def_config():

-    return Configuration(None, SRC_DIR.resolve() / 'settings')
+    cfg = Configuration(None, SRC_DIR.resolve() / 'settings')
+    cfg.set_libdirs(module='.', osm2pgsql='.',
+                    php=SRC_DIR / 'lib-php',
+                    sql=SRC_DIR / 'lib-sql',
+                    data=SRC_DIR / 'data')
+    return cfg
+
+
+@pytest.fixture
+def src_dir():
+    return SRC_DIR.resolve()
+
+
+@pytest.fixture
+def cli_call():
+    def _call_nominatim(*args):
+        return nominatim.cli.nominatim(
+                   module_dir='MODULE NOT AVAILABLE',
+                   osm2pgsql_path='OSM2PGSQL NOT AVAILABLE',
+                   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=args)
+
+    return _call_nominatim
+
+
+@pytest.fixture
+def property_table(table_factory):
+    table_factory('nominatim_properties', 'property TEXT, value TEXT')

 
 
 @pytest.fixture
 
 
 @pytest.fixture


@@ -150,7 +185,7 @@ def place_row(place_table, temp_db_cursor):
         temp_db_cursor.execute("INSERT INTO place VALUES (%s, %s, %s, %s, %s, %s, %s, %s, %s)",
                                (osm_id or next(idseq), osm_type, cls, typ, names,
                                 admin_level, address, extratags,
         temp_db_cursor.execute("INSERT INTO place VALUES (%s, %s, %s, %s, %s, %s, %s, %s, %s)",
                                (osm_id or next(idseq), osm_type, cls, typ, names,
                                 admin_level, address, extratags,

-                                geom or 'SRID=4326;POINT(0 0 )'))
+                                geom or 'SRID=4326;POINT(0 0)'))

 
     return _insert
 
 
     return _insert
 


@@ -158,34 +193,71 @@ def place_row(place_table, temp_db_cursor):
 def placex_table(temp_db_with_extensions, temp_db_conn):
     """ Create an empty version of the place table.
     """
 def placex_table(temp_db_with_extensions, temp_db_conn):
     """ Create an empty version of the place table.
     """

+    return mocks.MockPlacexTable(temp_db_conn)
+
+
+@pytest.fixture
+def osmline_table(temp_db_with_extensions, temp_db_conn):

     with temp_db_conn.cursor() as cur:
     with temp_db_conn.cursor() as cur:

-        cur.execute("""CREATE TABLE placex (
-                           place_id BIGINT NOT NULL,
+        cur.execute("""CREATE TABLE location_property_osmline (
+                           place_id BIGINT,
+                           osm_id BIGINT,

                            parent_place_id BIGINT,
                            parent_place_id BIGINT,

-                           linked_place_id BIGINT,
-                           importance FLOAT,
-                           indexed_date TIMESTAMP,

                            geometry_sector INTEGER,
                            geometry_sector INTEGER,

-                           rank_address SMALLINT,
-                           rank_search SMALLINT,
+                           indexed_date TIMESTAMP,
+                           startnumber INTEGER,
+                           endnumber INTEGER,

                            partition SMALLINT,
                            indexed_status SMALLINT,
                            partition SMALLINT,
                            indexed_status SMALLINT,

-                           osm_id int8,
-                           osm_type char(1),
-                           class text,
-                           type text,
-                           name hstore,
-                           admin_level smallint,
-                           address hstore,
-                           extratags hstore,
-                           geometry Geometry(Geometry,4326),
-                           wikipedia TEXT,
-                           country_code varchar(2),
-                           housenumber TEXT,
+                           linegeo GEOMETRY,
+                           interpolationtype TEXT,
+                           address HSTORE,

                            postcode TEXT,
                            postcode TEXT,

-                           centroid GEOMETRY(Geometry, 4326))
-                           """)
+                           country_code VARCHAR(2))""")

     temp_db_conn.commit()
 
 
     temp_db_conn.commit()
 
 

+@pytest.fixture
+def word_table(temp_db_conn):
+    return mocks.MockWordTable(temp_db_conn)
+
+
+@pytest.fixture
+def osm2pgsql_options(temp_db):
+    return dict(osm2pgsql='echo',
+                osm2pgsql_cache=10,
+                osm2pgsql_style='style.file',
+                threads=1,
+                dsn='dbname=' + temp_db,
+                flatnode_file='',
+                tablespaces=dict(slim_data='', slim_index='',
+                                 main_data='', main_index=''))
+
+@pytest.fixture
+def sql_preprocessor(temp_db_conn, tmp_path, monkeypatch, table_factory):
+    table_factory('country_name', 'partition INT', ((0, ), (1, ), (2, )))
+    cfg = Configuration(None, SRC_DIR.resolve() / 'settings')
+    cfg.set_libdirs(module='.', osm2pgsql='.', php=SRC_DIR / 'lib-php',
+                    sql=tmp_path, data=SRC_DIR / 'data')
+
+    return SQLPreprocessor(temp_db_conn, cfg)
+
+
+@pytest.fixture
+def tokenizer_mock(monkeypatch, property_table, temp_db_conn, tmp_path):
+    """ Sets up the configuration so that the test dummy tokenizer will be
+        loaded when the tokenizer factory is used. Also returns a factory
+        with which a new dummy tokenizer may be created.
+    """
+    monkeypatch.setenv('NOMINATIM_TOKENIZER', 'dummy')
+
+    def _import_dummy(module, *args, **kwargs):
+        return dummy_tokenizer
+
+    monkeypatch.setattr(nominatim.tokenizer.factory, "_import_tokenizer", _import_dummy)
+    properties.set_property(temp_db_conn, 'tokenizer', 'dummy')
+
+    def _create_tokenizer():
+        return dummy_tokenizer.DummyTokenizer(None, None)

 
 

+    return _create_tokenizer