Fix failing features

We now have two 'Path' links within .filters, so we need to disambiguate them in the step definition.
alphagov · May 6, 2014 · 5677432 · 5677432
1 parent 801ab87
commit 5677432
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/app/views/mappings/filter/_by_path_menu.html.erb b/app/views/mappings/filter/_by_path_menu.html.erb
@@ -1,4 +1,4 @@
-<li class="if-no-js-hide">
+<li class="if-no-js-hide filter-by-path">
   <% if @path_contains.present? %>
     <%= filter_remove_option_link(@site, 'Path', :path_contains) %>
   <% else %>

diff --git a/features/step_definitions/mappings_interaction_steps.rb b/features/step_definitions/mappings_interaction_steps.rb
@@ -59,7 +59,7 @@
 end
 
 When(/^I open the "(.*)" filter and filter by "(.*)"$/) do |filter_type, value|
-  within '.filters' do
+  within ".filters#{ ' .filter-by-path' if filter_type == 'Path' }" do
     click_link filter_type
     fill_in filter_type, with: value
     click_button 'Filter'
@@ -84,7 +84,9 @@
 end
 
 When(/^I remove the filter "(.*?)"$/) do |filter_type|
-  click_link filter_type
+  within ".filters#{ ' .filter-by-path' if filter_type == 'Path' }" do
+    click_link filter_type
+  end
 end
 
 When(/^I click the tag filter "(.*?)"$/) do |tag_filter|