Update the windows_firewall_profile resource to move the load_firewall_state method out of the action_class. Also add some firewall tests to the kitchen tests to avoid this issue in the future.

Signed-off-by: Davin Taddeo <davin@chef.io>

1 files changed, 10 insertions, 0 deletions

diff --git a/kitchen-tests/cookbooks/end_to_end/recipes/windows.rb b/kitchen-tests/cookbooks/end_to_end/recipes/windows.rb
index 3e069a43b0..33fc16e38a 100644
--- a/kitchen-tests/cookbooks/end_to_end/recipes/windows.rb
+++ b/kitchen-tests/cookbooks/end_to_end/recipes/windows.rb
@@ -32,6 +32,16 @@ windows_security_policy "EnableGuestAccount" do
   secvalue "1"
 end
 
+windows_firewall_profile "Domain" do
+  default_inbound_action "Allow"
+  default_outbound_action "Allow"
+  action :enable
+end
+
+windows_firewall_profile "Public" do
+  action :disable
+end
+
 users_manage "remove sysadmin" do
   group_name "sysadmin"
   group_id 2300