Opened 2 years ago

Closed 2 years ago

#27195 closed enhancement (implemented)

Rust: add tests for protover edge cases

Reported by: teor Owned by:
Priority: Medium Milestone: Tor: 0.3.5.x-final
Component: Core Tor/Tor Version:
Severity: Normal Keywords: 033-backport, 034-backport, rust, tor-test
Cc: Actual Points:
Parent ID: Points:
Reviewer: Sponsor:


There are more tests for #27164 and #27177 in branch prototest1 on ​

Since we backported the fixes, I suggest we backport the tests.

Original comment:

Child Tickets

Change History (5)

comment:1 Changed 2 years ago by teor

Status: newneeds_review

comment:2 Changed 2 years ago by chelseakomlo

These extra test cases look good to me. Ideally we would write some kind of fuzzer test for this kind of string-parsing/handling code, but for this particular change it looks good to merge in.

Setting this as merge-ready so Nick can take a look.

comment:3 Changed 2 years ago by chelseakomlo

Status: needs_reviewmerge_ready

comment:4 Changed 2 years ago by nickm

Added to github as for CI. Will merge once CI passes.

comment:5 Changed 2 years ago by nickm

Resolution: implemented
Status: merge_readyclosed

CI passed; branch merged!

Note: See TracTickets for help on using tickets.