[arvados] updated: 2.1.0-2617-g8609b8395

git repository hosting git at public.arvados.org
Fri Jun 24 14:05:28 UTC 2022


Summary of changes:
 services/api/test/unit/container_test.rb | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

       via  8609b839559fff26068ff831bbccd839ad4a762f (commit)
      from  20889c3022685d6fddd889ba2bbfe713335a4738 (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 8609b839559fff26068ff831bbccd839ad4a762f
Author: Tom Clegg <tom at curii.com>
Date:   Fri Jun 24 10:05:17 2022 -0400

    18948: Update test.
    
    Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tom at curii.com>

diff --git a/services/api/test/unit/container_test.rb b/services/api/test/unit/container_test.rb
index 685bd15e6..bcf99da2e 100644
--- a/services/api/test/unit/container_test.rb
+++ b/services/api/test/unit/container_test.rb
@@ -870,16 +870,14 @@ class ContainerTest < ActiveSupport::TestCase
     end
   end
 
-  test "Container only set exit code on complete" do
+  test "can only change exit code while running and at completion" do
     set_user_from_auth :active
     c, _ = minimal_new
     set_user_from_auth :dispatch1
     c.lock
+    check_illegal_updates c, [{exit_code: 1}]
     c.update_attributes! state: Container::Running
-
-    check_illegal_updates c, [{exit_code: 1},
-                              {exit_code: 1, state: Container::Cancelled}]
-
+    assert c.update_attributes(exit_code: 1)
     assert c.update_attributes(exit_code: 1, state: Container::Complete)
   end
 

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list