Merge pull request #4161 from akostadinov/cuces-screenshots

Fix screenshots on failing cucumbers tests

Author: akostadinov

features/support/hooks.rb

@@ -1,5 +1,11 @@
 # frozen_string_literal: true
 
+# IMPORTANT: make sure this hook is defined **before** the one that saves screenshots,
+# because the After hooks run in the reverse order in which they are defined.
+After("@javascript") do
+  Capybara.page&.driver&.quit
+end
+
 Around '@security' do |scenario, block|
   with_forgery_protection(&block)
 end
@@ -200,10 +206,6 @@
   end
 end
 
-After("@javascript") do
-  Capybara.page&.driver&.quit
-end
-
 After do |scenario|
   if ENV['FAIL_FAST']
     Cucumber.wants_to_quit = true if scenario.failed?