require 'integration_helper'
-require 'selenium-webdriver'
-require 'headless'
class CollectionsTest < ActionDispatch::IntegrationTest
setup do
- Capybara.current_driver = :rack_test
+ need_javascript
end
- test "Can copy a collection to a project" do
- Capybara.current_driver = Capybara.javascript_driver
+ # check_checkboxes_state asserts that the page holds at least one
+ # checkbox matching 'selector', and that all matching checkboxes
+ # are in state 'checkbox_status' (i.e. checked if true, unchecked otherwise)
+ def assert_checkboxes_state(selector, checkbox_status, msg=nil)
+ assert page.has_selector?(selector)
+ page.all(selector).each do |checkbox|
+ assert(checkbox.checked? == checkbox_status, msg)
+ end
+ end
+ test "Can copy a collection to a project" do
collection_uuid = api_fixture('collections')['foo_file']['uuid']
collection_name = api_fixture('collections')['foo_file']['name']
project_uuid = api_fixture('groups')['aproject']['uuid']
end
test "Collection page renders name" do
+ Capybara.current_driver = :rack_test
uuid = api_fixture('collections')['foo_file']['uuid']
coll_name = api_fixture('collections')['foo_file']['name']
visit page_with_token('active', "/collections/#{uuid}")
assert(page.has_link?('foo'), "Collection page did not include file link")
end
+ def check_sharing(want_state, link_regexp)
+ # We specifically want to click buttons. See #4291.
+ if want_state == :off
+ click_button "Unshare"
+ text_assertion = :assert_no_text
+ link_assertion = :assert_empty
+ else
+ click_button "Create sharing link"
+ text_assertion = :assert_text
+ link_assertion = :refute_empty
+ end
+ using_wait_time(Capybara.default_wait_time * 3) do
+ send(text_assertion, "Shared at:")
+ end
+ send(link_assertion, all("a").select { |a| a[:href] =~ link_regexp })
+ end
+
+ test "creating and uncreating a sharing link" do
+ coll_uuid = api_fixture("collections", "collection_owned_by_active", "uuid")
+ download_link_re =
+ Regexp.new(Regexp.escape("/collections/download/#{coll_uuid}/"))
+ visit page_with_token("active_trustedclient", "/collections/#{coll_uuid}")
+ within "#sharing-button" do
+ check_sharing(:on, download_link_re)
+ check_sharing(:off, download_link_re)
+ end
+ end
+
test "can download an entire collection with a reader token" do
+ Capybara.current_driver = :rack_test
+ CollectionsController.any_instance.
+ stubs(:file_enumerator).returns(["foo\n", "file\n"])
uuid = api_fixture('collections')['foo_file']['uuid']
token = api_fixture('api_client_authorizations')['active_all_collections']['api_token']
url_head = "/collections/download/#{uuid}/#{token}/"
# a very blunt approach.
assert_no_match(/<\s*meta[^>]+\bnofollow\b/i, page.html,
"wget prohibited from recursing the collection page")
- # TODO: When we can test against a Keep server, actually follow links
- # and check their contents, rather than testing the href directly
- # (this is too closely tied to implementation details).
+ # Look at all the links that wget would recurse through using our
+ # recommended options, and check that it's exactly the file list.
hrefs = page.all('a').map do |anchor|
link = anchor[:href] || ''
if link.start_with? url_head
end
assert_equal(['foo'], hrefs.compact.sort,
"download page did provide strictly file links")
+ within "#collection_files" do
+ click_link "foo"
+ assert_equal("foo\nfile\n", page.html)
+ end
end
test "can view empty collection" do
+ Capybara.current_driver = :rack_test
uuid = 'd41d8cd98f00b204e9800998ecf8427e+0'
visit page_with_token('active', "/collections/#{uuid}")
- assert page.has_text?('This collection is empty')
+ assert page.has_text?(/This collection is empty|The following collections have this content/)
end
test "combine selected collections into new collection" do
- headless = Headless.new
- headless.start
- Capybara.current_driver = :selenium
-
foo_collection = api_fixture('collections')['foo_file']
bar_collection = api_fixture('collections')['bar_file']
assert(page.has_text?('foo'), "Collection page did not include foo file")
assert(page.has_no_text?(bar_collection['name']), "Collection page did not include foo file")
assert(page.has_text?('bar'), "Collection page did not include bar file")
-
- headless.stop
+ assert(page.has_text?('Created new collection in your Home project'),
+ 'Not found flash message that new collection is created in Home project')
end
- test "combine selected collection files into new collection" do
- headless = Headless.new
- headless.start
- Capybara.current_driver = :selenium
+ [
+ ['active', 'foo_file', false],
+ ['active', 'foo_collection_in_aproject', true],
+ ['project_viewer', 'foo_file', false],
+ ['project_viewer', 'foo_collection_in_aproject', false], #aproject not writable
+ ].each do |user, collection, expect_collection_in_aproject|
+ test "combine selected collection files into new collection #{user} #{collection} #{expect_collection_in_aproject}" do
+ my_collection = api_fixture('collections')[collection]
- foo_collection = api_fixture('collections')['foo_file']
+ visit page_with_token(user, "/collections")
- visit page_with_token('active', "/collections")
+ # choose file from foo collection
+ within('tr', text: my_collection['uuid']) do
+ click_link 'Show'
+ end
- # choose file from foo collection
- within('tr', text: foo_collection['uuid']) do
- click_link 'Show'
+ # now in collection page
+ find('input[type=checkbox]').click
+
+ click_button 'Selection...'
+ within('.selection-action-container') do
+ click_link 'Create new collection with selected files'
+ end
+
+ # now in the newly created collection page
+ assert(page.has_text?('Copy to project'), "Copy to project text not found in new collection page")
+ assert(page.has_no_text?(my_collection['name']), "Collection page did not include foo file")
+ assert(page.has_text?('foo'), "Collection page did not include foo file")
+ if expect_collection_in_aproject
+ aproject = api_fixture('groups')['aproject']
+ assert page.has_text?("Created new collection in the project #{aproject['name']}"),
+ 'Not found flash message that new collection is created in aproject'
+ else
+ assert page.has_text?("Created new collection in your Home project"),
+ 'Not found flash message that new collection is created in Home project'
+ end
end
+ end
+
+ test "combine selected collection files from collection subdirectory" do
+ visit page_with_token('user1_with_load', "/collections/zzzzz-4zz18-filesinsubdir00")
# now in collection page
- find('input[type=checkbox]').click
+ input_files = page.all('input[type=checkbox]')
+ (0..input_files.count-1).each do |i|
+ input_files[i].click
+ end
click_button 'Selection...'
within('.selection-action-container') do
end
# now in the newly created collection page
- assert(page.has_text?('Copy to project'), "Copy to project text not found in new collection page")
- assert(page.has_no_text?(foo_collection['name']), "Collection page did not include foo file")
- assert(page.has_text?('foo'), "Collection page did not include foo file")
+ assert(page.has_text?('file_in_subdir1'), 'file not found - file_in_subdir1')
+ assert(page.has_text?('file1_in_subdir3.txt'), 'file not found - file1_in_subdir3.txt')
+ assert(page.has_text?('file2_in_subdir3.txt'), 'file not found - file2_in_subdir3.txt')
+ assert(page.has_text?('file1_in_subdir4.txt'), 'file not found - file1_in_subdir4.txt')
+ assert(page.has_text?('file2_in_subdir4.txt'), 'file not found - file1_in_subdir4.txt')
+ end
+
+ test "Collection portable data hash redirect" do
+ di = api_fixture('collections')['docker_image']
+ visit page_with_token('active', "/collections/#{di['portable_data_hash']}")
+
+ # check redirection
+ assert current_path.end_with?("/collections/#{di['uuid']}")
+ assert page.has_text?("docker_image")
+ assert page.has_text?("Activity")
+ assert page.has_text?("Sharing and permissions")
+ end
+
+ test "Collection portable data hash with multiple matches" do
+ pdh = api_fixture('collections')['baz_file']['portable_data_hash']
+ visit page_with_token('admin', "/collections/#{pdh}")
+
+ matches = api_fixture('collections').select {|k,v| v["portable_data_hash"] == pdh}
+ assert matches.size > 1
+
+ matches.each do |k,v|
+ assert page.has_link?(v["name"]), "Page /collections/#{pdh} should contain link '#{v['name']}'"
+ end
+ assert page.has_no_text?("Activity")
+ assert page.has_no_text?("Sharing and permissions")
+ end
+
+ test "Filtering collection files by regexp" do
+ col = api_fixture('collections', 'multilevel_collection_1')
+ visit page_with_token('active', "/collections/#{col['uuid']}")
+
+ # Filter file list to some but not all files in the collection
+ page.find_field('file_regex').set('file[12]')
+ assert page.has_text?("file1")
+ assert page.has_text?("file2")
+ assert page.has_no_text?("file3")
+
+ # Filter file list with a regex matching all files
+ page.find_field('file_regex').set('.*')
+ assert page.has_text?("file1")
+ assert page.has_text?("file2")
+ assert page.has_text?("file3")
+
+ # Filter file list to a regex matching no files
+ page.find_field('file_regex').set('file9')
+ assert page.has_no_text?("file1")
+ assert page.has_no_text?("file2")
+ assert page.has_no_text?("file3")
+ # make sure that we actually are looking at the collections
+ # page and not e.g. a fiddlesticks
+ assert page.has_text?("multilevel_collection_1")
+ assert page.has_text?(col['portable_data_hash'])
+
+ # Set filename filter to a syntactically invalid regex
+ # Page loads, but stops filtering after the last valid regex parse
+ page.find_field('file_regex').set('file[2')
+ assert page.has_text?("multilevel_collection_1")
+ assert page.has_text?(col['portable_data_hash'])
+ assert page.has_text?("file1")
+ assert page.has_text?("file2")
+ assert page.has_text?("file3")
+
+ # Test the "Select all" button
+
+ # Note: calling .set('') on a Selenium element is not sufficient
+ # to reset the field for this test, as it does not send any key
+ # events to the browser. To clear the field, we must instead send
+ # a backspace character.
+ # See https://selenium.googlecode.com/svn/trunk/docs/api/rb/Selenium/WebDriver/Element.html#clear-instance_method
+ page.find_field('file_regex').set("\b") # backspace
+ find('button#select-all').click
+ assert_checkboxes_state('input[type=checkbox]', true, '"select all" should check all checkboxes')
+
+ # Test the "Unselect all" button
+ page.find_field('file_regex').set("\b") # backspace
+ find('button#unselect-all').click
+ assert_checkboxes_state('input[type=checkbox]', false, '"unselect all" should clear all checkboxes')
+
+ # Filter files, then "select all", then unfilter
+ page.find_field('file_regex').set("\b") # backspace
+ find('button#unselect-all').click
+ page.find_field('file_regex').set('file[12]')
+ find('button#select-all').click
+ page.find_field('file_regex').set("\b") # backspace
+
+ # all "file1" and "file2" checkboxes must be selected
+ # all "file3" checkboxes must be clear
+ assert_checkboxes_state('[value*="file1"]', true, 'checkboxes for file1 should be selected after filtering')
+ assert_checkboxes_state('[value*="file2"]', true, 'checkboxes for file2 should be selected after filtering')
+ assert_checkboxes_state('[value*="file3"]', false, 'checkboxes for file3 should be clear after filtering')
+
+ # Select all files, then filter, then "unselect all", then unfilter
+ page.find_field('file_regex').set("\b") # backspace
+ find('button#select-all').click
+ page.find_field('file_regex').set('file[12]')
+ find('button#unselect-all').click
+ page.find_field('file_regex').set("\b") # backspace
+
+ # all "file1" and "file2" checkboxes must be clear
+ # all "file3" checkboxes must be selected
+ assert_checkboxes_state('[value*="file1"]', false, 'checkboxes for file1 should be clear after filtering')
+ assert_checkboxes_state('[value*="file2"]', false, 'checkboxes for file2 should be clear after filtering')
+ assert_checkboxes_state('[value*="file3"]', true, 'checkboxes for file3 should be selected after filtering')
+ end
+
+ test "Creating collection from list of filtered files" do
+ col = api_fixture('collections', 'collection_with_files_in_subdir')
+ visit page_with_token('user1_with_load', "/collections/#{col['uuid']}")
+ assert page.has_text?('file_in_subdir1'), 'expected file_in_subdir1 not found'
+ assert page.has_text?('file1_in_subdir3'), 'expected file1_in_subdir3 not found'
+ assert page.has_text?('file2_in_subdir3'), 'expected file2_in_subdir3 not found'
+ assert page.has_text?('file1_in_subdir4'), 'expected file1_in_subdir4 not found'
+ assert page.has_text?('file2_in_subdir4'), 'expected file2_in_subdir4 not found'
+
+ # Select all files but then filter them to files in subdir1, subdir2 or subdir3
+ find('button#select-all').click
+ page.find_field('file_regex').set('_in_subdir[123]')
+ assert page.has_text?('file_in_subdir1'), 'expected file_in_subdir1 not in filtered files'
+ assert page.has_text?('file1_in_subdir3'), 'expected file1_in_subdir3 not in filtered files'
+ assert page.has_text?('file2_in_subdir3'), 'expected file2_in_subdir3 not in filtered files'
+ assert page.has_no_text?('file1_in_subdir4'), 'file1_in_subdir4 found in filtered files'
+ assert page.has_no_text?('file2_in_subdir4'), 'file2_in_subdir4 found in filtered files'
+
+ # Create a new collection
+ click_button 'Selection...'
+ within('.selection-action-container') do
+ click_link 'Create new collection with selected files'
+ end
+
+ # now in the newly created collection page
+ assert page.has_text?('Content hash:'), 'not on new collection page'
+ assert page.has_no_text?(col['uuid']), 'new collection page has old collection uuid'
+ assert page.has_no_text?(col['portable_data_hash']), 'new collection page has old portable_data_hash'
- headless.stop
+ # must have files in subdir1 and subdir3 but not subdir4
+ assert page.has_text?('file_in_subdir1'), 'file_in_subdir1 missing from new collection'
+ assert page.has_text?('file1_in_subdir3'), 'file1_in_subdir3 missing from new collection'
+ assert page.has_text?('file2_in_subdir3'), 'file2_in_subdir3 missing from new collection'
+ assert page.has_no_text?('file1_in_subdir4'), 'file1_in_subdir4 found in new collection'
+ assert page.has_no_text?('file2_in_subdir4'), 'file2_in_subdir4 found in new collection'
end
end
projects / arvados.git / blobdiff