From b99a2eac6fd76cd58276acbcdcc74c09ec941020 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Mar=C3=ADa=20Checa?= <MariaCheca@users.noreply.github.com>
Date: Tue, 19 Dec 2017 19:10:43 +0100
Subject: [PATCH] Fixed tests

---
 app/views/relationable/_related_list.html.erb | 2 +-
 spec/shared/features/relationable.rb          | 4 ++++
 2 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/app/views/relationable/_related_list.html.erb b/app/views/relationable/_related_list.html.erb
index 606819624..860790715 100644
--- a/app/views/relationable/_related_list.html.erb
+++ b/app/views/relationable/_related_list.html.erb
@@ -1,6 +1,6 @@
 <ul class="related-content-list" id="related-content-list">
   <% @related_contents.each do |related| %>
-    <li>
+    <li id="related-content-<%= related.relate_content(relationable).id %>">
       <% if current_user %>
         <span id="<%= dom_id(related.relate_content(relationable)) %>" class="js-score-actions score-actions">
           <%= render 'relationable/score', related: related.relate_content(relationable) %>
diff --git a/spec/shared/features/relationable.rb b/spec/shared/features/relationable.rb
index f17544b38..6ff423445 100644
--- a/spec/shared/features/relationable.rb
+++ b/spec/shared/features/relationable.rb
@@ -25,6 +25,7 @@ shared_examples "relationable" do |relationable_model_name|
   end
 
   scenario 'related contents can be added' do
+    login_as(user)
     visit eval("#{relationable.class.name.downcase}_path(relationable)")
 
     expect(page).to have_selector('#related_content', visible: false)
@@ -57,6 +58,7 @@ shared_examples "relationable" do |relationable_model_name|
   end
 
   scenario 'if related content URL is invalid returns error' do
+    login_as(user)
     visit eval("#{relationable.class.name.downcase}_path(relationable)")
 
     click_on("Add related content")
@@ -76,6 +78,7 @@ shared_examples "relationable" do |relationable_model_name|
     visit eval("#{relationable.class.name.downcase}_path(relationable)")
 
     within("#related-content-list") do
+      find("#related-content-#{related_content.opposite_related_content.id}").hover
       find("#score-positive-related-#{related_content.opposite_related_content.id}").click
       expect(page).to_not have_css("#score-positive-related-#{related_content.opposite_related_content.id}")
     end
@@ -91,6 +94,7 @@ shared_examples "relationable" do |relationable_model_name|
     visit eval("#{relationable.class.name.downcase}_path(relationable)")
 
     within("#related-content-list") do
+      find("#related-content-#{related_content.opposite_related_content.id}").hover
       find("#score-negative-related-#{related_content.opposite_related_content.id}").click
       expect(page).to_not have_css("#score-negative-related-#{related_content.opposite_related_content.id}")
     end