puppet.git - Puppet repo

	Commit message (Collapse)	Author	Age	Files	Lines
*	Migrating Feature tests to spec	Luke Kanies	2009-08-02	1	-0/+54
\| \| \| \| \| \| \| \| \| \|	This was to fix a failing test/unit test. Test coverage is now a bit better, more maintainable, and I refactored the code just slightly to make it a bit cleaner. Signed-off-by: Luke Kanies <luke@madstop.com>
*	Ruby no longer clobbers puppet autoloading	Luke Kanies	2009-07-16	1	-0/+91
\| \| \| \| \| \| \| \| \|	We basically just make sure that we tell Ruby about files we've loaded, so you can 'require' these files and doing so will essentially no-op, rather than clobbering the already-loaded code. Signed-off-by: Luke Kanies <luke@madstop.com>
*	Adding a Spec lib directory and moving tmpfile to it	Luke Kanies	2009-05-26	1	-6/+3
\| \| \| \| \| \| \| \| \| \| \| \| \| \|	We had a common pattern for creating a temporary file during integration tests, and this just makes that common pattern explicit by moving it to a module in the newly-created lib directory in the spec directory. We definitely don't want to go overboard in using libraries in our tests, but sometimes it gets a bit excessive to completely avoid them. Signed-off-by: Luke Kanies <luke@madstop.com>
*	Fix failing test: file.close! and file.path ordering fix	Brice Figureau	2009-05-15	1	-1/+2
\| \| \| \| \| \| \| \| \| \| \|	On ruby 1.8.7 file.close! nils the internal file path. So the following pattern: file = temp file.close! file = file.path doesn't work. Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
*	Merge commit 'masterzen/tickets/1834'	Luke Kanies	2008-12-30	1	-1/+2
\|\
\| *	Fix #1834 part1 - Fix tempfile failing tests	Brice Figureau	2008-12-20	1	-1/+2
\| \| \| \| \| \| \| \|	Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* \|	Merge branch '0.24.x'	James Turnbull	2008-12-27	1	-0/+36
\|\ \ \| \|/ \|/\| \| \| \| \|	Conflicts: CHANGELOG
\| *	Not using a temporary file when locking files for writing.	Luke Kanies	2008-12-15	1	-3/+3
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The temporary file was not actually useful, because we could never really get atomic renames, for annoying, complicated reasons. This hopefully finally fixes #1812. Signed-off-by: Luke Kanies <luke@madstop.com>
\| *	Modifying the corruption-checking test.	Luke Kanies	2008-12-15	1	-3/+3
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	It is now more likely to fail if there's a problem, since the yaml should be corrupt, causing a yaml failure. Also removing the equivalent test from the Storage module. Signed-off-by: Luke Kanies <luke@madstop.com>
\| *	Preparing to fix #1812 - Moving locking code to a module	Luke Kanies	2008-12-12	1	-0/+36
\| \| \| \| \| \| \|	This moves the locking code out of Puppet::Util into a separate module, to make the code cleaner. Signed-off-by: Luke Kanies <luke@madstop.com>
*	Refactoring the Settings class to use Puppet::Resource	Luke Kanies	2008-12-18	1	-0/+29
	It also now uses the Catalog instead of the recursive TransObject stuff. Signed-off-by: Luke Kanies <luke@madstop.com>