Page MenuHomePhabricator
Differential D11082

test: remove duplicate assertions in util_tests
ClosedPublic
Actions

Authored by PiRK on Feb 21 2022, 09:13.

Details

Reviewers
Fabien
Group Reviewers
Restricted Project
Commits
rABC1b45438ad29e: test: remove duplicate assertions in util_tests
Summary

Co-authored-by: Kiminuo <kiminuo@protonmail.com>

This is a backport of core#21491
Depends on D11078 and D11081

Test Plan

ninja check

Diff Detail

Event Timeline

PiRK created this revision.Feb 21 2022, 09:13
Herald added a reviewer: Restricted Project. · View Herald TranscriptFeb 21 2022, 09:13
PiRK requested review of this revision.Feb 21 2022, 09:13
teamcity edited the summary of this revision. (Show Details)Feb 21 2022, 09:14
Fabien requested changes to this revision.Feb 21 2022, 09:20
Fabien added a subscriber: Fabien.

What about ParseUInt32 ?

This revision now requires changes to proceed.Feb 21 2022, 09:20
Harbormaster completed remote builds in B18316: Diff 32452.Feb 21 2022, 09:56
PiRK updated this revision to Diff 32455.EditedFeb 21 2022, 10:38

rebase after fixing D11081

Harbormaster completed remote builds in B18319: Diff 32455.Feb 21 2022, 11:00
Fabien accepted this revision.Feb 21 2022, 12:47
This revision is now accepted and ready to land.Feb 21 2022, 12:47
Closed by commit rABC1b45438ad29e: test: remove duplicate assertions in util_tests (authored by Jon Atack <jon@atack.com>, committed by PiRK). · Explain WhyFeb 21 2022, 13:25
This revision was automatically updated to reflect the committed changes.
PiRK added a commit: rABC1b45438ad29e: test: remove duplicate assertions in util_tests.

Revision Contents
Changeset List

PathSize
src/
test/
2 lines

Diff 32455

View Options

src/test/util_tests.cpp

Loading...