[ARVADOS] updated: e56010c12b4cc0695000d54291588fc713ed60b0

git at public.curoverse.com git at public.curoverse.com
Tue Sep 23 08:12:06 EDT 2014


Summary of changes:
 .../api/test/functional/arvados/v1/nodes_controller_test.rb  | 12 ++++++++++++
 1 file changed, 12 insertions(+)

       via  e56010c12b4cc0695000d54291588fc713ed60b0 (commit)
      from  f9b0d35bd35c3acb496d3a2afa1c1290f1ed7feb (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 e56010c12b4cc0695000d54291588fc713ed60b0
Author: Brett Smith <brett at curoverse.com>
Date:   Tue Sep 23 08:13:59 2014 -0400

    Fixup: Add Node-Job API test.
    
    I was worried this wouldn't pass.  Turns out it does.  The test is
    worth keeping around.

diff --git a/services/api/test/functional/arvados/v1/nodes_controller_test.rb b/services/api/test/functional/arvados/v1/nodes_controller_test.rb
index fdea25b..f7bc74e 100644
--- a/services/api/test/functional/arvados/v1/nodes_controller_test.rb
+++ b/services/api/test/functional/arvados/v1/nodes_controller_test.rb
@@ -159,4 +159,16 @@ class Arvados::V1::NodesControllerTest < ActionController::TestCase
     }
     assert_response 403
   end
+
+  test "job readable after updating other attributes" do
+    authorize_with :admin
+    post :update, {
+      id: nodes(:busy).uuid,
+      node: {last_ping_at: 1.second.ago},
+    }
+    assert_response :success
+    assert_equal(jobs(:nearly_finished_job).uuid,
+                 json_response["job"].andand["uuid"],
+                 "mismatched job UUID after ping update")
+  end
 end

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list