Added new test to cookies spec to cover bug on cookies banner and fixed it. See issue #2599.

Also, improved readability on cookies spec with some extracted methods.

app/assets/javascripts/darkswarm/cookies_banner/cookies_banner_controller.js.coffee

@@ -3,3 +3,4 @@ Darkswarm.controller "CookiesBannerCtrl", ($scope, CookiesBannerService, $http,
   $scope.acceptCookies = ->
     $http.post('/api/cookies/consent')
     CookiesBannerService.close()
+    CookiesBannerService.disable()

app/assets/javascripts/darkswarm/cookies_banner/cookies_banner_directive.js.coffee

@@ -2,5 +2,5 @@ Darkswarm.directive 'cookiesBanner', (CookiesBannerService) ->
   restrict: 'A'
   link: (scope, elm, attr)->
     return if not attr.cookiesBanner? || attr.cookiesBanner == 'false'
-    CookiesBannerService.setActive()
+    CookiesBannerService.enable()
     CookiesBannerService.open()

app/assets/javascripts/darkswarm/cookies_banner/cookies_banner_service.js.coffee

@@ -2,10 +2,10 @@ Darkswarm.factory "CookiesBannerService", (Navigation, $modal, $location, Redire
 
   new class CookiesBannerService
     modalMessage: null
-    isActive: false
+    isEnabled: false
 
     open: (path, template = 'darkswarm/cookies_banner/cookies_banner.html') =>
-      return unless @isActive
+      return unless @isEnabled
       @modalInstance = $modal.open
         templateUrl: template
         windowClass: "cookies-banner full"
@@ -13,8 +13,11 @@ Darkswarm.factory "CookiesBannerService", (Navigation, $modal, $location, Redire
         keyboard: false
 
     close: =>
-      return unless @isActive
+      return unless @isEnabled
       @modalInstance.close()
 
-    setActive: =>
-      @isActive = true
+    enable: =>
+      @isEnabled = true
+
+    disable: =>
+      @isEnabled = false

spec/features/consumer/cookies_spec.rb

@@ -5,30 +5,37 @@ feature "Cookies", js: true do
     describe "in the homepage" do
       before do
         Spree::Config[:cookies_consent_banner_toggle] = true
-        visit root_path
+        visit_root_path_and_wait
       end
 
       scenario "does not show after cookies are accepted" do
-        sleep 1
-        click_button I18n.t('legal.cookies_banner.cookies_accept_button')
-        sleep 2
+        accept_cookies_and_wait
         expect_not_visible_cookies_banner
 
-        visit root_path
-        sleep 1
+        visit_root_path_and_wait
         expect_not_visible_cookies_banner
       end
 
       scenario "banner contains cookies policy link that opens coookies policy page and closes banner" do
-        sleep 1
-        find("p.ng-binding > a", :text => "cookies policy").click
-        sleep 2
+        click_banner_cookies_policy_link_and_wait
         expect_visible_cookies_policy_page
         expect_not_visible_cookies_banner
 
-        find("a.close-reveal-modal").click
+        close_cookies_policy_page_and_wait
         expect_visible_cookies_banner
       end
+
+      scenario "does not show after cookies are accepted, and policy page is opened through the footer, and closed again (bug #2599)" do
+        accept_cookies_and_wait
+        expect_not_visible_cookies_banner
+        
+        click_footer_cookies_policy_link_and_wait
+        expect_visible_cookies_policy_page
+        expect_not_visible_cookies_banner
+
+        close_cookies_policy_page_and_wait
+        expect_not_visible_cookies_banner
+      end
     end
 
     describe "in product listing page" do
@@ -94,4 +101,29 @@ feature "Cookies", js: true do
   def accept_cookies_button_text
     I18n.t('legal.cookies_banner.cookies_accept_button')
   end
+
+  def visit_root_path_and_wait
+    visit root_path
+    sleep 1
+  end
+
+  def accept_cookies_and_wait
+    click_button accept_cookies_button_text
+    sleep 2
+  end
+
+  def click_banner_cookies_policy_link_and_wait
+    find("p.ng-binding > a", :text => "cookies policy").click
+    sleep 2
+  end
+
+  def click_footer_cookies_policy_link_and_wait
+    find("div > a", :text => "cookies policy").click
+    sleep 2
+  end
+
+  def close_cookies_policy_page_and_wait
+    find("a.close-reveal-modal").click
+    sleep 2
+  end
 end