6465-Moved test "project viewer can't see project sharing tab" from

projects integration test to projects controller test.

diff --git a/apps/workbench/test/controllers/projects_controller_test.rb b/apps/workbench/test/controllers/projects_controller_test.rb
index c695bd5e6bb3c9569a781fa4c7111d17602e38fe..ba4188e4c5cd57cb214a0e93a7254f13c24293be 100644 (file)
--- a/apps/workbench/test/controllers/projects_controller_test.rb
+++ b/apps/workbench/test/controllers/projects_controller_test.rb
@@ -345,10 +345,16 @@ class ProjectsControllerTest < ActionController::TestCase
     found.description = '"Link to object":' + api_fixture('groups')['asubproject']['uuid']
     found.save!
     get(:show, {id: project['uuid']}, session_for(:active))
-    puts @response.body
     assert_not_includes '"Link to object"', @response.body
     assert_match /href=.*Link to object.*\/a./, @response.body
     refute_empty css_select('[href="/groups/zzzzz-j7d0g-axqo7eu9pwvna1x"]')
   end
 
+  test "project viewer can't see project sharing tab" do
+    project = api_fixture('groups')['aproject']
+    get(:show, {id: project['uuid']}, session_for(:project_viewer))
+    refute_includes @response.body, '<div id ="Sharing"'
+    assert_includes @response.body, '<div id="Data_collections"'
+  end
+
 end

diff --git a/apps/workbench/test/integration/projects_test.rb b/apps/workbench/test/integration/projects_test.rb
index 366b1d3ddc05ebb047cd984a50d5b8f810c5f5dd..0efa9c18cb937adec05632f08dcbb6e6dcc79344 100644 (file)
--- a/apps/workbench/test/integration/projects_test.rb
+++ b/apps/workbench/test/integration/projects_test.rb
@@ -127,12 +127,6 @@ class ProjectsTest < ActionDispatch::IntegrationTest
                     text: group_name("anonymous_group"))
   end
 
-  test "project viewer can't see project sharing tab" do
-    show_object_using('project_viewer', 'groups', 'aproject', 'A Project')
-    assert(page.has_no_link?("Sharing"),
-           "read-only project user sees sharing tab")
-  end
-
   test "project owner can manage sharing for another user" do
     add_user = api_fixture('users')['future_project_user']
     new_name = ["first_name", "last_name"].map { |k| add_user[k] }.join(" ")