enable flake for bdd test code

[nominatim.git] / test / bdd / environment.py

diff --git a/test/bdd/environment.py b/test/bdd/environment.py
index 7535c5086133a1e9dd2a4ffb62c1f48ec278850a..bedbe8d2e95ca27fb1c1116fdddef71485ffcd66 100644 (file)
--- a/test/bdd/environment.py
+++ b/test/bdd/environment.py
@@ -2,43 +2,45 @@
 #
 # This file is part of Nominatim. (https://nominatim.org)
 #
 #
 # This file is part of Nominatim. (https://nominatim.org)
 #

-# Copyright (C) 2024 by the Nominatim developer community.
+# Copyright (C) 2025 by the Nominatim developer community.

 # For a full list of authors see the git log.
 from pathlib import Path
 import sys
 
 # For a full list of authors see the git log.
 from pathlib import Path
 import sys
 

-from behave import *
+from behave import *  # noqa

 
 sys.path.insert(1, str(Path(__file__, '..', '..', '..', 'src').resolve()))
 
 
 sys.path.insert(1, str(Path(__file__, '..', '..', '..', 'src').resolve()))
 

-from steps.geometry_factory import GeometryFactory
-from steps.nominatim_environment import NominatimEnvironment
+from steps.geometry_factory import GeometryFactory  # noqa: E402
+from steps.nominatim_environment import NominatimEnvironment  # noqa: E402

 
 TEST_BASE_DIR = Path(__file__, '..', '..').resolve()
 
 userconfig = {
 
 TEST_BASE_DIR = Path(__file__, '..', '..').resolve()
 
 userconfig = {

-    'REMOVE_TEMPLATE' : False,
-    'KEEP_TEST_DB' : False,
-    'DB_HOST' : None,
-    'DB_PORT' : None,
-    'DB_USER' : None,
-    'DB_PASS' : None,
-    'TEMPLATE_DB' : 'test_template_nominatim',
-    'TEST_DB' : 'test_nominatim',
-    'API_TEST_DB' : 'test_api_nominatim',
-    'API_TEST_FILE'  : TEST_BASE_DIR / 'testdb' / 'apidb-test-data.pbf',
-    'TOKENIZER' : None, # Test with a custom tokenizer
-    'STYLE' : 'extratags',
+    'REMOVE_TEMPLATE': False,
+    'KEEP_TEST_DB': False,
+    'DB_HOST': None,
+    'DB_PORT': None,
+    'DB_USER': None,
+    'DB_PASS': None,
+    'TEMPLATE_DB': 'test_template_nominatim',
+    'TEST_DB': 'test_nominatim',
+    'API_TEST_DB': 'test_api_nominatim',
+    'API_TEST_FILE': TEST_BASE_DIR / 'testdb' / 'apidb-test-data.pbf',
+    'TOKENIZER': None,  # Test with a custom tokenizer
+    'STYLE': 'extratags',

     'API_ENGINE': 'falcon'
 }
 
     'API_ENGINE': 'falcon'
 }
 

-use_step_matcher("re")
+
+use_step_matcher("re")  # noqa: F405
+

 
 def before_all(context):
     # logging setup
     context.config.setup_logging()
     # set up -D options
 
 def before_all(context):
     # logging setup
     context.config.setup_logging()
     # set up -D options

-    for k,v in userconfig.items():
+    for k, v in userconfig.items():

         context.config.userdata.setdefault(k, v)
     # Nominatim test setup
     context.nominatim = NominatimEnvironment(context.config.userdata)
         context.config.userdata.setdefault(k, v)
     # Nominatim test setup
     context.nominatim = NominatimEnvironment(context.config.userdata)


@@ -46,7 +48,7 @@ def before_all(context):
 
 
 def before_scenario(context, scenario):
 
 
 def before_scenario(context, scenario):

-    if not 'SQLITE' in context.tags \
+    if 'SQLITE' not in context.tags \

        and context.config.userdata['API_TEST_DB'].startswith('sqlite:'):
         context.scenario.skip("Not usable with Sqlite database.")
     elif 'DB' in context.tags:
        and context.config.userdata['API_TEST_DB'].startswith('sqlite:'):
         context.scenario.skip("Not usable with Sqlite database.")
     elif 'DB' in context.tags:


@@ -56,6 +58,7 @@ def before_scenario(context, scenario):
     elif 'UNKNOWNDB' in context.tags:
         context.nominatim.setup_unknown_db()
 
     elif 'UNKNOWNDB' in context.tags:
         context.nominatim.setup_unknown_db()
 

+

 def after_scenario(context, scenario):
     if 'DB' in context.tags:
         context.nominatim.teardown_db(context)
 def after_scenario(context, scenario):
     if 'DB' in context.tags:
         context.nominatim.teardown_db(context)