[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]
Re: [tor-bugs] #20037 [Metrics/CollecTor]: Add support for Bifroest's bridge descriptor tarballs
#20037: Add support for Bifroest's bridge descriptor tarballs
-------------------------------+------------------------------
Reporter: karsten | Owner:
Type: enhancement | Status: needs_review
Priority: Medium | Milestone:
Component: Metrics/CollecTor | Version:
Severity: Normal | Resolution:
Keywords: | Actual Points:
Parent ID: | Points:
Reviewer: | Sponsor:
-------------------------------+------------------------------
Comment (by iwakeh):
It would be great to have some tests for this addition. Is there time for
that?
If not these should be added as soon as possible.
Maybe add some more inline comments with possible test cases now that will
help adding tests later?
The tests will be very important when huge methods like in
BridgeSnapshotReader (just the constructor has more than 200 lines) or in
SanitizedBridgesWriter (almost 200 lines in one method) will undergo
refactoring.
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/20037#comment:2>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
_______________________________________________
tor-bugs mailing list
tor-bugs@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs