Clarify specs

rockandror · Sep 27, 2019 · 559456a · 559456a
1 parent 3bf2edf
commit 559456a
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 8 deletions.
diff --git a/app/views/admin/settings/_featured_settings_table.html.erb b/app/views/admin/settings/_featured_settings_table.html.erb
@@ -8,7 +8,7 @@
   </thead>
   <tbody>
     <% features.each do |feature| %>
-      <tr>
+      <tr class="setting">
         <td class="small-8">
           <strong><%= t("settings.#{feature.key}") %></strong>
           <br>

diff --git a/app/views/admin/settings/_mixed_settings_table.html.erb b/app/views/admin/settings/_mixed_settings_table.html.erb
@@ -8,7 +8,7 @@
   </thead>
   <tbody>
     <% settings.each do |setting| %>
-      <tr>
+      <tr class="setting">
         <td class="small-6">
           <strong><%= t("settings.#{setting.key}") %></strong>
           <br>

diff --git a/app/views/admin/settings/_settings_table.html.erb b/app/views/admin/settings/_settings_table.html.erb
@@ -7,7 +7,7 @@
   </thead>
   <tbody>
     <% settings.each do |setting| %>
-      <tr>
+      <tr class="setting">
         <td class="small-6">
           <strong><%= t("settings.#{setting.key}") %></strong>
           <br>

diff --git a/spec/features/admin/wizards/installer_spec.rb b/spec/features/admin/wizards/installer_spec.rb
@@ -28,7 +28,7 @@
       expect(page).to have_link("Back", href: new_admin_wizards_installer_path)
       expect(page).to have_link("Next", href: admin_wizards_installer_path(:participation_process))
       within "#installer-general-settings" do
-        expect(page).to have_css("table tbody tr", count: 2)
+        expect(page).to have_css(".setting", count: 2)
         expect(page).to have_content "Site name"
         expect(page).to have_content "Minimum age needed to participate"
       end
@@ -44,7 +44,7 @@
       expect(page).to have_link("Back", href: admin_wizards_installer_path(:general_settings))
       expect(page).to have_link("Next", href: admin_wizards_installer_path(:map))
       within "#installer-participation-process" do
-        expect(page).to have_css("table tbody tr", count: 5)
+        expect(page).to have_css(".setting", count: 5)
         expect(page).to have_content "Debates"
         expect(page).to have_content "Proposals"
         expect(page).to have_content "Number of supports necessary for approval of a Proposal"
@@ -63,7 +63,7 @@
       expect(page).to have_link("Back", href: admin_wizards_installer_path(:participation_process))
       expect(page).to have_link("Next", href: admin_wizards_installer_path(:smtp))
       within "#installer-map" do
-        expect(page).to have_css("table tbody tr", count: 4)
+        expect(page).to have_css(".setting", count: 4)
         expect(page).to have_content "Proposals and budget investments geolocation"
         expect(page).to have_content "Latitude"
         expect(page).to have_content "Longitude"
@@ -81,7 +81,7 @@
       expect(page).to have_link("Back", href: admin_wizards_installer_path(:map))
       expect(page).to have_link("Next", href: admin_wizards_installer_path(:regional))
       within "#installer-smtp" do
-        expect(page).to have_css("table tbody tr", count: 8)
+        expect(page).to have_css(".setting", count: 8)
         expect(page).to have_content "SMTP Configuration"
         expect(page).to have_content "SMTP Host"
         expect(page).to have_content "SMTP Port"
@@ -103,7 +103,7 @@
       expect(page).to have_link("Back", href: admin_wizards_installer_path(:smtp))
       expect(page).to have_link("Next", href: admin_wizards_installer_path(:finish))
       within "#installer-regional" do
-        expect(page).to have_css("table tbody tr", count: 29)
+        expect(page).to have_css(".setting", count: 29)
         expect(page).to have_content "Application default locale"
         expect(page).to have_content "Application available locales"
         expect(page).to have_content "Time Zone"