Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit cde65cd9 authored by Jiyong Park's avatar Jiyong Park Committed by Automerger Merge Worker
Browse files

Handle the case when non-optional props have the same value am: 5df5873d 

Original change: https://googleplex-android-review.googlesource.com/c/platform/build/+/12042312

Change-Id: I2d1dbddef59f581d83e22d13ef1ea22dd5fd3abe
parents b2a8bc84 5df5873d

tools/post_process_props.py

+2 −0
Original line number Diff line number Diff line
@@ -88,6 +88,8 @@ def override_optional_props(prop_list, allow_dup=False): 
    if len(overriding_props) > 1:

      # duplicated props are allowed when the all have the same value

      if all(overriding_props[0].value == p.value for p in overriding_props):

        for p in optional_props:

          p.delete("overridden by %s" % str(overriding_props[0]))

        continue

      # or if dup is explicitly allowed for compat reason

      if allow_dup:

tools/test_post_process_props.py

+6 −0
Original line number Diff line number Diff line
@@ -221,11 +221,17 @@ class PropListTestcase(unittest.TestCase): 
      stderr_redirect = io.StringIO()

      with contextlib.redirect_stderr(stderr_redirect):

        props = PropList("hello")

        optional_prop = props.get_props("foo")[2] # the last foo?=false one



        # we have duplicated foo=true and foo=true, but that's allowed

        # since they have the same value

        self.assertTrue(override_optional_props(props))



        # foo?=false should be commented out

        self.assertEqual("# Removed by post_process_props.py because " +

                         "overridden by foo=true\n#foo?=false",

                         str(optional_prop))



  def test_allowDuplicates(self):

    content = """

    # comment