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

Revert "Adds .js-class for specs"

Browse Source 
The `js-` prefix (which I admit I'm not fond of) is usually used to
indicate the class is used by JavaScript files, not for using it in test
files. And in all the other similar tests, we're using the `in-favor`
class instead of the `js-in-favor` class.

This reverts commit 83fe74d53.
This commit is contained in:
Javi Martín
2021-06-12 03:09:21 +02:00
parent f437ca988d
commit e4ad70fd82
2 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
spec/system/management/budget_investments_spec.rb
View File

@@ -277,7 +277,7 @@ describe "Budget Investments" do
expect(page).to have_content(budget_investment.title)
within("#budget-investments") do
find(".js-in-favor a").click
find("in-favor a").click
expect(page).to have_content "1 support"
expect(page).to have_content "You have already supported this investment project. Share it!"
@@ -302,7 +302,7 @@ describe "Budget Investments" do
expect(page).to have_css "h1", exact_text: budget_investment.title
find(".js-in-favor a").click
find(".in-favor a").click
expect(page).to have_content "1 support"
expect(page).to have_content "You have already supported this investment project. Share it!"
@@ -325,7 +325,7 @@ describe "Budget Investments" do
expect(page).to have_css "h1", exact_text: "Default heading investment"
accept_confirm { find(".js-in-favor a").click }
accept_confirm { find(".in-favor a").click }
expect(page).to have_content "1 support"
expect(page).to have_content "You have already supported this investment project. Share it!"