Skip to content
This repository was archived by the owner on Nov 24, 2025. It is now read-only.

Conversation

@zrhoffman
Copy link
Member

This PR adds test coverage for API v5 /statuses endpoint.


Which Traffic Control components are affected by this PR?

  • Traffic Ops

What is the best way to verify this PR?

Run unit tests

PR submission checklist

@zrhoffman zrhoffman added Traffic Ops related to Traffic Ops tests related to tests and/or testing infrastructure tech debt rework due to choosing easy/limited solution improvement The functionality exists but it could be improved in some way. labels Aug 11, 2023
@zrhoffman zrhoffman force-pushed the rfc-3339-statuses-unit-tests branch from d6cbcc9 to 7fbd466 Compare August 11, 2023 18:12
@codecov
Copy link

codecov bot commented Aug 11, 2023

Codecov Report

Merging #7710 (7fbd466) into master (48ec14e) will increase coverage by 2.66%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #7710      +/-   ##
============================================
+ Coverage     29.78%   32.45%   +2.66%     
  Complexity       98       98              
============================================
  Files           804      512     -292     
  Lines         86263    54106   -32157     
  Branches        965      965              
============================================
- Hits          25696    17561    -8135     
+ Misses        58426    35277   -23149     
+ Partials       2141     1268     -873     
Flag Coverage Δ
golib_unit ?
grove_unit ?
t3c_unit ?
traffic_monitor_unit ?
traffic_ops_integration 69.39% <ø> (ø)
traffic_ops_unit 22.15% <ø> (+0.03%) ⬆️
traffic_portal_v2 74.38% <ø> (ø)
traffic_stats_unit ?
unit_tests 28.15% <ø> (+1.19%) ⬆️
v3 57.79% <ø> (ø)
v4 79.18% <ø> (ø)
v5 78.52% <ø> (ø)

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

see 295 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@shamrickus shamrickus merged commit 9c4ba13 into apache:master Aug 14, 2023
cybertunnel pushed a commit to cybertunnel/trafficcontrol that referenced this pull request Aug 16, 2023
@zrhoffman zrhoffman deleted the rfc-3339-statuses-unit-tests branch August 28, 2023 19:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

improvement The functionality exists but it could be improved in some way. tech debt rework due to choosing easy/limited solution tests related to tests and/or testing infrastructure Traffic Ops related to Traffic Ops

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants