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

[DR-90265] Log error body to DataDog if Lighthouse returns 422 #18559

Merged
merged 4 commits into from
Sep 25, 2024

Merge branch 'master' into DR-90265-add-error-body-for-422-logs

ccc4e10
Select commit
Loading
Failed to load commit list.
Sign in for the full log view
Merged

[DR-90265] Log error body to DataDog if Lighthouse returns 422 #18559

Merge branch 'master' into DR-90265-add-error-body-for-422-logs
ccc4e10
Select commit
Loading
Failed to load commit list.
GitHub Actions / Test Results failed Sep 23, 2024 in 0s

1 fail, 5 skipped, 24 122 pass in 1h 0m 31s

    13 files  ± 0      13 suites  ±0   1h 0m 31s ⏱️ -39s
24 128 tests +10  24 122 ✅ +9  5 💤 ±0  1 ❌ +1 
24 422 runs  +10  24 416 ✅ +9  5 💤 ±0  1 ❌ +1 

Results for commit ccc4e10. ± Comparison against earlier commit 6208115.

Annotations

Check warning on line 0 in ./modules/mobile/spec/requests/mobile/v0/appointments/vaos_v2_spec.rb

See this annotation in the file changed.

@github-actions github-actions / Test Results

Mobile::V0::Appointments::VAOSV2 with VAOS GET /mobile/v0/appointments when custom error response is injected raises 418 custom error (modules.mobile.spec.requests.mobile.v0.appointments.vaos_v2_spec) failed

Test Results/rspec7.xml [took 0s]
Raw output
RSpec::Core::MultipleExceptionError
\e[1mFailure/Error: \e[0mUnable to find matching line from backtrace

  expected the response to have status code 418 but it was 401

  Top level errors member must be an array.