[ARVADOS] updated: a00eab83a1a2636f5c18be8109f73bf050d1ec88

[git at public.curoverse.com]

Summary of changes:
 apps/workbench/test/controllers/projects_controller_test.rb | 8 +++++++-
 apps/workbench/test/integration/projects_test.rb            | 6 ------
 2 files changed, 7 insertions(+), 7 deletions(-)

       via  a00eab83a1a2636f5c18be8109f73bf050d1ec88 (commit)
      from  de98daba898ad2dfa58c9e810d98dadf4d208b95 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit a00eab83a1a2636f5c18be8109f73bf050d1ec88
Author: Manoj <jonam33 at gmail.com>
Date:   Mon Jul 13 11:18:23 2015 -0400

    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 c695bd5..ba4188e 100644
--- 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 366b1d3..0efa9c1 100644
--- 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(" ")

-----------------------------------------------------------------------


hooks/post-receive
--