[metrics-bugs] #28114 [Metrics/Library]: Improve getter names for boolean fields in metrics-lib

Tor Bug Tracker & Wiki blackhole at torproject.org
Thu Oct 18 23:54:44 UTC 2018


#28114: Improve getter names for boolean fields in metrics-lib
-------------------------+---------------------------------
 Reporter:  efgyirfe784  |          Owner:  metrics-team
     Type:  enhancement  |         Status:  new
 Priority:  Medium       |      Component:  Metrics/Library
  Version:               |       Severity:  Normal
 Keywords:               |  Actual Points:
Parent ID:               |         Points:
 Reviewer:               |        Sponsor:
-------------------------+---------------------------------
 This ticket is created from #24419. #24419 notes that not all Onionoo
 methods could be changed to the isXY() pattern for boolean getter methods
 due to interfaces such as org.torproject.descriptor.NetworkStatusEntry.
 This ticket is to update metrics-libs accordingly. From #24419:


 >1. I did not change
 src/test/org/torproject/metrics/onionoo/updater/DummyStatusEntry's
 "getUnmeasured()" to "isUnmeasured()" because the interface
 org.torproject.descriptor.NetworkStatusEntry requires it, and I figured
 org.torproject.descriptor is outside the scope of this ticket.

 >>Agreed. We'd first have to change metrics-lib interfaces, put out a new
 major metrics-lib release, and then update the tests in Onionoo. Worth
 doing, though we would ideally combine this with other changes that
 require a major metrics-lib version bump.

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/28114>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online


More information about the metrics-bugs mailing list