From c340bd4be23bcb58fecafbc295c37247ebb74d27 Mon Sep 17 00:00:00 2001 From: Adam Spiers Date: Thu, 10 Apr 2014 23:28:42 +0100 Subject: [PATCH] avoid test messing with fixture Assigning to `fixture` means that the change to the fixture can leak out to other tests, so we assign to a temporary variable instead. --- spec/providers/group_spec.rb | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/spec/providers/group_spec.rb b/spec/providers/group_spec.rb index c9ba86e..2525b2c 100644 --- a/spec/providers/group_spec.rb +++ b/spec/providers/group_spec.rb @@ -40,10 +40,11 @@ describe "Chef::Provider::PacemakerGroup" do end it "should modify the group if it has different member resources" do - fixture.members = %w(resource1 resource3) - expected_configure_cmd_args = [fixture.reconfigure_command] + expected = fixture.dup + expected.members = %w(resource1 resource3) + expected_configure_cmd_args = [expected.reconfigure_command] test_modify(expected_configure_cmd_args) do - @resource.members fixture.members + @resource.members expected.members end end