consul/nairobi
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix test checking link in budget header

Browse Source 
The test was passing because we were using `within`, but actually the
`have_css` method doesn't support the `href:` argument.
This commit is contained in:
Javi Martín
2021-10-17 15:57:12 +02:00
parent 57fcdc402d
commit d7f26f012d
1 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
spec/components/budgets/budget_component_spec.rb
View File

@@ -23,16 +23,16 @@ describe Budgets::BudgetComponent do
it "shows budget main link when defined" do it "shows budget main link when defined" do
render_inline Budgets::BudgetComponent.new(budget) render_inline Budgets::BudgetComponent.new(budget)
within(".budget-header") do page.find(".budget-header") do |header|
expect(page).not_to have_css("a.main-link") expect(header).not_to have_css ".main-link"
end end
budget.update!(main_link_text: "Partitipate now!", main_link_url: "https://consulproject.org") budget.update!(main_link_text: "Participate now!", main_link_url: "https://consulproject.org")
render_inline Budgets::BudgetComponent.new(budget) render_inline Budgets::BudgetComponent.new(budget)
within(".budget-header") do page.find(".budget-header") do |header|
expect(page).to have_css("a.main-link", text: "Participate now!", href: "https://consulproject.org") expect(header).to have_link "Participate now!", href: "https://consulproject.org", class: "main-link"
end end
end end
end end