closes #4951
[arvados.git] / apps / workbench / test / integration / anonymous_access_test.rb
index 4ffd7ae6fa4ecd030dfd42a1e500e341d4ce37a5..2e6366215c5495657874e116d737f851b5f0cc24 100644 (file)
@@ -8,23 +8,10 @@ class AnonymousAccessTest < ActionDispatch::IntegrationTest
 
   setup do
     need_javascript
+    Rails.configuration.anonymous_user_token = api_fixture('api_client_authorizations')['anonymous']['api_token']
   end
 
-  def visit_publicly_accessible_project token=nil, use_config=true, path=nil
-    if use_config
-      Rails.configuration.anonymous_user_token = api_fixture('api_client_authorizations')['anonymous']['api_token']
-    else
-      Rails.configuration.anonymous_user_token = false
-    end
-
-    path = "/projects/#{api_fixture('groups')['anonymously_accessible_project']['uuid']}/?public_data=true" if !path
-
-    if token
-      visit page_with_token(token, path)
-    else
-      visit path
-    end
-  end
+  PUBLIC_PROJECT = "/projects/#{api_fixture('groups')['anonymously_accessible_project']['uuid']}"
 
   def verify_site_navigation_anonymous_enabled user, is_active
     if user
@@ -35,7 +22,6 @@ class AnonymousAccessTest < ActionDispatch::IntegrationTest
         assert_text 'indicate that you have read and accepted the user agreement'
       end
       within('.navbar-fixed-top') do
-        assert_no_text 'You are viewing public data'
         assert_selector 'a', text: "#{user['email']}"
         find('a', text: "#{user['email']}").click
         within('.dropdown-menu') do
@@ -45,14 +31,7 @@ class AnonymousAccessTest < ActionDispatch::IntegrationTest
     else  # anonymous
       assert_text 'Unrestricted public data'
       within('.navbar-fixed-top') do
-        assert_text 'You are viewing public data'
-        anonymous_user = api_fixture('users')['anonymous']
-        assert_selector 'a', "#{anonymous_user['email']}"
-        find('a', text: "#{anonymous_user['email']}").click
-        within('.dropdown-menu') do
-          assert_selector 'a', text: 'Log in'
-          assert_no_selector 'a', text: 'Log out'
-        end
+        assert_selector 'a', text: 'Log in'
       end
     end
   end
@@ -63,24 +42,18 @@ class AnonymousAccessTest < ActionDispatch::IntegrationTest
     ['active', api_fixture('users')['active'], true, true],
   ].each do |token, user, is_active|
     test "visit public project as user #{token.inspect} when anonymous browsing is enabled" do
-      visit_publicly_accessible_project token
+      if !token
+        visit PUBLIC_PROJECT
+      else
+        visit page_with_token(token, PUBLIC_PROJECT)
+      end
+
       verify_site_navigation_anonymous_enabled user, is_active
     end
   end
 
-  test "anonymous user visit public project when anonymous browsing not enabled and expect to see login page" do
-    visit_publicly_accessible_project nil, false
-    assert_text 'Please log in'
-  end
-
-  test "visit non-public project as anonymous when anonymous browsing is enabled and expect page not found" do
-    visit_publicly_accessible_project nil, true,
-        "/projects/#{api_fixture('groups')['aproject']['uuid']}/?public_data=true"
-    assert_text 'Not Found'
-  end
-
   test "selection actions when anonymous user accesses shared project" do
-    visit_publicly_accessible_project
+    visit PUBLIC_PROJECT
 
     assert_selector 'a', text: 'Data collections'
     assert_selector 'a', text: 'Jobs and pipelines'
@@ -100,28 +73,11 @@ class AnonymousAccessTest < ActionDispatch::IntegrationTest
     end
   end
 
-  [
-    ['All pipelines', 'Pipeline in publicly accessible project'],
-    ['All jobs', 'job submitted'],
-    ['All collections', 'GNU_General_Public_License,_version_3.pdf'],
-  ].each do |selector, expectation|
-    test "verify dashboard when anonymous user accesses shared project and click #{selector}" do
-      visit_publicly_accessible_project
-
-      # go to dashboard
-      click_link 'You are viewing public data'
-
-      assert_no_selector 'a', text: 'Run a pipeline'
-      assert_selector 'a', text: selector
-      click_link selector
-      assert_text expectation
-    end
-  end
-
   test "anonymous user accesses data collections tab in shared project" do
-    visit_publicly_accessible_project
+    visit PUBLIC_PROJECT
+    assert_text 'GNU General Public License'
 
-    assert_selector 'a', text: 'Data collections (1)'
+    assert_selector 'a', text: 'Data collections'
 
     # click on show collection
     within first('tr[data-kind="arvados#collection"]') do
@@ -145,11 +101,13 @@ class AnonymousAccessTest < ActionDispatch::IntegrationTest
     'pipelineInstance'
   ].each do |type|
     test "anonymous user accesses jobs and pipelines tab in shared project and clicks on #{type}" do
-      visit_publicly_accessible_project
+      visit PUBLIC_PROJECT
+      assert_text 'GNU General Public License'
+
       click_link 'Jobs and pipelines'
       assert_text 'Pipeline in publicly accessible project'
 
-      # click on type specified collection
+      # click on the specified job
       if type.include? 'job'
         verify_job_row type
       else
@@ -166,7 +124,6 @@ class AnonymousAccessTest < ActionDispatch::IntegrationTest
 
     assert_text 'zzzzz-tpzed-xurymjxw79nv3jz' # modified by user
     assert_no_selector 'a', text: 'zzzzz-tpzed-xurymjxw79nv3jz'
-    assert_no_selector 'a', text: 'Log'
     assert_no_selector 'a', text: 'Move job'
     assert_no_selector 'button', text: 'Cancel'
     assert_no_selector 'button', text: 'Re-run job'
@@ -185,9 +142,10 @@ class AnonymousAccessTest < ActionDispatch::IntegrationTest
   end
 
   test "anonymous user accesses pipeline templates tab in shared project" do
-    visit_publicly_accessible_project
+    visit PUBLIC_PROJECT
+    assert_text 'GNU General Public License'
 
-    assert_selector 'a', 'Pipeline templates (1)'
+    assert_selector 'a', text: 'Pipeline templates'
 
     click_link 'Pipeline templates'
     assert_text 'Pipeline template in publicly accessible project'
@@ -200,21 +158,4 @@ class AnonymousAccessTest < ActionDispatch::IntegrationTest
     assert_text 'script version'
     assert_no_selector 'a', text: 'Run this pipeline'
   end
-
-  [
-    '/users',
-    '/groups',
-  ].each do |page|
-    test "anonymous user accesses publicly accessible project and then traverses to #{page}" do
-      # when anonymous, first visit publicly accessible project
-      visit_publicly_accessible_project
-      visit page
-
-      if page == '/users'
-      assert_no_selector 'a', text: 'Add a new user'
-      elsif page == '/groups'
-        assert_no_selector 'button', text: 'Add a new group'
-      end
-    end
-  end
 end