[ARVADOS] updated: b5fd5a6835ccf930ca78f9b6376bd90d546ecaee

git at public.curoverse.com git at public.curoverse.com
Tue Sep 30 22:48:18 EDT 2014


Summary of changes:
 .../test/integration/pipeline_instances_test.rb    | 36 +++++++++-------------
 1 file changed, 15 insertions(+), 21 deletions(-)

       via  b5fd5a6835ccf930ca78f9b6376bd90d546ecaee (commit)
      from  92d5df330b86fb4c279a6e030258b860bed37358 (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 b5fd5a6835ccf930ca78f9b6376bd90d546ecaee
Author: Peter Amstutz <peter.amstutz at curoverse.com>
Date:   Tue Sep 30 22:48:13 2014 -0400

    4004: Refactor common "run pipeline" test code into method

diff --git a/apps/workbench/test/integration/pipeline_instances_test.rb b/apps/workbench/test/integration/pipeline_instances_test.rb
index 3e8663d..898dc8f 100644
--- a/apps/workbench/test/integration/pipeline_instances_test.rb
+++ b/apps/workbench/test/integration/pipeline_instances_test.rb
@@ -88,6 +88,19 @@ class PipelineInstancesTest < ActionDispatch::IntegrationTest
     assert_not page.has_text? 'Graph'
   end
 
+  def test_run_pipeline
+    # "Run" button present and enabled
+    page.assert_no_selector 'a.disabled,button.disabled', text: 'Run'
+    first('a,button', text: 'Run').click
+
+    # Pipeline is running. We have a "Pause" button instead now.
+    page.assert_no_selector 'a,button', text: 'Run'
+    page.assert_selector 'a,button', text: 'Pause'
+
+    # Since it is test env, no jobs are created to run. So, graph not visible
+    assert_not page.has_text? 'Graph'
+  end
+
   # Create a pipeline instance from within a project and run
   test 'Create pipeline inside a project and run' do
     visit page_with_token('active_trustedclient')
@@ -134,16 +147,7 @@ class PipelineInstancesTest < ActionDispatch::IntegrationTest
     end
     wait_for_ajax
 
-    # "Run" button present and enabled
-    page.assert_no_selector 'a.disabled,button.disabled', text: 'Run'
-    first('a,button', text: 'Run').click
-
-    # Pipeline is running. We have a "Pause" button instead now.
-    page.assert_no_selector 'a,button', text: 'Run'
-    page.assert_selector 'a,button', text: 'Pause'
-
-    # Since it is test env, no jobs are created to run. So, graph not visible
-    assert_not page.has_text? 'Graph'
+    test_run_pipeline
   end
 
   # Create a pipeline instance from within a project and run
@@ -174,19 +178,9 @@ class PipelineInstancesTest < ActionDispatch::IntegrationTest
     end
     wait_for_ajax
 
-    # "Run" button present and enabled
-    page.assert_no_selector 'a.disabled,button.disabled', text: 'Run'
-    first('a,button', text: 'Run').click
-
-    # Pipeline is running. We have a "Pause" button instead now.
-    page.assert_no_selector 'a,button', text: 'Run'
-    page.assert_selector 'a,button', text: 'Pause'
-
-    # Since it is test env, no jobs are created to run. So, graph not visible
-    assert_not page.has_text? 'Graph'
+    test_run_pipeline
   end
 
-
   test 'view pipeline with job and see graph' do
     visit page_with_token('active_trustedclient')
 

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list