[ARVADOS] updated: b0001b6d784024885d8b90277e4a9b5a2955a9b4

git at public.curoverse.com git at public.curoverse.com
Thu Dec 4 11:30:44 EST 2014


Summary of changes:
 apps/workbench/test/integration/report_issue_test.rb | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

       via  b0001b6d784024885d8b90277e4a9b5a2955a9b4 (commit)
      from  b629c24e4dc15257ae8d70f63d037c12668a491a (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 b0001b6d784024885d8b90277e4a9b5a2955a9b4
Author: Tim Pierce <twp at curoverse.com>
Date:   Thu Dec 4 11:26:58 2014 -0500

    4465: test for regex link targets
    
    The goal of this story is that the "report issue" dialog includes links
    to a Github or Redmine page corresponding to the software versions for
    Workbench and the API server, so the test should ensure not just that
    there's a link with a given text, but that its target is a Github page
    corresponding to a hexadecimal commit hash.

diff --git a/apps/workbench/test/integration/report_issue_test.rb b/apps/workbench/test/integration/report_issue_test.rb
index 7eb65a2..fcbf4e7 100644
--- a/apps/workbench/test/integration/report_issue_test.rb
+++ b/apps/workbench/test/integration/report_issue_test.rb
@@ -1,11 +1,8 @@
 require 'integration_helper'
 require 'selenium-webdriver'
 require 'headless'
-require_relative '../../app/helpers/version_helper'
 
 class ReportIssueTest < ActionDispatch::IntegrationTest
-  include VersionHelper
-
   setup do
     headless = Headless.new
     headless.start
@@ -38,10 +35,12 @@ class ReportIssueTest < ActionDispatch::IntegrationTest
       assert page.has_text?('Version / debugging info'), 'No text - Version / debugging info'
       assert page.has_no_text?('Report a problem'), 'Found text - Report a problem'
       assert page.has_no_text?('Describe the problem?'), 'Found text - Describe the problem'
-      assert page.has_link?(api_version), 'No link for API version'
-      assert page.has_link?(wb_version_text), 'No link for Workbench version'
       assert page.has_button?('Close'), 'No button - Close'
       assert page.has_no_button?('Send problem report'), 'Found button - Send problem report'
+      api_version_rgx = %r!^https://github.com/curoverse/arvados/tree/[0-9a-f]+/services/api$!
+      wb_version_rgx = %r!^https://github.com/curoverse/arvados/tree/[0-9a-f]+/apps/workbench$!
+      assert_not_empty(all("a").select { |a| a[:href] =~ wb_version_rgx })
+      assert_not_empty(all("a").select { |a| a[:href] =~ api_version_rgx })
       click_button 'Close'
     end
 

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list