[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