[ARVADOS] updated: c6452ff3cf71462a46c3d7584696e0948dcce4dc

[Git user]

Summary of changes:
 .../_extra_tab_line_buttons.html.erb                  | 16 ++++++++--------
 .../controllers/container_requests_controller_test.rb |  5 +++--
 services/api/test/fixtures/container_requests.yml     | 19 -------------------
 3 files changed, 11 insertions(+), 29 deletions(-)

       via  c6452ff3cf71462a46c3d7584696e0948dcce4dc (commit)
      from  d694a717acb2e577afa396aea140e5284b7f763d (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 c6452ff3cf71462a46c3d7584696e0948dcce4dc
Author: Lucas Di Pentima <lucas at curoverse.com>
Date:   Mon May 1 16:58:48 2017 -0300

    11185: Removed newly added completed-acr fixture and modified the already existing completed-older CR
    to be able to use it on the tests.
    Change the reset_form() JS function to something more specific to this usage, to avoid future
    conflicts.

diff --git a/apps/workbench/app/views/container_requests/_extra_tab_line_buttons.html.erb b/apps/workbench/app/views/container_requests/_extra_tab_line_buttons.html.erb
index 9cf03c8..049be75 100644
--- a/apps/workbench/app/views/container_requests/_extra_tab_line_buttons.html.erb
+++ b/apps/workbench/app/views/container_requests/_extra_tab_line_buttons.html.erb
@@ -1,16 +1,16 @@
 <% if @object.state == 'Final' %>
+<script type="application/javascript">
+  function reset_form_cr_reuse() {
+    $('#use_existing').removeAttr('checked');
+  }
+</script>
+
   <%= link_to raw('<i class="fa fa-fw fa-play"></i> Re-run...'),
       "#",
       {class: 'btn btn-sm btn-primary', 'data-toggle' => 'modal',
        'data-target' => '#clone-and-edit-modal-window',
        title: 'This will make a copy and take you there. You can then make any needed changes and run it'}  %>
 
-<script type="application/javascript">
-  function reset_form() {
-    $('#use_existing').removeAttr('checked');
-  }
-</script>
-
 <div id="clone-and-edit-modal-window" class="modal fade" role="dialog"
      aria-labelledby="myModalLabel" aria-hidden="true">
   <div class="modal-dialog">
@@ -19,7 +19,7 @@
     <%= form_tag copy_container_request_path do |f| %>
 
       <div class="modal-header">
-        <button type="button" class="close" onClick="reset_form()" data-dismiss="modal" aria-hidden="true">×</button>
+        <button type="button" class="close" onClick="reset_form_cr_reuse()" data-dismiss="modal" aria-hidden="true">×</button>
         <div>
           <div class="col-sm-6"> <h4 class="modal-title">Re-run container request</h4> </div>
         </div>
@@ -32,7 +32,7 @@
       </div>
 
       <div class="modal-footer">
-        <button class="btn btn-default" onClick="reset_form()" data-dismiss="modal" aria-hidden="true">Cancel</button>
+        <button class="btn btn-default" onClick="reset_form_cr_reuse()" data-dismiss="modal" aria-hidden="true">Cancel</button>
         <button type="submit" class="btn btn-primary" name="container_request[state]" value="Uncommitted">Copy and edit inputs</button>
       </div>
 
diff --git a/apps/workbench/test/controllers/container_requests_controller_test.rb b/apps/workbench/test/controllers/container_requests_controller_test.rb
index 804263c..01f740c 100644
--- a/apps/workbench/test/controllers/container_requests_controller_test.rb
+++ b/apps/workbench/test/controllers/container_requests_controller_test.rb
@@ -44,11 +44,12 @@ class ContainerRequestsControllerTest < ActionController::TestCase
   [
     ['completed', false, false],
     ['completed', true, false],
-    ['completed-acr', false, true],
-    ['completed-acr', true, true],
+    ['completed-older', false, true],
+    ['completed-older', true, true],
   ].each do |cr_fixture, reuse_enabled, uses_acr|
     test "container request #{uses_acr ? '' : 'not'} using arvados-cwl-runner copy #{reuse_enabled ? 'with' : 'without'} reuse enabled" do
       completed_cr = api_fixture('container_requests')[cr_fixture]
+      # Set up post request params
       copy_params = {id: completed_cr['uuid']}
       if reuse_enabled
         copy_params.merge!({use_existing: true})
diff --git a/services/api/test/fixtures/container_requests.yml b/services/api/test/fixtures/container_requests.yml
index b35829a..d5bb47f 100644
--- a/services/api/test/fixtures/container_requests.yml
+++ b/services/api/test/fixtures/container_requests.yml
@@ -109,25 +109,6 @@ completed-older:
   container_image: test
   cwd: test
   output_path: test
-  command: ["echo", "hello"]
-  container_uuid: zzzzz-dz642-compltcontainr2
-  runtime_constraints:
-    vcpus: 1
-    ram: 123
-
-completed-acr:
-  uuid: zzzzz-xvhdp-cr4completedacr
-  owner_uuid: zzzzz-tpzed-xurymjxw79nv3jz
-  name: completed
-  state: Final
-  priority: 1
-  created_at: 2016-01-11 11:11:11.111111111 Z
-  updated_at: 2016-01-11 11:11:11.111111111 Z
-  modified_at: 2016-01-11 11:11:11.111111111 Z
-  modified_by_user_uuid: zzzzz-tpzed-xurymjxw79nv3jz
-  container_image: test
-  cwd: test
-  output_path: test
   command: ["arvados-cwl-runner", "echo", "hello"]
   container_uuid: zzzzz-dz642-compltcontainr2
   runtime_constraints:

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


hooks/post-receive
--