Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WIP] Move dns_instrument from onCompleted to onHeadersReceived #1021

Draft
wants to merge 7 commits into
base: master
Choose a base branch
from

Merge branch 'master' into dns_instrument_fix

8a855a4
Select commit
Loading
Failed to load commit list.
Sign in for the full log view
Draft

[WIP] Move dns_instrument from onCompleted to onHeadersReceived #1021

Merge branch 'master' into dns_instrument_fix
8a855a4
Select commit
Loading
Failed to load commit list.
GitHub Actions / test/test_[f-h]* succeeded Feb 8, 2024 in 0s

16 passed, 0 failed and 1 skipped

Tests passed successfully

✅ junit-report.xml

17 tests were completed in 429s with 16 passed, 0 failed and 1 skipped.

Test suite Passed Failed Skipped Time
pytest 16✅ 1⚪ 429s

✅ pytest

test.test_http_instrumentation.TestHTTPInstrument
  ✅ test_worker_script_requests
  ✅ test_service_worker_requests
test.test_http_instrumentation.TestPOSTInstrument
  ✅ test_record_post_data_x_www_form_urlencoded
  ✅ test_record_post_data_text_plain
  ✅ test_record_post_data_multipart_formdata
  ✅ test_record_post_data_ajax
  ✅ test_record_post_data_ajax_no_key_value
  ✅ test_record_post_data_ajax_no_key_value_base64_encoded
  ✅ test_record_post_formdata
  ✅ test_record_binary_post_data
  ⚪ test_record_file_upload
test.test_http_instrumentation
  ✅ test_page_visit[True]
  ✅ test_page_visit[False]
  ✅ test_javascript_saving
  ✅ test_document_saving
  ✅ test_content_saving
  ✅ test_cache_hits_recorded