[metrics-bugs] #29425 [Metrics/Statistics]: Write integration tests for data-processing modules
Tor Bug Tracker & Wiki
blackhole at torproject.org
Thu Mar 7 15:51:02 UTC 2019
#29425: Write integration tests for data-processing modules
-------------------------------------+--------------------------
Reporter: karsten | Owner: karsten
Type: enhancement | Status: accepted
Priority: Medium | Milestone:
Component: Metrics/Statistics | Version:
Severity: Normal | Resolution:
Keywords: metrics-roadmap-2019-q2 | Actual Points:
Parent ID: | Points: 8
Reviewer: | Sponsor:
-------------------------------------+--------------------------
Comment (by karsten):
I finally got it all working! There are still minor differences between
macOS and Linux:
- `bandwidth.csv` relies on the import order of descriptors, because
either the first or last descriptor wins (I did not look at the code in
detail). The differences in the output are tiny, but detectable. However,
fixing this requires extensive changes to the bwhist module.
- `platforms.csv` and `userstats-combined.csv` rely on sorting order,
which is apparently platform-dependent. After sorting these files, they're
equivalent. This is unfortunate, but acceptable, because we do not
guarantee a specific sorting order. We could fix this by changing SQL
statements to something like ORDER BY LOWER(x).
I decided to ignore these differences and expect the output as Linux would
produce it.
I also made a temporary decision regarding git-annex vs. adding large
files to git by creating this relatively large repository on GitHub for
now: https://github.com/kloesing/metrics-test
I'd like to use these integration tests for some more actual metrics-web
changes, including the Java 8 Date/Time updates. It's okay for me to just
keep it on GitHub without review and later get it reviewed and on Tor's
Git. For example, we could do that in April with more time on our hands.
Hope that works for you!
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/29425#comment:13>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
More information about the metrics-bugs
mailing list