Presubmit checks for user actions intorduced in HTML files.

chrome/browser/resources/PRESUBMIT_test.py

Index: chrome/browser/resources/PRESUBMIT_test.py
diff --git a/chrome/browser/resources/PRESUBMIT_test.py b/chrome/browser/resources/PRESUBMIT_test.py
new file mode 100644
index 0000000000000000000000000000000000000000..f6f7e52e8e92557ce0dac60fb58d6e764c39a132
--- /dev/null
+++ b/chrome/browser/resources/PRESUBMIT_test.py
@@ -0,0 +1,77 @@
+# Copyright 2014 The Chromium Authors. All rights reserved.
+# Use of this source code is governed by a BSD-style license that can be
+# found in the LICENSE file.
+
+import os
+import sys
+import imp
+import unittest
+import PRESUBMIT
+
+sys.path.append(os.path.abspath(os.path.dirname(__file__) + '/../../../'))
+from PRESUBMIT_test_mocks import MockFile, MockInputApi, MockOutputApi
+
+_TEST_DATA_DIR = 'base/test/data/presubmit/'
+_ACTION_XML_PATH = _TEST_DATA_DIR + "actions.xml"
+
+class HTMLActionAdditionTest(unittest.TestCase):
+
+  def testActionXMLChanged(self):
+    mock_input_api = MockInputApi()
+    mock_input_api.files = [
+      MockFile('path/valid.html', ''),
+      MockFile('actions.xml', '') ]
+
+    self.assertEqual([], PRESUBMIT.CheckUserActionUpdate(mock_input_api,
+                                                         MockOutputApi(),
+                                                         _ACTION_XML_PATH))
+
+  def testValidChange_StartOfLine(self):
+    lines = ['<input id="testinput" pref="testpref"',
+             'metric="validaction" type="checkbox" dialog-pref>']
+    self.assertEqual([], self._testChange(lines))
+
+  def testValidChange_StartsWithSpace(self):
+    lines = ['<input id="testinput" pref="testpref"',
+             '  metric="validaction" type="checkbox" dialog-pref>']
+    self.assertEqual([], self._testChange(lines))
+
+  def testValidChange_Radio(self):
+    lines = ['<input id="testinput" pref="testpref"',
+             '  metric="validaction" type="radio" dialog-pref value="true">']
+    self.assertEqual([], self._testChange(lines))
+
+  def testValidChange_UsingDatatype(self):
+    lines = ['<input id="testinput" pref="testpref"',
+             '  metric="validaction" datatype="boolean" dialog-pref>']
+    self.assertEqual([], self._testChange(lines))
+
+  def testValidChange_NotBoolean(self):
+    lines = ['<input id="testinput" pref="testpref"',
+             '  metric="notboolean_validaction" dialog-pref>']
+    self.assertEqual([], self._testChange(lines))
+
+  def testInvalidChange(self):
+    lines = ['<input id="testinput" pref="testpref"',
+             'metric="invalidaction" type="checkbox" dialog-pref>']
+    warnings = self._testChange(lines)
+    self.assertEqual(1, len(warnings))
+
+  def testInValidChange_Radio(self):
+    lines = ['<input id="testinput" pref="testpref"',
+             '  metric="validaction" type="radio" dialog-pref value="string">']
+    warnings = self._testChange(lines)
+    self.assertEqual(1, len(warnings))
+
+  def _testChange(self, lines):
+    action_xml_path = _TEST_DATA_DIR + "actions.xml"
+    mock_input_api = MockInputApi()
+    mock_input_api.files = [MockFile('path/test.html', lines)]
+
+    return PRESUBMIT.CheckUserActionUpdate(mock_input_api,
+                                               MockOutputApi(),

[M-A Ruel, 2014/11/28 17:34:25]

alignment. That's one of the reason I always use +4 instead.

[gayane -on leave until 09-2017, 2014/11/28 20:43:14]

Done.

+                                               _ACTION_XML_PATH)
+
+
+if __name__ == '__main__':
+  unittest.main()