[ARVADOS] updated: 4ccbea9ef440a7e4252b0df5e710dcb767831c60

Git user git at public.curoverse.com
Mon Apr 24 23:50:54 EDT 2017


Summary of changes:
 .../app/assets/javascripts/edit_collection.js      |  12 ++
 .../app/controllers/keep_disks_controller.rb       |   1 +
 apps/workbench/app/controllers/users_controller.rb |   1 +
 apps/workbench/app/helpers/application_helper.rb   |   2 +
 apps/workbench/app/models/arvados_base.rb          |   4 +
 apps/workbench/app/models/arvados_resource_list.rb |   8 ++
 apps/workbench/app/models/job.rb                   |   2 +-
 apps/workbench/app/models/pipeline_instance.rb     |   7 +-
 apps/workbench/app/models/proxy_work_unit.rb       |   1 +
 apps/workbench/app/models/workflow.rb              |   8 ++
 .../app/views/collections/_show_files.html.erb     |  14 ++-
 .../app/views/workflows/_show_recent.html.erb      |  65 ++++++++++
 .../container_requests_controller_test.rb          |  21 ++++
 .../workbench/test/integration/collections_test.rb |   1 +
 .../test/integration/container_requests_test.rb    |  12 ++
 apps/workbench/test/integration/work_units_test.rb |  24 ++++
 .../test/unit/arvados_resource_list_test.rb        |   8 ++
 apps/workbench/test/unit/link_test.rb              |   3 +
 apps/workbench/test/unit/pipeline_instance_test.rb |   3 +
 apps/workbench/test/unit/work_unit_test.rb         |   3 +
 build/run-build-packages.sh                        |   2 -
 build/run-library.sh                               |   2 +
 build/run-tests.sh                                 |   2 +-
 sdk/cli/bin/crunch-job                             |   8 +-
 sdk/cli/test/binstub_arv-mount/arv-mount           |   1 +
 .../test/binstub_clean_fail/{mount => arv-mount}   |   0
 sdk/cli/test/test_crunch-job.rb                    |   4 +-
 sdk/cwl/arvados_cwl/__init__.py                    |  37 +++---
 sdk/cwl/arvados_cwl/crunch_script.py               |  12 +-
 sdk/cwl/arvados_cwl/fsaccess.py                    |  32 +++--
 sdk/cwl/arvados_cwl/runner.py                      |   2 +-
 sdk/cwl/setup.py                                   |   2 +-
 sdk/cwl/tests/listing-job.yml                      |   3 +
 sdk/cwl/tests/test_container.py                    |  12 +-
 sdk/cwl/tests/test_fsaccess.py                     |  28 +++++
 sdk/cwl/tests/test_job.py                          |   9 +-
 sdk/go/crunchrunner/crunchrunner.go                |  66 +++++-----
 sdk/go/crunchrunner/crunchrunner_test.go           |  51 ++++----
 sdk/go/dispatch/throttle_test.go                   |  29 +++--
 sdk/python/arvados/_ranges.py                      |   6 +-
 sdk/python/arvados/arvfile.py                      |  83 ++++++++++---
 sdk/python/arvados/commands/get.py                 |  16 ++-
 sdk/python/tests/test_arv_get.py                   |  49 ++++++--
 sdk/python/tests/test_arvfile.py                   | 134 ++++++++++++++++++++-
 sdk/python/tests/test_stream.py                    |   7 +-
 ...1_add_created_by_job_task_index_to_job_tasks.rb |   5 +
 ...0170419173712_add_object_owner_index_to_logs.rb |   5 +
 ...esting_container_index_to_container_requests.rb |   5 +
 services/api/db/structure.sql                      |  29 ++++-
 services/keep-web/handler.go                       |  14 +++
 services/keep-web/handler_test.go                  |  31 +++++
 51 files changed, 711 insertions(+), 175 deletions(-)
 create mode 100644 apps/workbench/app/views/workflows/_show_recent.html.erb
 create mode 100755 sdk/cli/test/binstub_arv-mount/arv-mount
 rename sdk/cli/test/binstub_clean_fail/{mount => arv-mount} (100%)
 create mode 100644 sdk/cwl/tests/listing-job.yml
 create mode 100644 sdk/cwl/tests/test_fsaccess.py
 create mode 100644 services/api/db/migrate/20170419173031_add_created_by_job_task_index_to_job_tasks.rb
 create mode 100644 services/api/db/migrate/20170419173712_add_object_owner_index_to_logs.rb
 create mode 100644 services/api/db/migrate/20170419175801_add_requesting_container_index_to_container_requests.rb

       via  4ccbea9ef440a7e4252b0df5e710dcb767831c60 (commit)
       via  c043e133b2646037ed630d571e91dbf77344f855 (commit)
       via  bf5d77baad2071af6eea514c76b4892cec4974a0 (commit)
       via  04bd6b08b9ac13d29ac05c9281850d430d71066d (commit)
       via  840b855ff0317e66f4176ae0f23e9785f72267b4 (commit)
       via  1220e2184449ccab288fa41de4749fb029cd317b (commit)
       via  17b80c32a5b177ee8c5f32b81dd0889f3399eee8 (commit)
       via  35c2572761bb060aa1c12f417f97aa9e1ccbe7eb (commit)
       via  b04638275cff9b393e1bc04136d44f361b999cf8 (commit)
       via  c39ba5193005a4e9f619901f8348f11fada88df0 (commit)
       via  2333472a4f517a227278f028bbbc4e72687c0e71 (commit)
       via  5180238a10bd15302a1c15b9a428f2fdeeabdf4e (commit)
       via  bfc9660a8b2467893baf131b20e83e76c41ae438 (commit)
       via  aed7702a67426dfd9d24b512c90df8e909162179 (commit)
       via  9aa83ad7b4de05dd2818885ed34111d4dcf322ea (commit)
       via  3c34e713aa343d56c7cea00a9c998b06dbf411d6 (commit)
       via  65e339856daf4b5c3a4a810cd3a5f1a8e386dc8c (commit)
       via  151df8c3b177e4971bfbdf68c87d89599dbe0812 (commit)
       via  e7284afa8ccb95994dcd2009015cafc6180e7187 (commit)
       via  82c40aa7d30cd8e68e2a1bdc0bd8bf03cdfea029 (commit)
       via  38a4b3c43a8a6cff5a00624436b8eaa5cbfbc76b (commit)
       via  f79536fda9dc40f480383caa69a35663702b2ba4 (commit)
       via  c14ae0edcb3c386e50f46218184e8dabcbc20a37 (commit)
       via  04951581a941697d68cdaf9af6661c3c412f1bce (commit)
       via  de99c0b2effdd43d3843f15475cee84dbde8add8 (commit)
       via  e3ac17f8a8aa439e21a8bf56a571f91a671313f7 (commit)
       via  244c47436f294638271eef637997fd00f7ca49f5 (commit)
       via  67ba19113789346005aa61d4234bc33c8677a85c (commit)
       via  9609f9a5a4776671f571f765a179506d26df56da (commit)
       via  03188ad6eb14ee3dcd6bdf74198624c9358936c5 (commit)
       via  692a66affa483483d67931a6b095b361bfd06d24 (commit)
       via  243701b66914b69c9f9a94364e84713c8fcbc1da (commit)
       via  ea40bcae7a5d247ba8667c8c866339c2a8424464 (commit)
       via  cf311e8e16ba74467c77b5353afedc29b40a6a41 (commit)
       via  20364bc1aed2dcad71b006f4314ba0720d20ae05 (commit)
       via  72a377c59d972dd64de6e500c140738ff4b3dea6 (commit)
       via  fc2eaa20275146369e451da6cf14f4461c68117c (commit)
       via  bf55ee80e71c6503292a44fb377927f2ed908f9b (commit)
       via  9cab6a09cca4153104694f0dd4644c5aa5f54b22 (commit)
       via  cbcb0fc8c3ebc85bf81ba9d50795d62db75efa6c (commit)
       via  e507e67ee50852fc44011127eb42f535fe2ae493 (commit)
       via  d64ed33e94700f8204ec8089c7b235cff918f9f7 (commit)
       via  88c26c6b2dd8752ec1ff8196f4af11369ea6adbb (commit)
       via  91f1e7009305109db0bfb6405adbd11357745bfc (commit)
       via  2001423a6eb7937a689414f3fa62be5b124812c1 (commit)
       via  3a88a5d8753593be89373c5b124fc5a8398457a0 (commit)
       via  defb299deab3be85644a7880cba492a73c2e7e62 (commit)
       via  5152e0bf159151eaf04d31b788d57bebf7ab089b (commit)
       via  a3488ff35d219b943f3f52c07914fddf2009a140 (commit)
       via  574dc168c559dd680d26be1306c4a3d8b936c6e8 (commit)
       via  da32ae2a73012ce55cb89b2de9a4716b2800eee1 (commit)
       via  2f06f0e4d5b9ef371463c65dd5cae19f5b385a17 (commit)
       via  29d77300205e485be6595d9cd9276c246f3f89b7 (commit)
       via  af78d389b0ad37858108b69fe5100443dc88eb4a (commit)
      from  93c92875aaebe5b06f8dbfe2822b59a772895c08 (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 4ccbea9ef440a7e4252b0df5e710dcb767831c60
Author: Lucas Di Pentima <lucas at curoverse.com>
Date:   Tue Apr 25 00:50:31 2017 -0300

    10111: Tests added

diff --git a/apps/workbench/test/controllers/container_requests_controller_test.rb b/apps/workbench/test/controllers/container_requests_controller_test.rb
index 70e042c..6f5a6da 100644
--- a/apps/workbench/test/controllers/container_requests_controller_test.rb
+++ b/apps/workbench/test/controllers/container_requests_controller_test.rb
@@ -56,4 +56,25 @@ class ContainerRequestsControllerTest < ActionController::TestCase
     assert_equal completed_cr['cmd'], copied_cr['cmd']
     assert_equal completed_cr['runtime_constraints']['ram'], copied_cr['runtime_constraints'][:ram]
   end
+
+  [
+    ['completed', true],
+    ['running', true],
+    ['queued', true],
+    ['uncommitted', false],
+  ].each do |cr_fixture, should_show|
+    test "provenance tab should #{should_show ? '' : 'not'} be shown on #{cr_fixture} container requests" do
+      cr = api_fixture('container_requests')[cr_fixture]
+      assert_not_nil cr
+      get(:show,
+          {id: cr['uuid']},
+          session_for(:active))
+      assert_response :success
+      if should_show
+        assert_includes @response.body, "href=\"#Provenance\""
+      else
+        assert_not_includes @response.body, "href=\"#Provenance\""
+      end
+    end
+  end
 end
diff --git a/apps/workbench/test/integration/container_requests_test.rb b/apps/workbench/test/integration/container_requests_test.rb
index bd3a813..46f7e17 100644
--- a/apps/workbench/test/integration/container_requests_test.rb
+++ b/apps/workbench/test/integration/container_requests_test.rb
@@ -109,4 +109,16 @@ class ContainerRequestsTest < ActionDispatch::IntegrationTest
     assert_text 'This workflow does not need any further inputs'
     page.assert_selector 'a', text: 'Run'
   end
+
+  test "Provenance graph shown on committed container requests" do
+    cr = api_fixture('container_requests', 'completed')
+    visit page_with_token("active", "/container_requests/#{cr['uuid']}")
+    assert page.has_text? 'Provenance'
+    click_link 'Provenance'
+    wait_for_ajax
+    # Check for provenance graph existance
+    page.assert_selector '#provenance_svg'
+    page.assert_selector 'ellipse+text', text: cr['name'], visible: false
+    page.assert_selector 'g.node>title', text: cr['uuid'], visible: false
+  end
 end

commit c043e133b2646037ed630d571e91dbf77344f855
Merge: 93c9287 bf5d77b
Author: Lucas Di Pentima <lucas at curoverse.com>
Date:   Mon Apr 24 17:07:19 2017 -0300

    10111: Merge branch 'master' into 10111-cr-provenance-graph


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


hooks/post-receive
-- 




More information about the arvados-commits mailing list