Presubmit checks for user actions intorduced in HTML files.

chrome/browser/resources/PRESUBMIT_test.py

+# 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__) + '/../../../'))

[M-A Ruel, 2014/11/26 20:43:27]

Normally, it's safer to do:

os.path.dirname(os.path.abspath(__file__))

Then you'd loop 4 times. You may want to consider doing this instead of
../../../ due to windows path length limitation.

[gayane -on leave until 09-2017, 2014/11/26 21:57:28]

Do you mean to have something like this?
sys.path.append(os.path.dirname(os.path.dirname(os.path.dirname(os.path.dirname(os.path.abspath(__file__))))))
from PRESUBMIT_test_mocks import MockFile, MockInputApi, MockOutputApi

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

Yes but if it works, it's fine with me.

+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', '') ]
+
+    warnings = PRESUBMIT.CheckUserActionUpdate(mock_input_api,
+                                               MockOutputApi(),
+                                               _ACTION_XML_PATH)
+    self.assertEqual(0, len(warnings))

[M-A Ruel, 2014/11/26 20:43:27]

I meant for this one too.

[gayane -on leave until 09-2017, 2014/11/26 21:57:28]

Done.

+
+  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(),
+                                               _ACTION_XML_PATH)
+
+
+if __name__ == '__main__':
+  unittest.main()