[ARVADOS] updated: fc1a62883352b129ae9fc92efdc1bd7ef16972d7

git at public.curoverse.com git at public.curoverse.com
Mon Nov 17 17:31:14 EST 2014


Summary of changes:
 services/api/test/unit/job_test.rb | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

       via  fc1a62883352b129ae9fc92efdc1bd7ef16972d7 (commit)
      from  2f69408f572f6615504a6c17aba8c27ecba031d6 (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 fc1a62883352b129ae9fc92efdc1bd7ef16972d7
Author: Brett Smith <brett at curoverse.com>
Date:   Mon Nov 17 17:31:12 2014 -0500

    4027: Add unit test suggested by code review.

diff --git a/services/api/test/unit/job_test.rb b/services/api/test/unit/job_test.rb
index 32f9e3e..65ca935 100644
--- a/services/api/test/unit/job_test.rb
+++ b/services/api/test/unit/job_test.rb
@@ -320,7 +320,7 @@ class JobTest < ActiveSupport::TestCase
       end
     end
 
-    test "updating job with SDK version '#{search}'" do
+    test "updating job from no SDK to version '#{search}'" do
       job = Job.create!(job_attrs)
       assert_nil job.arvados_sdk_version
       check_job_sdk_version(commit_hash) do
@@ -328,6 +328,15 @@ class JobTest < ActiveSupport::TestCase
         job
       end
     end
+
+    test "updating job from SDK version 'master' to '#{search}'" do
+      job = Job.create!(job_attrs(sdk_constraint("master")))
+      assert_equal(SDK_MASTER, job.arvados_sdk_version)
+      check_job_sdk_version(commit_hash) do
+        job.runtime_constraints = sdk_constraint(search)[:runtime_constraints]
+        job
+      end
+    end
   end
 
   test "clear the SDK version" do

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list