- ['/collections', '/'].each do |path|
- test "Flip persistent switch at #{path}" do
- Capybara.current_driver = Capybara.javascript_driver
- uuid = api_fixture('collections')['foo_file']['uuid']
- visit page_with_token('active', path)
- within "tr[data-object-uuid='#{uuid}']" do
- change_persist 'cache', 'persistent'
- end
- # Refresh page and make sure the change was committed.
- visit current_path
- within "tr[data-object-uuid='#{uuid}']" do
- change_persist 'persistent', 'cache'
+ test "can download an entire collection with a reader token" do
+ Capybara.current_driver = :rack_test
+
+ uuid = api_fixture('collections')['foo_file']['uuid']
+ token = api_fixture('api_client_authorizations')['active_all_collections']['api_token']
+ url_head = "/collections/download/#{uuid}/#{token}/"
+ visit url_head
+ # It seems that Capybara can't inspect tags outside the body, so this is
+ # 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).
+ hrefs = page.all('a').map do |anchor|
+ link = anchor[:href] || ''
+ if link.start_with? url_head
+ link[url_head.size .. -1]
+ elsif link.start_with? '/'
+ nil
+ else
+ link