Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(250)

Unified Diff: tools/telemetry/telemetry/internal/results/html_output_formatter_unittest.py

Issue 1309143006: [Telemetry] Update the ConfindenceInterval calculation in results.html to use (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address petrcermak's comment Created 5 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: tools/telemetry/telemetry/internal/results/html_output_formatter_unittest.py
diff --git a/tools/telemetry/telemetry/internal/results/html_output_formatter_unittest.py b/tools/telemetry/telemetry/internal/results/html_output_formatter_unittest.py
index 178dfd60ac0905a1feca5eb22756786a436d6e40..5ea8f5ba579c43fa5f26c31e5ab3cc55e2ff9c62 100644
--- a/tools/telemetry/telemetry/internal/results/html_output_formatter_unittest.py
+++ b/tools/telemetry/telemetry/internal/results/html_output_formatter_unittest.py
@@ -78,6 +78,7 @@ class HtmlOutputFormatterTest(unittest.TestCase):
"metrics": {
"a": {
"current": [3, 7],
+ "std": 2.8284271247461903,
"units": "seconds",
"important": True
},
@@ -88,11 +89,13 @@ class HtmlOutputFormatterTest(unittest.TestCase):
},
"a.http://www.bar.com/": {
"current": [7],
+ "std": 0.0,
"units": "seconds",
"important": False
},
"a.http://www.foo.com/": {
"current": [3],
+ "std": 0.0,
"units": "seconds",
"important": False
}
@@ -131,6 +134,7 @@ class HtmlOutputFormatterTest(unittest.TestCase):
"a": {
"current": [3, 7],
"units": "seconds",
+ "std": 2.8284271247461903,
"important": True
},
"telemetry_page_measurement_results.num_failed": {
@@ -140,11 +144,13 @@ class HtmlOutputFormatterTest(unittest.TestCase):
},
"a.http://www.bar.com/": {
"current": [7],
+ "std": 0.0,
"units": "seconds",
"important": False
},
"a.http://www.foo.com/": {
"current": [3],
+ "std": 0.0,
"units": "seconds",
"important": False
}
@@ -161,6 +167,7 @@ class HtmlOutputFormatterTest(unittest.TestCase):
"metrics": {
"a": {
"current": [4, 8],
+ 'std': 2.8284271247461903,
"units": "seconds",
"important": True
},
@@ -171,11 +178,13 @@ class HtmlOutputFormatterTest(unittest.TestCase):
},
"a.http://www.bar.com/": {
"current": [8],
+ "std": 0.0,
"units": "seconds",
"important": False
},
"a.http://www.foo.com/": {
"current": [4],
+ "std": 0.0,
"units": "seconds",
"important": False
}
@@ -213,6 +222,7 @@ class HtmlOutputFormatterTest(unittest.TestCase):
"metrics": {
"a": {
"current": [5, 9],
+ 'std': 2.8284271247461903,
"units": "seconds",
"important": True
},
@@ -223,11 +233,13 @@ class HtmlOutputFormatterTest(unittest.TestCase):
},
"a.http://www.bar.com/": {
"current": [9],
+ "std": 0.0,
"units": "seconds",
"important": False
},
"a.http://www.foo.com/": {
"current": [5],
+ "std": 0.0,
"units": "seconds",
"important": False
}

Powered by Google App Engine
This is Rietveld 408576698