unit test fix and chefstyle fixes

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

1 files changed, 10 insertions, 10 deletions

diff --git a/spec/unit/resource/windows_firewall_policy_spec.rb b/spec/unit/resource/windows_firewall_policy_spec.rb
index 7b548c950a..347234121c 100644
--- a/spec/unit/resource/windows_firewall_policy_spec.rb
+++ b/spec/unit/resource/windows_firewall_policy_spec.rb
@@ -36,39 +36,39 @@ describe Chef::Resource::WindowsFirewallProfile do
 
   it "the resource's default_inbound_action property only strings Block, Allow, or NotConfigured" do
     expect { resource.default_inbound_action "AllowSome" }.to raise_error(ArgumentError)
-    expect { resource.default_inbound_action "Block"}.not_to raise_error
+    expect { resource.default_inbound_action "Block" }.not_to raise_error
   end
   it "the resource's default_outbound_action property only accepts strings Block, Allow, or NotConfigured" do
     expect { resource.default_outbound_action "BlockMost" }.to raise_error(ArgumentError)
-    expect { resource.default_outbound_action "Allow"}.not_to raise_error
+    expect { resource.default_outbound_action "Allow" }.not_to raise_error
   end
   it "the resource's allow_inbound_rules property only accepts strings True, False, or NotConfigured" do
     expect { resource.allow_inbound_rules "Yes" }.to raise_error(ArgumentError)
-    expect { resource.allow_inbound_rules "True"}.not_to raise_error
+    expect { resource.allow_inbound_rules "True" }.not_to raise_error
   end
   it "the resource's allow_local_firewall_rules property only accepts strings True, False, or NotConfigured" do
     expect { resource.allow_local_firewall_rules "No" }.to raise_error(ArgumentError)
-    expect { resource.allow_local_firewall_rules "False"}.not_to raise_error
+    expect { resource.allow_local_firewall_rules "False" }.not_to raise_error
   end
   it "the resource's allow_local_ipsec_rules property only accepts strings True, False, or NotConfigured" do
     expect { resource.allow_local_ipsec_rules "Yes" }.to raise_error(ArgumentError)
-    expect { resource.allow_local_ipsec_rules "True"}.not_to raise_error
+    expect { resource.allow_local_ipsec_rules "True" }.not_to raise_error
   end
   it "the resource's allow_user_apps property only accepts strings True, False, or NotConfigured" do
     expect { resource.allow_user_apps "No" }.to raise_error(ArgumentError)
-    expect { resource.allow_user_apps "False"}.not_to raise_error
+    expect { resource.allow_user_apps "False" }.not_to raise_error
   end
   it "the resource's allow_user_ports property only accepts strings True, False, or NotConfigured" do
-    expect { resource.allow_user_ports nil }.to raise_error(ArgumentError)
-    expect { resource.allow_user_ports "NotConfigured"}.not_to raise_error
+    expect { resource.allow_user_ports false }.to raise_error(ArgumentError)
+    expect { resource.allow_user_ports "NotConfigured" }.not_to raise_error
   end
   it "the resource's allow_unicast_response property only accepts strings True, False, or NotConfigured" do
     expect { resource.allow_unicast_response true }.to raise_error(ArgumentError)
-    expect { resource.allow_unicast_response "True"}.not_to raise_error
+    expect { resource.allow_unicast_response "True" }.not_to raise_error
   end
   it "the resource's display_notification property only accepts strings True, False, or NotConfigured" do
     expect { resource.display_notification false }.to raise_error(ArgumentError)
-    expect { resource.display_notification "False"}.not_to raise_error
+    expect { resource.display_notification "False" }.not_to raise_error
   end
 
   it "sets the default action as :configure" do