A compliant node is one without any failed resources. However, because the
failed column on resource_statuses is nullable, it defaults to null. Since we
weren't specifying it in this factory, the resource status wasn't being
considered successful.

Paired-With: Ben Hengst

Signed-off-by: Nick Lewis <n...@puppetlabs.com>
---
Local-branch: ticket/next/7007
 spec/factories.rb |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/spec/factories.rb b/spec/factories.rb
index 178c786..dfe91b4 100644
--- a/spec/factories.rb
+++ b/spec/factories.rb
@@ -85,7 +85,8 @@ end
 
 Factory.define :compliant_node, :parent => :successful_node do |node|
   node.after_create do |node|
-    
node.last_apply_report.resource_statuses.generate().events.generate(:status => 
'success')
+    res = node.last_apply_report.resource_statuses.generate!(:failed => false)
+    res.events.generate!(:status => 'success')
   end
 end
 
-- 
1.7.5.1

-- 
You received this message because you are subscribed to the Google Groups 
"Puppet Developers" group.
To post to this group, send email to puppet-dev@googlegroups.com.
To unsubscribe from this group, send email to 
puppet-dev+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/puppet-dev?hl=en.

Reply via email to