Commit 9910968f authored by Grzegorz Bizon's avatar Grzegorz Bizon

Fix QA selector class used to click on QA elements

parent fc9ecdbb
...@@ -6,8 +6,8 @@ ...@@ -6,8 +6,8 @@
.dropdown-menu.projects-dropdown-menu .dropdown-menu.projects-dropdown-menu
= render "layouts/nav/projects_dropdown/show" = render "layouts/nav/projects_dropdown/show"
= nav_link(controller: ['dashboard/groups', 'explore/groups'], html_options: { class: "hidden-xs qa-groups-link" }) do = nav_link(controller: ['dashboard/groups', 'explore/groups'], html_options: { class: "hidden-xs" }) do
= link_to dashboard_groups_path, class: 'dashboard-shortcuts-groups', title: 'Groups' do = link_to dashboard_groups_path, class: 'dashboard-shortcuts-groups qa-groups-link', title: 'Groups' do
Groups Groups
= nav_link(path: 'dashboard#activity', html_options: { class: "visible-lg" }) do = nav_link(path: 'dashboard#activity', html_options: { class: "visible-lg" }) do
......
...@@ -41,7 +41,7 @@ module QA ...@@ -41,7 +41,7 @@ module QA
end end
def click_element(name) def click_element(name)
find(Page::Element.new(name).selector).click find(Page::Element.new(name).selector_css).click
end end
def self.path def self.path
......
...@@ -12,6 +12,10 @@ module QA ...@@ -12,6 +12,10 @@ module QA
"qa-#{@name.to_s.tr('_', '-')}" "qa-#{@name.to_s.tr('_', '-')}"
end end
def selector_css
".#{selector}"
end
def expression def expression
if @pattern.is_a?(String) if @pattern.is_a?(String)
@_regexp ||= Regexp.new(Regexp.escape(@pattern)) @_regexp ||= Regexp.new(Regexp.escape(@pattern))
......
describe QA::Page::Element do describe QA::Page::Element do
describe '#selector' do describe '#selector' do
it 'transform element name into QA-specific selector' do it 'transforms element name into QA-specific selector' do
expect(described_class.new(:sign_in_button).selector) expect(described_class.new(:sign_in_button).selector)
.to eq 'qa-sign-in-button' .to eq 'qa-sign-in-button'
end end
end end
describe '#selector_css' do
it 'transforms element name into QA-specific clickable css selector' do
expect(described_class.new(:sign_in_button).selector_css)
.to eq '.qa-sign-in-button'
end
end
context 'when pattern is an expression' do context 'when pattern is an expression' do
subject { described_class.new(:something, /button 'Sign in'/) } subject { described_class.new(:something, /button 'Sign in'/) }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment