stackforge/cookbook-pacemaker
Code Issues Proposed changes

avoid test messing with fixture

Browse Source 
Assigning to `fixture` means that the change to the fixture can leak
out to other tests, so we assign to a temporary variable instead.
This commit is contained in:
Adam Spiers
2014-04-10 23:28:42 +01:00
parent c08fb8a3bd
commit c340bd4be2
1 changed files with 4 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
spec/providers/group_spec.rb
View File

@@ -40,10 +40,11 @@ describe "Chef::Provider::PacemakerGroup" do
end end
it "should modify the group if it has different member resources" do it "should modify the group if it has different member resources" do
fixture.members = %w(resource1 resource3) expected = fixture.dup
expected_configure_cmd_args = [fixture.reconfigure_command] expected.members = %w(resource1 resource3)
expected_configure_cmd_args = [expected.reconfigure_command]
test_modify(expected_configure_cmd_args) do test_modify(expected_configure_cmd_args) do
@resource.members fixture.members @resource.members expected.members
end end
end end