SONAR-4095 Refactoring of the resource viewer in order to use accordion navigation

1 files changed, 12 insertions, 12 deletions

diff --git a/plugins/sonar-core-plugin/src/main/resources/org/sonar/plugins/core/web/tests_viewer.html.erb b/plugins/sonar-core-plugin/src/main/resources/org/sonar/plugins/core/web/tests_viewer.html.erb
index db5484695cc..122fd22b7cc 100644
--- a/plugins/sonar-core-plugin/src/main/resources/org/sonar/plugins/core/web/tests_viewer.html.erb
+++ b/plugins/sonar-core-plugin/src/main/resources/org/sonar/plugins/core/web/tests_viewer.html.erb
@@ -1,8 +1,8 @@
-<div class="tab_header tests_viewer" id="tests_header">
+<div class="tab_header tests_viewer tests_header">
   <table>
     <tr>
       <% test_success_density = measure('test_success_density') %>
-      <td id="test_success_density" class="big"><%= test_success_density ? number_with_precision(test_success_density.value(), :precision => 1) : "100.0" -%>%</td>
+      <td class="test_success_density big"><%= test_success_density ? number_with_precision(test_success_density.value(), :precision => 1) : "100.0" -%>%</td>
       <td class="sep"></td>
       <td class="name"><%= message('test_viewer.tests') -%>:</td>
       <%
@@ -13,18 +13,18 @@
            tests_display = tests_display + " (+" + format_measure(skipped_tests) + " " + message('test_viewer.skipped') + ")"
          end
       %>
-      <td id="tests" class="value"><%= tests_display -%></td>
+      <td class="tests value"><%= tests_display -%></td>
       <td class="sep"></td>
       <td class="name"><%= message('test_viewer.failures') -%>/<%= message('test_viewer.errors') -%>:</td>
       <%
          test_failures = measure('test_failures') ? format_measure(measure('test_failures')) : "0"
          test_errors = measure('test_errors') ? format_measure(measure('test_errors')) : "0"
       %>
-      <td id="test_failures_errors" class="value"><%= test_failures -%>/<%= test_errors -%></td>
+      <td class="test_failures_errors value"><%= test_failures -%>/<%= test_errors -%></td>
       <td class="sep"></td>
       <td class="name"><%= message('test_viewer.duration') -%>:</td>
       <% test_execution_time = measure('test_execution_time') %>
-      <td id="test_execution_time" class="value"><%= test_execution_time ? format_measure(test_execution_time) : "-" -%></td>
+      <td class="test_execution_time value"><%= test_execution_time ? format_measure(test_execution_time) : "-" -%></td>
     </tr>
   </table>
 </div>
@@ -73,7 +73,7 @@
    test_cases = test_cases.sort_by { |test_case| test_case[:name] }
 %>
 <% if !test_cases.empty? %>
-  <table class="data width100 tests_viewer" id="test_data">
+  <table class="data width100 tests_viewer test_data">
     <thead>
     <tr>
       <th class="thin">&nbsp;</th>
@@ -112,24 +112,24 @@
         <td id="test_time_<%= index -%>" class="thin right" nowrap><%= test_case[:time] -%> ms</td>
         <% if has_covered_lines %>
           <td id="test_covered_lines_<%= index -%>" class="thin right" nowrap>
-            <a id="test_covered_lines_link_<%= index -%>" href="#"
-               onclick="openTestWorkingView('<%= @snapshot.id -%>', '<%= test_case[:name] -%>', 'testcase');"><%= number_with_precision(test_case[:covered_lines], :precision => 0) -%>
+            <a class="test_covered_lines_link_<%= index -%>" href="<%= ApplicationController.root_context -%>/test/testcase/?sid=<%= @snapshot.id -%>&test=<%= test_case[:name] -%>"
+               onclick="openAccordionItem(this.href, this); return false;"><%= number_with_precision(test_case[:covered_lines], :precision => 0) -%>
             </a>
           </td>
         <% end %>
         <td>
-        <span id="test_name_<%= index -%>">
+        <span class="test_name_<%= index -%>">
           <% name = test_case[:name] %>
           <% if test_case[:message] %>
-            <a id="test_expandLink_<%= index -%>" href="#" onclick="expandTests(<%= index -%>);"><%= name -%></a>
-            <a id="test_collapseLink_<%= index -%>" class="collapse" href="#" onclick="collapseTests(<%= index -%>);" style="display: none;"><%= name -%></a>
+            <a class="test_expandLink_<%= index -%>" href="#" onclick="expandTests(<%= index -%>);"><%= name -%></a>
+            <a class="test_collapseLink_<%= index -%>" class="collapse" href="#" onclick="collapseTests(<%= index -%>);" style="display: none;"><%= name -%></a>
           <% else %>
             <%= name -%>
           <% end %>
         </span>
 
           <% if test_case[:message] %>
-            <div id="test_message_<%= index -%>" class="message" style="display:none; margin-top:5px;">
+            <div class="test_message_<%= index -%>" class="message" style="display:none; margin-top:5px;">
               <% message = test_case[:message] %>
               <% if !message.nil? %>
                 <span><%= h message -%></span>