From 41b9b49ada8de34be83b3e91efca427edda9e19c Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Oliver=20G=C3=BCnther?= <mail@oliverguenther.de>
Date: Tue, 24 May 2016 11:09:15 +0200
Subject: [PATCH] Fix specs

---
 app/views/documents/show.html.erb             | 14 +++++------
 spec/controllers/documents_controller_spec.rb | 23 ++++++++++++++-----
 2 files changed, 24 insertions(+), 13 deletions(-)

diff --git a/app/views/documents/show.html.erb b/app/views/documents/show.html.erb
index 8e0e390d6b..b1034b6298 100644
--- a/app/views/documents/show.html.erb
+++ b/app/views/documents/show.html.erb
@@ -35,14 +35,14 @@ See doc/COPYRIGHT.rdoc for more details.
             subtitle: "#{@document.category.name} - #{format_date @document.created_on}" do %>
   <% if authorize_for(:documents, :edit) %>
     <li class="toolbar-item">
-      <%= link_to({:controller => '/documents', :action => 'edit', :id => @document}, class: 'button', accesskey: accesskey(:edit)) do %>
+      <%= link_to({controller: '/documents', action: 'edit', id: @document}, class: 'button', accesskey: accesskey(:edit)) do %>
         <i class="button--icon icon-edit"></i> <%= l(:button_edit) %>
       <% end %>
     </li>
   <% end %>
   <% if authorize_for(:documents, :destroy) %>
     <li class="toolbar-item">
-      <%= link_to({:controller => '/documents', :action => 'destroy', :id => @document}, class: 'button', data: { :confirm => l(:text_are_you_sure) }, :method => :delete) do %>
+      <%= link_to({controller: '/documents', action: 'destroy', id: @document}, class: 'button', data: { confirm: l(:text_are_you_sure) }, method: :delete) do %>
         <i class="button--icon icon-delete"></i> <%= l(:button_delete) %>
       <% end %>
     </li>
@@ -50,18 +50,18 @@ See doc/COPYRIGHT.rdoc for more details.
 <% end %>
 
 <div class="wiki">
-<%= textilizable @document.description, :attachments => @document.attachments %>
+<%= format_text @document.description, attachments: @document.attachments %>
 </div>
 
 <h3><%= l(:label_attachment_plural) %></h3>
 <%= link_to_attachments @document %>
 
 <% if authorize_for('documents', 'add_attachment') %>
-<p><%= link_to l(:label_attachment_new), {}, :onclick => "Element.show('add_attachment_form'); Element.hide(this); Element.scrollTo('add_attachment_form'); return false;",
-                                             :id => 'attach_files_link' %></p>
-  <%= form_tag(add_attachment_document_path(@document), method: :post, multipart: true, :id => "add_attachment_form", :style => "display:none;") do %>
+<p><%= link_to l(:label_attachment_new), {}, onclick: "Element.show('add_attachment_form'); Element.hide(this); Element.scrollTo('add_attachment_form'); return false;",
+                                             id: 'attach_files_link' %></p>
+  <%= form_tag(add_attachment_document_path(@document), method: :post, multipart: true, id: "add_attachment_form", style: "display:none;") do %>
   <div class="box">
-  <p><%= render :partial => 'attachments/form' %></p>
+  <p><%= render partial: 'attachments/form' %></p>
   </div>
   <%= styled_button_tag l(:button_add), class: "-highlight -with-icon icon-checkmark" %>
 
diff --git a/spec/controllers/documents_controller_spec.rb b/spec/controllers/documents_controller_spec.rb
index b849c096eb..6f9c70c853 100644
--- a/spec/controllers/documents_controller_spec.rb
+++ b/spec/controllers/documents_controller_spec.rb
@@ -53,14 +53,25 @@ describe DocumentsController do
   end
 
   describe "index" do
+    let(:long_description) {
+      <<-LOREM.strip_heredoc
+        Lorem ipsum dolor sit amet, consectetur adipiscing elit.\
+        Ut egestas, mi vehicula varius varius, ipsum massa fermentum orci,\
+        eget tristique ante sem vel mi. Nulla facilisi.\
+        Donec enim libero, luctus ac sagittis sit amet, vehicula sagittis magna.\
+        Duis ultrices molestie ante, eget scelerisque sem iaculis vitae.\
+        Etiam fermentum mauris vitae metus pharetra condimentum fermentum est pretium.\
+        Proin sollicitudin elementum quam quis pharetra.\
+        Aenean facilisis nunc quis elit volutpat mollis.\
+        Aenean eleifend varius euismod. Ut dolor est, congue eget dapibus eget, elementum eu odio.\
+        Integer et lectus neque, nec scelerisque nisi. EndOfLineHere
+
+        Praesent a nunc lorem, ac porttitor eros.
+      LOREM
+    }
 
     before do
-      document.update_attributes(description:<<LOREM)
-Lorem ipsum dolor sit amet, consectetur adipiscing elit. Ut egestas, mi vehicula varius varius, ipsum massa fermentum orci, eget tristique ante sem vel mi. Nulla facilisi. Donec enim libero, luctus ac sagittis sit amet, vehicula sagittis magna. Duis ultrices molestie ante, eget scelerisque sem iaculis vitae. Etiam fermentum mauris vitae metus pharetra condimentum fermentum est pretium. Proin sollicitudin elementum quam quis pharetra.  Aenean facilisis nunc quis elit volutpat mollis. Aenean eleifend varius euismod. Ut dolor est, congue eget dapibus eget, elementum eu odio. Integer et lectus neque, nec scelerisque nisi. EndOfLineHere
-
-Vestibulum non velit mi. Aliquam scelerisque libero ut nulla fringilla a sollicitudin magna rhoncus.  Praesent a nunc lorem, ac porttitor eros. Sed ac diam nec neque interdum adipiscing quis quis justo. Donec arcu nunc, fringilla eu dictum at, venenatis ac sem. Vestibulum quis elit urna, ac mattis sapien. Lorem ipsum dolor sit amet, consectetur adipiscing elit.
-LOREM
-
+      document.update_attributes(description: long_description)
       get :index, project_id: project.identifier
     end