From: radhika <radhika@curoverse.com>
Date: Thu, 25 Sep 2014 16:54:35 +0000 (-0400)
Subject: 3889: split the test "update a job without failing script_version check" into two... 
X-Git-Tag: 1.1.0~2151^2~21
X-Git-Url: https://git.arvados.org/arvados.git/commitdiff_plain/081ffd25912e960854427a7c0531c6dbb78d1b4c

3889: split the test "update a job without failing script_version check" into two tests instead of two action in one test.
---

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 03ee3df2e2..ca6112a5dd 100644
--- a/services/api/test/functional/arvados/v1/jobs_controller_test.rb
+++ b/services/api/test/functional/arvados/v1/jobs_controller_test.rb
@@ -102,22 +102,17 @@ 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
+  ['admin', 'active'].each do |user|
+    test "#{user} 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(user).uuid
+        }
       }
-    }
-    assert_response :success
+      assert_response :success
+    end
   end
 
   test "search jobs by uuid with >= query" do