[ARVADOS] updated: 6ae89f63b495395880b34645c84c813cbfacee78

[git at public.curoverse.com]

Summary of changes:
 services/api/test/fixtures/jobs.yml                |   22 ++++++++++++++++++++
 .../functional/arvados/v1/jobs_controller_test.rb  |   17 +++++++++++++++
 2 files changed, 39 insertions(+), 0 deletions(-)

       via  6ae89f63b495395880b34645c84c813cbfacee78 (commit)
      from  c67624630594b481aa96d548282187720601abea (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 6ae89f63b495395880b34645c84c813cbfacee78
Author: Tom Clegg <tom at curoverse.com>
Date:   Wed Jan 29 12:08:54 2014 -0800

    Add functional test for "skip script_version validation when not
    changing that attribute".

diff --git a/services/api/test/fixtures/jobs.yml b/services/api/test/fixtures/jobs.yml
index b6a08fe..39900ee 100644
--- a/services/api/test/fixtures/jobs.yml
+++ b/services/api/test/fixtures/jobs.yml
@@ -39,3 +39,25 @@ running_cancelled:
     running: 1
     done: 1
   runtime_constraints: {}
+
+uses_nonexistent_script_version:
+  uuid: zzzzz-8i9sb-7m339pu0x9mla88
+  owner_uuid: zzzzz-tpzed-xurymjxw79nv3jz
+  cancelled_at: ~
+  cancelled_by_user_uuid: ~
+  cancelled_by_client_uuid: ~
+  script_version: 7def43a4d3f20789dda4700f703b5514cc3ed250
+  started_at: <%= 3.minute.ago.to_s(:db) %>
+  finished_at: <%= 2.minute.ago.to_s(:db) %>
+  running: false
+  success: true
+  output: d41d8cd98f00b204e9800998ecf8427e+0
+  priority: ~
+  log: d41d8cd98f00b204e9800998ecf8427e+0
+  is_locked_by_uuid: ~
+  tasks_summary:
+    failed: 0
+    todo: 0
+    running: 0
+    done: 1
+  runtime_constraints: {}
diff --git a/services/api/test/functional/arvados/v1/jobs_controller_test.rb b/services/api/test/functional/arvados/v1/jobs_controller_test.rb
index 3296161..912a901 100644
--- a/services/api/test/functional/arvados/v1/jobs_controller_test.rb
+++ b/services/api/test/functional/arvados/v1/jobs_controller_test.rb
@@ -54,4 +54,21 @@ class Arvados::V1::JobsControllerTest < ActionController::TestCase
     assert_not_nil job['cancelled_at'], 'un-cancelled job stays cancelled'
   end
 
+  test "update a job without failing script_version check" do
+    authorize_with :admin
+    put :update, {
+      id: jobs(:uses_nonexistent_script_version).uuid,
+      job: {
+        owner_uuid: users(:admin).uuid
+      }
+    }
+    assert_response :success
+    put :update, {
+      id: jobs(:uses_nonexistent_script_version).uuid,
+      job: {
+        owner_uuid: users(:active).uuid
+      }
+    }
+    assert_response :success
+  end
 end

-----------------------------------------------------------------------


hooks/post-receive
--