X-Git-Url: https://git.arvados.org/arvados.git/blobdiff_plain/3db1a8bbb9f14799e0aabd12cff3c980c7da0167..4218f74266f3fee7c618c887bea6a804005c89b9:/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..f9f5addb15 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 @@ -57,11 +60,11 @@ class WorkUnitsTest < ActionDispatch::IntegrationTest end [ - ['jobs', 'running_job_with_components', true], - ['pipeline_instances', 'components_is_jobspec', false], + ['jobs', 'running_job_with_components', true, true], + ['pipeline_instances', 'components_is_jobspec', true, true], ['containers', 'running', false], ['container_requests', 'running', true], - ].each do |type, fixture, cancelable| + ].each do |type, fixture, cancelable, confirm_cancellation| test "cancel button for #{type}/#{fixture}" do if cancelable need_selenium 'to cancel' @@ -72,11 +75,31 @@ class WorkUnitsTest < ActionDispatch::IntegrationTest assert_text 'created_at' if cancelable - assert_selector 'button', text: 'Cancel' - click_button 'Cancel' + assert_text 'priority: 1' if type.include?('container') + if type.include?('pipeline') + assert_selector 'a', text: 'Pause' + first('a,link', text: 'Pause').click + else + assert_selector 'button', text: 'Cancel' + first('a,button', text: 'Cancel').click + end + if confirm_cancellation + alert = page.driver.browser.switch_to.alert + alert.accept + end wait_for_ajax end - assert_no_selector 'button', text: 'Cancel' + + if type.include?('pipeline') + assert_selector 'a', text: 'Resume' + assert_no_selector 'a', text: 'Pause' + elsif type.include?('job') + assert_text 'Cancelled' + assert_text 'Paused' # this job has a pipeline child which was also cancelled + assert_no_selector 'button', text: 'Cancel' + elsif cancelable + assert_text 'priority: 0' + end end end @@ -104,8 +127,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 +136,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 +149,133 @@ 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 + + test 'Run from workflows index page' do + visit page_with_token('active', '/workflows') + + wf_count = page.all('a[data-original-title="show workflow"]').count + assert_equal true, wf_count>0 + + # Run one of the workflows + wf_name = 'Workflow with input specifications' + within('tr', text: wf_name) do + find('a,button', text: 'Run').click + end + + # Choose project for the container_request being created + within('.modal-dialog') do + find('.selectable', text: 'A Project').click + find('button', text: 'Choose').click + end + + # In newly created container_request page now + assert_text 'A Project' # CR created in "A Project" + assert_text "This container request was created from the workflow #{wf_name}" + assert_match /Provide a value for .* then click the \"Run\" button to start the workflow/, page.text + end end