jro_apps/redmine Commit - r13265:1316f6b49199

Removed some test contexts....

Jean-Philippe Lang -

r13265:1316f6b49199

parent child

Context file:

r13265:1316f6b49199

Collapse all files

test/unit/lib/redmine/menu_manager/menu_helper_test.rb 0 -20

                  end
                end
-               context "MenuManager#current_menu_item" do
-                 should "be tested"
-               end
-               context "MenuManager#render_main_menu" do
-                 should "be tested"
-               end
-               context "MenuManager#render_menu" do
-                 should "be tested"
-               end
-               context "MenuManager#menu_item_and_children" do
-                 should "be tested"
-               end
-               context "MenuManager#extract_node_details" do
-                 should "be tested"
-               end
                def test_render_single_menu_node
                  node = Redmine::MenuManager::MenuItem.new(:testing, '/test', { })
                  @output_buffer = render_single_menu_node(node, 'This is a test', node.url, false)

test/unit/lib/redmine/menu_manager/menu_item_test.rb 0 -8

                  menu.push(:child2_menu, '/test', { :parent => :parent})
                end
-               context "MenuItem#caption" do
-                 should "be tested"
-               end
-               context "MenuItem#html_options" do
-                 should "be tested"
-               end
                # context new menu item
                def test_new_menu_item_should_require_a_name
                  assert_raises ArgumentError do

test/unit/user_test.rb +16 -33

                  end
                end
-               context "#allowed_to?" do
-                 context "with a unique project" do
-                   should "return false if project is archived" do
+               test "#allowed_to? for archived project should return false" do
-                     project = Project.find(1)
-                     Project.any_instance.stubs(:status).returns(Project::STATUS_ARCHIVED)
-                     assert_equal false, @admin.allowed_to?(:view_issues, Project.find(1))
+                 project.archive
+                 project.reload
+                 assert_equal false, @admin.allowed_to?(:view_issues, project)
-                   end
-                   should "return false for write action if project is closed" do
+               test "#allowed_to? for closed project should return true for read actions" do
-                     project = Project.find(1)
-                     Project.any_instance.stubs(:status).returns(Project::STATUS_CLOSED)
-                     assert_equal false, @admin.allowed_to?(:edit_project, Project.find(1))
+                 project.close
+                 project.reload
+                 assert_equal false, @admin.allowed_to?(:edit_project, project)
+                 assert_equal true, @admin.allowed_to?(:view_project, project)
-                   end
-                   should "return true for read action if project is closed" do
-                     project = Project.find(1)
-                     Project.any_instance.stubs(:status).returns(Project::STATUS_CLOSED)
-                     assert_equal true, @admin.allowed_to?(:view_project, Project.find(1))
-                   end
-                   should "return false if related module is disabled" do
+               test "#allowed_to? for project with module disabled should return false" do
-                     project = Project.find(1)
-                     project.enabled_module_names = ["issue_tracking"]
-                     assert_equal true, @admin.allowed_to?(:add_issues, project)
-                     assert_equal false, @admin.allowed_to?(:view_wiki_pages, project)
-                   end
-                   should "authorize nearly everything for admin users" do
+               test "#allowed_to? for admin users should return true" do
-                     project = Project.find(1)
-                     assert ! @admin.member_of?(project)
-                     %w(edit_issues delete_issues manage_news add_documents manage_wiki).each do |p|
-                     end
-                   end
-                   should "authorize normal users depending on their roles" do
+               test "#allowed_to? for normal users" do
-                     project = Project.find(1)
-                     assert_equal true, @jsmith.allowed_to?(:delete_messages, project)    #Manager
-                     assert_equal false, @dlopper.allowed_to?(:delete_messages, project) #Developper
-                   end
-                 end
-                 context "with multiple projects" do
-                   should "return false if array is empty" do
+               test "#allowed_to? with empty array should return false" do
-                     assert_equal false, @admin.allowed_to?(:view_project, [])
-                   end
-                   should "return true only if user has permission on all these projects" do
+               test "#allowed_to? with multiple projects" do
-                     assert_equal true, @admin.allowed_to?(:view_project, Project.all.to_a)
-                     assert_equal false, @dlopper.allowed_to?(:view_project, Project.all.to_a) #cannot see Project(2)
-                     assert_equal true, @jsmith.allowed_to?(:edit_issues, @jsmith.projects.to_a) #Manager or Developer everywhere
-                     assert_equal false, @jsmith.allowed_to?(:delete_issue_watchers, @jsmith.projects.to_a) #Dev cannot delete_issue_watchers
-                   end
-                   should "behave correctly with arrays of 1 project" do
-                     assert_equal false, User.anonymous.allowed_to?(:delete_issues, [Project.first])
-                   end
-                 end
-                 context "with options[:global]" do
-                   should "authorize if user has at least one role that has this permission" do
+               test "#allowed_to? with with options[:global] should return true if user has one role with the permission" do
-                     @dlopper2 = User.find(5) #only Developper on a project, not Manager anywhere
-                     @anonymous = User.find(6)
-                     assert_equal true, @jsmith.allowed_to?(:delete_issue_watchers, nil, :global => true)
-                     assert_equal false, @anonymous.allowed_to?(:add_issues, nil, :global => true)
-                     assert_equal true, @anonymous.allowed_to?(:view_issues, nil, :global => true)
-                   end
-                 end
-               end
                # this is just a proxy method, the test only calls it to ensure it doesn't break trivially
-               context "#allowed_to_globally?" do
-                 should "proxy to #allowed_to? and reflect global permissions" do
+               test "#allowed_to_globally?" do
-                   @dlopper2 = User.find(5) #only Developper on a project, not Manager anywhere
-                   @anonymous = User.find(6)
-                   assert_equal true, @jsmith.allowed_to_globally?(:delete_issue_watchers)
-                   assert_equal false, @anonymous.allowed_to_globally?(:add_issues)
-                   assert_equal true, @anonymous.allowed_to_globally?(:view_issues)
-                 end
-               end
                context "User#notify_about?" do
                  context "Issues" do

General Comments 0

Write
Preview

You need to be logged in to leave comments. Login now

No TODOs yet

	Site-wide shortcuts
/	Use quick search box
g h	Goto home page
g g	Goto my private gists page
g G	Goto my public gists page
g 0-9	Goto bookmarked items from 0-9
n r	New repository page
n g	New gist page

	Repositories
g s	Goto summary page
g c	Goto changelog page
g f	Goto files page
g F	Goto files page with file search activated
g p	Goto pull requests page
g o	Goto repository settings
g O	Goto repository access permissions settings
t s	Toggle sidebar on some pages