Skip to content

Commit 1d11e23

Browse files
committed
add a test for rendering page header without breadcrumbs
1 parent c81040e commit 1d11e23

1 file changed

Lines changed: 18 additions & 1 deletion

File tree

test/components/primer/open_project/page_header_test.rb

Lines changed: 18 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ def test_raises_if_no_title_provided
1010
render_inline(Primer::OpenProject::PageHeader.new)
1111
end
1212

13-
assert_equal("PageHeader needs a title. Please use the `with_title`` slot", err.message)
13+
assert_equal("PageHeader needs a title. Please use the `with_title` slot", err.message)
1414
end
1515

1616
def test_renders_title
@@ -279,6 +279,23 @@ def test_skips_a_breadcrumb_item_for_mobile
279279
assert_selector("nav[aria-label='Breadcrumb'].PageHeader-breadcrumbs .breadcrumb-item.text-bold a[href='#']")
280280
end
281281

282+
def test_renders_without_breadcrumbs
283+
render_inline(Primer::OpenProject::PageHeader.new) do |header|
284+
header.with_title { "Hello" }
285+
end
286+
287+
# Title is rendered
288+
assert_text("Hello")
289+
assert_selector(".PageHeader-title")
290+
291+
# Breadcrumbs and parent link are not rendered
292+
assert_no_selector(".PageHeader-breadcrumbs")
293+
assert_no_selector(".PageHeader-parentLink")
294+
295+
# Context bar should not be rendered when no breadcrumbs or parent link exist
296+
assert_no_selector(".PageHeader-contextBar")
297+
end
298+
282299
private
283300

284301
def breadcrumb_elements

0 commit comments

Comments
 (0)