Changeset 85657 in webkit

Timestamp:

May 3, 2011 1:57:14 PM (13 years ago)

Author:

caio.oliveira@openbossa.org

Message:

2011-05-03 Caio Marcelo de Oliveira Filho <​caio.oliveira@openbossa.org>

Reviewed by Adam Roben.

Add tests to PrettyPatch
​https://bugs.webkit.org/show_bug.cgi?id=57298

Verify that 'prettify' parse the right number of files and parts
(add/remove/shared) by checking a set of patches available in
bugs.webkit.org.

• PrettyPatch/PrettyPatch.rb: exported a couple a globals with statistics information for our test script.
• PrettyPatch/PrettyPatch_test.rb: Added.

Location:

trunk/Websites/bugs.webkit.org

Files:

• ChangeLog (modified) (1 diff)
• PrettyPatch/PrettyPatch.rb (modified) (3 diffs)
• PrettyPatch/PrettyPatch_test.rb (added)

trunk/Websites/bugs.webkit.org/ChangeLog

@@ +1 @@
+2011-05-03  Caio Marcelo de Oliveira Filho  <caio.oliveira@openbossa.org>
+
+        Reviewed by Adam Roben.
+
+        Add tests to PrettyPatch
+        https://bugs.webkit.org/show_bug.cgi?id=57298
+
+        Verify that 'prettify' parse the right number of files and parts
+        (add/remove/shared) by checking a set of patches available in
+        bugs.webkit.org.
+
+        * PrettyPatch/PrettyPatch.rb: exported a couple a globals with
+        statistics information for our test script.
+        * PrettyPatch/PrettyPatch_test.rb: Added.
+
 2011-04-22  Alexey Proskuryakov  <ap@apple.com>
 

trunk/Websites/bugs.webkit.org/PrettyPatch/PrettyPatch.rb

@@ -13 +13 @@
 
     def self.prettify(string)
+        $last_prettify_file_count = -1
+        $last_prettify_part_count = { "remove" => 0, "add" => 0, "shared" => 0 }
         string = normalize_line_ending(string)
         fileDiffs = FileDiff.parse(string)
@@ -28 +30 @@
         end
 
+        $last_prettify_file_count = fileDiffs.count
         str += fileDiffs.collect{ |diff| diff.to_html }.join
     end
@@ -683 +686 @@
 
         def initialize(className, container)
+            $last_prettify_part_count[className] += 1
             @className = className
             @lines = []