From: Tom Clegg <tom@curoverse.com>
Date: Wed, 5 Jul 2017 21:34:11 +0000 (-0400)
Subject: 11927: Eliminate unnecessary use of selenium.
X-Git-Tag: 1.1.0~153^2
X-Git-Url: https://git.arvados.org/arvados.git/commitdiff_plain/e914c48b16e6bb8199ebad89096a09e477a7efe8

11927: Eliminate unnecessary use of selenium.

This test case works without selenium, if we don't try to click an
"edit" link that's disabled because we already clicked it.

Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tom@curoverse.com>
---

diff --git a/apps/workbench/test/integration/collections_test.rb b/apps/workbench/test/integration/collections_test.rb
index af8e1313f8..8619858dfe 100644
--- a/apps/workbench/test/integration/collections_test.rb
+++ b/apps/workbench/test/integration/collections_test.rb
@@ -426,8 +426,6 @@ class CollectionsTest < ActionDispatch::IntegrationTest
   end
 
   test "collection tags tab" do
-    need_selenium
-
     visit page_with_token('active', '/collections/zzzzz-4zz18-bv31uwvy3neko21')
 
     click_link 'Tags'
@@ -447,8 +445,6 @@ class CollectionsTest < ActionDispatch::IntegrationTest
     assert_selector 'a', text: 'Cancel'
 
     # add two tags
-    first('.edit-collection-tags').click
-
     first('.glyphicon-plus').click
     first('.collection-tag-field-key').click
     first('.collection-tag-field-key').set('key 1')