[recipes.cfg] ONLY support jsonpb.

unittests/package_test.py

 #!/usr/bin/env python
 # Copyright 2015 The LUCI Authors. All rights reserved.
 # Use of this source code is governed under the Apache License, Version 2.0
 # that can be found in the LICENSE file.
 
 import copy
 import doctest
 import os
 import subprocess
 import unittest
@@ skipping 396 matching lines @@
         'repo/root', self.proto_file, allow_fetch=False)
 
   def test_dump_load_inverses(self):
     # Doubles as a test for equality reflexivity.
     package_spec = package.PackageSpec.load_proto(self.proto_file)
     self.assertEqual(self.proto_file.to_raw(package_spec.dump()),
                      self.proto_text)
     self.assertEqual(package.PackageSpec.load_proto(self.proto_file),
                      package_spec)
 
-  def test_dump_round_trips_text(self):
-    proto_text = """
-api_version: 1
-""".lstrip()
-    proto_file = MockProtoFile('repo/root/infra/config/recipes.cfg', proto_text)
-    package_spec = package.PackageSpec.load_proto(proto_file)
-    self.assertEqual(proto_file.to_raw(package_spec.dump()), proto_text)
-
-  def test_dump_round_trips_json(self):
+  def test_dump_round_trips(self):
     proto_text = """
 {"api_version": 1}
 """.lstrip()
     proto_file = MockProtoFile('repo/root/infra/config/recipes.cfg', proto_text)
     package_spec = package.PackageSpec.load_proto(proto_file)
     self.assertEqual(proto_file.to_raw(package_spec.dump()),
                      '{\n  "api_version": 1\n}')
 
   def test_no_version(self):
-    proto_text = """\
-{
+    proto_text = """{
   "project_id": "foo",
   "recipes_path": "path/to/recipes"
 }
 """
     proto_file = MockProtoFile('repo/root/infra/config/recipes.cfg', proto_text)
 
     with self.assertRaises(AssertionError):
       package.PackageSpec.load_proto(proto_file)
 
   def test_unsupported_version(self):
-    proto_text = """\
-api_version: 99999999
-project_id: "fizzbar"
-recipes_path: "path/to/recipes"
-"""
+    proto_text = """{
+  "api_version": 99999999,
+  "project_id": "fizzbar",
+  "recipes_path": "path/to/recipes"
+}"""
     proto_file = MockProtoFile('repo/root/infra/config/recipes.cfg', proto_text)
 
     with self.assertRaises(AssertionError):
       package.PackageSpec.load_proto(proto_file)
 
 
 class TestPackageDeps(MockIOThings, unittest.TestCase):
 
   def test_create_with_overrides(self):
-    base_proto_text = """
-api_version: 1
-project_id: "base_package"
-recipes_path: "path/to/recipes"
-deps {
-  project_id: "foo"
-  url: "https://repo.com/foo.git"
-  branch: "foobranch"
-  revision: "deadd00d"
+    base_proto_text = """{
+  "api_version": 1,
+  "project_id": "base_package",
+  "recipes_path": "path/to/recipes",
+  "deps": [
+    {
+      "project_id": "foo",
+      "url": "https://repo.com/foo.git",
+      "branch": "foobranch",
+      "revision": "deadd00d"
+    }
+  ]
 }
 """
     base_proto_file = MockProtoFile('base/infra/config/recipes.cfg',
                                     base_proto_text)
 
-    foo_proto_text = """
-api_version: 1
-project_id: "foo"
-recipes_path: "path/to/recipes"
-"""
+    foo_proto_text = """{
+  "api_version": 1,
+  "project_id": "foo",
+  "recipes_path": "path/to/recipes"
+}"""
     foo_proto_file = MockProtoFile('foo/infra/config/recipes.cfg',
                                    foo_proto_text)
 
     with mock.patch.object(package.GitRepoSpec, 'checkout') as checkout:
       with mock.patch.object(package.PathRepoSpec, 'proto_file',
                              return_value=foo_proto_file):
         deps = package.PackageDeps.create('base', base_proto_file, overrides={
           'foo': '/path/to/local/foo',
         })
 
       foo_deps = deps.get_package('foo')
       self.assertIsInstance(foo_deps.repo_spec, package.PathRepoSpec)
       self.assertEqual(foo_deps.repo_spec.path, '/path/to/local/foo')
       self.assertFalse(checkout.called)
 
 
 def load_tests(_loader, tests, _ignore):
   tests.addTests(doctest.DocTestSuite(package))
   return tests
 
 
 if __name__ == '__main__':
   result = unittest.main()