[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