test: Fix flaky test_external_crash_reporter_http when running with kcov (#1419)

* Initial plan

* Fix crash test assertions to handle kcov's exit code behavior

Co-authored-by: vaind <[email protected]>

* Skip entire crash tests when running with kcov instead of conditional assertions

Co-authored-by: vaind <[email protected]>

* Only skip test_external_crash_reporter_http when running with kcov

Co-authored-by: vaind <[email protected]>

* Update tests/test_integration_http.py

* Update tests/test_integration_http.py

* Update tests/test_integration_http.py

---------

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: vaind <[email protected]>

Author: Copilot

tests/test_integration_http.py

@@ -39,7 +39,7 @@
     assert_attachment_view_hierarchy,
     assert_logs,
 )
-from .conditions import has_http, has_breakpad, has_files
+from .conditions import has_http, has_breakpad, has_files, is_kcov
 
 pytestmark = pytest.mark.skipif(not has_http, reason="tests need http")
 
@@ -206,6 +206,7 @@ def test_user_report_http(cmake, httpserver):
     assert_user_report(envelope)
 
 
+@pytest.mark.skipif(is_kcov, reason="kcov exits with 0 even when the process crashes")
 @pytest.mark.parametrize(
     "build_args",
     [