Skip to content

Conversation

@gordonwatts
Copy link
Collaborator

If ServiceX responds with a > 400 error on the transform WebAPI, properly display the error message.

@codecov
Copy link

codecov bot commented May 12, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 73.88%. Comparing base (f4eeab3) to head (6159129).

Additional details and impacted files
@@             Coverage Diff              @@
##           3.0_develop     #378   +/-   ##
============================================
  Coverage        73.88%   73.88%           
============================================
  Files               42       42           
  Lines             2309     2309           
============================================
  Hits              1706     1706           
  Misses             603      603           
Flag Coverage Δ
unittests 73.88% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@BenGalewsky BenGalewsky self-requested a review May 12, 2024 07:13
Copy link
Contributor

@BenGalewsky BenGalewsky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah! Thanks

@BenGalewsky BenGalewsky merged commit e015c93 into 3.0_develop May 12, 2024
@BenGalewsky BenGalewsky deleted the bug/dump_error_message branch May 12, 2024 07:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants