diff options
| author | Luke Kanies <luke@puppetlabs.com> | 2010-06-10 20:57:41 -0700 |
|---|---|---|
| committer | test branch <puppet-dev@googlegroups.com> | 2010-02-17 06:50:53 -0800 |
| commit | 8f3e8bb31d513f67ea28a5f249aa850789a10ff2 (patch) | |
| tree | 7bc17146f574667822b062e3e32653fe91d3c580 /spec/integration | |
| parent | d6407f46f1743b9f3916d74bc0ed521fb5bf259d (diff) | |
Working #3139 - ResourceHarness does caching
This is again about moving transactional behaviour out
of Puppet::Type and into the transactional code.
I initially moved this code into Resource::Status,
but it seemed to make more sense in the Harness - the
Status object should be thin and have no code
that doesn't make sense on both sides of the pipe,
it seemed.
The interface gets a bit uglier as a result, but this
is all about good design != good OO (because we're
increasing the argument count of methods by moving
them outside of the target class).
Signed-off-by: Luke Kanies <luke@puppetlabs.com>
Diffstat (limited to 'spec/integration')
| -rwxr-xr-x | spec/integration/transaction.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/spec/integration/transaction.rb b/spec/integration/transaction.rb index f68287432..6a4dbae67 100755 --- a/spec/integration/transaction.rb +++ b/spec/integration/transaction.rb @@ -242,6 +242,6 @@ describe Puppet::Transaction do trans = catalog.apply - trans.resource_harness.should be_scheduled(resource) + trans.resource_harness.should be_scheduled(trans.resource_status(resource), resource) end end |