From 4f314bf6ff62b419ef1a38f32117dcf8df224941 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Javi=20Mart=C3=ADn?= Date: Thu, 23 Sep 2021 17:09:55 +0200 Subject: [PATCH] Add missing expectations in investments test We ended the test with two requests and no expectations. Debugging shows sometimes the test was hanging forever [1], probably due to simultaenous requests. So now we're adding expectations after each request. [1] https://github.com/consul/consul/runs/3687898744 --- spec/system/budgets/investments_spec.rb | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/spec/system/budgets/investments_spec.rb b/spec/system/budgets/investments_spec.rb index cc81b9f61..ddc666256 100644 --- a/spec/system/budgets/investments_spec.rb +++ b/spec/system/budgets/investments_spec.rb @@ -1128,7 +1128,12 @@ describe "Budget Investments" do accept_confirm { click_link("Delete") } end + expect(page).to have_content "Investment project deleted succesfully" + visit user_path(user, tab: :budget_investments) + + expect(page).to have_content "User has no public activity" + expect(page).not_to have_content investment1.title end end