Add JSON support to lint-test-expectations

Tools/Scripts/webkitpy/layout_tests/lint_test_expectations_unittest.py

 # Copyright (C) 2012 Google Inc. All rights reserved.
 #
 # Redistribution and use in source and binary forms, with or without
 # modification, are permitted provided that the following conditions are
 # met:
 #
 #     * Redistributions of source code must retain the above copyright
 # notice, this list of conditions and the following disclaimer.
 #     * Redistributions in binary form must reproduce the above
 # copyright notice, this list of conditions and the following disclaimer
@@ skipping 76 matching lines @@
                                                FakePort(host, 'b', 'path-to-b'),
                                                FakePort(host, 'b-win', 'path-to-b')))
 
         logging_stream = StringIO.StringIO()
         options = optparse.Values({'platform': None})
         logger, handler = lint_test_expectations.set_up_logging(logging_stream)
         try:
             res = lint_test_expectations.lint(host, options)
         finally:
             lint_test_expectations.tear_down_logging(logger, handler)
-        self.assertEqual(res, 0)
+        self.assertEqual(res, [])
         self.assertEqual(host.ports_parsed, ['a', 'b', 'b-win'])
 
     def test_lint_test_files(self):
         logging_stream = StringIO.StringIO()
         options = optparse.Values({'platform': 'test-mac-leopard'})
         host = MockHost()
 
         # pylint appears to complain incorrectly about the method overrides pylint: disable=E0202,C0322
         # FIXME: incorrect complaints about spacing pylint: disable=C0322
         host.port_factory.all_port_names = lambda platform=None: [platform]
 
         logger, handler = lint_test_expectations.set_up_logging(logging_stream)
         try:
             res = lint_test_expectations.lint(host, options)
-            self.assertEqual(res, 0)
+            self.assertEqual(res, [])
         finally:
             lint_test_expectations.tear_down_logging(logger, handler)
 
 
     def test_lint_test_files__errors(self):
         options = optparse.Values({'platform': 'test', 'debug_rwt_logging': False})
         host = MockHost()
 
         # FIXME: incorrect complaints about spacing pylint: disable=C0322
         port = host.port_factory.get(options.platform, options=options)
@@ skipping 33 matching lines @@
         finally:
             lint_test_expectations.tear_down_logging(logger, handler)
 
 
 class MainTest(unittest.TestCase):
     # unused args pylint: disable=W0613
 
     def setUp(self):
         self.orig_lint_fn = lint_test_expectations.lint
         self.orig_check_fn = lint_test_expectations.check_virtual_test_suites
-        lint_test_expectations.check_virtual_test_suites = lambda host, options: False
+        lint_test_expectations.check_virtual_test_suites = lambda host, options: []
 
         self.stdout = StringIO.StringIO()
         self.stderr = StringIO.StringIO()
 
     def tearDown(self):
         lint_test_expectations.lint = self.orig_lint_fn
         lint_test_expectations.check_virtual_test_suites = self.orig_check_fn
 
     def test_success(self):
-        lint_test_expectations.lint = lambda host, options: False
+        lint_test_expectations.lint = lambda host, options: []
         res = lint_test_expectations.main(['--platform', 'test'], self.stdout, self.stderr)
         self.assertTrue('Lint succeeded' in self.stderr.getvalue())
         self.assertEqual(res, 0)
 
     def test_failure(self):
-        lint_test_expectations.lint = lambda host, options: True
+        lint_test_expectations.lint = lambda host, options: ['test failure']
         res = lint_test_expectations.main(['--platform', 'test'], self.stdout, self.stderr)
         self.assertTrue('Lint failed' in self.stderr.getvalue())
         self.assertEqual(res, 1)
 
     def test_interrupt(self):
         def interrupting_lint(host, options):
             raise KeyboardInterrupt
 
         lint_test_expectations.lint = interrupting_lint
         res = lint_test_expectations.main([], self.stdout, self.stderr)
         self.assertEqual(res, lint_test_expectations.INTERRUPTED_EXIT_STATUS)
 
     def test_exception(self):
         def exception_raising_lint(host, options):
             assert False
         lint_test_expectations.lint = exception_raising_lint
         res = lint_test_expectations.main([], self.stdout, self.stderr)
         self.assertEqual(res, lint_test_expectations.EXCEPTIONAL_EXIT_STATUS)