Add comparison url

Author: adangel

lib/pmdtester/builders/summary_report_builder.rb

@@ -57,7 +57,7 @@ def write_index(target_root, base_details, patch_details, projects)
       end
 
       env = {
-        'comparison_url' => nil, # todo
+        'comparison_url' => create_comparison_url(base_details, patch_details),
         'base' => to_liquid(base_details, BASE_CONFIG_NAME),
         'patch' => to_liquid(patch_details, PATCH_CONFIG_NAME),
         'projects' => projects
@@ -66,9 +66,15 @@ def write_index(target_root, base_details, patch_details, projects)
       render_and_write('project_index.html', "#{target_root}/index.html", env)
     end
 
+    def create_comparison_url(base_details, patch_details)
+      base = CGI.escape(base_details.branch_name)
+      patch = CGI.escape(patch_details.branch_last_sha)
+      "https://github.com/pmd/pmd/compare/#{base}...#{patch}"
+    end
+
     def to_liquid(details, config_name)
       {
-        'tree_url' => "https://github.com/pmd/pmd/tree/#{details.branch_last_sha}",
+        'tree_url' => "https://github.com/pmd/pmd/tree/#{CGI.escape(details.branch_last_sha)}",
         'name' => details.branch_name,
         'tip' => {
           'sha' => details.branch_last_sha,

test/resources/summary_report_builder/expected_index.html

@@ -28,11 +28,11 @@ <h2>Branch details</h2>
                 <tbody>
                 <tr>
                     <td class="item">Branch info</td>
-                    <td><a href="https://github.com/pmd/pmd/tree/test sha">test_branch test sha</a>: test message</td>
+                    <td><a href="https://github.com/pmd/pmd/tree/test+sha">test_branch test sha</a>: test message</td>
                     <td>
-                        <a href="https://github.com/pmd/pmd/tree/test sha">test_branch test sha</a>: test message
+                        <a href="https://github.com/pmd/pmd/tree/test+sha">test_branch test sha</a>: test message
                         <span class="external-link-secondary"><a href="https://github.com/pmd/pmd/pull/42">PR #42</a></span>
-                        <span class="external-link-secondary"><a href="">[Compare]</a></span>
+                        <span class="external-link-secondary"><a href="https://github.com/pmd/pmd/compare/test_branch...test+sha">[Compare]</a></span>
                     </td>
                 </tr>
                 <tr>