Issue #17887 has been updated by Dominic Cleal.

Status changed from Unreviewed to In Topic Branch Pending Review
Target version set to 3.0.x
Branch set to https://github.com/puppetlabs/puppet/pull/1307


----------------------------------------
Bug #17887: Mocha expectation failure during property sync results in 
"Puppet::Util::Log requires a message"
https://projects.puppetlabs.com/issues/17887#change-77909

Author: Dominic Cleal
Status: In Topic Branch Pending Review
Priority: Low
Assignee: Dominic Cleal
Category: transactions
Target version: 3.0.x
Affected Puppet version: development
Keywords: resource harness, transaction
Branch: https://github.com/puppetlabs/puppet/pull/1307


During provider development and testing with rspec+mocha and using a test 
catalog, an expectation failure can cause the following log message:

    Could not evaluate: Puppet::Util::Log requires a message

This occurs if the property sync calls a method that has a mocha expectation 
set, but has reached the max call count or similar.

When testing a catalog from a test, the `Puppet::Transaction::ResourceHarness` 
calls `apply_parameter` for a given parameter, which then calls 
`property.sync`.  If the mocha expectation fails inside this sync, the `rescue` 
block is never called (it isn't an exception) but the `ensure` block is still 
run.  This calls `send_log` on the event, but the message is uninitialised at 
that point so the logging fails with the above error.


-- 
You have received this notification because you have either subscribed to it, 
or are involved in it.
To change your notification preferences, please click here: 
http://projects.puppetlabs.com/my/account

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

Reply via email to