X-Git-Url: https://git.arvados.org/arvados.git/blobdiff_plain/3db1a8bbb9f14799e0aabd12cff3c980c7da0167..965565ddc62635928a6b043158fd683738961c8c:/apps/workbench/test/integration/work_units_test.rb diff --git a/apps/workbench/test/integration/work_units_test.rb b/apps/workbench/test/integration/work_units_test.rb index 7d19fcc9d7..5b5848ee77 100644 --- a/apps/workbench/test/integration/work_units_test.rb +++ b/apps/workbench/test/integration/work_units_test.rb @@ -1,6 +1,9 @@ +require 'helpers/fake_websocket_helper' require 'integration_helper' class WorkUnitsTest < ActionDispatch::IntegrationTest + include FakeWebsocketHelper + setup do need_javascript end @@ -71,12 +74,14 @@ class WorkUnitsTest < ActionDispatch::IntegrationTest visit page_with_token "active", "/#{type}/#{obj['uuid']}" assert_text 'created_at' + if cancelable + assert_text 'priority: 1' if type.include?('container') assert_selector 'button', text: 'Cancel' - click_button 'Cancel' + first('a,button', text: 'Cancel').click wait_for_ajax end - assert_no_selector 'button', text: 'Cancel' + assert_text 'priority: 0' if cancelable and type.include?('container') end end @@ -104,8 +109,8 @@ class WorkUnitsTest < ActionDispatch::IntegrationTest end [ - ['Two Part Pipeline Template', 'part-one', 'Provide a value for the following'], - ['Workflow with input specifications', 'this workflow has inputs specified', 'Provide a value for the following'], + ['Pipeline with default input specifications', 'part-one', 'Provide values for the following'], + ['Workflow with default input specifications', 'this workflow has inputs specified', 'Provide a value for the following'], ].each do |template_name, preview_txt, process_txt| test "run a process using template #{template_name} from dashboard" do visit page_with_token('admin') @@ -113,7 +118,7 @@ class WorkUnitsTest < ActionDispatch::IntegrationTest within('.recent-processes-actions') do assert page.has_link?('All processes') - find('a', text: 'Run a pipeline').click + find('a', text: 'Run a process').click end # in the chooser, verify preview and click Next button @@ -126,6 +131,109 @@ class WorkUnitsTest < ActionDispatch::IntegrationTest # in the process page now assert_text process_txt assert_selector 'a', text: template_name + + assert_equal "Set value for ex_string_def", find('div.form-group > div > p.form-control-static > a', text: "hello-testing-123")[:"data-title"] + + page.assert_selector 'a.disabled,button.disabled', text: 'Run' + end + end + + test 'display container state changes in Container Request live log' do + use_fake_websocket_driver + c = api_fixture('containers')['queued'] + cr = api_fixture('container_requests')['queued'] + visit page_with_token('active', '/container_requests/'+cr['uuid']) + click_link('Log') + + # The attrs of the "terminal window" text div in the log tab + # indicates which objects' events are worth displaying. Events + # that arrive too early (before that div exists) are not + # shown. For the user's sake, these early logs should also be + # retrieved and shown one way or another -- but in this particular + # test, we are only interested in logs that arrive by + # websocket. Therefore, to avoid races, we wait for the log tab to + # display before sending any events. + assert_text 'Recent logs' + + [[{ + event_type: 'dispatch', + properties: { + text: "dispatch logged a fake message\n", + }, + }, "dispatch logged"], + [{ + event_type: 'update', + properties: { + old_attributes: {state: 'Locked'}, + new_attributes: {state: 'Queued'}, + }, + }, "Container #{c['uuid']} was returned to the queue"], + [{ + event_type: 'update', + properties: { + old_attributes: {state: 'Queued'}, + new_attributes: {state: 'Locked'}, + }, + }, "Container #{c['uuid']} was taken from the queue by a dispatch process"], + [{ + event_type: 'crunch-run', + properties: { + text: "according to fake crunch-run,\nsome setup stuff happened on the compute node\n", + }, + }, "setup stuff happened"], + [{ + event_type: 'update', + properties: { + old_attributes: {state: 'Locked'}, + new_attributes: {state: 'Running'}, + }, + }, "Container #{c['uuid']} started"], + [{ + event_type: 'update', + properties: { + old_attributes: {state: 'Running'}, + new_attributes: {state: 'Complete', exit_code: 1}, + }, + }, "Container #{c['uuid']} finished with exit code 1 (failure)"], + # It's unrealistic for state to change again once it's Complete, + # but the logging code doesn't care, so we do it to keep the test + # simple. + [{ + event_type: 'update', + properties: { + old_attributes: {state: 'Running'}, + new_attributes: {state: 'Cancelled'}, + }, + }, "Container #{c['uuid']} was cancelled"], + ].each do |send_event, expect_log_text| + assert_no_text(expect_log_text) + fake_websocket_event(send_event.merge(object_uuid: c['uuid'])) + assert_text(expect_log_text) + end + end + + [ + ['jobs', 'active', 'running_job_with_components', 'component1', '/jobs/zzzzz-8i9sb-jyq01m7in1jlofj#Log'], + ['pipeline_instances', 'active', 'pipeline_in_running_state', 'foo', '/jobs/zzzzz-8i9sb-pshmckwoma9plh7#Log'], + ['pipeline_instances', nil, 'pipeline_in_publicly_accessible_project_but_other_objects_elsewhere', 'foo', 'Log unavailable'], + ].each do |type, token, fixture, child, log_link| + test "link_to_log for #{fixture} for #{token}" do + obj = api_fixture(type)[fixture] + if token + visit page_with_token token, "/#{type}/#{obj['uuid']}" + else + Rails.configuration.anonymous_user_token = + api_fixture("api_client_authorizations", "anonymous", "api_token") + visit "/#{type}/#{obj['uuid']}" + end + + click_link(child) + + if token + assert_selector "a[href=\"#{log_link}\"]" + else + assert_text log_link + end end end end