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

Unified Diff: dashboard/dashboard/pinpoint/handlers/quest_generator_test.py

Issue 2998283002: [pinpoint] Add GTest support to quest_generator. (Closed)
Patch Set: Rebase Created 3 years, 4 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: dashboard/dashboard/pinpoint/handlers/quest_generator_test.py
diff --git a/dashboard/dashboard/pinpoint/handlers/quest_generator_test.py b/dashboard/dashboard/pinpoint/handlers/quest_generator_test.py
index 1aede9becbf4605a016435a37b39cd7a2bcf4850..45e3b37dbf1db0578c9e89a557918f702f63c11e 100644
--- a/dashboard/dashboard/pinpoint/handlers/quest_generator_test.py
+++ b/dashboard/dashboard/pinpoint/handlers/quest_generator_test.py
@@ -8,7 +8,7 @@ from dashboard.pinpoint.handlers import quest_generator
from dashboard.pinpoint.models import quest
-_MIN_RUN_TEST_ARGUMENTS = [
+_MIN_TELEMETRY_RUN_TEST_ARGUMENTS = [
'speedometer', '--pageset-repeat', '20', '--browser', 'release',
'-v', '--upload-results', '--output-format', 'chartjson',
'--isolated-script-test-output', '${ISOLATED_OUTDIR}/output.json',
@@ -17,7 +17,7 @@ _MIN_RUN_TEST_ARGUMENTS = [
]
-_ALL_RUN_TEST_ARGUMENTS = [
+_ALL_TELEMETRY_RUN_TEST_ARGUMENTS = [
'speedometer', '--story-filter', 'http://www.fifa.com/',
'--pageset-repeat', '50', '--browser', 'release',
'-v', '--upload-results', '--output-format', 'chartjson',
@@ -27,6 +27,22 @@ _ALL_RUN_TEST_ARGUMENTS = [
]
+_MIN_GTEST_RUN_TEST_ARGUMENTS = [
+ '--gtest_repeat', '20',
+ '--isolated-script-test-output', '${ISOLATED_OUTDIR}/output.json',
+ '--isolated-script-test-chartjson-output',
+ '${ISOLATED_OUTDIR}/chartjson-output.json',
+]
+
+
+_ALL_GTEST_RUN_TEST_ARGUMENTS = [
+ '--gtest_filter', 'test_name', '--gtest_repeat', '50',
+ '--isolated-script-test-output', '${ISOLATED_OUTDIR}/output.json',
+ '--isolated-script-test-chartjson-output',
+ '${ISOLATED_OUTDIR}/chartjson-output.json',
+]
+
+
class FindIsolateTest(unittest.TestCase):
def testMissingArguments(self):
@@ -52,7 +68,7 @@ class FindIsolateTest(unittest.TestCase):
(arguments, expected_quests))
-class TelemetryRunTestQuest(unittest.TestCase):
+class TelemetryRunTest(unittest.TestCase):
def testMissingArguments(self):
arguments = {
@@ -86,7 +102,7 @@ class TelemetryRunTestQuest(unittest.TestCase):
expected_quests = [
quest.FindIsolate('chromium-rel-mac11-pro', 'telemetry_perf_tests'),
- quest.RunTest({}, _MIN_RUN_TEST_ARGUMENTS),
+ quest.RunTest({}, _MIN_TELEMETRY_RUN_TEST_ARGUMENTS),
]
self.assertEqual(quest_generator.GenerateQuests(arguments),
(arguments, expected_quests))
@@ -104,13 +120,48 @@ class TelemetryRunTestQuest(unittest.TestCase):
expected_quests = [
quest.FindIsolate('chromium-rel-mac11-pro', 'telemetry_perf_tests'),
- quest.RunTest({'key': 'value'}, _ALL_RUN_TEST_ARGUMENTS),
+ quest.RunTest({'key': 'value'}, _ALL_TELEMETRY_RUN_TEST_ARGUMENTS),
+ ]
+ self.assertEqual(quest_generator.GenerateQuests(arguments),
+ (arguments, expected_quests))
+
+
+class GTestRunTest(unittest.TestCase):
+
+ def testMinimumArguments(self):
+ arguments = {
+ 'configuration': 'chromium-rel-mac11-pro',
+ 'target': 'net_perftests',
+ 'dimensions': '{}',
+ }
+
+ expected_quests = [
+ quest.FindIsolate('chromium-rel-mac11-pro', 'net_perftests'),
+ quest.RunTest({}, _MIN_GTEST_RUN_TEST_ARGUMENTS),
+ ]
+ print quest_generator.GenerateQuests(arguments)[1][1]._extra_args
+ self.assertEqual(quest_generator.GenerateQuests(arguments),
+ (arguments, expected_quests))
+
+ def testAllArguments(self):
+ arguments = {
+ 'configuration': 'chromium-rel-mac11-pro',
+ 'target': 'net_perftests',
+ 'dimensions': '{"key": "value"}',
+ 'test': 'test_name',
+ 'repeat_count': '50',
+ }
+
+ expected_quests = [
+ quest.FindIsolate('chromium-rel-mac11-pro', 'net_perftests'),
+ quest.RunTest({'key': 'value'}, _ALL_GTEST_RUN_TEST_ARGUMENTS),
]
+ print quest_generator.GenerateQuests(arguments)[1][1]._extra_args
self.assertEqual(quest_generator.GenerateQuests(arguments),
(arguments, expected_quests))
-class ReadChartJsonValueQuest(unittest.TestCase):
+class ReadChartJsonValue(unittest.TestCase):
def testMinimumArguments(self):
arguments = {
@@ -124,7 +175,7 @@ class ReadChartJsonValueQuest(unittest.TestCase):
expected_quests = [
quest.FindIsolate('chromium-rel-mac11-pro', 'telemetry_perf_tests'),
- quest.RunTest({}, _MIN_RUN_TEST_ARGUMENTS),
+ quest.RunTest({}, _MIN_TELEMETRY_RUN_TEST_ARGUMENTS),
quest.ReadChartJsonValue('pcv1-cold@@timeToFirst', None),
]
self.assertEqual(quest_generator.GenerateQuests(arguments),
@@ -144,8 +195,55 @@ class ReadChartJsonValueQuest(unittest.TestCase):
expected_quests = [
quest.FindIsolate('chromium-rel-mac11-pro', 'telemetry_perf_tests'),
- quest.RunTest({'key': 'value'}, _ALL_RUN_TEST_ARGUMENTS),
+ quest.RunTest({'key': 'value'}, _ALL_TELEMETRY_RUN_TEST_ARGUMENTS),
quest.ReadChartJsonValue('pcv1-cold@@timeTo', 'http://www.fifa.com/'),
]
self.assertEqual(quest_generator.GenerateQuests(arguments),
(arguments, expected_quests))
+
+
+class ReadGraphJsonValue(unittest.TestCase):
+
+ def testMissingArguments(self):
+ arguments = {
+ 'configuration': 'chromium-rel-mac11-pro',
+ 'target': 'net_perftests',
+ 'dimensions': '{"key": "value"}',
+ 'test': 'test_name',
+ 'repeat_count': '50',
+ 'trace': 'trace_name',
+ }
+
+ with self.assertRaises(TypeError):
+ quest_generator.GenerateQuests(arguments)
+
+ arguments = {
+ 'configuration': 'chromium-rel-mac11-pro',
+ 'target': 'net_perftests',
+ 'dimensions': '{"key": "value"}',
+ 'test': 'test_name',
+ 'repeat_count': '50',
+ 'chart': 'chart_name',
+ }
+
+ with self.assertRaises(TypeError):
+ quest_generator.GenerateQuests(arguments)
+
+ def testAllArguments(self):
+ arguments = {
+ 'configuration': 'chromium-rel-mac11-pro',
+ 'target': 'net_perftests',
+ 'dimensions': '{"key": "value"}',
+ 'test': 'test_name',
+ 'repeat_count': '50',
+ 'chart': 'chart_name',
+ 'trace': 'trace_name',
+ }
+
+ expected_quests = [
+ quest.FindIsolate('chromium-rel-mac11-pro', 'net_perftests'),
+ quest.RunTest({'key': 'value'}, _ALL_GTEST_RUN_TEST_ARGUMENTS),
+ quest.ReadGraphJsonValue('chart_name', 'trace_name'),
+ ]
+ self.assertEqual(quest_generator.GenerateQuests(arguments),
+ (arguments, expected_quests))
« no previous file with comments | « dashboard/dashboard/pinpoint/handlers/quest_generator.py ('k') | dashboard/dashboard/pinpoint/models/quest/__init__.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698