Merge "Connectivity metrics: simplify ConnectivityMetricsEvent"

This commit is contained in:
Hugo Benichi
2017-03-16 00:37:43 +00:00
committed by Android (Google) Code Review
2 changed files with 8 additions and 4 deletions

View File

@@ -315,14 +315,15 @@ public class IpConnectivityMetricsTest extends TestCase {
} }
static ConnectivityMetricsEvent expectedEvent(int timestamp) { static ConnectivityMetricsEvent expectedEvent(int timestamp) {
return new ConnectivityMetricsEvent((long)timestamp, 0, 0, FAKE_EV); ConnectivityMetricsEvent ev = new ConnectivityMetricsEvent();
ev.timestamp = timestamp;
ev.data = FAKE_EV;
return ev;
} }
/** Outer equality for ConnectivityMetricsEvent to avoid overriding equals() and hashCode(). */ /** Outer equality for ConnectivityMetricsEvent to avoid overriding equals() and hashCode(). */
static void assertEventsEqual(ConnectivityMetricsEvent expected, ConnectivityMetricsEvent got) { static void assertEventsEqual(ConnectivityMetricsEvent expected, ConnectivityMetricsEvent got) {
assertEquals(expected.timestamp, got.timestamp); assertEquals(expected.timestamp, got.timestamp);
assertEquals(expected.componentTag, got.componentTag);
assertEquals(expected.eventTag, got.eventTag);
assertEquals(expected.data, got.data); assertEquals(expected.data, got.data);
} }

View File

@@ -28,7 +28,10 @@ abstract public class MetricsTestUtil {
} }
static ConnectivityMetricsEvent ev(Parcelable p) { static ConnectivityMetricsEvent ev(Parcelable p) {
return new ConnectivityMetricsEvent(1L, 0, 0, p); ConnectivityMetricsEvent ev = new ConnectivityMetricsEvent();
ev.timestamp = 1L;
ev.data = p;
return ev;
} }
static ConnectivityMetricsEvent describeIpEvent(Consumer<Parcel>... fs) { static ConnectivityMetricsEvent describeIpEvent(Consumer<Parcel>... fs) {