coverage/index.html in trusty-cms-3.1.5 vs coverage/index.html in trusty-cms-3.1.6

- old
+ new

@@ -12,31 +12,31 @@ <body> <div id="loading"> <img src="./assets/0.10.2/loading.gif" alt="loading"/> </div> <div id="wrapper" style="display:none;"> - <div class="timestamp">Generated <abbr class="timeago" title="2017-11-22T09:17:07-05:00">2017-11-22T09:17:07-05:00</abbr></div> + <div class="timestamp">Generated <abbr class="timeago" title="2018-01-03T14:38:22-05:00">2018-01-03T14:38:22-05:00</abbr></div> <ul class="group_tabs"></ul> <div id="content"> <div class="file_list_container" id="AllFiles"> <h2> <span class="group_name">All Files</span> - (<span class="covered_percent"><span class="yellow">82.89%</span></span> + (<span class="covered_percent"><span class="yellow">82.77%</span></span> covered at <span class="covered_strength"> <span class="green"> - 14.77 + 14.76 </span> </span> hits/line) </h2> <a name="AllFiles"></a> <div> <b>51</b> files in total. - <b>1368</b> relevant lines. + <b>1370</b> relevant lines. <span class="green"><b>1134</b> lines covered</span> and - <span class="red"><b>234</b> lines missed </span> + <span class="red"><b>236</b> lines missed </span> </div> <table class="file_list"> <thead> <tr> <th>File</th> @@ -80,16 +80,16 @@ <td>1.0</td> </tr> <tr> <td class="strong"><a href="#eef04f79443b3c1cf5f1a94e49da169bb9bf9edf" class="src_link" title="app/controllers/admin/pages_controller.rb">app/controllers/admin/pages_controller.rb</a></td> - <td class="red strong">51.39 %</td> - <td>114</td> - <td>72</td> + <td class="red strong">50.0 %</td> + <td>116</td> + <td>74</td> <td>37</td> - <td>35</td> - <td>6.4</td> + <td>37</td> + <td>6.3</td> </tr> <tr> <td class="strong"><a href="#ffc0ea6768b2d6709d4896039faa4e70ac4ac143" class="src_link" title="app/controllers/admin/preferences_controller.rb">app/controllers/admin/preferences_controller.rb</a></td> <td class="yellow strong">84.21 %</td> @@ -567,11 +567,11 @@ </div> <div id="footer"> - Generated by <a href="http://github.com/colszowka/simplecov">simplecov</a> v0.15.1 + Generated by <a href="http://github.com/colszowka/simplecov">simplecov</a> v0.13.0 and simplecov-html v0.10.2<br/> using RSpec </div> <div class="source_files"> @@ -1048,15 +1048,15 @@ <div class="source_table" id="eef04f79443b3c1cf5f1a94e49da169bb9bf9edf"> <div class="header"> <h3>app/controllers/admin/pages_controller.rb</h3> - <h4><span class="red">51.39 %</span> covered</h4> + <h4><span class="red">50.0 %</span> covered</h4> <div> - <b>72</b> relevant lines. + <b>74</b> relevant lines. <span class="green"><b>37</b> lines covered</span> and - <span class="red"><b>35</b> lines missed.</span> + <span class="red"><b>37</b> lines missed.</span> </div> </div> <pre> <ol> @@ -1494,254 +1494,266 @@ </li> <li class="missed" data-hits="0" data-linenumber="73"> - <code class="ruby"> Page.transaction do</code> + <code class="ruby"> params.permit!</code> </li> <li class="missed" data-hits="0" data-linenumber="74"> - <code class="ruby"> page_class = Page.descendants.include?(model_class) ? model_class : Page</code> + <code class="ruby"> Page.transaction do</code> </li> <li class="missed" data-hits="0" data-linenumber="75"> - <code class="ruby"> if request.referer =~ %r{/admin/pages/(\d+)/edit}</code> + <code class="ruby"> page_class = Page.descendants.include?(model_class) ? model_class : Page</code> </li> <li class="missed" data-hits="0" data-linenumber="76"> - <code class="ruby"> page = Page.find($1).becomes(page_class)</code> + <code class="ruby"> if request.referer =~ %r{/admin/pages/(\d+)/edit}</code> </li> <li class="missed" data-hits="0" data-linenumber="77"> - <code class="ruby"> page.update_attributes(params[:page])</code> + <code class="ruby"> page = Page.find($1).becomes(page_class)</code> </li> <li class="missed" data-hits="0" data-linenumber="78"> + <code class="ruby"> layout_id = page.layout_id</code> + </li> + + <li class="missed" data-hits="0" data-linenumber="79"> + + + <code class="ruby"> page.update_attributes(params[:page])</code> + </li> + + <li class="missed" data-hits="0" data-linenumber="80"> + + <code class="ruby"> page.published_at ||= Time.now</code> </li> - <li class="never" data-hits="" data-linenumber="79"> + <li class="never" data-hits="" data-linenumber="81"> <code class="ruby"> else</code> </li> - <li class="missed" data-hits="0" data-linenumber="80"> + <li class="missed" data-hits="0" data-linenumber="82"> <code class="ruby"> page = page_class.new(params[:page])</code> </li> - <li class="missed" data-hits="0" data-linenumber="81"> + <li class="missed" data-hits="0" data-linenumber="83"> <code class="ruby"> page.published_at = page.updated_at = page.created_at = Time.now</code> </li> - <li class="missed" data-hits="0" data-linenumber="82"> + <li class="missed" data-hits="0" data-linenumber="84"> <code class="ruby"> page.parent = Page.find($1) if request.referer =~ %r{/admin/pages/(\d+)/children/new}</code> </li> - <li class="never" data-hits="" data-linenumber="83"> + <li class="never" data-hits="" data-linenumber="85"> <code class="ruby"> end</code> </li> - <li class="missed" data-hits="0" data-linenumber="84"> + <li class="missed" data-hits="0" data-linenumber="86"> <code class="ruby"> page.pagination_parameters = pagination_parameters</code> </li> - <li class="missed" data-hits="0" data-linenumber="85"> + <li class="missed" data-hits="0" data-linenumber="87"> <code class="ruby"> process_with_exception(page)</code> </li> - <li class="never" data-hits="" data-linenumber="86"> + <li class="never" data-hits="" data-linenumber="88"> <code class="ruby"> end</code> </li> - <li class="never" data-hits="" data-linenumber="87"> + <li class="never" data-hits="" data-linenumber="89"> <code class="ruby"> end</code> </li> - <li class="never" data-hits="" data-linenumber="88"> + <li class="never" data-hits="" data-linenumber="90"> <code class="ruby"></code> </li> - <li class="covered" data-hits="1" data-linenumber="89"> + <li class="covered" data-hits="1" data-linenumber="91"> <span class="hits">1</span> <code class="ruby"> def process_with_exception(page)</code> </li> - <li class="missed" data-hits="0" data-linenumber="90"> + <li class="missed" data-hits="0" data-linenumber="92"> <code class="ruby"> page.process(request, response)</code> </li> - <li class="missed" data-hits="0" data-linenumber="91"> + <li class="missed" data-hits="0" data-linenumber="93"> <code class="ruby"> @performed_render = true</code> </li> - <li class="missed" data-hits="0" data-linenumber="92"> + <li class="missed" data-hits="0" data-linenumber="94"> <code class="ruby"> render template: &#39;site/show_page&#39;, layout: false</code> </li> - <li class="missed" data-hits="0" data-linenumber="93"> + <li class="missed" data-hits="0" data-linenumber="95"> <code class="ruby"> raise PreviewStop</code> </li> - <li class="never" data-hits="" data-linenumber="94"> + <li class="never" data-hits="" data-linenumber="96"> <code class="ruby"> end</code> </li> - <li class="never" data-hits="" data-linenumber="95"> + <li class="never" data-hits="" data-linenumber="97"> <code class="ruby"></code> </li> - <li class="covered" data-hits="1" data-linenumber="96"> + <li class="covered" data-hits="1" data-linenumber="98"> <span class="hits">1</span> <code class="ruby"> def count_deleted_pages</code> </li> - <li class="covered" data-hits="1" data-linenumber="97"> + <li class="covered" data-hits="1" data-linenumber="99"> <span class="hits">1</span> <code class="ruby"> @count = model.children.count + 1</code> </li> - <li class="never" data-hits="" data-linenumber="98"> + <li class="never" data-hits="" data-linenumber="100"> <code class="ruby"> end</code> </li> - <li class="never" data-hits="" data-linenumber="99"> + <li class="never" data-hits="" data-linenumber="101"> <code class="ruby"></code> </li> - <li class="covered" data-hits="1" data-linenumber="100"> + <li class="covered" data-hits="1" data-linenumber="102"> <span class="hits">1</span> <code class="ruby"> def initialize_meta_rows_and_buttons</code> </li> - <li class="covered" data-hits="4" data-linenumber="101"> + <li class="covered" data-hits="4" data-linenumber="103"> <span class="hits">4</span> <code class="ruby"> @buttons_partials ||= []</code> </li> - <li class="covered" data-hits="4" data-linenumber="102"> + <li class="covered" data-hits="4" data-linenumber="104"> <span class="hits">4</span> <code class="ruby"> @meta ||= []</code> </li> - <li class="covered" data-hits="4" data-linenumber="103"> + <li class="covered" data-hits="4" data-linenumber="105"> <span class="hits">4</span> <code class="ruby"> @meta &lt;&lt; {:field =&gt; &quot;slug&quot;, :type =&gt; &quot;text_field&quot;, :args =&gt; [{:class =&gt; &#39;textbox&#39;, :maxlength =&gt; 100}]}</code> </li> - <li class="covered" data-hits="4" data-linenumber="104"> + <li class="covered" data-hits="4" data-linenumber="106"> <span class="hits">4</span> <code class="ruby"> @meta &lt;&lt; {:field =&gt; &quot;breadcrumb&quot;, :type =&gt; &quot;text_field&quot;, :args =&gt; [{:class =&gt; &#39;textbox&#39;, :maxlength =&gt; 160}]}</code> </li> - <li class="never" data-hits="" data-linenumber="105"> + <li class="never" data-hits="" data-linenumber="107"> <code class="ruby"> end</code> </li> - <li class="never" data-hits="" data-linenumber="106"> + <li class="never" data-hits="" data-linenumber="108"> <code class="ruby"></code> </li> - <li class="covered" data-hits="1" data-linenumber="107"> + <li class="covered" data-hits="1" data-linenumber="109"> <span class="hits">1</span> <code class="ruby"> def verify_page_class(page_class)</code> </li> - <li class="missed" data-hits="0" data-linenumber="108"> + <li class="missed" data-hits="0" data-linenumber="110"> <code class="ruby"> if page_class.constantize.ancestors.include?(Page)</code> </li> - <li class="missed" data-hits="0" data-linenumber="109"> + <li class="missed" data-hits="0" data-linenumber="111"> <code class="ruby"> page_class.constantize</code> </li> - <li class="never" data-hits="" data-linenumber="110"> + <li class="never" data-hits="" data-linenumber="112"> <code class="ruby"> else</code> </li> - <li class="missed" data-hits="0" data-linenumber="111"> + <li class="missed" data-hits="0" data-linenumber="113"> <code class="ruby"> raise &quot;I&#39;m not allowed to constantize #{page_class}!&quot;</code> </li> - <li class="never" data-hits="" data-linenumber="112"> + <li class="never" data-hits="" data-linenumber="114"> <code class="ruby"> end</code> </li> - <li class="never" data-hits="" data-linenumber="113"> + <li class="never" data-hits="" data-linenumber="115"> <code class="ruby"> end</code> </li> - <li class="never" data-hits="" data-linenumber="114"> + <li class="never" data-hits="" data-linenumber="116"> <code class="ruby">end</code> </li> @@ -3437,11 +3449,11 @@ <span class="hits">1</span> <code class="ruby"> def permitted_params</code> </li> - <li class="covered" data-hits="20" data-linenumber="244"> - <span class="hits">20</span> + <li class="covered" data-hits="18" data-linenumber="244"> + <span class="hits">18</span> <code class="ruby"> model_symbols = ActiveRecord::Base.descendants.map{|a| a.name.underscore.to_sym}</code> </li> <li class="covered" data-hits="14" data-linenumber="245">