google-nomulus/javatests/google/registry/dns
guyben 8cfb748316 Remove TLD @Parameter from inside the DnsMetrics
Currently, the TLD is a "hidden" part of the metric - we record it, but by looking at the call site you can't see that we record it.

Also, it's injected from the query / POST parameter, so we might not even be aware of what the value is and it might not be the value we wanted.

Instead, making it explicit in the Metric call. That way it's also more similar to the "logging" statements that record the same data but have to explicitly output the TLD.

It also makes the tests better, as we test that we indeed record the correct TLD

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=229550115
2019-01-17 19:12:41 -05:00
..
writer Make loadByForeignKey() and related methods return Optional 2018-12-14 11:24:49 -05:00
BUILD Clean up annotation imports in BUILD files 2018-06-27 15:28:53 -04:00
DnsInjectionTest.java Remove more unnecessary "throws" declarations 2018-06-27 15:28:52 -04:00
DnsQueueTest.java Remove more unnecessary "throws" declarations 2018-06-27 15:28:52 -04:00
DnsTestComponent.java Update copyright year on all license headers 2017-02-02 16:27:22 -05:00
PublishDnsUpdatesActionTest.java Remove TLD @Parameter from inside the DnsMetrics 2019-01-17 19:12:41 -05:00
ReadDnsQueueActionTest.java Remove more unnecessary "throws" declarations 2018-06-27 15:28:52 -04:00
RefreshDnsActionTest.java Remove unnecessary "throws" declarations 2018-06-18 18:17:56 -04:00