agent_features: Skip all failing windows tests for now

TimPansino · TimPansino · commit 286a60acc952 · 2025-08-13T15:06:07.000-07:00
diff --git a/tests/agent_features/conftest.py b/tests/agent_features/conftest.py
@@ -12,6 +12,9 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
+import sys
+
+import pytest
 from testing_support.fixture.event_loop import event_loop
 from testing_support.fixtures import collector_agent_registration_fixture, collector_available_fixture
 from testing_support.fixtures import newrelic_caplog as caplog
@@ -31,3 +34,5 @@
 collector_agent_registration = collector_agent_registration_fixture(
     app_name="Python Agent Test (agent_features)", default_settings=_default_settings
 )
+
+FAILING_ON_WINDOWS = pytest.mark.xfail(sys.platform == "win32", reason="TODO: Fix this test on Windows")
diff --git a/tests/agent_features/test_async_generator_trace.py b/tests/agent_features/test_async_generator_trace.py
@@ -13,10 +13,10 @@
 # limitations under the License.
 
 import functools
-import sys
 import time
 
 import pytest
+from conftest import FAILING_ON_WINDOWS
 from testing_support.fixtures import capture_transaction_metrics, validate_tt_parenting
 from testing_support.validators.validate_transaction_errors import validate_transaction_errors
 from testing_support.validators.validate_transaction_metrics import validate_transaction_metrics
@@ -209,6 +209,7 @@ async def _test():
     event_loop.run_until_complete(_test())
 
 
+@FAILING_ON_WINDOWS
 @validate_tt_parenting(("TransactionNode", [("FunctionNode", [("FunctionNode", [])])]))
 @validate_transaction_metrics(
     "test_async_generator_parents",
diff --git a/tests/agent_features/test_attribute.py b/tests/agent_features/test_attribute.py
@@ -14,6 +14,7 @@
 
 import pytest
 import webtest
+from conftest import FAILING_ON_WINDOWS
 from testing_support.fixtures import (
     override_application_settings,
     validate_agent_attribute_types,
@@ -60,6 +61,7 @@ def target_wsgi_application(environ, start_response):
 _forgone_intrinsics = []
 
 
+@FAILING_ON_WINDOWS
 @validate_attributes("intrinsic", _required_intrinsics, _forgone_intrinsics)
 def test_intrinsics():
     target_application = webtest.TestApp(target_wsgi_application)
@@ -82,6 +84,7 @@ def test_intrinsics():
 _forgone_agent = []
 
 
+@FAILING_ON_WINDOWS
 @validate_attributes("agent", _required_agent, _forgone_agent)
 def test_agent():
     target_application = webtest.TestApp(target_wsgi_application)
diff --git a/tests/agent_features/test_attributes_in_action.py b/tests/agent_features/test_attributes_in_action.py
@@ -51,6 +51,7 @@
 except SyntaxError:
     normal_asgi_application = None
 
+from conftest import FAILING_ON_WINDOWS
 
 URL_PARAM = "some_key"
 URL_PARAM2 = "second_key"
@@ -196,6 +197,7 @@ def test_error_in_transaction_default_settings(normal_application):
 _expected_attributes = {"agent": TRACE_ERROR_AGENT_KEYS, "user": USER_ATTRS, "intrinsic": ["trip_id"]}
 
 
+@FAILING_ON_WINDOWS
 @cat_enabled
 @validate_transaction_trace_attributes(_expected_attributes, _expected_absent_attributes)
 @override_application_settings({})
@@ -263,6 +265,7 @@ def test_error_in_transaction_exclude_request_params(normal_application):
 _expected_attributes = {"agent": TRACE_ERROR_AGENT_KEYS, "user": USER_ATTRS, "intrinsic": ["trip_id"]}
 
 
+@FAILING_ON_WINDOWS
 @validate_transaction_trace_attributes(_expected_attributes, _expected_absent_attributes)
 @override_application_settings(_override_settings)
 def test_transaction_trace_exclude_request_params(normal_application):
@@ -285,6 +288,7 @@ def test_error_in_transaction_capture_params_exclude_request_params(normal_appli
 _override_settings = {"capture_params": True, "transaction_tracer.attributes.exclude": ["request.parameters.*"]}
 
 
+@FAILING_ON_WINDOWS
 @validate_transaction_trace_attributes(_expected_attributes, _expected_absent_attributes)
 @override_application_settings(_override_settings)
 def test_transaction_trace_capture_params_exclude_request_params(normal_application):
@@ -312,6 +316,7 @@ def test_error_in_transaction_include_request_params(normal_application):
 _expected_attributes = {"agent": AGENT_KEYS_ALL, "user": USER_ATTRS, "intrinsic": ["trip_id"]}
 
 
+@FAILING_ON_WINDOWS
 @validate_transaction_trace_attributes(_expected_attributes)
 @override_application_settings(_override_settings)
 def test_transaction_trace_include_request_params(normal_application):
@@ -393,6 +398,7 @@ def test_error_in_transaction_include_exclude(normal_application):
 }
 
 
+@FAILING_ON_WINDOWS
 @validate_transaction_trace_attributes(_expected_attributes, _expected_absent_attributes)
 @override_application_settings(_override_settings)
 def test_transaction_trace_include_exclude(normal_application):
@@ -467,6 +473,7 @@ def test_error_in_transaction_exclude_user_attribute(normal_application):
 _expected_attributes = {"agent": TRACE_ERROR_AGENT_KEYS, "user": ["sunshine"], "intrinsic": ["trip_id"]}
 
 
+@FAILING_ON_WINDOWS
 @validate_transaction_trace_attributes(_expected_attributes, _expected_absent_attributes)
 @override_application_settings(_override_settings)
 def test_transaction_trace_exclude_user_attribute(normal_application):
@@ -545,6 +552,7 @@ def test_error_in_transaction_exclude_agent_attribute(normal_application):
 }
 
 
+@FAILING_ON_WINDOWS
 @validate_transaction_trace_attributes(_expected_attributes, _expected_absent_attributes)
 @override_application_settings(_override_settings)
 def test_transaction_trace_exclude_agent_attribute(normal_application):
@@ -602,6 +610,7 @@ def test_error_in_transaction_deprecated_capture_params_true(normal_application)
 _expected_attributes = {"agent": AGENT_KEYS_ALL, "user": USER_ATTRS, "intrinsic": ["trip_id"]}
 
 
+@FAILING_ON_WINDOWS
 @validate_transaction_trace_attributes(_expected_attributes)
 @override_application_settings(_override_settings)
 def test_transaction_trace_deprecated_capture_params_true(normal_application):
@@ -659,6 +668,7 @@ def test_error_in_transaction_deprecated_capture_params_false(normal_application
 _expected_attributes = {"agent": TRACE_ERROR_AGENT_KEYS, "user": USER_ATTRS, "intrinsic": ["trip_id"]}
 
 
+@FAILING_ON_WINDOWS
 @validate_transaction_trace_attributes(_expected_attributes, _expected_absent_attributes)
 @override_application_settings(_override_settings)
 def test_transaction_trace_deprecated_capture_params_false(normal_application):
@@ -718,6 +728,7 @@ def test_error_in_transaction_exclude_intrinsic(normal_application):
 _expected_attributes = {"agent": TRACE_ERROR_AGENT_KEYS, "user": USER_ATTRS, "intrinsic": ["trip_id"]}
 
 
+@FAILING_ON_WINDOWS
 @validate_transaction_trace_attributes(_expected_attributes, _expected_absent_attributes)
 @override_application_settings(_override_settings)
 def test_transaction_trace_exclude_intrinsic(normal_application):
@@ -774,6 +785,7 @@ def test_error_in_transaction_attributes_disabled(normal_application):
 _override_settings = {"transaction_tracer.attributes.enabled": False}
 
 
+@FAILING_ON_WINDOWS
 @validate_transaction_trace_attributes(_expected_attributes, _expected_absent_attributes)
 @override_application_settings(_override_settings)
 def test_transaction_trace_attributes_disabled(normal_application):
@@ -831,6 +843,7 @@ def test_error_in_transaction_error_param_excluded(normal_application):
 _expected_attributes = {"agent": TRANS_EVENT_AGENT_KEYS, "user": USER_ATTRS, "intrinsic": []}
 
 
+@FAILING_ON_WINDOWS
 @validate_transaction_trace_attributes(_expected_attributes)
 @validate_transaction_event_attributes(_expected_attributes)
 @override_application_settings(_override_settings)
diff --git a/tests/agent_features/test_collector_payloads.py b/tests/agent_features/test_collector_payloads.py
@@ -14,6 +14,7 @@
 
 import pytest
 import webtest
+from conftest import FAILING_ON_WINDOWS
 from testing_support.fixtures import override_application_settings
 from testing_support.sample_applications import simple_app, simple_custom_event_app, simple_exceptional_app
 from testing_support.validators.validate_custom_event_collector_json import validate_custom_event_collector_json
@@ -46,11 +47,13 @@ def test_error_event_json():
         pass
 
 
+@FAILING_ON_WINDOWS
 @validate_tt_collector_json()
 def test_transaction_trace_json():
     normal_application.get("/")
 
 
+@FAILING_ON_WINDOWS
 @validate_tt_collector_json(exclude_request_uri=True)
 @override_application_settings({"attributes.exclude": {"request.uri"}})
 def test_transaction_trace_json_no_request_uri():
diff --git a/tests/agent_features/test_coroutine_trace.py b/tests/agent_features/test_coroutine_trace.py
@@ -19,6 +19,7 @@
 import time
 
 import pytest
+from conftest import FAILING_ON_WINDOWS
 from testing_support.fixtures import capture_transaction_metrics, validate_tt_parenting
 from testing_support.validators.validate_transaction_errors import validate_transaction_errors
 from testing_support.validators.validate_transaction_metrics import validate_transaction_metrics
@@ -75,6 +76,7 @@ def _test():
     assert full_metrics[metric_key].total_call_time >= 0.2
 
 
+@FAILING_ON_WINDOWS
 @validate_tt_parenting(("TransactionNode", [("FunctionNode", [("FunctionNode", []), ("FunctionNode", [])])]))
 @validate_transaction_metrics(
     "test_coroutine_siblings",
@@ -246,6 +248,7 @@ def coro():
     gen.close()
 
 
+@FAILING_ON_WINDOWS
 @validate_tt_parenting(("TransactionNode", [("FunctionNode", [("FunctionNode", [])])]))
 @validate_transaction_metrics(
     "test_coroutine_parents",
diff --git a/tests/agent_features/test_coroutine_transaction.py b/tests/agent_features/test_coroutine_transaction.py
@@ -13,9 +13,9 @@
 # limitations under the License.
 
 import asyncio
-import sys
 
 import pytest
+from conftest import FAILING_ON_WINDOWS
 from testing_support.fixtures import capture_transaction_metrics, override_generic_settings
 from testing_support.validators.validate_transaction_errors import validate_transaction_errors
 
@@ -91,6 +91,7 @@ async def task():
     test_matrix.append(native_coroutine_test)
 
 
+@FAILING_ON_WINDOWS
 @pytest.mark.parametrize("num_coroutines", (2,))
 @pytest.mark.parametrize("create_test_task", test_matrix)
 @pytest.mark.parametrize(
diff --git a/tests/agent_features/test_error_events.py b/tests/agent_features/test_error_events.py
@@ -16,6 +16,7 @@
 import time
 
 import webtest
+from conftest import FAILING_ON_WINDOWS
 from testing_support.fixtures import (
     cat_enabled,
     make_cross_agent_headers,
@@ -74,6 +75,7 @@ def test_transaction_error_event_no_extra_attributes():
 }
 
 
+@FAILING_ON_WINDOWS
 @validate_error_event_sample_data(required_attrs=_intrinsic_attributes, required_user_attrs=True)
 def test_transaction_error_event_lotsa_attributes():
     test_environ = {
@@ -97,6 +99,7 @@ def test_transaction_error_event_lotsa_attributes():
 }
 
 
+@FAILING_ON_WINDOWS
 @validate_error_event_sample_data(required_attrs=_intrinsic_attributes, required_user_attrs=True)
 def test_transaction_error_background_task():
     test_environ = {"err_message": ERR_MESSAGE, "external": "2", "db": "2", "newrelic.set_background_task": True}
diff --git a/tests/agent_features/test_function_trace.py b/tests/agent_features/test_function_trace.py
@@ -14,6 +14,7 @@
 
 import time
 
+from conftest import FAILING_ON_WINDOWS
 from testing_support.fixtures import validate_tt_parenting
 from testing_support.validators.validate_transaction_metrics import validate_transaction_metrics
 
@@ -88,6 +89,7 @@ def test_function_trace_leading_slash_on_group():
 )
 
 
+@FAILING_ON_WINDOWS
 @validate_transaction_metrics(
     "test_function_trace:test_async_trace_overlapping_children",
     scoped_metrics=_test_async_trace_overlapping_children_scoped_metrics,
diff --git a/tests/agent_features/test_high_security_mode.py b/tests/agent_features/test_high_security_mode.py
@@ -17,6 +17,7 @@
 
 import pytest
 import webtest
+from conftest import FAILING_ON_WINDOWS
 from testing_support.fixtures import (
     override_application_settings,
     override_generic_settings,
@@ -705,6 +706,7 @@ def test_http_referrer_url_is_sanitized_in_hsm():
     target_application.get("/", extra_environ=environ)
 
 
+@FAILING_ON_WINDOWS
 @pytest.mark.parametrize("hsm_enabled", [True, False])
 def test_function_trace_params_dropped_in_hsm(hsm_enabled):
     @background_task()
@@ -722,6 +724,7 @@ def _test():
     _test()
 
 
+@FAILING_ON_WINDOWS
 @pytest.mark.parametrize("hsm_enabled", [True, False])
 def test_message_trace_params_dropped_in_hsm(hsm_enabled):
     @background_task()
diff --git a/tests/agent_features/test_span_events.py b/tests/agent_features/test_span_events.py
@@ -15,6 +15,7 @@
 import sys
 
 import pytest
+from conftest import FAILING_ON_WINDOWS
 from testing_support.fixtures import dt_enabled, function_not_called, override_application_settings
 from testing_support.validators.validate_span_events import validate_span_events
 from testing_support.validators.validate_transaction_event_attributes import validate_transaction_event_attributes
@@ -465,6 +466,7 @@ def __exit__(self, *args):
         pass
 
 
+@FAILING_ON_WINDOWS
 @pytest.mark.parametrize(
     "trace_type,args",
     (
diff --git a/tests/agent_features/test_synthetics.py b/tests/agent_features/test_synthetics.py
@@ -14,6 +14,7 @@
 
 import pytest
 import webtest
+from conftest import FAILING_ON_WINDOWS
 from testing_support.external_fixtures import validate_synthetics_external_trace_header
 from testing_support.fixtures import cat_enabled, make_synthetics_headers, override_application_settings
 from testing_support.validators.validate_synthetics_event import validate_synthetics_event
@@ -195,6 +196,7 @@ def test_valid_synthetics_in_transaction_trace():
     response = target_application.get("/", headers=headers)
 
 
+@FAILING_ON_WINDOWS
 @validate_synthetics_transaction_trace([], _test_valid_synthetics_tt_required, should_exist=False)
 @override_application_settings(_override_settings)
 def test_no_synthetics_in_transaction_trace():
diff --git a/tests/agent_features/test_transaction_event_data_and_some_browser_stuff_too.py b/tests/agent_features/test_transaction_event_data_and_some_browser_stuff_too.py
@@ -15,6 +15,7 @@
 import json
 
 import webtest
+from conftest import FAILING_ON_WINDOWS
 from testing_support.fixtures import override_application_settings, validate_transaction_event_sample_data
 from testing_support.sample_applications import fully_featured_app, user_attributes_added
 from testing_support.validators.validate_transaction_event_attributes import validate_transaction_event_attributes
@@ -382,6 +383,7 @@ def test_no_database_or_external_attributes_in_analytics():
 _intrinsic_attributes = {"name": "WebTransaction/Uri/db", "databaseCallCount": 2}
 
 
+@FAILING_ON_WINDOWS
 @validate_transaction_event_sample_data(required_attrs=_intrinsic_attributes, required_user_attrs=_user_attributes)
 def test_database_attributes_in_analytics():
     """Make database calls in the transaction and check if the analytic
@@ -407,6 +409,7 @@ def test_database_attributes_in_analytics():
 _intrinsic_attributes = {"name": "WebTransaction/Uri/ext", "externalCallCount": 2}
 
 
+@FAILING_ON_WINDOWS
 @validate_transaction_event_sample_data(required_attrs=_intrinsic_attributes, required_user_attrs=_user_attributes)
 def test_external_attributes_in_analytics():
     """Make external calls in the transaction and check if the analytic
@@ -432,6 +435,7 @@ def test_external_attributes_in_analytics():
 _intrinsic_attributes = {"name": "WebTransaction/Uri/dbext", "databaseCallCount": 2, "externalCallCount": 2}
 
 
+@FAILING_ON_WINDOWS
 @validate_transaction_event_sample_data(required_attrs=_intrinsic_attributes, required_user_attrs=_user_attributes)
 def test_database_and_external_attributes_in_analytics():
     """Make external calls and database calls in the transaction and check if
diff --git a/tests/agent_features/test_transaction_trace_segments.py b/tests/agent_features/test_transaction_trace_segments.py
diff --git a/tests/agent_features/test_wsgi_attributes.py b/tests/agent_features/test_wsgi_attributes.py
