converted the documents_controller_test to rspec.

app/views/documents/edit.html.erb

@@ -12,7 +12,7 @@ See doc/COPYRIGHT.rdoc for more details.
 
 <h2><%= Document.model_name.human %></h2>
 
-<%= form_tag(document_path(@document),
+<%= form_tag(project_document_path(@document),
              :method => :put,
              :class => "tabular") do %>
   <%= render :partial => 'form' %>

spec/assets/attachments/testfile.txt

@@ -0,0 +1,2 @@
+this is a text file for upload tests
+with multiple lines

spec/controllers/documents_controller_spec.rb

@@ -0,0 +1,164 @@
+#-- copyright
+# OpenProject is a project management system.
+#
+# Copyright (C) 2012-2013 the OpenProject Team
+#
+# This program is free software; you can redistribute it and/or
+# modify it under the terms of the GNU General Public License version 3.
+#
+# See doc/COPYRIGHT.rdoc for more details.
+#++
+require File.dirname(__FILE__) + '/../spec_helper'
+
+
+describe DocumentsController do
+
+  render_views
+
+  let(:admin)           { FactoryGirl.create(:admin)}
+  let(:project)         { FactoryGirl.create(:project, name: "Test Project")}
+  let(:user)            { FactoryGirl.create(:user)}
+  let(:role)            { FactoryGirl.create(:role, :permissions => [:view_documents]) }
+
+  let(:default_category){ FactoryGirl.create(:document_category, project: project, name: "Default Category")}
+
+
+  before do
+    User.stub(:current).and_return admin
+  end
+
+  describe "index" do
+
+    let(:document) { FactoryGirl.create(:document, title: "Sample Document", project: project, category: default_category) }
+
+    before :each 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
+
+      get :index, :project_id => project.identifier
+
+    end
+
+    it "should render the index-template successfully" do
+
+      expect(response).to be_success
+      expect(response).to render_template("index")
+    end
+
+    it "should group documents by category, if no other sorting is given " do
+
+      expect(assigns(:grouped)).not_to  be_nil
+      expect(assigns(:grouped).keys.map(&:name)).to eql [default_category.name]
+    end
+
+    it "should render documents with long descriptions properly" do
+
+      response.body.should have_css('.wiki p')
+      response.body.should have_css('.wiki p', text: (document.description.split("\n").first + '...'))
+      response.body.should have_css('.wiki p', text: /EndOfLineHere.../)
+
+    end
+
+  end
+
+
+  describe "create" do
+
+    let(:document_attributes) { FactoryGirl.attributes_for(:document, title: "New Document", project_id: project.id, category_id: default_category.id) }
+
+
+    before :each do
+      ActionMailer::Base.deliveries.clear
+      Setting.notified_events = Setting.notified_events.dup << 'document_added'
+    end
+
+    it "should create a new document with valid arguments" do
+      expect do
+        post :create, :project_id => project.identifier,
+             :document => FactoryGirl.attributes_for(:document, title: "New Document", project_id: project.id, category_id: default_category.id)
+             #:attachments => {'1' => FactoryGirl.attributes_for(:attachment, :author => admin)}
+
+
+      end.to change{Document.count}.by 1
+    end
+
+    it "should create a new document with valid arguments" do
+      expect do
+        post :create,
+             :project_id => project.identifier,
+             :document => document_attributes
+
+      end.to change{Document.count}.by 1
+    end
+
+    describe "with attachments" do
+
+      before :each do
+        notify_project = project
+        notify_member  = FactoryGirl.create(:member, :project => notify_project, :user => user, :roles => [role])
+
+        post :create,
+             :project_id => notify_project.identifier,
+             :document => FactoryGirl.attributes_for(:document, title: "New Document", project_id: notify_project.id, category_id: default_category.id),
+             :attachments => {'1' => {description: "sample file", file: file_attachment}}
+      end
+
+      it "should add an attachment" do
+        document = Document.last
+
+        expect(document.attachments.count).to eql 1
+        attachment = document.attachments.first
+        expect(attachment.description).to eql "sample file"
+        expect(attachment.filename).to eql "testfile.txt"
+      end
+
+      it "should redirect to the documents-page" do
+        expect(response).to redirect_to project_documents_path(project.identifier)
+      end
+
+      it "should send out mails with notifications to members of the project with :view_documents-permission" do
+
+        document = Document.last
+        expect(ActionMailer::Base.deliveries.size).to eql 1
+      end
+
+
+    end
+
+  end
+
+  describe "destroy" do
+
+    let(:document) { FactoryGirl.create(:document, title: "Sample Document", project: project, category: default_category) }
+
+
+    before :each do
+      document
+    end
+
+    it "should delete the document and redirect back to documents-page of the project" do
+      expect{
+        delete :destroy, :id => document.id
+      }.to change{Document.count}.by -1
+
+      expect(response).to redirect_to "/projects/#{project.identifier}/documents"
+      expect{Document.find(document.id)}.to raise_error
+    end
+  end
+
+
+
+  def file_attachment
+    test_document = "#{OpenProject::Documents::Engine.root}/spec/assets/attachments/testfile.txt"
+    Rack::Test::UploadedFile.new(test_document, "text/plain")
+  end
+
+end
+
+
+
+
+

spec/models/document_category_spec.rb

@@ -34,6 +34,17 @@ describe DocumentCategory do
     expect(DocumentCategory.new.option_name).to eql  :enumeration_doc_categories
   end
 
+  it "should only allow one category to be the default-category" do
+    old_default = FactoryGirl.create :document_category, :name => "old default", :project => project, :is_default => true
+
+    expect{
+      new_default = FactoryGirl.create :document_category, :name => "new default", :project => project, :is_default => true
+      old_default.reload
+    }.to change{old_default.is_default?}.from(true).to(false)
+
+
+  end
+
 end