Skip to content
Snippets Groups Projects
Unverified Commit e3bbf853 authored by Daniel Davison's avatar Daniel Davison Committed by GitLab
Browse files

Merge branch 'jmd/e2e-expand-breadbcrumbs-for-text-check' into 'master'

E2E: Always expand breadcrumbs prior to checking for text

See merge request https://gitlab.com/gitlab-org/gitlab/-/merge_requests/168412



Merged-by: default avatarDan Davison <ddavison@gitlab.com>
Approved-by: default avatarSofia Vistas <svistas@gitlab.com>
Approved-by: default avatarDan Davison <ddavison@gitlab.com>
Co-authored-by: default avatarJohn McDonnell <jmcdonnell@gitlab.com>
parents 040a0613 03b02dab
No related branches found
No related tags found
No related merge requests found
Loading
Loading
@@ -15,6 +15,12 @@ def self.included(base)
end
 
def has_breadcrumb?(text)
# In some situations the length of the breadcrumbs may lead to it being condensed. For these situations
# open the dropdown toggle which should allow us to see the all components of the breadcrumb.
if has_no_element?('breadcrumb-links', text: text, wait: 0)
within_element('breadcrumb-links') { click_element('base-dropdown-toggle') }
end
has_element?('breadcrumb-links', text: text)
end
end
Loading
Loading
Loading
Loading
@@ -31,11 +31,7 @@ module QA
Page::Project::Menu.perform(&:click_project)
 
Page::Project::Show.perform do |project|
# TODO breadcrumb test should be updated.
# I'm commenting it out for now. I will create a follow-up issue to fix this.
# https://gitlab.com/gitlab-org/gitlab/-/issues/497758#note_2145018495
# expect(project).to have_breadcrumb(target_group.path)
expect(project).to have_breadcrumb(target_group.path)
expect(project).to have_readme_content(readme_content)
end
end
Loading
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment