From f50d4fdee0528d1c600aec20828296e135b3d031 Mon Sep 17 00:00:00 2001
From: Matt-Yorkley <9029026+Matt-Yorkley@users.noreply.github.com>
Date: Tue, 3 Aug 2021 13:35:27 +0100
Subject: [PATCH] Update enterprises spec

The bug mentioned here is no longer present (two alert boxes opened instead of one), and the old hack for Poltergeist is no longer needed. :tada:
---
 spec/features/admin/enterprises_spec.rb | 10 ++--------
 1 file changed, 2 insertions(+), 8 deletions(-)

diff --git a/spec/features/admin/enterprises_spec.rb b/spec/features/admin/enterprises_spec.rb
index 0a9e0e2445..360a751e0a 100644
--- a/spec/features/admin/enterprises_spec.rb
+++ b/spec/features/admin/enterprises_spec.rb
@@ -424,24 +424,18 @@ feature '
 
       fill_in 'enterprise_producer_properties_attributes_0_value', with: "NASAA 12345"
 
-      # Because poltergist does not support form onchange event
-      # We need trigger the change manually
-      page.evaluate_script("angular.element(enterprise_form).scope().setFormDirty()")
       click_button 'Update'
 
       expect(supplier1.producer_properties.reload.count).to eq(1)
 
       # -- Destroy
-      pp = supplier1.producer_properties.first
       within(".side_menu") do
         click_link "Properties"
       end
 
-      # Bug: https://github.com/openfoodfoundation/openfoodnetwork/issues/2453
       accept_alert do
-        accept_alert do
-          within("#spree_producer_property_#{pp.id}") { page.find('a.remove_fields').click }
-        end
+        property = supplier1.producer_properties.first
+        within("#spree_producer_property_#{property.id}") { page.find('a.remove_fields').click }
       end
 
       click_button 'Update'