X-Git-Url: https://git.arvados.org/arvados.git/blobdiff_plain/cc6f3141d4e55c8adab282762bad9a8643336346..c50fe608334f9fc2e19969f41c04d671cb9c0a19:/services/api/test/unit/job_test.rb diff --git a/services/api/test/unit/job_test.rb b/services/api/test/unit/job_test.rb index 357e010e5c..a70c56f53a 100644 --- a/services/api/test/unit/job_test.rb +++ b/services/api/test/unit/job_test.rb @@ -25,7 +25,7 @@ class JobTest < ActiveSupport::TestCase assert_nil job.docker_image_locator end - { 'name' => [:links, :docker_image_collection_repository, :name], + { 'name' => [:links, :docker_image_collection_tag, :name], 'hash' => [:links, :docker_image_collection_hash, :name], 'locator' => [:collections, :docker_image, :portable_data_hash], }.each_pair do |spec_type, (fixture_type, fixture_name, fixture_attr)| @@ -70,7 +70,7 @@ class JobTest < ActiveSupport::TestCase end test "can't locate a Docker image with a nonexistent tag" do - image_repo = links(:docker_image_collection_repository).name + image_repo = links(:docker_image_collection_tag).name image_tag = '__nonexistent tag__' job = Job.new job_attrs(runtime_constraints: {'docker_image' => image_repo, @@ -154,4 +154,57 @@ class JobTest < ActiveSupport::TestCase assert_not_empty job.errors, "validation failure did not provide errors" end end + + [ + # Each test case is of the following format + # Array of parameters where each parameter is of the format: + # attr name to be changed, attr value, and array of expectations (where each expectation is an array) + [['running', false, [['state', 'Queued']]]], + [['state', 'Running', [['started_at', 'not_nil']]]], + [['is_locked_by_uuid', 'use_current_user_uuid', [['state', 'Queued']]], ['state', 'Running', [['running', true], ['started_at', 'not_nil'], ['success', 'nil']]]], + [['running', false, [['state', 'Queued']]], ['state', 'Complete', [['success', true]]]], + [['running', true, [['state', 'Running']]], ['cancelled_at', Time.now, [['state', 'Cancelled']]]], + [['running', true, [['state', 'Running']]], ['state', 'Cancelled', [['cancelled_at', 'not_nil']]]], + [['running', true, [['state', 'Running']]], ['success', true, [['state', 'Complete']]]], + [['running', true, [['state', 'Running']]], ['success', false, [['state', 'Failed']]]], + [['running', true, [['state', 'Running']]], ['state', 'Complete', [['success', true],['finished_at', 'not_nil']]]], + [['running', true, [['state', 'Running']]], ['state', 'Failed', [['success', false],['finished_at', 'not_nil']]]], + [['running', true, [['state', 'Running']]], ['running', false, [['state', 'Queued']]]], + [['cancelled_at', Time.now, [['state', 'Cancelled']]], ['success', false, [['state', 'Cancelled'],['finished_at', 'nil'], ['cancelled_at', 'not_nil']]]], + [['cancelled_at', Time.now, [['state', 'Cancelled'],['running', false]]], ['success', true, [['state', 'Cancelled'],['running', false],['finished_at', 'nil'],['cancelled_at', 'not_nil']]]], + # potential migration cases + [['state', nil, [['state', 'Queued']]]], + [['state', nil, [['state', 'Queued']]], ['cancelled_at', Time.now, [['state', 'Cancelled']]]], + [['running', true, [['state', 'Running']]], ['state', nil, [['state', 'Running']]]], + ].each do |parameters| + test "verify job status #{parameters}" do + job = Job.create! job_attrs + assert job.valid?, job.errors.full_messages.to_s + assert_equal 'Queued', job.state, "job.state" + + parameters.each do |parameter| + expectations = parameter[2] + if parameter[1] == 'use_current_user_uuid' + parameter[1] = Thread.current[:user].uuid + end + + if expectations.instance_of? Array + job[parameter[0]] = parameter[1] + assert_equal true, job.save, job.errors.full_messages.to_s + expectations.each do |expectation| + if expectation[1] == 'not_nil' + assert_not_nil job[expectation[0]], expectation[0] + elsif expectation[1] == 'nil' + assert_nil job[expectation[0]], expectation[0] + else + assert_equal expectation[1], job[expectation[0]], expectation[0] + end + end + else + raise 'I do not know how to handle this expectation' + end + end + end + end + end