fix(NGUI-386): Improve pytest configuration to fail on unknown marker and fix missing deps (#178)

Author: lkrzyzanek

libs/next_gen_ui_agent/BUILD

@@ -16,7 +16,11 @@ python_tests(
     sources=[
         "**/*_test.py",
     ],
-    dependencies=[":lib", ":test_resources"],
+    dependencies=[
+        ":lib",
+        ":test_resources",
+        "libs/3rdparty/python:pytest",
+    ],
 )
 
 resources(

libs/next_gen_ui_agent/component_selection_test.py

@@ -62,12 +62,14 @@ async def test_component_selection_run_OK() -> None:
     llm = FakeMessagesListChatModel(responses=[msg])  # type: ignore
     inference = LangChainModelInference(llm)
 
-    component_selection = OnestepLLMCallComponentSelectionStrategy(False)
+    component_selection = OnestepLLMCallComponentSelectionStrategy(
+        input_data_json_wrapping=False
+    )
     result = await component_selection.component_selection_run(
         inference, user_input, input_data
     )
     assert result.component == "one-card"
-    # assert json_data are not wrapped even it is enabled and type is provided, as there is only one item in the array
+    # assert json_data are not wrapped as it is disabled
     assert result.json_data == json.loads(movies_data)
 
 

libs/next_gen_ui_langgraph/BUILD

@@ -6,6 +6,9 @@ python_sources(
 # This target sets the metadata for all the Python test files in this directory.
 python_tests(
     name="tests",
+    dependencies=[
+        "libs/3rdparty/python:pytest",
+    ],
 )
 
 # This target allows us to build a `.whl` bdist and a `.tar.gz` sdist by auto-generating

libs/next_gen_ui_llama_stack/BUILD

@@ -12,6 +12,7 @@ python_sources(
 python_tests(
     name="tests",
     dependencies=[
+        "libs/3rdparty/python:pytest",
         # Has to be explicitly noted because of missing trans deps in llama-stack-client
         "libs/3rdparty/python:llama-stack-client",
     ],

libs/next_gen_ui_llama_stack_embedded/BUILD

@@ -12,6 +12,7 @@ python_sources(
 python_tests(
     name="tests",
     dependencies=[
+        "libs/3rdparty/python:pytest",
         # Has to be explicitly noted because of missing trans deps in llama-stack-client
         "libs/3rdparty/python:llama-stack-client",
         "libs/3rdparty/python:llama-stack",

pants.ci.toml

@@ -9,7 +9,9 @@ remote_cache_read = false
 remote_cache_write = false
 
 [pytest]
-args = ["-vv", "--no-header"]
+args = ["--no-header", "-vv", "--strict-markers"]
+execution_slot_var = "TEST_EXECUTION_SLOT"
+requirements = ["//libs/3rdparty/python:pytest"]
 
 [tailor]
 ignore_paths = [

pants.toml

@@ -55,7 +55,7 @@ ignore_paths = [
 ]
 
 [pytest]
-args = ["--no-header", "-vv"]
+args = ["--no-header", "-vv", "--strict-markers"]
 execution_slot_var = "TEST_EXECUTION_SLOT"
 requirements = ["//libs/3rdparty/python:pytest"]