15499: Make reuse tests fail to expose the bug.
authorLucas Di Pentima <ldipentima@veritasgenetics.com>
Fri, 6 Sep 2019 21:28:39 +0000 (18:28 -0300)
committerPeter Amstutz <pamstutz@veritasgenetics.com>
Wed, 11 Sep 2019 19:09:24 +0000 (15:09 -0400)
Arvados-DCO-1.1-Signed-off-by: Lucas Di Pentima <ldipentima@veritasgenetics.com>

services/api/test/unit/container_request_test.rb

index 69e277cc2c000ae627c4b1193c629ac8cf99e7dc..7b75ee5134527c9ab2bf19590b9731bc54d77080 100644 (file)
@@ -630,6 +630,8 @@ class ContainerRequestTest < ActiveSupport::TestCase
       set_user_from_auth :active
       cr1 = create_minimal_req!(common_attrs.merge({state: ContainerRequest::Committed,
                                                     environment: env1}))
       set_user_from_auth :active
       cr1 = create_minimal_req!(common_attrs.merge({state: ContainerRequest::Committed,
                                                     environment: env1}))
+      run_container(cr1)
+      cr1.reload
       if use_existing.nil?
         # Testing with use_existing default value
         cr2 = create_minimal_req!(common_attrs.merge({state: ContainerRequest::Uncommitted,
       if use_existing.nil?
         # Testing with use_existing default value
         cr2 = create_minimal_req!(common_attrs.merge({state: ContainerRequest::Uncommitted,
@@ -855,8 +857,7 @@ class ContainerRequestTest < ActiveSupport::TestCase
       c.update_attributes!(state: Container::Running)
       c.update_attributes!(state: Container::Complete,
                            exit_code: 0,
       c.update_attributes!(state: Container::Running)
       c.update_attributes!(state: Container::Complete,
                            exit_code: 0,
-                           output: '1f4b0bc7583c2a7f9102c395f4ffc5e3+45',
-                           log: 'fa7aeb5140e2848d39b416daeef4ffc5+45')
+                           output: '1f4b0bc7583c2a7f9102c395f4ffc5e3+45')
       c
     end
   end
       c
     end
   end