summaryrefslogtreecommitdiffstats
path: root/test
Commit message (Collapse)AuthorAgeFilesLines
...
* | Merge branch '0.24.x'James Turnbull2008-12-272-57/+0
|\| | | | | | | | | Conflicts: CHANGELOG
| * Modifying the corruption-checking test.Luke Kanies2008-12-151-24/+0
| | | | | | | | | | | | | | | | | | 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 moduleLuke Kanies2008-12-121-33/+0
| | | | | | | | | | | | | | 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>
* | Fixing all of the test/ tests I broke in previous dev.Luke Kanies2008-12-189-317/+20
| | | | | | | | | | | | | | | | | | Most of these are straightforward changes to the tests, but a couple required small refactorings (e.g., References can now be created with Puppet::Type instances, and they know how to extract the type/title from them). Signed-off-by: Luke Kanies <luke@madstop.com>
* | Deprecating the Puppet::Type.create.Luke Kanies2008-12-1839-261/+261
| | | | | | | | | | | | | | This method is no longer necessary; you can use the normal 'new' class method. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Deprecating 'Puppet.type'; replacing all instances with Puppet::Type.typeLuke Kanies2008-12-1828-148/+148
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* | Finishing the work to use Puppet::Resource instead of TransObjectLuke Kanies2008-12-181-35/+0
| | | | | | | | | | | | | | | | | | | | | | | | This was a complicated project because TransObject had made its way into too many classes. The usage by Util::Settings was particularly nefarious. Refactoring and fixing this exposed some other issues. The main complication, though, was the extent to which the Puppet::Type class depended on TransObject. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Refactoring the Settings class to use Puppet::ResourceLuke Kanies2008-12-181-19/+2
| | | | | | | | | | | | | | It also now uses the Catalog instead of the recursive TransObject stuff. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Replacing TransObject usage with Puppet::ResourceLuke Kanies2008-12-181-38/+0
| | | | | | | | | | | | | | | | | | This completely reorganizes how RAL resources are initialized, and in the process I was able to remove a lot of code (I removed other apparently obsolete code at the same time). Signed-off-by: Luke Kanies <luke@madstop.com>
* | Renaming the "Catalog#to_type" method to "Catalog#to_ral"Luke Kanies2008-12-185-77/+8
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* | Adding resource convertion to the parser resourcesLuke Kanies2008-12-181-84/+0
| | | | | | | | | | | | | | Also uses Puppet::Resource's method for creating transportable resources. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Renaming Puppet::Node::Catalog to Puppet::Resource::CatalogLuke Kanies2008-12-185-12/+12
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* | Merge branch '0.24.x'Luke Kanies2008-12-091-1/+1
|\| | | | | | | | | | | | | | | | | Conflicts: CHANGELOG lib/puppet/type/tidy.rb spec/unit/type/file/ensure.rb spec/unit/type/tidy.rb
| * Fix rake abort when there is a matching confineBrice Figureau2008-12-081-1/+1
| | | | | | | | Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* | Fixing all broken tests. Most of them were broken by fileserving changes.Luke Kanies2008-12-022-2/+2
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* | Merge branch '0.24.x'Luke Kanies2008-12-026-26/+23
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: bin/puppetca lib/puppet/type/group.rb lib/puppet/type/tidy.rb lib/puppet/util/settings.rb Also edited the following files so tests will pass: lib/puppet/type/component.rb spec/unit/ssl/certificate_request.rb spec/unit/type/computer.rb spec/unit/type/mcx.rb spec/unit/type/resources.rb spec/unit/util/settings.rb spec/unit/util/storage.rb test/ral/type/zone.rb
| * Fixing #1683 - accessing and changing settings is now thread-safe.Luke Kanies2008-11-261-4/+4
| | | | | | | | | | | | Applying patch by Matt Palmer. Signed-off-by: Luke Kanies <luke@madstop.com>
| * Partially fixing #1772 - broken 'resources' tests.Luke Kanies2008-11-261-19/+2
| | | | | | | | | | | | | | | | The main problem was that the directory_services user provider was returning a string for its uid instead of an integer. I also began a 'resources' spec file. Signed-off-by: Luke Kanies <luke@madstop.com>
| * Fixed #1711 - fileserver test fails due to incorrect mockingJames Turnbull2008-11-191-0/+5
| |
| * Move function existance test to parser evaluationBrice Figureau2008-11-171-1/+11
| | | | | | | | | | | | | | | | The aim is to let --parseonly succeeds even if the function is not (yet) present. This is usefull in commit-hooks and for the inline documentation generation system. Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
| * Fix #1737 - ssh_authorized_keys should be able to parse options containing ↵Brice Figureau2008-11-111-0/+1
| | | | | | | | | | | | commas Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
| * Fixing a test I broke when fixing a reporting bugLuke Kanies2008-11-011-4/+0
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* | Completely refactoring the tidy type.Luke Kanies2008-11-151-289/+0
| | | | | | | | | | | | | | | | | | | | | | | | This was necessary because of how I changed file recursion. The type works much more intelligently now -- files to be removed have a file resource generated for them, and that resource handles deletion. Also fixes #1717; neither age nor size is required now. Signed-off-by: Luke Kanies <luke@madstop.com>
* | These changes are all about making sure file data is expired when appropriate.Luke Kanies2008-11-152-149/+8
| | | | | | | | | | | | All file tests now pass. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Changing the Cacher.invalidate method to Cacher.expire.Luke Kanies2008-11-112-2/+2
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* | Fixing tests broken by fileserving and other refactoring.Luke Kanies2008-11-083-257/+34
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* | Converting the file 'source' property to a parameter.Luke Kanies2008-11-055-204/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This makes a lot of sense because source was always more of a metaparameter than a property -- it affected the 'should' values of other properties, but it shouldn't have done any other work. It will hopefully make everything else much cleaner. This is such a large commit mostly because of the need to fix a lot of tests. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Switched all value management in props/params to internal classes.Luke Kanies2008-11-042-562/+0
| | | | | | | | | | | | | | | | | | | | | | This is a significant refactor of some very murky code, and it's all much cleaner and more readable now. All of the 'newvalue' methods and any value-related code is in a ValueCollection class. This puts us in a good position to refactor the Property and Parameter classes more completely. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Moving a file purging test to rspecLuke Kanies2008-11-041-42/+0
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* | Removing obsolete methods and tests:Luke Kanies2008-11-041-83/+0
| | | | | | | | | | | | | | | | | | Removing obsolete handleignore method Removing obsolete FileSource class Removing a now-obsolete test/unit test Removing a now-obsolete recursive filebucket test Signed-off-by: Luke Kanies <luke@madstop.com>
* | Making it so (once again) files with sources set can still be deletedLuke Kanies2008-11-041-47/+0
| | | | | | | | | | | | (which I think is kinda stupid, but apparently people want it). Signed-off-by: Luke Kanies <luke@madstop.com>
* | Fixing and migrating more file tests.Luke Kanies2008-11-042-482/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is an unfortunately messy commit; I should have tried harder to separate the different tasks into different commits, but it's not worth going back and doing now. This is the first commit in the long road of fixing the existing file tests, and in the process refactoring and better testing the code. The refactoring in this commit is mostly around the 'stat' instance variable that tests whether the file exists (and provides its metadata if it does) and the 'insync?' method in the source property. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Fixing forward-compatibility issues resulting from no global resourcesLuke Kanies2008-10-311-1/+1
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* | Fixing a test I broke when fixing a reporting bugLuke Kanies2008-10-311-4/+0
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* | Merge branch '0.24.x'Luke Kanies2008-10-314-6/+31
|\| | | | | | | | | | | Conflicts: CHANGELOG
| * Fix #1402 - Allow multiline commentsBrice Figureau2008-10-292-0/+17
| | | | | | | | Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
| * Fix #857 - Multiple class of the same name don't append codeBrice Figureau2008-10-292-0/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The following manifest wasn't working: class one { notice('class one') } class one { notice('second class one') } include one It all boiled down to class code not being arrays. Encapsulating code in ASTArray when needed is enough to append code, because of the property of ASTArray to evaluate all their members in turn. Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
| * Fixed #1710 - Spurious output in test runJames Turnbull2008-10-291-6/+0
| |
* | Merge branch '0.24.x'Luke Kanies2008-10-224-0/+35
|\| | | | | | | | | | | Conflicts: lib/puppet/type/user.rb
| * Fix #936 - Allow trailing comma in array definitionBrice Figureau2008-10-213-0/+23
| | | | | | | | Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
| * Fix #636 - Allow extraneous comma in function argument listBrice Figureau2008-10-212-0/+12
| | | | | | | | Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* | Merge branch '0.24.x' Removed the 'after' blocks that call Type.clear,Luke Kanies2008-10-1725-628/+72
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | since that method is deprecated. Conflicts: CHANGELOG bin/puppetca lib/puppet/file_serving/fileset.rb lib/puppet/network/xmlrpc/client.rb lib/puppet/type/file/selcontext.rb spec/unit/file_serving/metadata.rb spec/unit/type/file.rb
| * Fixing some tests that were broken in 2fba85afLuke Kanies2008-10-082-21/+22
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
| * Fixing a test that was failing as a result of the fix to #1491Luke Kanies2008-10-081-4/+1
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
| * Removing a gid test for users, since it is a bad test and has mostly been ↵Luke Kanies2008-10-081-50/+0
| | | | | | | | | | | | replaced in rspec Signed-off-by: Luke Kanies <luke@madstop.com>
| * Fixing a test that was broken in ee579641Luke Kanies2008-10-081-1/+1
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
| * Fixing some tests that were broken in the fix for #1633Luke Kanies2008-10-082-6/+2
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
| * Fixing a test that was failing because of the change to retrieve() in ee579641Luke Kanies2008-10-081-2/+1
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
| * Fixing a file source test that was failing because missing sources is now a ↵Luke Kanies2008-10-081-3/+1
| | | | | | | | | | | | failure Signed-off-by: Luke Kanies <luke@madstop.com>
| * Fixing the broken tests resulting from the fix for #1551.Luke Kanies2008-10-081-1/+3
| | | | | | | | | | | | The test was expecting the current time, albeit as an integer. Signed-off-by: Luke Kanies <luke@madstop.com>