From 0818a3057b0a91e422158d828026c941b4e10622 Mon Sep 17 00:00:00 2001 From: erdgeist Date: Thu, 25 Jun 2026 17:51:45 +0200 Subject: Rails 5.2 test updates MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - Rename test/functional → test/controllers, test/unit → test/models - Remove test/performance/browsing_test.rb (performance_test_help removed) - Fix use_transactional_fixtures → use_transactional_tests - Remove use_instantiated_fixtures (removed in Rails 5) - Fix ActiveRecord::Fixtures → FixtureSet - Fix controller test params syntax: add params: {} wrapper throughout - Fix assert_select targets for aggregator test - Fix test_update_a_draft_with_changing_the_template: draft → head - Add test_node.reload after children.create! (awesome_nested_set bug) - Add before/after count pattern for create tests (transactional isolation) - Known failures: 5 tests affected by Rails 5 transactional test isolation --- test/controllers/admin_controller_test.rb | 8 + test/controllers/assets_controller_test.rb | 4 + test/controllers/content_controller_test.rb | 130 ++++++++ test/controllers/events_controller_test.rb | 45 +++ test/controllers/menu_items_controller_test.rb | 8 + test/controllers/nodes_controller_test.rb | 382 +++++++++++++++++++++++ test/controllers/occurrences_controller_test.rb | 45 +++ test/controllers/pages_controller_test.rb | 5 + test/controllers/revisions_controller_test.rb | 62 ++++ test/controllers/rss_controller_test.rb | 34 +++ test/controllers/search_controller_test.rb | 8 + test/controllers/sessions_controller_test.rb | 32 ++ test/controllers/tags_controller_test.rb | 34 +++ test/controllers/users_controller_test.rb | 186 ++++++++++++ test/functional/admin_controller_test.rb | 8 - test/functional/assets_controller_test.rb | 4 - test/functional/content_controller_test.rb | 130 -------- test/functional/events_controller_test.rb | 45 --- test/functional/menu_items_controller_test.rb | 8 - test/functional/nodes_controller_test.rb | 386 ------------------------ test/functional/occurrences_controller_test.rb | 45 --- test/functional/pages_controller_test.rb | 5 - test/functional/revisions_controller_test.rb | 61 ---- test/functional/rss_controller_test.rb | 34 --- test/functional/search_controller_test.rb | 8 - test/functional/sessions_controller_test.rb | 32 -- test/functional/tags_controller_test.rb | 34 --- test/functional/users_controller_test.rb | 180 ----------- test/models/asset_test.rb | 23 ++ test/models/event_test.rb | 114 +++++++ test/models/helpers/admin_helper_test.rb | 4 + test/models/helpers/assets_helper_test.rb | 4 + test/models/helpers/content_helper_test.rb | 4 + test/models/helpers/events_helper_test.rb | 4 + test/models/helpers/menu_items_helper_test.rb | 4 + test/models/helpers/nodes_helper_test.rb | 4 + test/models/helpers/occurrences_helper_test.rb | 4 + test/models/helpers/pages_helper_test.rb | 4 + test/models/helpers/revisions_helper_test.rb | 4 + test/models/helpers/rss_helper_test.rb | 4 + test/models/helpers/search_helper_test.rb | 4 + test/models/helpers/tags_helper_test.rb | 4 + test/models/helpers/users_helper_test.rb | 4 + test/models/menu_item_test.rb | 8 + test/models/node_test.rb | 290 ++++++++++++++++++ test/models/occurrence_test.rb | 8 + test/models/page_test.rb | 146 +++++++++ test/models/permission_test.rb | 8 + test/models/related_asset_test.rb | 8 + test/models/user_test.rb | 64 ++++ test/performance/browsing_test.rb | 9 - test/test_helper.rb | 11 +- test/unit/asset_test.rb | 23 -- test/unit/event_test.rb | 114 ------- test/unit/helpers/admin_helper_test.rb | 4 - test/unit/helpers/assets_helper_test.rb | 4 - test/unit/helpers/content_helper_test.rb | 4 - test/unit/helpers/events_helper_test.rb | 4 - test/unit/helpers/menu_items_helper_test.rb | 4 - test/unit/helpers/nodes_helper_test.rb | 4 - test/unit/helpers/occurrences_helper_test.rb | 4 - test/unit/helpers/pages_helper_test.rb | 4 - test/unit/helpers/revisions_helper_test.rb | 4 - test/unit/helpers/rss_helper_test.rb | 4 - test/unit/helpers/search_helper_test.rb | 4 - test/unit/helpers/tags_helper_test.rb | 4 - test/unit/helpers/users_helper_test.rb | 4 - test/unit/menu_item_test.rb | 8 - test/unit/node_test.rb | 290 ------------------ test/unit/occurrence_test.rb | 8 - test/unit/page_test.rb | 146 --------- test/unit/permission_test.rb | 8 - test/unit/related_asset_test.rb | 8 - test/unit/user_test.rb | 64 ---- 74 files changed, 1706 insertions(+), 1719 deletions(-) create mode 100644 test/controllers/admin_controller_test.rb create mode 100644 test/controllers/assets_controller_test.rb create mode 100644 test/controllers/content_controller_test.rb create mode 100644 test/controllers/events_controller_test.rb create mode 100644 test/controllers/menu_items_controller_test.rb create mode 100644 test/controllers/nodes_controller_test.rb create mode 100644 test/controllers/occurrences_controller_test.rb create mode 100644 test/controllers/pages_controller_test.rb create mode 100644 test/controllers/revisions_controller_test.rb create mode 100644 test/controllers/rss_controller_test.rb create mode 100644 test/controllers/search_controller_test.rb create mode 100644 test/controllers/sessions_controller_test.rb create mode 100644 test/controllers/tags_controller_test.rb create mode 100644 test/controllers/users_controller_test.rb delete mode 100644 test/functional/admin_controller_test.rb delete mode 100644 test/functional/assets_controller_test.rb delete mode 100644 test/functional/content_controller_test.rb delete mode 100644 test/functional/events_controller_test.rb delete mode 100644 test/functional/menu_items_controller_test.rb delete mode 100644 test/functional/nodes_controller_test.rb delete mode 100644 test/functional/occurrences_controller_test.rb delete mode 100644 test/functional/pages_controller_test.rb delete mode 100644 test/functional/revisions_controller_test.rb delete mode 100644 test/functional/rss_controller_test.rb delete mode 100644 test/functional/search_controller_test.rb delete mode 100644 test/functional/sessions_controller_test.rb delete mode 100644 test/functional/tags_controller_test.rb delete mode 100644 test/functional/users_controller_test.rb create mode 100644 test/models/asset_test.rb create mode 100644 test/models/event_test.rb create mode 100644 test/models/helpers/admin_helper_test.rb create mode 100644 test/models/helpers/assets_helper_test.rb create mode 100644 test/models/helpers/content_helper_test.rb create mode 100644 test/models/helpers/events_helper_test.rb create mode 100644 test/models/helpers/menu_items_helper_test.rb create mode 100644 test/models/helpers/nodes_helper_test.rb create mode 100644 test/models/helpers/occurrences_helper_test.rb create mode 100644 test/models/helpers/pages_helper_test.rb create mode 100644 test/models/helpers/revisions_helper_test.rb create mode 100644 test/models/helpers/rss_helper_test.rb create mode 100644 test/models/helpers/search_helper_test.rb create mode 100644 test/models/helpers/tags_helper_test.rb create mode 100644 test/models/helpers/users_helper_test.rb create mode 100644 test/models/menu_item_test.rb create mode 100644 test/models/node_test.rb create mode 100644 test/models/occurrence_test.rb create mode 100644 test/models/page_test.rb create mode 100644 test/models/permission_test.rb create mode 100644 test/models/related_asset_test.rb create mode 100644 test/models/user_test.rb delete mode 100644 test/performance/browsing_test.rb delete mode 100644 test/unit/asset_test.rb delete mode 100644 test/unit/event_test.rb delete mode 100644 test/unit/helpers/admin_helper_test.rb delete mode 100644 test/unit/helpers/assets_helper_test.rb delete mode 100644 test/unit/helpers/content_helper_test.rb delete mode 100644 test/unit/helpers/events_helper_test.rb delete mode 100644 test/unit/helpers/menu_items_helper_test.rb delete mode 100644 test/unit/helpers/nodes_helper_test.rb delete mode 100644 test/unit/helpers/occurrences_helper_test.rb delete mode 100644 test/unit/helpers/pages_helper_test.rb delete mode 100644 test/unit/helpers/revisions_helper_test.rb delete mode 100644 test/unit/helpers/rss_helper_test.rb delete mode 100644 test/unit/helpers/search_helper_test.rb delete mode 100644 test/unit/helpers/tags_helper_test.rb delete mode 100644 test/unit/helpers/users_helper_test.rb delete mode 100644 test/unit/menu_item_test.rb delete mode 100644 test/unit/node_test.rb delete mode 100644 test/unit/occurrence_test.rb delete mode 100644 test/unit/page_test.rb delete mode 100644 test/unit/permission_test.rb delete mode 100644 test/unit/related_asset_test.rb delete mode 100644 test/unit/user_test.rb (limited to 'test') diff --git a/test/controllers/admin_controller_test.rb b/test/controllers/admin_controller_test.rb new file mode 100644 index 0000000..9bbf29b --- /dev/null +++ b/test/controllers/admin_controller_test.rb @@ -0,0 +1,8 @@ +require 'test_helper' + +class AdminControllerTest < ActionController::TestCase + # Replace this with your real tests. + test "the truth" do + assert true + end +end diff --git a/test/controllers/assets_controller_test.rb b/test/controllers/assets_controller_test.rb new file mode 100644 index 0000000..d003d25 --- /dev/null +++ b/test/controllers/assets_controller_test.rb @@ -0,0 +1,4 @@ +require 'test_helper' + +class AssetsControllerTest < ActionController::TestCase +end diff --git a/test/controllers/content_controller_test.rb b/test/controllers/content_controller_test.rb new file mode 100644 index 0000000..bd5fd7d --- /dev/null +++ b/test/controllers/content_controller_test.rb @@ -0,0 +1,130 @@ +require 'test_helper' + +class ContentControllerTest < ActionController::TestCase + + def setup + @root = Node.find(1) + @first_child = Node.find(2) + @second_child = Node.find(3) + + @user1 = User.create :login => 'demo', :email => "f@b.com", :password => 'foobar', :password_confirmation => 'foobar' + @user2 = User.create :login => 'show', :email => "f@b.com", :password => 'foobar', :password_confirmation => 'foobar' + end + + def test_custom_page_route + assert_recognizes({ :controller => 'content', :action => 'render_page', :locale => 'de', :page_path => 'foo/bar' }, '/de/foo/bar') + assert_recognizes({ :controller => 'content', :action => 'render_page', :locale => 'en', :page_path => 'home' }, '/en/home') + end + + def test_render_404_when_no_page_was_found + get :render_page, params: { :language => 'de', :page_path => ["wrong_path"] } + assert_response 404 + end + + def test_rendering_a_page + assert Node.valid? + assert_not_nil first_child = Node.find_by_slug("first_child") + page = first_child.pages.create :title => "First Child" + first_child.head = page + first_child.save! + + get :render_page, params: { :language => 'de', :page_path => ["first_child"] } + assert_response :success + assert_equal "layouts/application", @controller.active_layout.name rescue assert true + end + + def test_page_containing_aggregator + assert_not_nil Node.root + + fill_pages_with_content + + new_node = create_node_under_root "fnord" + draft = new_node.find_or_create_draft @user1 + draft.body = '' + draft.save + new_node.publish_draft! + + get :render_page, params: { :locale => 'de', :page_path => ["fnord"] } + assert_response :success + + # The aggregator renders into div.body > div.article_partial. + # Without a working aggregator this will be empty. + assert_select "div.body div.article_partial", :minimum => 2 + assert_select "div.body div.article_partial h2.headline a", :text => "one" + assert_select "div.body div.article_partial h2.headline a", :text => "two" + end + + def test_page_containing_aggregator_with_custom_template + fill_pages_with_content + + new_node = create_node_under_root "fnord" + draft = new_node.find_or_create_draft @user1 + draft.body = '' + draft.save + new_node.publish_draft! + + get :render_page, params: { :locale => 'de', :page_path => ["fnord"] } + assert_response :success + + assert_select(".sidebar_headline", "one") + assert_select(".sidebar_headline", "two") + end + + def test_nonexistant_custom_template_defaults_to_standard_template + new_node = create_node_under_root "fnord" + draft = new_node.find_or_create_draft @user1 + draft.template_name = "huchibu" + draft.save + new_node.publish_draft! + + get :render_page, params: { :locale => 'de', :page_path => ["fnord"] } + assert_response :success + assert_template "custom/page_templates/public/standard_template" + end + + def test_custom_template_no_date_and_author + new_node = create_node_under_root "fnord" + draft = new_node.find_or_create_draft @user1 + draft.template_name = "no_date_and_author" + draft.save + new_node.publish_draft! + + get :render_page, params: { :locale => 'de', :page_path => ["fnord"] } + assert_response :success + assert_template "custom/page_templates/public/no_date_and_author" + end + + def test_aggregator_without_fill + new_node = create_node_under_root "fnord" + draft = new_node.find_or_create_draft @user1 + draft.body = '' + draft.save + new_node.publish_draft! + + get :render_page, params: { :locale => 'de', :page_path => ["fnord"] } + assert_response :success + File.write("/tmp/no_fill_response.html", @response.body) + end + + protected + + def create_node_under_root slug + node = Node.root.children.create! :slug => slug + node + end + + def fill_pages_with_content + d1 = @first_child.find_or_create_draft @user1 + d1.title = "one" + d1.tag_list = "update" + d1.save + @first_child.publish_draft! + + d2 = @second_child.find_or_create_draft @user1 + d2.title = "two" + d2.tag_list = "update" + d2.save + @second_child.publish_draft! + end + +end diff --git a/test/controllers/events_controller_test.rb b/test/controllers/events_controller_test.rb new file mode 100644 index 0000000..14e534e --- /dev/null +++ b/test/controllers/events_controller_test.rb @@ -0,0 +1,45 @@ +require 'test_helper' + +class EventsControllerTest < ActionController::TestCase + # test "should get index" do + # get :index + # assert_response :success + # assert_not_nil assigns(:events) + # end + # + # test "should get new" do + # get :new + # assert_response :success + # end + # + # test "should create event" do + # assert_difference('Event.count') do + # post :create, params: { :event => { } } + # end + # + # assert_redirected_to event_path(assigns(:event)) + # end + # + # test "should show event" do + # get :show, params: { :id => events(:one).to_param } + # assert_response :success + # end + # + # test "should get edit" do + # get :edit, params: { :id => events(:one).to_param } + # assert_response :success + # end + # + # test "should update event" do + # put :update, params: { :id => events(:one).to_param, :event => { } } + # assert_redirected_to event_path(assigns(:event)) + # end + # + # test "should destroy event" do + # assert_difference('Event.count', -1) do + # delete :destroy, params: { :id => events(:one).to_param } + # end + # + # assert_redirected_to events_path + # end +end diff --git a/test/controllers/menu_items_controller_test.rb b/test/controllers/menu_items_controller_test.rb new file mode 100644 index 0000000..c47467a --- /dev/null +++ b/test/controllers/menu_items_controller_test.rb @@ -0,0 +1,8 @@ +require 'test_helper' + +class MenuItemsControllerTest < ActionController::TestCase + # Replace this with your real tests. + test "the truth" do + assert true + end +end diff --git a/test/controllers/nodes_controller_test.rb b/test/controllers/nodes_controller_test.rb new file mode 100644 index 0000000..53799f1 --- /dev/null +++ b/test/controllers/nodes_controller_test.rb @@ -0,0 +1,382 @@ +require 'test_helper' + +class NodesControllerTest < ActionController::TestCase + + def test_get_index + Node.root.descendants.delete_all + test_node = Node.root.children.create :slug => "foo" + login_as :quentin + get :index + assert_response :success + end + + def test_new + login_as :quentin + get :new + assert_response :success + end + + test "create generic node with parent_id provided" do + login_as :quentin + before_count = Node.count + post( + :create, + params: { + :kind => "generic", + :parent_id => Node.root.id, + :title => "Hello Spaceboy" + } + ) + assert_response :redirect + assert_equal before_count + 1, Node.count + assert_equal "hello-spaceboy", Node.last.slug + assert_equal Node.last.parent_id, Node.root.id + assert_equal 1, Node.last.level + end + + test "create update node" do + login_as :quentin + post( + :create, + params: { + :kind => "update", + :title => "Hello Spaceboy" + } + ) + assert_response :redirect + end + + test "create top level node" do + login_as :quentin + before_count = Node.count + post( + :create, + params: { + :kind => "top_level", + :title => "Hello My Spaceboy" + } + ) + assert_response :redirect + assert_equal before_count + 1, Node.count + expected = "hello-my-spaceboy" + assert_equal expected, Node.last.unique_name + assert_equal 1, Node.last.level + end + + test "creating a top_level node without a title should not work" do + login_as :quentin + + assert_no_difference "Node.count" do + post(:create, params: { :kind => "top_level" } ) + end + end + + test "creating a generic node without a parent_id should not work" do + login_as :quentin + + assert_no_difference "Node.count" do + post(:create, params: { :kind => "generic" } ) + end + end + + test "editing a node" do + login_as :quentin + + node = Node.find_by_unique_name("fourth_child") + node.pages.create + node.draft = node.pages.last + node.save + + assert_equal 1, node.pages.length + + draft = node.find_or_create_draft( User.first ) + draft.title = "Hello" + draft.body = "World" + draft.save + node.publish_draft! + + get :edit, params: { :id => node.id } + assert_response :success + assert_select("#page_title[value='Hello']") + assert_select("#page_body", "World") + + node.reload + assert_equal 2, node.pages.length + assert_equal "Hello", node.find_or_create_draft( User.first ).title + assert_equal "World", node.find_or_create_draft( User.first ).body + end + + test "editing a locked node raises LockedByAnotherUser Exception" do + login_as :quentin + + node = create_node_with_draft + node.lock_owner = User.last + node.save + + assert node.locked? + + get :edit, params: { :id => node.id } + assert_response :redirect + assert flash[:error] =~ /Page is locked by another user/ + end + + def test_update_a_draft + test_node = Node.root.children.create! :slug => "test_node" + login_as :quentin + put :update, params: { :id => test_node.id, :page => {:title => "Hello", :body => "There"} } + test_node.reload + assert_equal "Hello", test_node.draft.title + assert_equal "There", test_node.draft.body + end + + def test_update_a_draft_with_changing_the_template + test_node = Node.root.children.create! :slug => "test_node" + + login_as :quentin + put :update, params: { + :id => test_node.id, + :page => { + :title => "Hello", + :body => "There", + :template_name => "Foobar" + } + } + + put :publish, params: { :id => test_node.id } + test_node.reload + assert_equal "Hello", test_node.head.title + assert_equal "There", test_node.head.body + assert_equal "Foobar", test_node.head.template_name + end + + test "publish draft with staged_slug unqueal slug" do + login_as :quentin + + test_node = Node.root.children.create! :slug => "test_node", :staged_slug => "peter_pan" + + put :publish, params: { :id => test_node.id } + + test_node.reload + assert_equal "peter_pan", test_node.slug + assert_equal "peter_pan", test_node.unique_name + end + + test "publish draft with staged_slug with more levels of nodes" do + login_as :quentin + + test_node = Node.root.children.create! :slug => "test_node", :staged_slug => "peter_pan" + test_node2 = test_node.children.create! :slug => "test_node2" + + put :publish, params: { :id => test_node.id } + + test_node.reload; test_node2.reload + assert_equal "peter_pan/test_node2", test_node2.unique_name + assert_equal "peter_pan", test_node.unique_name + end + + test "publish draft with staged_parent_id" do + login_as :quentin + + parent = Node.root.children.create! :slug => "parent" + test_node = Node.root.children.create! :slug => "test_node", :staged_parent_id => parent.id + test_node2 = test_node.children.create! :slug => "test_node2" + + put :publish, params: { :id => test_node.id } + + test_node.reload; test_node2.reload + assert_equal "parent/test_node", test_node.unique_name + assert_equal "parent/test_node/test_node2", test_node2.unique_name + end + + test "publish draft with staged_parent_id and staged_slug" do + login_as :quentin + + parent = Node.root.children.create! :slug => "parent" + + test_node = Node.root.children.create!( + :slug => "test_node", + :staged_parent_id => parent.id, + :staged_slug => "peter_pan" + ) + + test_node2 = test_node.children.create! :slug => "test_node2" + + put :publish, params: { :id => test_node.id } + + test_node.reload; test_node2.reload + assert_equal "parent/peter_pan", test_node.unique_name + assert_equal "parent/peter_pan/test_node2", test_node2.unique_name + end + + test "show node with empty draft" do + login_as :quentin + assert_not_nil node = create_node_with_draft + get :show, params: { :id => node.id } + assert_response :success + end + + test "show node with published draft" do + login_as :quentin + node = create_node_with_published_page + get :show, params: { :id => node.id } + assert_response :success + assert_select "td", :text => "Test", :count => 3 + end + + test "unlocking a locked node" do + login_as :quentin + node = create_node_with_published_page + node.find_or_create_draft User.first + + assert node.locked? + + put :unlock, params: { :id => node.id } + assert_response :redirect + assert !node.reload.locked? + end + + test "unlocking an already unlocked node" do + login_as :quentin + node = create_node_with_published_page + + put :unlock, params: { :id => node.id } + assert_response :redirect + assert_equal "Already unlocked", flash[:notice] + end + + test "updating a node by changing its parent" do + Node.root.descendants.destroy_all + login_as :quentin + node = create_node_with_published_page + node.find_or_create_draft User.first + + other_node = Node.root.children.create( :slug => "other" ) + + node.staged_parent_id = other_node.id + node.publish_draft! + + assert Node.valid? + end + + test "editing the initial draft sets the author to current_user" do + login_as :quentin + Node.root.descendants.destroy_all + node = create_node_with_draft + get :edit, params: { :id => node.id } + node.reload + assert_equal "quentin", node.draft.user.login + end + + test "updating the author of a node with existing head" do + login_as :quentin + Node.root.descendants.destroy_all + node = create_node_with_published_page + assert_equal "quentin", node.head.user.login + node.find_or_create_draft users(:quentin) + assert node.draft.valid? + assert node.valid? + + put :update, params: { :id => node.id, :page => {:user_id => users(:aaron).id} } + assert_response :redirect + assert_equal "aaron", node.reload.draft.user.login + end + + test "updating an existing page should not modify published_at" do + login_as :quentin + Node.root.descendants.destroy_all + node = create_node_with_published_page + + get :edit, params: { :id => node.id } + assert_response :success + + put :publish, params: { :id => node.id } + + node.reload + assert_equal node.pages[0].published_at, node.pages[1].published_at + end + + test "updating an exisiting page should not alter the author" do + login_as :aaron + Node.root.descendants.destroy_all + node = create_node_with_published_page + get :edit, params: { :id => node.id } + + put :publish, params: { :id => node.id } + + node.reload + assert_equal node.pages[0].user, node.pages[1].user + end + + test "editor and author are the same on a new node" do + login_as :quentin + node = create_node_with_draft + get :edit, params: { :id => node.id } + + node.reload + assert_equal "quentin", node.draft.user.login + assert_equal "quentin", node.draft.editor.login + end + + test "creating new draft alters the editor but keeps the author" do + node = create_node_with_published_page + assert_equal "quentin", node.head.user.login + + login_as :aaron + get :edit, params: {:id => node.id } + + node.reload + assert_equal "quentin", node.head.user.login + assert_equal "aaron", node.draft.editor.login + end + + test "unlocking and relocking changes editor if done by another user" do + node = create_node_with_published_page + draft = node.find_or_create_draft users(:quentin) + assert_equal draft.user.login, draft.editor.login + assert node.locked? + node.unlock! + + login_as :aaron + get :edit, params: { :id => node.id } + + node.reload + assert_equal "quentin", node.draft.user.login + assert_equal "aaron", node.draft.editor.login + end + + test "destroy a published node" do + node = create_node_with_published_page + node.destroy + + login_as :quentin + get :index + end + + test "no dangling pages remain after node removal" do + node = create_node_with_published_page + page_id = node.pages.first.id + node.destroy + + assert_raises(ActiveRecord::RecordNotFound) do + assert Page.find page_id + end + end + + test "can remove a node with an event" do + node = create_node_with_published_page + Event.create!( + :start_time => "2009-01-01T15:23:42".to_time, + :end_time => "2009-01-01T20:05:23".to_time, + :url => "http://events.ccc.de/congress/2082", + :latitude => 52.525308, + :longitude => 13.378944, + :allday => true, + :node_id => node.id + ) + node.destroy + + login_as :quentin + get :index + end + +end diff --git a/test/controllers/occurrences_controller_test.rb b/test/controllers/occurrences_controller_test.rb new file mode 100644 index 0000000..87f8bdb --- /dev/null +++ b/test/controllers/occurrences_controller_test.rb @@ -0,0 +1,45 @@ +require 'test_helper' + +class OccurrencesControllerTest < ActionController::TestCase + # test "should get index" do + # get :index + # assert_response :success + # assert_not_nil assigns(:occurrences) + # end + # + # test "should get new" do + # get :new + # assert_response :success + # end + # + # test "should create occurrence" do + # assert_difference('Occurrence.count') do + # post :create, params: { :occurrence => { } } + # end + # + # assert_redirected_to occurrence_path(assigns(:occurrence)) + # end + # + # test "should show occurrence" do + # get :show, params: { :id => occurrences(:one).to_param } + # assert_response :success + # end + # + # test "should get edit" do + # get :edit, params: { :id => occurrences(:one).to_param } + # assert_response :success + # end + # + # test "should update occurrence" do + # put :update, params: { :id => occurrences(:one).to_param, :occurrence => { } } + # assert_redirected_to occurrence_path(assigns(:occurrence)) + # end + # + # test "should destroy occurrence" do + # assert_difference('Occurrence.count', -1) do + # delete params: { :destroy, :id => occurrences(:one).to_param } + # end + # + # assert_redirected_to occurrences_path + # end +end diff --git a/test/controllers/pages_controller_test.rb b/test/controllers/pages_controller_test.rb new file mode 100644 index 0000000..3879014 --- /dev/null +++ b/test/controllers/pages_controller_test.rb @@ -0,0 +1,5 @@ +require 'test_helper' + +class PagesControllerTest < ActionController::TestCase + # will be removed anyway +end diff --git a/test/controllers/revisions_controller_test.rb b/test/controllers/revisions_controller_test.rb new file mode 100644 index 0000000..385e458 --- /dev/null +++ b/test/controllers/revisions_controller_test.rb @@ -0,0 +1,62 @@ +require 'test_helper' + +class RevisionsControllerTest < ActionController::TestCase + + def setup + Node.root.descendants.destroy_all + @user = User.find_by_login("aaron") + @node = Node.root.children.create!( :slug => "version_me" ) + + draft = @node.draft + draft.body = "first" + @node.publish_draft! + @node.find_or_create_draft @user + draft = @node.draft + draft.update_attributes(:body => "second") + @node.publish_draft! + end + + test "setup" do + assert_equal 2, Node.count + assert_equal 2, @node.pages.count + assert_equal ["first", "second"], @node.pages.map {|p| p.body} + end + + test "get list of revisions for a given node" do + login_as :quentin + get :index, params: { :node_id => @node.id } + assert_response :success + assert_select ".revision", 2 + end + + test "showing one revision" do + login_as :quentin + get :show, params: { :node_id => @node.id, :id => @node.pages.last.id } + assert_response :success + assert_select "strong", "Body" + assert_select "td", {:count => 1, :text => "second"} + end + + test "diffing two revisions" do + login_as :quentin + post( + :diff, params: { + :node_id => @node.id, + :start_revision => @node.pages.first.revision, + :end_revision => @node.pages.last.revision + } + ) + assert_response :success + end + + test "restoring a revision" do + assert_equal "second", @node.head.body + + login_as :aaron + put( :restore, params: { :node_id => @node.id, :id => @node.pages.first.id } ) + + @node.reload + assert_equal @node.head, @node.pages.first + assert_equal "first", @node.head.reload.body + end +end diff --git a/test/controllers/rss_controller_test.rb b/test/controllers/rss_controller_test.rb new file mode 100644 index 0000000..7e28844 --- /dev/null +++ b/test/controllers/rss_controller_test.rb @@ -0,0 +1,34 @@ +require 'test_helper' + +class RssControllerTest < ActionController::TestCase + + def setup + @user = User.create :login => 'rsstest', :email => 'rsstest@example.com', + :password => 'foobar', :password_confirmation => 'foobar' + @node = Node.root.children.create! :slug => 'rss_test_node' + draft = @node.find_or_create_draft @user + draft.title = "RSS Update Article" + draft.tag_list = "update" + draft.save + @node.publish_draft! + end + + test "updates feed contains tagged pages" do + begin + get :updates, params: { format: :xml } + rescue ActionView::Template::Error => e + raise unless e.message =~ /superclass mismatch/ + end + assert assigns(:items).any?, "Expected at least one page tagged with 'update'" + end + + test "updates feed is limited to 20 items" do + begin + get :updates, params: { format: :xml } + rescue ActionView::Template::Error => e + raise unless e.message =~ /superclass mismatch/ + end + assert assigns(:items).length <= 20 + end + +end diff --git a/test/controllers/search_controller_test.rb b/test/controllers/search_controller_test.rb new file mode 100644 index 0000000..49bb14f --- /dev/null +++ b/test/controllers/search_controller_test.rb @@ -0,0 +1,8 @@ +require 'test_helper' + +class SearchControllerTest < ActionController::TestCase + # Replace this with your real tests. + test "the truth" do + assert true + end +end diff --git a/test/controllers/sessions_controller_test.rb b/test/controllers/sessions_controller_test.rb new file mode 100644 index 0000000..bfcc647 --- /dev/null +++ b/test/controllers/sessions_controller_test.rb @@ -0,0 +1,32 @@ +require File.dirname(__FILE__) + '/../test_helper' +require 'sessions_controller' + +# Re-raise errors caught by the controller. +class SessionsController; def rescue_action(e) raise e end; end + +class SessionsControllerTest < ActionController::TestCase + # Be sure to include AuthenticatedTestHelper in test/test_helper.rb instead + # Then, you can remove it from this and the units test. + include AuthenticatedTestHelper + + fixtures :users + + def test_should_login_and_redirect + post :create, params: { login: 'quentin', password: 'monkey' } + assert session[:user_id] + assert_response :redirect + end + + def test_should_fail_login_and_not_redirect + post :create, params: { login: 'quentin', password: 'bad password' } + assert_nil session[:user_id] + assert_response :success + end + + def test_should_logout + login_as :quentin + get :destroy + assert_nil session[:user_id] + assert_response :redirect + end +end diff --git a/test/controllers/tags_controller_test.rb b/test/controllers/tags_controller_test.rb new file mode 100644 index 0000000..95c0d31 --- /dev/null +++ b/test/controllers/tags_controller_test.rb @@ -0,0 +1,34 @@ +require 'test_helper' + +class TagsControllerTest < ActionController::TestCase + + def setup + @user = User.create :login => 'tagtest', :email => 'tagtest@example.com', + :password => 'foobar', :password_confirmation => 'foobar' + @node = Node.root.children.create! :slug => 'tag_test_node' + draft = @node.find_or_create_draft @user + draft.title = "Tagged Article" + draft.tag_list = "testtag" + draft.save + @node.publish_draft! + end + + test "show returns pages tagged with the requested tag" do + get :show, params: { id: 'testtag', locale: 'de' } + assert_response :success + assert assigns(:pages).any?, "Expected at least one page tagged with 'testtag'" + assert assigns(:pages).all? { |p| p.is_a?(Page) } + end + + test "show with unknown tag returns empty collection" do + get :show, params: { id: 'nonexistent_tag_xyz', locale: 'de' } + assert_response :success + assert assigns(:pages).empty? + end + + test "show with invalid tag characters returns 400" do + get :show, params: { id: '', locale: 'de' } + assert_response 400 + end + +end diff --git a/test/controllers/users_controller_test.rb b/test/controllers/users_controller_test.rb new file mode 100644 index 0000000..3ace95c --- /dev/null +++ b/test/controllers/users_controller_test.rb @@ -0,0 +1,186 @@ +require 'test_helper' + +class UsersControllerTest < ActionController::TestCase + + test "get index as regular user renders stripped partial" do + login_as :quentin + get :index + assert_response :success + assert_select "a", { :count => 0, :text => "Destroy" } + end + + test "get index as admin user renders admin partial" do + login_as :aaron + get :index + assert_response :success + assert_select "a", "destroy" + assert_select "a", "show", "Show Link is missing" + end + + test "get new when logged in as admin" do + login_as :aaron + get :new + assert_response :success + end + + test "get new without being logged in as admin redirects back to index" do + login_as :quentin + get :new + assert_response :redirect + assert_redirected_to users_path + assert_equal( + "Sorry, you need to be an admin for this action", + flash[:notice] + ) + end + + test "creating new users being logged in as admin" do + login_as :aaron + assert_difference "User.count", +1 do + post :create, params: { + :user => { + :login => "peter", + :email => "foo@bar.com", + :password => "xxxzzz", + :password_confirmation => "xxxzzz" + } + } + end + + assert_redirected_to user_path(User.last) + assert !User.last.admin + end + + test "creating new admin users being logged in as admin" do + login_as :aaron + assert_difference "User.count", +1 do + post :create, params: { + :user => { + :login => "peter", + :email => "foo@bar.com", + :password => "xxxzzz", + :password_confirmation => "xxxzzz", + :admin => true + } + } + end + + assert_redirected_to user_path(User.last) + assert User.last.admin + end + + test "creating new users not being logged as regular user wont work" do + login_as :quentin + assert_no_difference "User.count" do + post :create, params: { + :user => { + :login => "peter", + :email => "foo@bar.com", + :password => "xxxzzz", + :password_confirmation => "xxxzzz" + } + } + end + + assert_redirected_to users_path + assert_equal( + "Sorry, you need to be an admin for this action", + flash[:notice] + ) + end + + test "get edit of another user being logged in as regular user wont work" do + login_as :quentin + get :edit, params: { :id => User.find_by_login("aaron").id } + assert_redirected_to users_path + assert_equal( + "Sorry, you need to be an admin for this action", + flash[:notice] + ) + end + + test "get edit of another user being logged in as admin user" do + login_as :aaron + get :edit, params: { :id => User.find_by_login("quentin").id } + assert_response :success + end + + test "editing own user details is allowed" do + login_as :quentin + get :edit, params: { :id => User.find_by_login("quentin").id } + assert_response :success + end + + test "updating an user when being logged in as regular user wont work" do + user = User.find_by_login("aaron") + login_as :quentin + put :update, params: { :id => user.id, :user => {:login => "random"} } + assert_redirected_to users_path + assert_equal( + "Sorry, you need to be an admin for this action", + flash[:notice] + ) + end + + test "updating an user when being login in as admin user" do + user = User.find_by_login("quentin") + login_as :aaron + put :update, params: { :id => user.id, :user => {:login => "random"} } + assert_redirected_to user_path(user) + assert_equal "random", user.reload.login + end + + test "updating own user details is allowd" do + user = User.find_by_login("quentin") + login_as :quentin + put :update, params: { :id => user.id, :user => {:login => "random"} } + assert_redirected_to user_path(user) + assert_equal "random", user.reload.login + end + + test "showing a user" do + login_as :quentin + get :show, params: { :id => User.find_by_login("aaron").id } + assert_response :success + end + + test "destroying an user being logged in as regular user wont work" do + login_as :quentin + assert_no_difference "User.count" do + delete :destroy, params: { :id => User.find_by_login("aaron").id } + end + assert_redirected_to users_path + assert_equal( + "Sorry, you need to be an admin for this action", + flash[:notice] + ) + end + + test "destroying an user being logged in as admin user" do + login_as :aaron + assert_difference "User.count", -1 do + delete :destroy, params: { :id => User.find_by_login("quentin").id } + end + assert_redirected_to users_path + end + + test "admin user can promote regular users to admins" do + login_as :aaron + user = users(:quentin) + put :update, params: { :id => user.id, :user => {:admin => true} } + + user.reload + assert_equal true, user.is_admin? + end + + test "regular users cannot promote themselves to admins" do + login_as :quentin + user = users(:quentin) + put :update, params: { :id => user.id, :user => {:admin => true} } + + user.reload + assert_equal false, user.is_admin? + end + + +end diff --git a/test/functional/admin_controller_test.rb b/test/functional/admin_controller_test.rb deleted file mode 100644 index 9bbf29b..0000000 --- a/test/functional/admin_controller_test.rb +++ /dev/null @@ -1,8 +0,0 @@ -require 'test_helper' - -class AdminControllerTest < ActionController::TestCase - # Replace this with your real tests. - test "the truth" do - assert true - end -end diff --git a/test/functional/assets_controller_test.rb b/test/functional/assets_controller_test.rb deleted file mode 100644 index d003d25..0000000 --- a/test/functional/assets_controller_test.rb +++ /dev/null @@ -1,4 +0,0 @@ -require 'test_helper' - -class AssetsControllerTest < ActionController::TestCase -end diff --git a/test/functional/content_controller_test.rb b/test/functional/content_controller_test.rb deleted file mode 100644 index 106f10d..0000000 --- a/test/functional/content_controller_test.rb +++ /dev/null @@ -1,130 +0,0 @@ -require 'test_helper' - -class ContentControllerTest < ActionController::TestCase - - def setup - @root = Node.find(1) - @first_child = Node.find(2) - @second_child = Node.find(3) - - @user1 = User.create :login => 'demo', :email => "f@b.com", :password => 'foobar', :password_confirmation => 'foobar' - @user2 = User.create :login => 'show', :email => "f@b.com", :password => 'foobar', :password_confirmation => 'foobar' - end - - def test_custom_page_route - assert_recognizes({ :controller => 'content', :action => 'render_page', :locale => 'de', :page_path => 'foo/bar' }, '/de/foo/bar') - assert_recognizes({ :controller => 'content', :action => 'render_page', :locale => 'en', :page_path => 'home' }, '/en/home') - end - - def test_render_404_when_no_page_was_found - get :render_page, :language => 'de', :page_path => ["wrong_path"] - assert_response 404 - end - - def test_rendering_a_page - assert Node.valid? - assert_not_nil first_child = Node.find_by_slug("first_child") - page = first_child.pages.create :title => "First Child" - first_child.head = page - first_child.save! - - get :render_page, :language => 'de', :page_path => ["first_child"] - assert_response :success - assert_equal "layouts/application", @controller.active_layout.name rescue assert true - end - - def test_page_containing_aggregator - assert_not_nil Node.root - - fill_pages_with_content - - new_node = create_node_under_root "fnord" - draft = new_node.find_or_create_draft @user1 - draft.body = '' - draft.save - new_node.publish_draft! - - get :render_page, :locale => 'de', :page_path => ["fnord"] - assert_response :success - - # The aggregator renders into div.body > div.article_partial. - # Without a working aggregator this will be empty. - assert_select "div.body div.article_partial", :minimum => 2 - assert_select "div.body div.article_partial h2.headline a", :text => "one" - assert_select "div.body div.article_partial h2.headline a", :text => "two" - end - - def test_page_containing_aggregator_with_custom_template - fill_pages_with_content - - new_node = create_node_under_root "fnord" - draft = new_node.find_or_create_draft @user1 - draft.body = '' - draft.save - new_node.publish_draft! - - get :render_page, :locale => 'de', :page_path => ["fnord"] - assert_response :success - - assert_select(".sidebar_headline", "one") - assert_select(".sidebar_headline", "two") - end - - def test_nonexistant_custom_template_defaults_to_standard_template - new_node = create_node_under_root "fnord" - draft = new_node.find_or_create_draft @user1 - draft.template_name = "huchibu" - draft.save - new_node.publish_draft! - - get :render_page, :locale => 'de', :page_path => ["fnord"] - assert_response :success - assert_template "custom/page_templates/public/standard_template" - end - - def test_custom_template_no_date_and_author - new_node = create_node_under_root "fnord" - draft = new_node.find_or_create_draft @user1 - draft.template_name = "no_date_and_author" - draft.save - new_node.publish_draft! - - get :render_page, :locale => 'de', :page_path => ["fnord"] - assert_response :success - assert_template "custom/page_templates/public/no_date_and_author" - end - - def test_aggregator_without_fill - new_node = create_node_under_root "fnord" - draft = new_node.find_or_create_draft @user1 - draft.body = '' - draft.save - new_node.publish_draft! - - get :render_page, :locale => 'de', :page_path => ["fnord"] - assert_response :success - File.write("/tmp/no_fill_response.html", @response.body) - end - - protected - - def create_node_under_root slug - node = Node.root.children.create! :slug => slug - node - end - - def fill_pages_with_content - d1 = @first_child.find_or_create_draft @user1 - d1.title = "one" - d1.tag_list = "update" - d1.save - @first_child.publish_draft! - - d2 = @second_child.find_or_create_draft @user1 - d2.title = "two" - d2.tag_list = "update" - d2.save - @second_child.publish_draft! - end - -end diff --git a/test/functional/events_controller_test.rb b/test/functional/events_controller_test.rb deleted file mode 100644 index 5698c7b..0000000 --- a/test/functional/events_controller_test.rb +++ /dev/null @@ -1,45 +0,0 @@ -require 'test_helper' - -class EventsControllerTest < ActionController::TestCase - # test "should get index" do - # get :index - # assert_response :success - # assert_not_nil assigns(:events) - # end - # - # test "should get new" do - # get :new - # assert_response :success - # end - # - # test "should create event" do - # assert_difference('Event.count') do - # post :create, :event => { } - # end - # - # assert_redirected_to event_path(assigns(:event)) - # end - # - # test "should show event" do - # get :show, :id => events(:one).to_param - # assert_response :success - # end - # - # test "should get edit" do - # get :edit, :id => events(:one).to_param - # assert_response :success - # end - # - # test "should update event" do - # put :update, :id => events(:one).to_param, :event => { } - # assert_redirected_to event_path(assigns(:event)) - # end - # - # test "should destroy event" do - # assert_difference('Event.count', -1) do - # delete :destroy, :id => events(:one).to_param - # end - # - # assert_redirected_to events_path - # end -end diff --git a/test/functional/menu_items_controller_test.rb b/test/functional/menu_items_controller_test.rb deleted file mode 100644 index c47467a..0000000 --- a/test/functional/menu_items_controller_test.rb +++ /dev/null @@ -1,8 +0,0 @@ -require 'test_helper' - -class MenuItemsControllerTest < ActionController::TestCase - # Replace this with your real tests. - test "the truth" do - assert true - end -end diff --git a/test/functional/nodes_controller_test.rb b/test/functional/nodes_controller_test.rb deleted file mode 100644 index d53fde2..0000000 --- a/test/functional/nodes_controller_test.rb +++ /dev/null @@ -1,386 +0,0 @@ -require 'test_helper' - -class NodesControllerTest < ActionController::TestCase - - def test_get_index - Node.root.descendants.delete_all - test_node = Node.root.children.create :slug => "foo" - login_as :quentin - get :index - assert_response :success - end - - def test_new - login_as :quentin - get :new - assert_response :success - end - - test "create generic node with parent_id provided" do - login_as :quentin - assert_difference "Node.count", +1 do - post( - :create, - :kind => "generic", - :parent_id => Node.root.id, - :title => "Hello Spaceboy" - ) - end - - assert_response :redirect - assert_equal "hello-spaceboy", Node.last.slug - assert_equal Node.last.parent_id, Node.root.id - assert_equal 1, Node.last.level - end - - test "create update node" do - login_as :quentin - #difference of three because "updates" and "2009" node get created as well - assert_difference "Node.count", +3 do - post( - :create, - :kind => "update", - :title => "Hello Spaceboy" - ) - end - - assert_response :redirect - expected = "updates/#{Time.now.year.to_s}/hello-spaceboy" - assert_equal expected, Node.last.unique_name - assert_equal 3, Node.last.level - end - - test "create top level node" do - login_as :quentin - - assert_difference "Node.count", +1 do - post( - :create, - :kind => "top_level", - :title => "Hello My Spaceboy" - ) - end - - assert_response :redirect - expected = "hello-my-spaceboy" - assert_equal expected, Node.last.unique_name - assert_equal 1, Node.last.level - end - - test "creating a top_level node without a title should not work" do - login_as :quentin - - assert_no_difference "Node.count" do - post(:create, :kind => "top_level") - end - end - - test "creating a generic node without a parent_id should not work" do - login_as :quentin - - assert_no_difference "Node.count" do - post(:create, :kind => "generic") - end - end - - test "editing a node" do - login_as :quentin - - node = Node.find_by_unique_name("fourth_child") - node.pages.create - node.draft = node.pages.last - node.save - - assert_equal 1, node.pages.length - - draft = node.find_or_create_draft( User.first ) - draft.title = "Hello" - draft.body = "World" - draft.save - node.publish_draft! - - get :edit, :id => node.id - assert_response :success - assert_select("#page_title[value='Hello']") - assert_select("#page_body", "World") - - node.reload - assert_equal 2, node.pages.length - assert_equal "Hello", node.find_or_create_draft( User.first ).title - assert_equal "World", node.find_or_create_draft( User.first ).body - end - - test "editing a locked node raises LockedByAnotherUser Exception" do - login_as :quentin - - node = create_node_with_draft - node.lock_owner = User.last - node.save - - assert node.locked? - - get :edit, :id => node.id - assert_response :redirect - assert flash[:error] =~ /Page is locked by another user/ - end - - def test_update_a_draft - test_node = Node.root.children.create! :slug => "test_node" - - login_as :quentin - put :update, :id => test_node.id, :page => {:title => "Hello", :body => "There"} - - assert_equal "Hello", test_node.draft.title - assert_equal "There", test_node.draft.body - end - - def test_update_a_draft_with_changing_the_template - test_node = Node.root.children.create! :slug => "test_node" - - login_as :quentin - put :update, { - :id => test_node.id, - :page => { - :title => "Hello", - :body => "There", - :template_name => "Foobar" - } - } - - test_node.reload - assert_equal "Hello", test_node.draft.title - assert_equal "There", test_node.draft.body - assert_equal "Foobar", test_node.draft.template_name - end - - - test "publish draft with staged_slug unqueal slug" do - login_as :quentin - - test_node = Node.root.children.create! :slug => "test_node", :staged_slug => "peter_pan" - - put :publish, :id => test_node.id - - test_node.reload - assert_equal "peter_pan", test_node.slug - assert_equal "peter_pan", test_node.unique_name - end - - test "publish draft with staged_slug with more levels of nodes" do - login_as :quentin - - test_node = Node.root.children.create! :slug => "test_node", :staged_slug => "peter_pan" - test_node2 = test_node.children.create! :slug => "test_node2" - - put :publish, :id => test_node.id - - test_node.reload; test_node2.reload - assert_equal "peter_pan/test_node2", test_node2.unique_name - assert_equal "peter_pan", test_node.unique_name - end - - test "publish draft with staged_parent_id" do - login_as :quentin - - parent = Node.root.children.create! :slug => "parent" - test_node = Node.root.children.create! :slug => "test_node", :staged_parent_id => parent.id - test_node2 = test_node.children.create! :slug => "test_node2" - - put :publish, :id => test_node.id - - test_node.reload; test_node2.reload - assert_equal "parent/test_node", test_node.unique_name - assert_equal "parent/test_node/test_node2", test_node2.unique_name - end - - test "publish draft with staged_parent_id and staged_slug" do - login_as :quentin - - parent = Node.root.children.create! :slug => "parent" - - test_node = Node.root.children.create!( - :slug => "test_node", - :staged_parent_id => parent.id, - :staged_slug => "peter_pan" - ) - - test_node2 = test_node.children.create! :slug => "test_node2" - - put :publish, :id => test_node.id - - test_node.reload; test_node2.reload - assert_equal "parent/peter_pan", test_node.unique_name - assert_equal "parent/peter_pan/test_node2", test_node2.unique_name - end - - test "show node with empty draft" do - login_as :quentin - assert_not_nil node = create_node_with_draft - get :show, :id => node.id - assert_response :success - end - - test "show node with published draft" do - login_as :quentin - node = create_node_with_published_page - get :show, :id => node.id - assert_response :success - assert_select "td", :text => "Test", :count => 3 - end - - test "unlocking a locked node" do - login_as :quentin - node = create_node_with_published_page - node.find_or_create_draft User.first - - assert node.locked? - - get :unlock, :id => node.id - assert_response :redirect - assert !node.reload.locked? - end - - test "unlocking an already unlocked node" do - login_as :quentin - node = create_node_with_published_page - - get :unlock, :id => node.id - assert_response :redirect - assert_equal "Already unlocked", flash[:notice] - end - - test "updating a node by changing its parent" do - Node.root.descendants.destroy_all - login_as :quentin - node = create_node_with_published_page - node.find_or_create_draft User.first - - other_node = Node.root.children.create( :slug => "other" ) - - node.staged_parent_id = other_node.id - node.publish_draft! - - assert Node.valid? - end - - test "editing the initial draft sets the author to current_user" do - login_as :quentin - Node.root.descendants.destroy_all - node = create_node_with_draft - get :edit, :id => node.id - assert_equal "quentin", node.draft.user.login - end - - test "updating the author of a node with existing head" do - login_as :quentin - Node.root.descendants.destroy_all - node = create_node_with_published_page - assert_equal "quentin", node.head.user.login - node.find_or_create_draft users(:quentin) - assert node.draft.valid? - assert node.valid? - - put :update, :id => node.id, :page => {:user_id => users(:aaron).id} - assert_response :redirect - assert_equal "aaron", node.reload.draft.user.login - end - - test "updating an existing page should not modify published_at" do - login_as :quentin - Node.root.descendants.destroy_all - node = create_node_with_published_page - - get :edit, :id => node.id - assert_response :success - - put :publish, :id => node.id - - node.reload - assert_equal node.pages[0].published_at, node.pages[1].published_at - end - - test "updating an exisiting page should not alter the author" do - login_as :aaron - Node.root.descendants.destroy_all - node = create_node_with_published_page - get :edit, :id => node.id - - put :publish, :id => node.id - - node.reload - assert_equal node.pages[0].user, node.pages[1].user - end - - test "editor and author are the same on a new node" do - login_as :quentin - node = create_node_with_draft - get :edit, :id => node.id - - node.reload - assert_equal "quentin", node.draft.user.login - assert_equal "quentin", node.draft.editor.login - end - - test "creating new draft alters the editor but keeps the author" do - node = create_node_with_published_page - assert_equal "quentin", node.head.user.login - - login_as :aaron - get :edit, :id => node.id - - node.reload - assert_equal "quentin", node.head.user.login - assert_equal "aaron", node.draft.editor.login - end - - test "unlocking and relocking changes editor if done by another user" do - node = create_node_with_published_page - draft = node.find_or_create_draft users(:quentin) - assert_equal draft.user.login, draft.editor.login - assert node.locked? - node.unlock! - - login_as :aaron - get :edit, :id => node.id - - node.reload - assert_equal "quentin", node.draft.user.login - assert_equal "aaron", node.draft.editor.login - end - - test "destroy a published node" do - node = create_node_with_published_page - node.destroy - - login_as :quentin - get :index - end - - test "no dangling pages remain after node removal" do - node = create_node_with_published_page - page_id = node.pages.first.id - node.destroy - - assert_raises(ActiveRecord::RecordNotFound) do - assert Page.find page_id - end - end - - test "can remove a node with an event" do - node = create_node_with_published_page - Event.create!( - :start_time => "2009-01-01T15:23:42".to_time, - :end_time => "2009-01-01T20:05:23".to_time, - :url => "http://events.ccc.de/congress/2082", - :latitude => 52.525308, - :longitude => 13.378944, - :allday => true, - :node_id => node.id - ) - node.destroy - - login_as :quentin - get :index - end - -end diff --git a/test/functional/occurrences_controller_test.rb b/test/functional/occurrences_controller_test.rb deleted file mode 100644 index 0b00e0e..0000000 --- a/test/functional/occurrences_controller_test.rb +++ /dev/null @@ -1,45 +0,0 @@ -require 'test_helper' - -class OccurrencesControllerTest < ActionController::TestCase - # test "should get index" do - # get :index - # assert_response :success - # assert_not_nil assigns(:occurrences) - # end - # - # test "should get new" do - # get :new - # assert_response :success - # end - # - # test "should create occurrence" do - # assert_difference('Occurrence.count') do - # post :create, :occurrence => { } - # end - # - # assert_redirected_to occurrence_path(assigns(:occurrence)) - # end - # - # test "should show occurrence" do - # get :show, :id => occurrences(:one).to_param - # assert_response :success - # end - # - # test "should get edit" do - # get :edit, :id => occurrences(:one).to_param - # assert_response :success - # end - # - # test "should update occurrence" do - # put :update, :id => occurrences(:one).to_param, :occurrence => { } - # assert_redirected_to occurrence_path(assigns(:occurrence)) - # end - # - # test "should destroy occurrence" do - # assert_difference('Occurrence.count', -1) do - # delete :destroy, :id => occurrences(:one).to_param - # end - # - # assert_redirected_to occurrences_path - # end -end diff --git a/test/functional/pages_controller_test.rb b/test/functional/pages_controller_test.rb deleted file mode 100644 index 3879014..0000000 --- a/test/functional/pages_controller_test.rb +++ /dev/null @@ -1,5 +0,0 @@ -require 'test_helper' - -class PagesControllerTest < ActionController::TestCase - # will be removed anyway -end diff --git a/test/functional/revisions_controller_test.rb b/test/functional/revisions_controller_test.rb deleted file mode 100644 index 43001df..0000000 --- a/test/functional/revisions_controller_test.rb +++ /dev/null @@ -1,61 +0,0 @@ -require 'test_helper' - -class RevisionsControllerTest < ActionController::TestCase - - def setup - Node.root.descendants.destroy_all - @user = User.find_by_login("aaron") - @node = Node.root.children.create!( :slug => "version_me" ) - - draft = @node.draft - draft.body = "first" - @node.publish_draft! - @node.find_or_create_draft @user - draft = @node.draft - draft.update_attributes(:body => "second") - @node.publish_draft! - end - - test "setup" do - assert_equal 2, Node.count - assert_equal 2, @node.pages.count - assert_equal ["first", "second"], @node.pages.map {|p| p.body} - end - - test "get list of revisions for a given node" do - login_as :quentin - get :index, :node_id => @node.id - assert_response :success - assert_select ".revision", 2 - end - - test "showing one revision" do - login_as :quentin - get :show, :node_id => @node.id, :id => @node.pages.last.id - assert_response :success - assert_select "strong", "Body" - assert_select "td", {:count => 1, :text => "second"} - end - - test "diffing two revisions" do - login_as :quentin - post( - :diff, - :node_id => @node.id, - :start_revision => @node.pages.first.revision, - :end_revision => @node.pages.last.revision - ) - assert_response :success - end - - test "restoring a revision" do - assert_equal "second", @node.head.body - - login_as :aaron - put( :restore, :node_id => @node.id, :id => @node.pages.first.id ) - - @node.reload - assert_equal @node.head, @node.pages.first - assert_equal "first", @node.head.reload.body - end -end diff --git a/test/functional/rss_controller_test.rb b/test/functional/rss_controller_test.rb deleted file mode 100644 index acf7369..0000000 --- a/test/functional/rss_controller_test.rb +++ /dev/null @@ -1,34 +0,0 @@ -require 'test_helper' - -class RssControllerTest < ActionController::TestCase - - def setup - @user = User.create :login => 'rsstest', :email => 'rsstest@example.com', - :password => 'foobar', :password_confirmation => 'foobar' - @node = Node.root.children.create! :slug => 'rss_test_node' - draft = @node.find_or_create_draft @user - draft.title = "RSS Update Article" - draft.tag_list = "update" - draft.save - @node.publish_draft! - end - - test "updates feed contains tagged pages" do - begin - get :updates, :format => :xml - rescue ActionView::Template::Error => e - raise unless e.message =~ /superclass mismatch/ - end - assert assigns(:items).any?, "Expected at least one page tagged with 'update'" - end - - test "updates feed is limited to 20 items" do - begin - get :updates, :format => :xml - rescue ActionView::Template::Error => e - raise unless e.message =~ /superclass mismatch/ - end - assert assigns(:items).length <= 20 - end - -end diff --git a/test/functional/search_controller_test.rb b/test/functional/search_controller_test.rb deleted file mode 100644 index 49bb14f..0000000 --- a/test/functional/search_controller_test.rb +++ /dev/null @@ -1,8 +0,0 @@ -require 'test_helper' - -class SearchControllerTest < ActionController::TestCase - # Replace this with your real tests. - test "the truth" do - assert true - end -end diff --git a/test/functional/sessions_controller_test.rb b/test/functional/sessions_controller_test.rb deleted file mode 100644 index 6baff5c..0000000 --- a/test/functional/sessions_controller_test.rb +++ /dev/null @@ -1,32 +0,0 @@ -require File.dirname(__FILE__) + '/../test_helper' -require 'sessions_controller' - -# Re-raise errors caught by the controller. -class SessionsController; def rescue_action(e) raise e end; end - -class SessionsControllerTest < ActionController::TestCase - # Be sure to include AuthenticatedTestHelper in test/test_helper.rb instead - # Then, you can remove it from this and the units test. - include AuthenticatedTestHelper - - fixtures :users - - def test_should_login_and_redirect - post :create, :login => 'quentin', :password => 'monkey' - assert session[:user_id] - assert_response :redirect - end - - def test_should_fail_login_and_not_redirect - post :create, :login => 'quentin', :password => 'bad password' - assert_nil session[:user_id] - assert_response :success - end - - def test_should_logout - login_as :quentin - get :destroy - assert_nil session[:user_id] - assert_response :redirect - end -end diff --git a/test/functional/tags_controller_test.rb b/test/functional/tags_controller_test.rb deleted file mode 100644 index 23049b9..0000000 --- a/test/functional/tags_controller_test.rb +++ /dev/null @@ -1,34 +0,0 @@ -require 'test_helper' - -class TagsControllerTest < ActionController::TestCase - - def setup - @user = User.create :login => 'tagtest', :email => 'tagtest@example.com', - :password => 'foobar', :password_confirmation => 'foobar' - @node = Node.root.children.create! :slug => 'tag_test_node' - draft = @node.find_or_create_draft @user - draft.title = "Tagged Article" - draft.tag_list = "testtag" - draft.save - @node.publish_draft! - end - - test "show returns pages tagged with the requested tag" do - get :show, :id => 'testtag', :locale => 'de' - assert_response :success - assert assigns(:pages).any?, "Expected at least one page tagged with 'testtag'" - assert assigns(:pages).all? { |p| p.is_a?(Page) } - end - - test "show with unknown tag returns empty collection" do - get :show, :id => 'nonexistent_tag_xyz', :locale => 'de' - assert_response :success - assert assigns(:pages).empty? - end - - test "show with invalid tag characters returns 400" do - get :show, :id => '', :locale => 'de' - assert_response 400 - end - -end diff --git a/test/functional/users_controller_test.rb b/test/functional/users_controller_test.rb deleted file mode 100644 index 7511170..0000000 --- a/test/functional/users_controller_test.rb +++ /dev/null @@ -1,180 +0,0 @@ -require 'test_helper' - -class UsersControllerTest < ActionController::TestCase - - test "get index as regular user renders stripped partial" do - login_as :quentin - get :index - assert_response :success - assert_select "a", { :count => 0, :text => "Destroy" } - end - - test "get index as admin user renders admin partial" do - login_as :aaron - get :index - assert_response :success - assert_select "a", "destroy" - assert_select "a", "show", "Show Link is missing" - end - - test "get new when logged in as admin" do - login_as :aaron - get :new - assert_response :success - end - - test "get new without being logged in as admin redirects back to index" do - login_as :quentin - get :new - assert_response :redirect - assert_redirected_to users_path - assert_equal( - "Sorry, you need to be an admin for this action", - flash[:notice] - ) - end - - test "creating new users being logged in as admin" do - login_as :aaron - assert_difference "User.count", +1 do - post :create, :user => { - :login => "peter", - :email => "foo@bar.com", - :password => "xxxzzz", - :password_confirmation => "xxxzzz" - } - end - - assert_redirected_to user_path(User.last) - assert !User.last.admin - end - - test "creating new admin users being logged in as admin" do - login_as :aaron - assert_difference "User.count", +1 do - post :create, :user => { - :login => "peter", - :email => "foo@bar.com", - :password => "xxxzzz", - :password_confirmation => "xxxzzz", - :admin => true - } - end - - assert_redirected_to user_path(User.last) - assert User.last.admin - end - - test "creating new users not being logged as regular user wont work" do - login_as :quentin - assert_no_difference "User.count" do - post :create, :user => { - :login => "peter", - :email => "foo@bar.com", - :password => "xxxzzz", - :password_confirmation => "xxxzzz" - } - end - - assert_redirected_to users_path - assert_equal( - "Sorry, you need to be an admin for this action", - flash[:notice] - ) - end - - test "get edit of another user being logged in as regular user wont work" do - login_as :quentin - get :edit, :id => User.find_by_login("aaron").id - assert_redirected_to users_path - assert_equal( - "Sorry, you need to be an admin for this action", - flash[:notice] - ) - end - - test "get edit of another user being logged in as admin user" do - login_as :aaron - get :edit, :id => User.find_by_login("quentin").id - assert_response :success - end - - test "editing own user details is allowed" do - login_as :quentin - get :edit, :id => User.find_by_login("quentin").id - assert_response :success - end - - test "updating an user when being logged in as regular user wont work" do - user = User.find_by_login("aaron") - login_as :quentin - put :update, :id => user.id, :user => {:login => "random"} - assert_redirected_to users_path - assert_equal( - "Sorry, you need to be an admin for this action", - flash[:notice] - ) - end - - test "updating an user when being login in as admin user" do - user = User.find_by_login("quentin") - login_as :aaron - put :update, :id => user.id, :user => {:login => "random"} - assert_redirected_to user_path(user) - assert_equal "random", user.reload.login - end - - test "updating own user details is allowd" do - user = User.find_by_login("quentin") - login_as :quentin - put :update, :id => user.id, :user => {:login => "random"} - assert_redirected_to user_path(user) - assert_equal "random", user.reload.login - end - - test "showing a user" do - login_as :quentin - get :show, :id => User.find_by_login("aaron").id - assert_response :success - end - - test "destroying an user being logged in as regular user wont work" do - login_as :quentin - assert_no_difference "User.count" do - delete :destroy, :id => User.find_by_login("aaron").id - end - assert_redirected_to users_path - assert_equal( - "Sorry, you need to be an admin for this action", - flash[:notice] - ) - end - - test "destroying an user being logged in as admin user" do - login_as :aaron - assert_difference "User.count", -1 do - delete :destroy, :id => User.find_by_login("quentin").id - end - assert_redirected_to users_path - end - - test "admin user can promote regular users to admins" do - login_as :aaron - user = users(:quentin) - put :update, :id => user.id, :user => {:admin => true} - - user.reload - assert_equal true, user.is_admin? - end - - test "regular users cannot promote themselves to admins" do - login_as :quentin - user = users(:quentin) - put :update, :id => user.id, :user => {:admin => true} - - user.reload - assert_equal false, user.is_admin? - end - - -end diff --git a/test/models/asset_test.rb b/test/models/asset_test.rb new file mode 100644 index 0000000..a1041e4 --- /dev/null +++ b/test/models/asset_test.rb @@ -0,0 +1,23 @@ +require 'test_helper' + +class AssetTest < ActiveSupport::TestCase + + test "related assets get destroyed when assets get destroyed" do + Asset.delete_all + RelatedAsset.delete_all + + assert asset = Asset.create + assert node = Node.root.children.create( :slug => "asset" ) + assert_equal [], node.draft.assets + + draft = node.draft + draft.assets << asset + assert_equal 1, draft.assets.length + + asset.destroy + draft.reload + assert_equal 0, draft.assets.length + assert_equal 0, RelatedAsset.count + end + +end diff --git a/test/models/event_test.rb b/test/models/event_test.rb new file mode 100644 index 0000000..f310af8 --- /dev/null +++ b/test/models/event_test.rb @@ -0,0 +1,114 @@ +require 'test_helper' + +class EventTest < ActiveSupport::TestCase + + def setup + Page.delete_all + @cal_node = Node.root.children.create! :slug => "calendar" + @draft = @cal_node.find_or_create_draft User.first + @draft.title = "99C3" + @draft.abstract = "The 99th Chaos Comunication Congress" + @draft.body = "Its totally freakin awesome" + @draft.save + @cal_node.publish_draft! + @cal_node.head.reload + end + + test 'verfy setup data' do + assert_not_nil @cal_node + assert_not_nil @cal_node.head + end + + test 'creating an event with malformed rrule raises exception' do + assert_raise(ArgumentError) do + Event.create!( + :start_time => "2009-01-01T15:23:42".to_time, + :end_time => "2009-01-01T20:05:23".to_time, + :url => "http://events.ccc.de/congress/2082", + :latitude => 52.525308, + :longitude => 13.378944, + :rrule => "FOOBAR", + :allday => false, + :custom_rrule => false, + :node_id => @cal_node.id + ) + end + end + + test 'create day event for node with one occurrence' do + assert_not_nil event = Event.create!( + :start_time => "2009-01-01T15:23:42".to_time, + :end_time => "2009-01-01T20:05:23".to_time, + :url => "http://events.ccc.de/congress/2082", + :latitude => 52.525308, + :longitude => 13.378944, + :rrule => nil, + :allday => false, + :custom_rrule => false, + :node_id => @cal_node.id + ) + + assert_equal 1, Occurrence.count + assert_equal event.start_time, Occurrence.first.start_time + assert_equal event.end_time, Occurrence.first.end_time + end + + test 'create day event with weekly reoccurrence and checking data' do + assert_not_nil event = Event.create!( + :start_time => "2009-01-01T15:23:42".to_time, + :end_time => "2009-01-01T20:05:23".to_time, + :url => "http://events.ccc.de/congress/2082", + :latitude => 52.525308, + :longitude => 13.378944, + :rrule => "FREQ=WEEKLY;INTERVAL=1", + :allday => false, + :custom_rrule => false, + :node_id => @cal_node.id + ) + + assert_not_nil scoped_occurrences = event.occurrences_in_range( + "2009-01-01".to_time, "2009-12-31".to_time + ) + + assert_equal 52, scoped_occurrences.length + + assert_equal "2009-12-24T15:23:42".to_time, scoped_occurrences[51].start_time + assert_equal "2009-12-24T20:05:23".to_time, scoped_occurrences[51].end_time + assert_equal @cal_node.event, scoped_occurrences[51].event + assert_equal @cal_node, scoped_occurrences[51].node + + assert_equal "2009-03-19T15:23:42".to_time, scoped_occurrences[11].start_time + assert_equal "2009-03-19T20:05:23".to_time, scoped_occurrences[11].end_time + assert_equal @cal_node.event, scoped_occurrences[11].event + assert_equal @cal_node, scoped_occurrences[11].node + + assert_equal "2009-01-01T15:23:42".to_time, scoped_occurrences[0].start_time + assert_equal "2009-01-01T20:05:23".to_time, scoped_occurrences[0].end_time + assert_equal @cal_node.event, scoped_occurrences[11].event + assert_equal @cal_node, scoped_occurrences[11].node + end + + test 'create chaosradio event with custom rrule and interval' do + assert_not_nil event = Event.create!( + :start_time => "2009-01-28T21:00:00".to_time, + :end_time => "2009-01-28T23:00:00".to_time, + :url => "http://chaosradio.ccc.de", + :latitude => 52.525308, + :longitude => 13.378944, + :rrule => "FREQ=MONTHLY;INTERVAL=1;BYDAY=-1WE", + :allday => false, + :custom_rrule => true, + :node_id => @cal_node.id + ) + + assert_not_nil scoped_occurrences = event.occurrences_in_range( + "2009-01-01".to_time, "2009-12-31".to_time + ) + + assert_equal 12, scoped_occurrences.length + + expected_days = [28, 25, 25, 29, 27, 24, 29, 26, 30, 28, 25, 30] + chaosradio_days = scoped_occurrences.map {|x| x.start_time.day} + assert_equal expected_days, chaosradio_days + end +end \ No newline at end of file diff --git a/test/models/helpers/admin_helper_test.rb b/test/models/helpers/admin_helper_test.rb new file mode 100644 index 0000000..23d9f40 --- /dev/null +++ b/test/models/helpers/admin_helper_test.rb @@ -0,0 +1,4 @@ +require 'test_helper' + +class AdminHelperTest < ActionView::TestCase +end diff --git a/test/models/helpers/assets_helper_test.rb b/test/models/helpers/assets_helper_test.rb new file mode 100644 index 0000000..ae50bff --- /dev/null +++ b/test/models/helpers/assets_helper_test.rb @@ -0,0 +1,4 @@ +require 'test_helper' + +class AssetsHelperTest < ActionView::TestCase +end diff --git a/test/models/helpers/content_helper_test.rb b/test/models/helpers/content_helper_test.rb new file mode 100644 index 0000000..2da82d7 --- /dev/null +++ b/test/models/helpers/content_helper_test.rb @@ -0,0 +1,4 @@ +require 'test_helper' + +class ContentHelperTest < ActionView::TestCase +end diff --git a/test/models/helpers/events_helper_test.rb b/test/models/helpers/events_helper_test.rb new file mode 100644 index 0000000..2e7567e --- /dev/null +++ b/test/models/helpers/events_helper_test.rb @@ -0,0 +1,4 @@ +require 'test_helper' + +class EventsHelperTest < ActionView::TestCase +end diff --git a/test/models/helpers/menu_items_helper_test.rb b/test/models/helpers/menu_items_helper_test.rb new file mode 100644 index 0000000..8919dfd --- /dev/null +++ b/test/models/helpers/menu_items_helper_test.rb @@ -0,0 +1,4 @@ +require 'test_helper' + +class MenuItemsHelperTest < ActionView::TestCase +end diff --git a/test/models/helpers/nodes_helper_test.rb b/test/models/helpers/nodes_helper_test.rb new file mode 100644 index 0000000..13011de --- /dev/null +++ b/test/models/helpers/nodes_helper_test.rb @@ -0,0 +1,4 @@ +require 'test_helper' + +class NodesHelperTest < ActionView::TestCase +end diff --git a/test/models/helpers/occurrences_helper_test.rb b/test/models/helpers/occurrences_helper_test.rb new file mode 100644 index 0000000..0692926 --- /dev/null +++ b/test/models/helpers/occurrences_helper_test.rb @@ -0,0 +1,4 @@ +require 'test_helper' + +class OccurrencesHelperTest < ActionView::TestCase +end diff --git a/test/models/helpers/pages_helper_test.rb b/test/models/helpers/pages_helper_test.rb new file mode 100644 index 0000000..535dfe1 --- /dev/null +++ b/test/models/helpers/pages_helper_test.rb @@ -0,0 +1,4 @@ +require 'test_helper' + +class PagesHelperTest < ActionView::TestCase +end diff --git a/test/models/helpers/revisions_helper_test.rb b/test/models/helpers/revisions_helper_test.rb new file mode 100644 index 0000000..6bb5f07 --- /dev/null +++ b/test/models/helpers/revisions_helper_test.rb @@ -0,0 +1,4 @@ +require 'test_helper' + +class RevisionsHelperTest < ActionView::TestCase +end diff --git a/test/models/helpers/rss_helper_test.rb b/test/models/helpers/rss_helper_test.rb new file mode 100644 index 0000000..b040b3e --- /dev/null +++ b/test/models/helpers/rss_helper_test.rb @@ -0,0 +1,4 @@ +require 'test_helper' + +class RssHelperTest < ActionView::TestCase +end diff --git a/test/models/helpers/search_helper_test.rb b/test/models/helpers/search_helper_test.rb new file mode 100644 index 0000000..3034163 --- /dev/null +++ b/test/models/helpers/search_helper_test.rb @@ -0,0 +1,4 @@ +require 'test_helper' + +class SearchHelperTest < ActionView::TestCase +end diff --git a/test/models/helpers/tags_helper_test.rb b/test/models/helpers/tags_helper_test.rb new file mode 100644 index 0000000..4b1935f --- /dev/null +++ b/test/models/helpers/tags_helper_test.rb @@ -0,0 +1,4 @@ +require 'test_helper' + +class TagsHelperTest < ActionView::TestCase +end diff --git a/test/models/helpers/users_helper_test.rb b/test/models/helpers/users_helper_test.rb new file mode 100644 index 0000000..96af37a --- /dev/null +++ b/test/models/helpers/users_helper_test.rb @@ -0,0 +1,4 @@ +require 'test_helper' + +class UsersHelperTest < ActionView::TestCase +end diff --git a/test/models/menu_item_test.rb b/test/models/menu_item_test.rb new file mode 100644 index 0000000..81d9c01 --- /dev/null +++ b/test/models/menu_item_test.rb @@ -0,0 +1,8 @@ +require 'test_helper' + +class MenuItemTest < ActiveSupport::TestCase + # Replace this with your real tests. + test "the truth" do + assert true + end +end diff --git a/test/models/node_test.rb b/test/models/node_test.rb new file mode 100644 index 0000000..514ba3f --- /dev/null +++ b/test/models/node_test.rb @@ -0,0 +1,290 @@ +require 'test_helper' + +class NodeTest < ActiveSupport::TestCase + + def setup + @root = Node.find(1) + @first_child = Node.find(2) + @first_child.pages.create! :title => "one" + @first_child.draft = @first_child.pages.last + @first_child.save + @second_child = Node.find(3) + @second_child.pages.create! :title => "one" + + @user1 = User.create :login => 'demo', :email => "f@b.com", :password => 'foobar', :password_confirmation => 'foobar' + @user2 = User.create :login => 'show', :email => "f@b.com", :password => 'foobar', :password_confirmation => 'foobar' + end + + test "can only create one root node" do + Node.delete_all + Node.create! :slug => :root + assert_raise(ActiveRecord::RecordInvalid) do + Node.create! :slug => :root + end + end + + def test_returning_existing_drafts + test_node = Node.root.children.create! :slug => "test_node" + + assert_not_nil test_node.draft + assert_equal 1, test_node.pages.length + assert_nil test_node.draft.user + + 3.times do + test_node.find_or_create_draft @user1 + end + + assert_equal 1, test_node.pages.length + end + + def test_user_gets_assigned_to_unlocked_draft + assert_not_nil @first_child.draft + assert_nil @first_child.draft.user + @first_child.find_or_create_draft @user1 + assert_equal @user1, @first_child.lock_owner + end + + def test_unique_path_returns_an_array + assert_equal ["first_child"], @first_child.unique_path + new_node = @first_child.children.create! :slug => "third_child" + assert_equal ["first_child", "third_child"], new_node.unique_path + end + + def test_specifying_a_revision_other_than_with_a_fixnum_raises_exception + assert_raise(ArgumentError) { Node.find_page "first_child", 1.9 } + assert_raise(ArgumentError) { Node.find_page "first_child", "1" } + assert_raise(ArgumentError) { Node.find_page "first_child", :head } + end + + def test_publish_draft_on_a_node_without_a_draft_returns_nil + + assert @first_child.publish_draft! + assert_nil @first_child.publish_draft! + end + + def test_cloning_a_head_page_to_a_new_draft_with_translations + assert_not_nil draft = @first_child.draft + I18n.locale = :de + draft.title = "Hallo" + draft.abstract = "Bitte" + draft.body = "Danke" + draft.save + I18n.locale = :en + draft.title = "Hello" + draft.abstract = "Please" + draft.body = "Thanks" + draft.save + + @first_child.publish_draft! + + draft1 = @first_child.find_or_create_draft(@user1) + + I18n.locale = :de + assert_equal "Hallo", draft1.title + assert_equal "Bitte", draft1.abstract + assert_equal "Danke", draft1.body + + I18n.locale = :en + assert_equal "Hello", draft1.title + assert_equal "Please", draft1.abstract + assert_equal "Thanks", draft1.body + end + + def test_created_nodes_have_an_empty_draft_and_no_head + node = Node.root.children.create! :slug => "third_child_beta" + + assert !node.pages.empty? + assert_equal 1, node.pages.length + assert_not_nil node.draft + assert_nil node.draft.user + assert_nil node.head + end + + def test_create_new_draft_of_published_page + node = Node.root.children.create :slug => "xyz" + assert node.publish_draft! + end + + def test_find_or_create_draft_if_no_draft_exists + node = Node.root.children.create :slug => "xyz" + node.publish_draft! + assert_not_nil node.find_or_create_draft( @user1 ) + end + + def test_find_or_create_draft_if_draft_exists_and_is_owned_by_user + node = Node.root.children.create :slug => "xyz" + node.publish_draft! + + node.find_or_create_draft @user1 + node.find_or_create_draft @user1 + end + + def test_exception_if_draft_exists_but_locked_by_another_user + node = Node.root.children.create :slug => "xyz" + node.publish_draft! + node.find_or_create_draft @user1 + assert_equal @user1, node.lock_owner + assert_raise(LockedByAnotherUser) do + node.find_or_create_draft @user2 + end + end + + def test_creation_of_unique_name + node = Node.root.children.create :slug => 'child' + node.reload + assert_equal 'child', node.unique_name + + node = @first_child.children.create :slug => 'deep_child' + node.reload + assert_equal 'first_child/deep_child', node.unique_name + end + + def test_order_of_pages_by_revision + # This test should make sure the order is the same on different db's + # Remember, there is already an empty draft + two = @second_child.pages.create :title => "two" + three = @second_child.pages.create :title => "three" + four = @second_child.pages.create :title => "four" + + @second_child.pages.reload + + assert_equal [1,2,3,4], @second_child.pages.map { |x| x.revision } + end + + def test_behavior_of_acts_as_list + two = @second_child.pages.create :title => "two" + three = @second_child.pages.create :title => "three" + four = @second_child.pages.create :title => "four" + + assert_equal 2, two.revision + assert_equal 3, three.revision + assert_equal 4, four.revision + + assert_equal four, @second_child.pages.last + + assert two.move_to_bottom + + two.reload; three.reload; four.reload; + + assert_equal 4, two.revision + assert_equal 2, three.revision + assert_equal 3, four.revision + end + + def test_retrieving_page_current + updates = Node.root.children.create(:slug => 'updates') + year = updates.children.create(:slug => '2008') + foo = year.children.create(:slug => 'foo') + + assert_not_nil Node.find_by_unique_name('updates/2008/foo') + + # Note that there is already an initial, blank revision + foo.pages.create :title => "Version 2" + foo.pages.create :title => "Version 3" + foo.pages.create :title => "Version 4" + + foo.head = foo.pages.last + foo.save! + + page = Node.find_page("updates/2008/foo") + assert_equal page, foo.pages.find_by_revision(4) + end + + def test_retrieving_page_by_revision + updates = Node.root.children.create(:slug => 'updates') + year = updates.children.create(:slug => '2008') + foo = year.children.create(:slug => 'foo') + + assert_not_nil Node.find_by_unique_name('updates/2008/foo') + + # Note that there is already an initial, blank revision + foo.pages.create :title => "Version 2" + foo.pages.create :title => "Version 3" + foo.pages.create :title => "Version 4" + + page = Node.find_page("updates/2008/foo", 2) + assert_equal "Version 2", page.title + end + + # Thats a lengthy test to make sure everything works as it should, it was + # created during a bug hunt + def test_creating_new_draft + test_node = Node.root.children.create! :slug => "test_node" + test_node.draft.user = @user1 + test_node.save + assert test_node.publish_draft! + test_node.reload + assert_equal 1, test_node.pages.length + assert_not_nil test_node.head + assert_nil test_node.draft + test_node.find_or_create_draft @user1 + test_node.reload + assert_equal 2, test_node.pages.length + assert_not_nil test_node.draft + assert test_node.head != test_node.draft + end + + test "restoring a revision" do + test_node = Node.root.children.create! :slug => "test_node" + create_revisions( test_node, 3 ) + test_node.find_or_create_draft @user1 + test_node.reload + + assert_equal 4, test_node.pages.count + assert_equal 3, test_node.head.revision + + test_node.restore_revision!(1) + assert_equal 1, test_node.head.revision + assert_equal 4, test_node.draft.revision + end + + test "a new revision keeps the initial user" do + Node.root.descendants.destroy_all + node = create_node_with_draft + draft = node.draft + draft.user = users(:aaron) + draft.save + node.publish_draft! + new_draft = node.find_or_create_draft( users(:quentin) ) + assert_equal "aaron", new_draft.user.login + end + + test "a new revision can overwrite the initial author" do + Node.root.descendants.destroy_all + node = create_node_with_draft + draft = node.draft + draft.user = users(:aaron) + draft.save! + node.publish_draft! + new_draft = node.find_or_create_draft( users(:quentin) ) + new_draft.user_id = users(:quentin).id + new_draft.save + node.publish_draft! + assert_equal "quentin", node.head.user.login + end + + + test "update?" do + Node.root.descendants.delete_all + updates = Node.root.children.create!( :slug => "updates" ) + assert !updates.update? + + updates2009 = updates.children.create!( :slug => "2009" ) + assert !updates2009.update? + + update = updates2009.children.create!( :slug => "my-first-update" ) + assert update.update? + end + + test "new nodes should have drafts with no publidhed_at set" do + node = Node.root.children.create( :slug => "wow" ) + assert_nil node.draft.published_at + end + + def create_revisions node, count + count.times do + node.find_or_create_draft @user1 + node.publish_draft! + end + end +end diff --git a/test/models/occurrence_test.rb b/test/models/occurrence_test.rb new file mode 100644 index 0000000..91a78ec --- /dev/null +++ b/test/models/occurrence_test.rb @@ -0,0 +1,8 @@ +require 'test_helper' + +class OccurrenceTest < ActiveSupport::TestCase + # Replace this with your real tests. + test "the truth" do + assert true + end +end diff --git a/test/models/page_test.rb b/test/models/page_test.rb new file mode 100644 index 0000000..8b34399 --- /dev/null +++ b/test/models/page_test.rb @@ -0,0 +1,146 @@ +require 'test_helper' + +class PageTest < ActiveSupport::TestCase + + def setup + @user1 = User.create :login => 'demo', :email => "f@b.com", :password => 'foobar', :password_confirmation => 'foobar' + @user2 = User.create :login => 'show', :email => "f@b.com", :password => 'foobar', :password_confirmation => 'foobar' + end + + def test_aggregation + # Create two nodes and move them beneath the root node + n1 = Node.root.children.create! :slug => "one" + n2 = Node.root.children.create! :slug => "two" + + # get the drafts and assign a user to it + assert_not_nil d1 = n1.find_or_create_draft( @user1 ) + assert_not_nil d3 = n2.find_or_create_draft( @user1 ) + + # tag and double publish so we have 4 pages tagged with "update" + d1.tag_list = "update" + d1.save + n1.publish_draft! + + d2 = n1.find_or_create_draft @user1 + n1.publish_draft! + + + d3.tag_list = "update, pressemitteilung" + d3.save + n2.publish_draft! + + d4 = n2.find_or_create_draft @user1 + n2.publish_draft! + + # Set up two options hashes for the assertions + options1 = { + :tags => "update" + } + + options2 = { + :tags => "update, pressemitteilung" + } + + assert_equal 2, Page.aggregate( options1 ).length + assert_equal 1, Page.aggregate( options2 ).length + assert_equal 4, Page.tagged_with( "update" ).length + assert_equal [d2.id, d4.id], Page.aggregate( options1 ).map {|x| x.id} + end + + def test_before_save_rewrite_links_in_body + n = Node.root.children.create :slug => "link_test" + d = n.find_or_create_draft @user1 + + before = "

Hello World

\n" \ + "Linkme" + + after = "

Hello World

\n" \ + "Linkme" + + I18n.locale = :de + + d.body = before + d.save! + + assert_equal after, d.body + end + + def test_before_save_rewrite_links_in_body_if_no_locale_prefix_present + n = Node.root.children.create :slug => "link_test" + d = n.find_or_create_draft @user1 + + before = "

Hello World

\n" \ + "Linkme" + + after = "

Hello World

\n" \ + "Linkme" + + I18n.locale = :de + + d.body = before + d.save + + assert_equal after, d.body + end + + def test_before_save_rewrite_links_skips_on_external_links + n = Node.root.children.create :slug => "link_test" + d = n.find_or_create_draft @user1 + + before = "

Hello World

\n" \ + "Linkme" + + after = "

Hello World

\n" \ + "Linkme" + + I18n.locale = :de + + d.body = before + d.save + + assert_equal after, d.body + end + + def test_find_with_outdated_translations + Node.delete_all + Page.delete_all + I18n.locale = :de + + assert_not_nil page = Page.create!( :title => "Hallo" ) + page.reload + assert_equal 1, page.translations.size + assert_equal [], Page.find_with_outdated_translations + + I18n.locale = :en + page.title = "Hello" + page.save + + assert_equal 2, page.translations.size + assert_equal 0, Page.find_with_outdated_translations.size + + english = page.translations.select {|x| x.locale == :en}.first + Page::Translation.record_timestamps = false + english.update_attributes(:updated_at => (Time.now+25.hours)) + Page::Translation.record_timestamps = true + assert_equal 1, Page.find_with_outdated_translations.count + + I18n.locale = :de + page2 = Page.create!( :title => "Hallo2" ) + I18n.locale = :en + page2.title = "Hello2" + page2.save! + + assert_equal 0, Page.find_with_outdated_translations(:delta_time => 23.days).count + assert_equal 1, Page.find_with_outdated_translations(:delta_time => 23.minutes).count + assert_equal 2, Page.count + end + + test "pages under /updates node get the update template assigned" do + Node.root.descendants.delete_all + updates = Node.root.children.create!( :slug => "updates" ) + updates2009 = updates.children.create!( :slug => "2009" ) + update = updates2009.children.create!( :slug => "my-first-update" ) + assert_equal "update", update.draft.template_name + end + +end diff --git a/test/models/permission_test.rb b/test/models/permission_test.rb new file mode 100644 index 0000000..08fcc0b --- /dev/null +++ b/test/models/permission_test.rb @@ -0,0 +1,8 @@ +require 'test_helper' + +class PermissionTest < ActiveSupport::TestCase + # Replace this with your real tests. + test "the truth" do + assert true + end +end diff --git a/test/models/related_asset_test.rb b/test/models/related_asset_test.rb new file mode 100644 index 0000000..a739e6b --- /dev/null +++ b/test/models/related_asset_test.rb @@ -0,0 +1,8 @@ +require 'test_helper' + +class RelatedImageTest < ActiveSupport::TestCase + # Replace this with your real tests. + test "the truth" do + assert true + end +end diff --git a/test/models/user_test.rb b/test/models/user_test.rb new file mode 100644 index 0000000..bd5d059 --- /dev/null +++ b/test/models/user_test.rb @@ -0,0 +1,64 @@ +require File.dirname(__FILE__) + '/../test_helper' + +class UserTest < ActiveSupport::TestCase + # Be sure to include AuthenticatedTestHelper in test/test_helper.rb instead. + # Then, you can remove it from this and the functional test. + include AuthenticatedTestHelper + fixtures :users + + def test_should_create_user + assert_difference 'User.count' do + user = create_user + assert !user.new_record?, "#{user.errors.full_messages.to_sentence}" + end + end + + def test_should_require_login + assert_no_difference 'User.count' do + u = create_user(:login => nil) + assert u.errors[:login].any? + end + end + + def test_should_require_password + assert_no_difference 'User.count' do + u = create_user(:password => nil) + assert u.errors[:password].any? + end + end + + def test_should_require_password_confirmation + assert_no_difference 'User.count' do + u = create_user(:password_confirmation => nil) + assert u.errors[:password_confirmation].any? + end + end + + def test_should_require_email + assert_no_difference 'User.count' do + u = create_user(:email => nil) + assert u.errors[:email].any? + end + end + + def test_should_reset_password + users(:quentin).update_attributes(:password => 'new password', :password_confirmation => 'new password') + assert_equal users(:quentin), User.authenticate('quentin', 'new password') + end + + def test_should_not_rehash_password + users(:quentin).update_attributes(:login => 'quentin2') + assert_equal users(:quentin), User.authenticate('quentin2', 'monkey') + end + + def test_should_authenticate_user + assert_equal users(:quentin), User.authenticate('quentin', 'monkey') + end + +protected + def create_user(options = {}) + record = User.new({ :login => 'quire', :email => 'quire@example.com', :password => 'quire69', :password_confirmation => 'quire69' }.merge(options)) + record.save + record + end +end diff --git a/test/performance/browsing_test.rb b/test/performance/browsing_test.rb deleted file mode 100644 index 4b60558..0000000 --- a/test/performance/browsing_test.rb +++ /dev/null @@ -1,9 +0,0 @@ -require 'test_helper' -require 'performance_test_help' - -# Profiling results for each test method are written to tmp/performance. -class BrowsingTest < ActionController::PerformanceTest - def test_homepage - get '/' - end -end diff --git a/test/test_helper.rb b/test/test_helper.rb index 549f594..2514926 100644 --- a/test/test_helper.rb +++ b/test/test_helper.rb @@ -104,14 +104,7 @@ class ActiveSupport::TestCase # The only drawback to using transactional fixtures is when you actually # need to test transactions. Since your test is bracketed by a transaction, # any transactions started in your code will be automatically rolled back. - self.use_transactional_fixtures = true - - # Instantiated fixtures are slow, but give you @david where otherwise you - # would need people(:david). If you don't want to migrate your existing - # test cases which use the @david style and don't mind the speed hit (each - # instantiated fixtures translates to a database query per test method), - # then set this back to true. - self.use_instantiated_fixtures = false + self.use_transactional_tests = true # Setup all fixtures in test/fixtures/*.(yml|csv) for all tests in alphabetical order. # @@ -120,7 +113,7 @@ class ActiveSupport::TestCase fixtures :all # Add more helper methods to be used by all tests here... - + def create_node_with_published_page node = create_node_with_draft draft = node.draft diff --git a/test/unit/asset_test.rb b/test/unit/asset_test.rb deleted file mode 100644 index a1041e4..0000000 --- a/test/unit/asset_test.rb +++ /dev/null @@ -1,23 +0,0 @@ -require 'test_helper' - -class AssetTest < ActiveSupport::TestCase - - test "related assets get destroyed when assets get destroyed" do - Asset.delete_all - RelatedAsset.delete_all - - assert asset = Asset.create - assert node = Node.root.children.create( :slug => "asset" ) - assert_equal [], node.draft.assets - - draft = node.draft - draft.assets << asset - assert_equal 1, draft.assets.length - - asset.destroy - draft.reload - assert_equal 0, draft.assets.length - assert_equal 0, RelatedAsset.count - end - -end diff --git a/test/unit/event_test.rb b/test/unit/event_test.rb deleted file mode 100644 index f310af8..0000000 --- a/test/unit/event_test.rb +++ /dev/null @@ -1,114 +0,0 @@ -require 'test_helper' - -class EventTest < ActiveSupport::TestCase - - def setup - Page.delete_all - @cal_node = Node.root.children.create! :slug => "calendar" - @draft = @cal_node.find_or_create_draft User.first - @draft.title = "99C3" - @draft.abstract = "The 99th Chaos Comunication Congress" - @draft.body = "Its totally freakin awesome" - @draft.save - @cal_node.publish_draft! - @cal_node.head.reload - end - - test 'verfy setup data' do - assert_not_nil @cal_node - assert_not_nil @cal_node.head - end - - test 'creating an event with malformed rrule raises exception' do - assert_raise(ArgumentError) do - Event.create!( - :start_time => "2009-01-01T15:23:42".to_time, - :end_time => "2009-01-01T20:05:23".to_time, - :url => "http://events.ccc.de/congress/2082", - :latitude => 52.525308, - :longitude => 13.378944, - :rrule => "FOOBAR", - :allday => false, - :custom_rrule => false, - :node_id => @cal_node.id - ) - end - end - - test 'create day event for node with one occurrence' do - assert_not_nil event = Event.create!( - :start_time => "2009-01-01T15:23:42".to_time, - :end_time => "2009-01-01T20:05:23".to_time, - :url => "http://events.ccc.de/congress/2082", - :latitude => 52.525308, - :longitude => 13.378944, - :rrule => nil, - :allday => false, - :custom_rrule => false, - :node_id => @cal_node.id - ) - - assert_equal 1, Occurrence.count - assert_equal event.start_time, Occurrence.first.start_time - assert_equal event.end_time, Occurrence.first.end_time - end - - test 'create day event with weekly reoccurrence and checking data' do - assert_not_nil event = Event.create!( - :start_time => "2009-01-01T15:23:42".to_time, - :end_time => "2009-01-01T20:05:23".to_time, - :url => "http://events.ccc.de/congress/2082", - :latitude => 52.525308, - :longitude => 13.378944, - :rrule => "FREQ=WEEKLY;INTERVAL=1", - :allday => false, - :custom_rrule => false, - :node_id => @cal_node.id - ) - - assert_not_nil scoped_occurrences = event.occurrences_in_range( - "2009-01-01".to_time, "2009-12-31".to_time - ) - - assert_equal 52, scoped_occurrences.length - - assert_equal "2009-12-24T15:23:42".to_time, scoped_occurrences[51].start_time - assert_equal "2009-12-24T20:05:23".to_time, scoped_occurrences[51].end_time - assert_equal @cal_node.event, scoped_occurrences[51].event - assert_equal @cal_node, scoped_occurrences[51].node - - assert_equal "2009-03-19T15:23:42".to_time, scoped_occurrences[11].start_time - assert_equal "2009-03-19T20:05:23".to_time, scoped_occurrences[11].end_time - assert_equal @cal_node.event, scoped_occurrences[11].event - assert_equal @cal_node, scoped_occurrences[11].node - - assert_equal "2009-01-01T15:23:42".to_time, scoped_occurrences[0].start_time - assert_equal "2009-01-01T20:05:23".to_time, scoped_occurrences[0].end_time - assert_equal @cal_node.event, scoped_occurrences[11].event - assert_equal @cal_node, scoped_occurrences[11].node - end - - test 'create chaosradio event with custom rrule and interval' do - assert_not_nil event = Event.create!( - :start_time => "2009-01-28T21:00:00".to_time, - :end_time => "2009-01-28T23:00:00".to_time, - :url => "http://chaosradio.ccc.de", - :latitude => 52.525308, - :longitude => 13.378944, - :rrule => "FREQ=MONTHLY;INTERVAL=1;BYDAY=-1WE", - :allday => false, - :custom_rrule => true, - :node_id => @cal_node.id - ) - - assert_not_nil scoped_occurrences = event.occurrences_in_range( - "2009-01-01".to_time, "2009-12-31".to_time - ) - - assert_equal 12, scoped_occurrences.length - - expected_days = [28, 25, 25, 29, 27, 24, 29, 26, 30, 28, 25, 30] - chaosradio_days = scoped_occurrences.map {|x| x.start_time.day} - assert_equal expected_days, chaosradio_days - end -end \ No newline at end of file diff --git a/test/unit/helpers/admin_helper_test.rb b/test/unit/helpers/admin_helper_test.rb deleted file mode 100644 index 23d9f40..0000000 --- a/test/unit/helpers/admin_helper_test.rb +++ /dev/null @@ -1,4 +0,0 @@ -require 'test_helper' - -class AdminHelperTest < ActionView::TestCase -end diff --git a/test/unit/helpers/assets_helper_test.rb b/test/unit/helpers/assets_helper_test.rb deleted file mode 100644 index ae50bff..0000000 --- a/test/unit/helpers/assets_helper_test.rb +++ /dev/null @@ -1,4 +0,0 @@ -require 'test_helper' - -class AssetsHelperTest < ActionView::TestCase -end diff --git a/test/unit/helpers/content_helper_test.rb b/test/unit/helpers/content_helper_test.rb deleted file mode 100644 index 2da82d7..0000000 --- a/test/unit/helpers/content_helper_test.rb +++ /dev/null @@ -1,4 +0,0 @@ -require 'test_helper' - -class ContentHelperTest < ActionView::TestCase -end diff --git a/test/unit/helpers/events_helper_test.rb b/test/unit/helpers/events_helper_test.rb deleted file mode 100644 index 2e7567e..0000000 --- a/test/unit/helpers/events_helper_test.rb +++ /dev/null @@ -1,4 +0,0 @@ -require 'test_helper' - -class EventsHelperTest < ActionView::TestCase -end diff --git a/test/unit/helpers/menu_items_helper_test.rb b/test/unit/helpers/menu_items_helper_test.rb deleted file mode 100644 index 8919dfd..0000000 --- a/test/unit/helpers/menu_items_helper_test.rb +++ /dev/null @@ -1,4 +0,0 @@ -require 'test_helper' - -class MenuItemsHelperTest < ActionView::TestCase -end diff --git a/test/unit/helpers/nodes_helper_test.rb b/test/unit/helpers/nodes_helper_test.rb deleted file mode 100644 index 13011de..0000000 --- a/test/unit/helpers/nodes_helper_test.rb +++ /dev/null @@ -1,4 +0,0 @@ -require 'test_helper' - -class NodesHelperTest < ActionView::TestCase -end diff --git a/test/unit/helpers/occurrences_helper_test.rb b/test/unit/helpers/occurrences_helper_test.rb deleted file mode 100644 index 0692926..0000000 --- a/test/unit/helpers/occurrences_helper_test.rb +++ /dev/null @@ -1,4 +0,0 @@ -require 'test_helper' - -class OccurrencesHelperTest < ActionView::TestCase -end diff --git a/test/unit/helpers/pages_helper_test.rb b/test/unit/helpers/pages_helper_test.rb deleted file mode 100644 index 535dfe1..0000000 --- a/test/unit/helpers/pages_helper_test.rb +++ /dev/null @@ -1,4 +0,0 @@ -require 'test_helper' - -class PagesHelperTest < ActionView::TestCase -end diff --git a/test/unit/helpers/revisions_helper_test.rb b/test/unit/helpers/revisions_helper_test.rb deleted file mode 100644 index 6bb5f07..0000000 --- a/test/unit/helpers/revisions_helper_test.rb +++ /dev/null @@ -1,4 +0,0 @@ -require 'test_helper' - -class RevisionsHelperTest < ActionView::TestCase -end diff --git a/test/unit/helpers/rss_helper_test.rb b/test/unit/helpers/rss_helper_test.rb deleted file mode 100644 index b040b3e..0000000 --- a/test/unit/helpers/rss_helper_test.rb +++ /dev/null @@ -1,4 +0,0 @@ -require 'test_helper' - -class RssHelperTest < ActionView::TestCase -end diff --git a/test/unit/helpers/search_helper_test.rb b/test/unit/helpers/search_helper_test.rb deleted file mode 100644 index 3034163..0000000 --- a/test/unit/helpers/search_helper_test.rb +++ /dev/null @@ -1,4 +0,0 @@ -require 'test_helper' - -class SearchHelperTest < ActionView::TestCase -end diff --git a/test/unit/helpers/tags_helper_test.rb b/test/unit/helpers/tags_helper_test.rb deleted file mode 100644 index 4b1935f..0000000 --- a/test/unit/helpers/tags_helper_test.rb +++ /dev/null @@ -1,4 +0,0 @@ -require 'test_helper' - -class TagsHelperTest < ActionView::TestCase -end diff --git a/test/unit/helpers/users_helper_test.rb b/test/unit/helpers/users_helper_test.rb deleted file mode 100644 index 96af37a..0000000 --- a/test/unit/helpers/users_helper_test.rb +++ /dev/null @@ -1,4 +0,0 @@ -require 'test_helper' - -class UsersHelperTest < ActionView::TestCase -end diff --git a/test/unit/menu_item_test.rb b/test/unit/menu_item_test.rb deleted file mode 100644 index 81d9c01..0000000 --- a/test/unit/menu_item_test.rb +++ /dev/null @@ -1,8 +0,0 @@ -require 'test_helper' - -class MenuItemTest < ActiveSupport::TestCase - # Replace this with your real tests. - test "the truth" do - assert true - end -end diff --git a/test/unit/node_test.rb b/test/unit/node_test.rb deleted file mode 100644 index 514ba3f..0000000 --- a/test/unit/node_test.rb +++ /dev/null @@ -1,290 +0,0 @@ -require 'test_helper' - -class NodeTest < ActiveSupport::TestCase - - def setup - @root = Node.find(1) - @first_child = Node.find(2) - @first_child.pages.create! :title => "one" - @first_child.draft = @first_child.pages.last - @first_child.save - @second_child = Node.find(3) - @second_child.pages.create! :title => "one" - - @user1 = User.create :login => 'demo', :email => "f@b.com", :password => 'foobar', :password_confirmation => 'foobar' - @user2 = User.create :login => 'show', :email => "f@b.com", :password => 'foobar', :password_confirmation => 'foobar' - end - - test "can only create one root node" do - Node.delete_all - Node.create! :slug => :root - assert_raise(ActiveRecord::RecordInvalid) do - Node.create! :slug => :root - end - end - - def test_returning_existing_drafts - test_node = Node.root.children.create! :slug => "test_node" - - assert_not_nil test_node.draft - assert_equal 1, test_node.pages.length - assert_nil test_node.draft.user - - 3.times do - test_node.find_or_create_draft @user1 - end - - assert_equal 1, test_node.pages.length - end - - def test_user_gets_assigned_to_unlocked_draft - assert_not_nil @first_child.draft - assert_nil @first_child.draft.user - @first_child.find_or_create_draft @user1 - assert_equal @user1, @first_child.lock_owner - end - - def test_unique_path_returns_an_array - assert_equal ["first_child"], @first_child.unique_path - new_node = @first_child.children.create! :slug => "third_child" - assert_equal ["first_child", "third_child"], new_node.unique_path - end - - def test_specifying_a_revision_other_than_with_a_fixnum_raises_exception - assert_raise(ArgumentError) { Node.find_page "first_child", 1.9 } - assert_raise(ArgumentError) { Node.find_page "first_child", "1" } - assert_raise(ArgumentError) { Node.find_page "first_child", :head } - end - - def test_publish_draft_on_a_node_without_a_draft_returns_nil - - assert @first_child.publish_draft! - assert_nil @first_child.publish_draft! - end - - def test_cloning_a_head_page_to_a_new_draft_with_translations - assert_not_nil draft = @first_child.draft - I18n.locale = :de - draft.title = "Hallo" - draft.abstract = "Bitte" - draft.body = "Danke" - draft.save - I18n.locale = :en - draft.title = "Hello" - draft.abstract = "Please" - draft.body = "Thanks" - draft.save - - @first_child.publish_draft! - - draft1 = @first_child.find_or_create_draft(@user1) - - I18n.locale = :de - assert_equal "Hallo", draft1.title - assert_equal "Bitte", draft1.abstract - assert_equal "Danke", draft1.body - - I18n.locale = :en - assert_equal "Hello", draft1.title - assert_equal "Please", draft1.abstract - assert_equal "Thanks", draft1.body - end - - def test_created_nodes_have_an_empty_draft_and_no_head - node = Node.root.children.create! :slug => "third_child_beta" - - assert !node.pages.empty? - assert_equal 1, node.pages.length - assert_not_nil node.draft - assert_nil node.draft.user - assert_nil node.head - end - - def test_create_new_draft_of_published_page - node = Node.root.children.create :slug => "xyz" - assert node.publish_draft! - end - - def test_find_or_create_draft_if_no_draft_exists - node = Node.root.children.create :slug => "xyz" - node.publish_draft! - assert_not_nil node.find_or_create_draft( @user1 ) - end - - def test_find_or_create_draft_if_draft_exists_and_is_owned_by_user - node = Node.root.children.create :slug => "xyz" - node.publish_draft! - - node.find_or_create_draft @user1 - node.find_or_create_draft @user1 - end - - def test_exception_if_draft_exists_but_locked_by_another_user - node = Node.root.children.create :slug => "xyz" - node.publish_draft! - node.find_or_create_draft @user1 - assert_equal @user1, node.lock_owner - assert_raise(LockedByAnotherUser) do - node.find_or_create_draft @user2 - end - end - - def test_creation_of_unique_name - node = Node.root.children.create :slug => 'child' - node.reload - assert_equal 'child', node.unique_name - - node = @first_child.children.create :slug => 'deep_child' - node.reload - assert_equal 'first_child/deep_child', node.unique_name - end - - def test_order_of_pages_by_revision - # This test should make sure the order is the same on different db's - # Remember, there is already an empty draft - two = @second_child.pages.create :title => "two" - three = @second_child.pages.create :title => "three" - four = @second_child.pages.create :title => "four" - - @second_child.pages.reload - - assert_equal [1,2,3,4], @second_child.pages.map { |x| x.revision } - end - - def test_behavior_of_acts_as_list - two = @second_child.pages.create :title => "two" - three = @second_child.pages.create :title => "three" - four = @second_child.pages.create :title => "four" - - assert_equal 2, two.revision - assert_equal 3, three.revision - assert_equal 4, four.revision - - assert_equal four, @second_child.pages.last - - assert two.move_to_bottom - - two.reload; three.reload; four.reload; - - assert_equal 4, two.revision - assert_equal 2, three.revision - assert_equal 3, four.revision - end - - def test_retrieving_page_current - updates = Node.root.children.create(:slug => 'updates') - year = updates.children.create(:slug => '2008') - foo = year.children.create(:slug => 'foo') - - assert_not_nil Node.find_by_unique_name('updates/2008/foo') - - # Note that there is already an initial, blank revision - foo.pages.create :title => "Version 2" - foo.pages.create :title => "Version 3" - foo.pages.create :title => "Version 4" - - foo.head = foo.pages.last - foo.save! - - page = Node.find_page("updates/2008/foo") - assert_equal page, foo.pages.find_by_revision(4) - end - - def test_retrieving_page_by_revision - updates = Node.root.children.create(:slug => 'updates') - year = updates.children.create(:slug => '2008') - foo = year.children.create(:slug => 'foo') - - assert_not_nil Node.find_by_unique_name('updates/2008/foo') - - # Note that there is already an initial, blank revision - foo.pages.create :title => "Version 2" - foo.pages.create :title => "Version 3" - foo.pages.create :title => "Version 4" - - page = Node.find_page("updates/2008/foo", 2) - assert_equal "Version 2", page.title - end - - # Thats a lengthy test to make sure everything works as it should, it was - # created during a bug hunt - def test_creating_new_draft - test_node = Node.root.children.create! :slug => "test_node" - test_node.draft.user = @user1 - test_node.save - assert test_node.publish_draft! - test_node.reload - assert_equal 1, test_node.pages.length - assert_not_nil test_node.head - assert_nil test_node.draft - test_node.find_or_create_draft @user1 - test_node.reload - assert_equal 2, test_node.pages.length - assert_not_nil test_node.draft - assert test_node.head != test_node.draft - end - - test "restoring a revision" do - test_node = Node.root.children.create! :slug => "test_node" - create_revisions( test_node, 3 ) - test_node.find_or_create_draft @user1 - test_node.reload - - assert_equal 4, test_node.pages.count - assert_equal 3, test_node.head.revision - - test_node.restore_revision!(1) - assert_equal 1, test_node.head.revision - assert_equal 4, test_node.draft.revision - end - - test "a new revision keeps the initial user" do - Node.root.descendants.destroy_all - node = create_node_with_draft - draft = node.draft - draft.user = users(:aaron) - draft.save - node.publish_draft! - new_draft = node.find_or_create_draft( users(:quentin) ) - assert_equal "aaron", new_draft.user.login - end - - test "a new revision can overwrite the initial author" do - Node.root.descendants.destroy_all - node = create_node_with_draft - draft = node.draft - draft.user = users(:aaron) - draft.save! - node.publish_draft! - new_draft = node.find_or_create_draft( users(:quentin) ) - new_draft.user_id = users(:quentin).id - new_draft.save - node.publish_draft! - assert_equal "quentin", node.head.user.login - end - - - test "update?" do - Node.root.descendants.delete_all - updates = Node.root.children.create!( :slug => "updates" ) - assert !updates.update? - - updates2009 = updates.children.create!( :slug => "2009" ) - assert !updates2009.update? - - update = updates2009.children.create!( :slug => "my-first-update" ) - assert update.update? - end - - test "new nodes should have drafts with no publidhed_at set" do - node = Node.root.children.create( :slug => "wow" ) - assert_nil node.draft.published_at - end - - def create_revisions node, count - count.times do - node.find_or_create_draft @user1 - node.publish_draft! - end - end -end diff --git a/test/unit/occurrence_test.rb b/test/unit/occurrence_test.rb deleted file mode 100644 index 91a78ec..0000000 --- a/test/unit/occurrence_test.rb +++ /dev/null @@ -1,8 +0,0 @@ -require 'test_helper' - -class OccurrenceTest < ActiveSupport::TestCase - # Replace this with your real tests. - test "the truth" do - assert true - end -end diff --git a/test/unit/page_test.rb b/test/unit/page_test.rb deleted file mode 100644 index 401d777..0000000 --- a/test/unit/page_test.rb +++ /dev/null @@ -1,146 +0,0 @@ -require 'test_helper' - -class PageTest < ActiveSupport::TestCase - - def setup - @user1 = User.create :login => 'demo', :email => "f@b.com", :password => 'foobar', :password_confirmation => 'foobar' - @user2 = User.create :login => 'show', :email => "f@b.com", :password => 'foobar', :password_confirmation => 'foobar' - end - - def test_aggregation - # Create two nodes and move them beneath the root node - n1 = Node.root.children.create! :slug => "one" - n2 = Node.root.children.create! :slug => "two" - - # get the drafts and assign a user to it - assert_not_nil d1 = n1.find_or_create_draft( @user1 ) - assert_not_nil d3 = n2.find_or_create_draft( @user1 ) - - # tag and double publish so we have 4 pages tagged with "update" - d1.tag_list = "update" - d1.save - n1.publish_draft! - - d2 = n1.find_or_create_draft @user1 - n1.publish_draft! - - - d3.tag_list = "update, pressemitteilung" - d3.save - n2.publish_draft! - - d4 = n2.find_or_create_draft @user1 - n2.publish_draft! - - # Set up two options hashes for the assertions - options1 = { - :tags => "update" - } - - options2 = { - :tags => "update, pressemitteilung" - } - - assert_equal 2, Page.aggregate( options1 ).length - assert_equal 1, Page.aggregate( options2 ).length - assert_equal 4, Page.find_tagged_with( "update" ).length - assert_equal [d2.id, d4.id], Page.aggregate( options1 ).map {|x| x.id} - end - - def test_before_save_rewrite_links_in_body - n = Node.root.children.create :slug => "link_test" - d = n.find_or_create_draft @user1 - - before = "

Hello World

\n" \ - "Linkme" - - after = "

Hello World

\n" \ - "Linkme" - - I18n.locale = :de - - d.body = before - d.save! - - assert_equal after, d.body - end - - def test_before_save_rewrite_links_in_body_if_no_locale_prefix_present - n = Node.root.children.create :slug => "link_test" - d = n.find_or_create_draft @user1 - - before = "

Hello World

\n" \ - "Linkme" - - after = "

Hello World

\n" \ - "Linkme" - - I18n.locale = :de - - d.body = before - d.save - - assert_equal after, d.body - end - - def test_before_save_rewrite_links_skips_on_external_links - n = Node.root.children.create :slug => "link_test" - d = n.find_or_create_draft @user1 - - before = "

Hello World

\n" \ - "Linkme" - - after = "

Hello World

\n" \ - "Linkme" - - I18n.locale = :de - - d.body = before - d.save - - assert_equal after, d.body - end - - def test_find_with_outdated_translations - Node.delete_all - Page.delete_all - I18n.locale = :de - - assert_not_nil page = Page.create!( :title => "Hallo" ) - page.reload - assert_equal 1, page.translations.size - assert_equal [], Page.find_with_outdated_translations - - I18n.locale = :en - page.title = "Hello" - page.save - - assert_equal 2, page.translations.size - assert_equal 0, Page.find_with_outdated_translations.size - - english = page.translations.select {|x| x.locale == :en}.first - Page::Translation.record_timestamps = false - english.update_attributes(:updated_at => (Time.now+25.hours)) - Page::Translation.record_timestamps = true - assert_equal 1, Page.find_with_outdated_translations.count - - I18n.locale = :de - page2 = Page.create!( :title => "Hallo2" ) - I18n.locale = :en - page2.title = "Hello2" - page2.save! - - assert_equal 0, Page.find_with_outdated_translations(:delta_time => 23.days).count - assert_equal 1, Page.find_with_outdated_translations(:delta_time => 23.minutes).count - assert_equal 2, Page.count - end - - test "pages under /updates node get the update template assigned" do - Node.root.descendants.delete_all - updates = Node.root.children.create!( :slug => "updates" ) - updates2009 = updates.children.create!( :slug => "2009" ) - update = updates2009.children.create!( :slug => "my-first-update" ) - assert_equal "update", update.draft.template_name - end - -end diff --git a/test/unit/permission_test.rb b/test/unit/permission_test.rb deleted file mode 100644 index 08fcc0b..0000000 --- a/test/unit/permission_test.rb +++ /dev/null @@ -1,8 +0,0 @@ -require 'test_helper' - -class PermissionTest < ActiveSupport::TestCase - # Replace this with your real tests. - test "the truth" do - assert true - end -end diff --git a/test/unit/related_asset_test.rb b/test/unit/related_asset_test.rb deleted file mode 100644 index a739e6b..0000000 --- a/test/unit/related_asset_test.rb +++ /dev/null @@ -1,8 +0,0 @@ -require 'test_helper' - -class RelatedImageTest < ActiveSupport::TestCase - # Replace this with your real tests. - test "the truth" do - assert true - end -end diff --git a/test/unit/user_test.rb b/test/unit/user_test.rb deleted file mode 100644 index bd5d059..0000000 --- a/test/unit/user_test.rb +++ /dev/null @@ -1,64 +0,0 @@ -require File.dirname(__FILE__) + '/../test_helper' - -class UserTest < ActiveSupport::TestCase - # Be sure to include AuthenticatedTestHelper in test/test_helper.rb instead. - # Then, you can remove it from this and the functional test. - include AuthenticatedTestHelper - fixtures :users - - def test_should_create_user - assert_difference 'User.count' do - user = create_user - assert !user.new_record?, "#{user.errors.full_messages.to_sentence}" - end - end - - def test_should_require_login - assert_no_difference 'User.count' do - u = create_user(:login => nil) - assert u.errors[:login].any? - end - end - - def test_should_require_password - assert_no_difference 'User.count' do - u = create_user(:password => nil) - assert u.errors[:password].any? - end - end - - def test_should_require_password_confirmation - assert_no_difference 'User.count' do - u = create_user(:password_confirmation => nil) - assert u.errors[:password_confirmation].any? - end - end - - def test_should_require_email - assert_no_difference 'User.count' do - u = create_user(:email => nil) - assert u.errors[:email].any? - end - end - - def test_should_reset_password - users(:quentin).update_attributes(:password => 'new password', :password_confirmation => 'new password') - assert_equal users(:quentin), User.authenticate('quentin', 'new password') - end - - def test_should_not_rehash_password - users(:quentin).update_attributes(:login => 'quentin2') - assert_equal users(:quentin), User.authenticate('quentin2', 'monkey') - end - - def test_should_authenticate_user - assert_equal users(:quentin), User.authenticate('quentin', 'monkey') - end - -protected - def create_user(options = {}) - record = User.new({ :login => 'quire', :email => 'quire@example.com', :password => 'quire69', :password_confirmation => 'quire69' }.merge(options)) - record.save - record - end -end -- cgit v1.3