params
end
- def expected_contents(params, token)
- unless token.is_a? String
- token = params[:api_token] || token[:arvados_api_token]
- end
- [token, params[:uuid], params[:file]].join('/')
- end
-
def assert_hash_includes(actual_hash, expected_hash, msg=nil)
expected_hash.each do |key, value|
assert_equal(value, actual_hash[key], msg)
test "inactive user can retrieve user agreement" do
ua_collection = api_fixture('collections')['user_agreement']
+ # Here we don't test whether the agreement can be retrieved from
+ # Keep. We only test that show_file decides to send file content,
+ # so we use the file content stub.
+ stub_file_content
get :show_file, {
uuid: ua_collection['uuid'],
file: ua_collection['manifest_text'].match(/ \d+:\d+:(\S+)/)[1]
show_collection({uuid: NONEXISTENT_COLLECTION, id: NONEXISTENT_COLLECTION},
:active, 404)
end
+
+ test "use a reasonable read buffer even if client requests a huge range" do
+ fakefiledata = mock
+ IO.expects(:popen).returns(fakefiledata)
+ fakefiledata.expects(:read).twice.with() do |length|
+ # Fail the test if read() is called with length>1MiB:
+ length < 2**20
+ end.returns("foo\n", nil)
+ fakefiledata.expects(:close)
+ foo_file = api_fixture('collections')['foo_file']
+ @request.headers['Range'] = 'bytes=0-4294967296/*'
+ get :show_file, {
+ uuid: foo_file['uuid'],
+ file: foo_file['manifest_text'].match(/ \d+:\d+:(\S+)/)[1]
+ }, session_for(:active)
+ end
end