From d0ca64a9a4d68ee76b519b164e85314d3194bbff Mon Sep 17 00:00:00 2001 From: Michael Comella Date: Mon, 2 Aug 2021 15:20:53 -0700 Subject: [PATCH] [fenix] Closes https://github.com/mozilla-mobile/fenix/issues/20631: remove @Ignore on StorageStats tests. The errors that caused this to be @Ignored were addressed by a recent PR landing on master (i.e. the one that renewed the probes this test is testing). --- .../java/org/mozilla/fenix/perf/StorageStatsMetricsTest.kt | 3 --- 1 file changed, 3 deletions(-) diff --git a/app/src/test/java/org/mozilla/fenix/perf/StorageStatsMetricsTest.kt b/app/src/test/java/org/mozilla/fenix/perf/StorageStatsMetricsTest.kt index a9c6cc076a..e66a69fb84 100644 --- a/app/src/test/java/org/mozilla/fenix/perf/StorageStatsMetricsTest.kt +++ b/app/src/test/java/org/mozilla/fenix/perf/StorageStatsMetricsTest.kt @@ -16,7 +16,6 @@ import mozilla.components.support.test.robolectric.testContext import org.junit.Assert.assertEquals import org.junit.Assert.assertTrue import org.junit.Before -import org.junit.Ignore import org.junit.Rule import org.junit.Test import org.junit.runner.RunWith @@ -42,7 +41,6 @@ class StorageStatsMetricsTest { } @Test - @Ignore("These metrics have expired.") fun `WHEN reporting THEN the values from the storageStats are accumulated`() { every { storageStats.appBytes } returns 100 every { storageStats.cacheBytes } returns 200 @@ -56,7 +54,6 @@ class StorageStatsMetricsTest { } @Test - @Ignore("These metrics have expired.") fun `WHEN reporting THEN the query duration is measured`() { StorageStatsMetrics.reportSync(mockContext) assertTrue(Metrics.queryStatsDuration.testHasValue())