From c2526c6111ba0f78d8478997d1bf129e37a9352c Mon Sep 17 00:00:00 2001
From: Benjamin Neff <benjamin@coding4coffee.ch>
Date: Fri, 11 Aug 2017 21:25:00 +0200
Subject: [PATCH] Don't wait for .loading class, it's sometimes already removed
again
---
features/desktop/notifications.feature | 1 -
features/step_definitions/notifications_steps.rb | 4 ----
2 files changed, 5 deletions(-)
diff --git a/features/desktop/notifications.feature b/features/desktop/notifications.feature
index 13fc31a6b1..7726d3d870 100644
--- a/features/desktop/notifications.feature
+++ b/features/desktop/notifications.feature
@@ -177,6 +177,5 @@ Feature: Notifications
When I wait for notifications to load
Then there should be 10 notifications loaded
When I scroll down on the notifications dropdown
- Then the notification dropdown should load more notifications
When I wait for notifications to load
Then there should be 15 notifications loaded
diff --git a/features/step_definitions/notifications_steps.rb b/features/step_definitions/notifications_steps.rb
index 65aa57ad0e..bd38c03f00 100644
--- a/features/step_definitions/notifications_steps.rb
+++ b/features/step_definitions/notifications_steps.rb
@@ -25,10 +25,6 @@ And "I scroll down on the notifications dropdown" do
page.execute_script("$('.notifications').scrollTop(350)")
end
-Then "the notification dropdown should load more notifications" do
- expect(find("#notification-dropdown")).to have_css(".loading")
-end
-
Then "the notification dropdown should be visible" do
expect(find(:css, "#notification-dropdown")).to be_visible
end
--
GitLab