Clean up UpdateCampaign test to make it more readable
Created by: mrnugget
This is preparation for the work to fix #7915. The old test was really hard to read (I can only imagine how much harder it was to read for someone that didn't write it) and thus really hard to extend. The new one is much easier to extend and to reason about.
Merge request reports
Activity
Created by: codecov[bot]
Codecov Report
Merging #8138 into master will decrease coverage by
0.14%
. The diff coverage isn/a
.@@ Coverage Diff @@ ## master #8138 +/- ## ========================================== - Coverage 40.83% 40.68% -0.15% ========================================== Files 1286 1286 Lines 67526 67310 -216 Branches 6314 6314 ========================================== - Hits 27571 27383 -188 + Misses 37398 37380 -18 + Partials 2557 2547 -10
Impacted Files Coverage Δ internal/httpcli/client.go 38.88% <0%> (-22.93%)
cmd/frontend/backend/go_importers.go 61.42% <0%> (-7.96%)
internal/src-cli/version.go 53.7% <0%> (-5.76%)
internal/extsvc/phabricator/client.go 89.06% <0%> (-1.91%)
internal/extsvc/bitbucketcloud/client.go 69.69% <0%> (-1.42%)
enterprise/internal/a8n/service.go 36.04% <0%> (-0.59%)
...rprise/cmd/frontend/auth/openidconnect/provider.go 56.06% <0%> (+2.43%)
Please register or sign in to reply