Reenable notifications feature, don't use javascript in step definitions if you can avoid it?

2 files changed, 6 insertions, 6 deletions

diff --git a/features/notifications.feature b/features/notifications.feature
index 0d8743961..ad63a9754 100644
--- a/features/notifications.feature
+++ b/features/notifications.feature
@@ -23,7 +23,7 @@ Background:
   Then I should see "started sharing with you"
 
   Scenario: notification popup
-    #When I sign in as "alice@alice.alice"
-    #And I click the notification badge
-    #And I wait for the ajax to finish
-    #Then the notification dropdown should be visible
+    When I sign in as "alice@alice.alice"
+    And I click the notification badge
+    And I wait for the ajax to finish
+    Then the notification dropdown should be visible
diff --git a/features/step_definitions/custom_web_steps.rb b/features/step_definitions/custom_web_steps.rb
index 55f3ea3bb..135776487 100644
--- a/features/step_definitions/custom_web_steps.rb
+++ b/features/step_definitions/custom_web_steps.rb
@@ -188,9 +188,9 @@ When /^I wait for (\d+) seconds$/ do |seconds|
 end
 
 When /^I click the notification badge$/ do
-  evaluate_script("$('#notification_badge a').click();");
+  find(:css, "#notification_badge a").click
 end
 
 Then /^the notification dropdown should be visible$/ do
-  evaluate_script("$('#notification_dropdown').css('display') === 'block'")
+  find(:css, "#notification_dropdown").should be_visible
 end