2011-01-06 Julie Parent <jparent@chromium.org>
authorjparent@chromium.org <jparent@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 7 Jan 2011 07:31:14 +0000 (07:31 +0000)
committerjparent@chromium.org <jparent@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 7 Jan 2011 07:31:14 +0000 (07:31 +0000)
        Reviewed by Eric Seidel.

        Test Result Server not removing old results.
        https://bugs.webkit.org/show_bug.cgi?id=52012

        * TestResultServer/model/jsonresults.py:
        Fix off-by-one: Truncate if >= num_runs, not just >.
        * TestResultServer/model/jsonresults_unittest.py:
        Add unit test to test that merging in a new result with the same value
        as the last result causes previous result to fall off.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@75234 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Tools/ChangeLog
Tools/TestResultServer/model/jsonresults.py
Tools/TestResultServer/model/jsonresults_unittest.py

index 1b94216..b2c1535 100644 (file)
@@ -1,3 +1,16 @@
+2011-01-06  Julie Parent  <jparent@chromium.org>
+
+        Reviewed by Eric Seidel.
+
+        Test Result Server not removing old results.
+        https://bugs.webkit.org/show_bug.cgi?id=52012
+
+        * TestResultServer/model/jsonresults.py:
+        Fix off-by-one: Truncate if >= num_runs, not just >.
+        * TestResultServer/model/jsonresults_unittest.py:
+        Add unit test to test that merging in a new result with the same value
+        as the last result causes previous result to fall off.
+
 2011-01-06  James Kozianski  <koz@chromium.org>
 
         Reviewed by Mihai Parparita.
index 8e507a2..d61a860 100755 (executable)
@@ -305,7 +305,7 @@ class JsonResults(object):
         for result in encoded_list:
             num_builds = num_builds + result[0]
             index = index + 1
-            if num_builds > num_runs:
+            if num_builds >= num_runs:
                 return encoded_list[:index]
 
         return encoded_list
index 03f1b46..eb8c864 100755 (executable)
@@ -322,6 +322,18 @@ class JsonResultsTest(unittest.TestCase):
             (["3", "2", "1"], [["001.html", "[1,\"T\"],[" + max_builds + ",\"F\"]", "[1,1],[" + max_builds + ",0]"]]),
             int(max_builds))
 
+        # Test that merging in a new result of the same type as the last result
+        # causes old results to fall off.
+        max_builds = str(jsonresults.JSON_RESULTS_MAX_BUILDS_SMALL)
+        self._test_merge(
+            # Aggregated results
+            (["2", "1"], [["001.html", "[" + max_builds + ",\"F\"],[1,\"N\"]", "[" + max_builds + ",0],[1,1]"]]),
+            # Incremental results
+            (["3"], [["001.html", "[1,\"F\"]", "[1,0]"]]),
+            # Expected results
+            (["3", "2", "1"], [["001.html", "[" + max_builds + ",\"F\"]", "[" + max_builds + ",0]"]]),
+            int(max_builds))
+
         # Get test name list only. Don't include non-test-list data and
         # of test result details.
         self._test_get_test_list(