[test+presubmit] Remove duplicate test status file entries

tools/testrunner/local/statusfile.py

 # Copyright 2012 the V8 project authors. 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 in the documentation and/or other materials provided
@@ skipping 138 matching lines @@
       if rule[-1] == '*':
         _ParseOutcomeList(rule, section[rule], wildcards, variables)
       else:
         _ParseOutcomeList(rule, section[rule], rules, variables)
   return rules, wildcards
 
 
 def PresubmitCheck(path):
   contents = ReadContent(path)
   root_prefix = os.path.basename(os.path.dirname(path)) + "/"
+  def _Assert(check, message, status):

[Michael Achenbach, 2015/11/27 14:24:20]

nit: naming (optional). Since it's nested, the _ is not necessary and functions
(at least nested) usually start with small letter. Unfortunately "assert" is
already taken ;) Maybe call it check?

[Jakob Kummerow, 2015/11/27 14:43:26]

Compromise: _assert (to emphasize that it's a customized variant of the default
"assert").

+    if not check:
+      print("%s: Error: %s" % (path, message))
+      status["success"] = False
 
+  status = {"success": True}

[Michael Achenbach, 2015/11/27 14:24:20]

As talked offline. You could move this a few lines up and then skip passing it
as additional parameter.

[Jakob Kummerow, 2015/11/27 14:43:26]

Done.

   try:
     for section in contents:
-      assert type(section) == list
-      assert len(section) == 2
+      _Assert(type(section) == list, "Section must be a list", status)

[Michael Achenbach, 2015/11/27 14:24:20]

Just for the record. Also as discussed offline. If any of these checks really
fails, the subsequent ones might run into less readable errors.

E.g. a section that is an int will trigger the first _Assert and then cause an
exception on the len call below...

[Jakob Kummerow, 2015/11/27 14:43:26]

Keeping as is, reasoning: it's rather unlikely that someone would run into
these, so we shouldn't overengineer it.

+      _Assert(len(section) == 2, "Section list must have exactly 2 entries",
+              status)
       section = section[1]
-      assert type(section) == dict
+      _Assert(type(section) == dict,
+              "Second entry of section must be a dictionary", status)
       for rule in section:
-        assert type(rule) == str
-        assert not rule.startswith(root_prefix), (
-            "Suite name prefix must not be used in status files")
-        assert not rule.endswith('.js'), (
-            ".js extension must not be used in status files.")
-    return True
+        _Assert(type(rule) == str, "Rule key must be a string", status)
+        _Assert(not rule.startswith(root_prefix),
+                "Suite name prefix must not be used in rule keys", status)
+        _Assert(not rule.endswith('.js'),
+                ".js extension must not be used in rule keys.", status)
+    return status["success"]
   except Exception as e:
     print e
     return False