summaryrefslogtreecommitdiffstats
path: root/spec
Commit message (Collapse)AuthorAgeFilesLines
* The 'Environment' class can now calculate its modulepath.Luke Kanies2009-02-181-0/+42
| | | | | | | | | | | This includes adding PUPPETLIB from the shell environment. I'm moving responsibility for this from the Module class, because nearly every method in Puppet::Module accepted 'environment' as its argument, which is a good sign that it's on the wrong class. Signed-off-by: Luke Kanies <luke@madstop.com>
* Added support for finding modules from an environmentLuke Kanies2009-02-181-0/+38
| | | | | | | | This uses the environment to search for the modules, rather than relying on the Puppet::Module class to know how to handle environments. Signed-off-by: Luke Kanies <luke@madstop.com>
* Adding support for finding all modules in a given path.Luke Kanies2009-02-181-0/+87
| | | | | | | | This 'each_module' method will be used by environments to find all or a given module, and will likely eventually be used internally, too. Signed-off-by: Luke Kanies <luke@madstop.com>
* reformatting the environment testsLuke Kanies2009-02-181-15/+15
| | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* Supporting multiple paths for searching for files.Luke Kanies2009-02-181-18/+22
| | | | | | | This is, once again, used for plugins, which needs to search across multiple modules' plugin directories. Signed-off-by: Luke Kanies <luke@madstop.com>
* Adding support for merging multiple filesets.Luke Kanies2009-02-181-0/+43
| | | | | | | This is required for plugins, which recurse across multiple directories. Signed-off-by: Luke Kanies <luke@madstop.com>
* Adding new methods to Puppet::Module.Luke Kanies2009-02-181-0/+37
| | | | | | | | | | | | There are now boolean methods to test whether a given kind of file is present in a given module. E.g, you can do: Puppet::Module.new("mod", "/my/path").plugins? There are also accessor-style methods that return the full path for a given kind of file. Signed-off-by: Luke Kanies <luke@madstop.com>
* Refactoring Puppet::Module a bit.Luke Kanies2009-02-181-5/+5
| | | | | | | No behaviour change here, just some internal changes to make way for methods I want to add. Signed-off-by: Luke Kanies <luke@madstop.com>
* Fixing fileserving to support strings or symbolsLuke Kanies2009-02-182-4/+8
| | | | | | | When used internally we would use symbols, but the REST transfers need to support strings. Signed-off-by: Luke Kanies <luke@madstop.com>
* Adding clarity to query string handling in REST callsLuke Kanies2009-02-183-15/+120
| | | | | | | | | | | | We previously only handled simple strings as values, but we know handle true and false as booleans, we URI-escape all strings, and we can yaml-encode and then escape arrays of strings. This could get abused a bit, in that we're just yaml-dumping anything that's an array, but it should be pretty safe. Mmmm, should. Signed-off-by: Luke Kanies <luke@madstop.com>
* Stubbing tests that were affecting other testsLuke Kanies2009-02-188-4/+30
| | | | | | | | | | | These stubs are all for global configuration state. It's a known problem that these have to be stubbed, but we haven't been able to find a design yet that fixes this. At least these tests are no longer breaking other tests. Signed-off-by: Luke Kanies <luke@madstop.com>
* Fixing a warning in a testLuke Kanies2009-02-181-2/+2
| | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* Fixing a syntactically invalid application testLuke Kanies2009-02-181-3/+3
| | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* Move --version handling to Puppet::ApplicationBrice Figureau2009-02-166-40/+13
| | | | Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* Move puppetd to the Application Controller paradigmBrice Figureau2009-02-161-0/+487
| | | | Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* Move puppetdoc to the Application Controller paradigmBrice Figureau2009-02-161-0/+343
| | | | Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* Move ralsh to the Application Controller paradigmBrice Figureau2009-02-161-0/+230
| | | | Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* Move puppetrun to Application Controller paradigmBrice Figureau2009-02-161-0/+271
| | | | Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* Move pi to the Application Controller paradigmBrice Figureau2009-02-161-0/+84
| | | | Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* Move puppetmasterd to Puppet::ApplicationBrice Figureau2009-02-161-0/+334
| | | | Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* Move puppet to the Application Controller paradigmBrice Figureau2009-02-161-0/+302
| | | | Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* Move filebucket to the Application Controller paradigmBrice Figureau2009-02-161-0/+230
| | | | Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* Move puppetca to the Application Controller paradigmBrice Figureau2009-02-161-0/+134
| | | | Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* Introducing the Application ControllerBrice Figureau2009-02-161-0/+405
| | | | Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* Fixing broken filetype tests resulting from the loss of Type[]Luke Kanies2009-02-141-11/+1
| | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* Fixing some tests that somehow broke in the merge to masterLuke Kanies2009-02-141-2/+7
| | | | | | | | | | I expect these failed in master but not 0.24.x because I'm better about stubbing calls to Facter in master. I stubbed these enough that they should continue to work. Signed-off-by: Luke Kanies <luke@madstop.com>
* Removing a test that was too dependant on order.Luke Kanies2009-02-141-4/+0
| | | | | | | It wasn't a terribly useful test anyway, since no one depends on that individual configuration. Signed-off-by: Luke Kanies <luke@madstop.com>
* Merge branch '0.24.x'Luke Kanies2009-02-141-2/+2
|\
| * Minor fix to launchd testsJames Turnbull2009-02-151-2/+2
| |
* | Merge branch '0.24.x'Luke Kanies2009-02-144-10/+31
|\| | | | | | | | | Conflicts: test/ral/manager/type.rb
| * Fix spec test for launchd service provider to work with new service status ↵Nigel Kersten2009-02-151-1/+9
| | | | | | | | | | | | method and add two new status tests. Now ensures that given the correct input, the correct status for the launchd service is returned.
| * Fixing password validation to support symbols.Luke Kanies2009-02-141-4/+8
| | | | | | | | | | | | | | My fix for #1920 broke when you set :absent or :present. Signed-off-by: Luke Kanies <luke@madstop.com>
| * Only backing up within parsedfile when managing filesLuke Kanies2009-02-141-3/+12
| | | | | | | | | | | | | | | | | | | | | | I was getting failing tests because I was using non-files for testing and they didn't back up the same, not surprisingly. This moves the 'backup' method to the :flat filetype and then only backs up if the filetype supports it. Signed-off-by: Luke Kanies <luke@madstop.com>
| * Fixing a test broken by a log demotionLuke Kanies2009-02-141-2/+2
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* | Failure to find node facts is now a failure.Luke Kanies2009-02-141-0/+5
| | | | | | | | | | | | | | | | It was previously just a warning, but the node is essentially non-functional without the facts, so it makes more sense for it to be a warning. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Fixing #1527 - Failing Facter does not hurt PuppetLuke Kanies2009-02-142-0/+17
| | | | | | | | | | | | | | | | At this point, the server's behaviour is a bit undefined if it tries to compile the catalog with no facts locally. The next commits will fix that. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Using Puppet::Type.new instead of createLuke Kanies2009-02-141-10/+10
| | | | | | | | | | | | | | create() got deprecated and I apparently missed changing this. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Actualling syncing facts and pluginsLuke Kanies2009-02-143-2/+8
| | | | | | | | | | | | | | | | Also fixing the argument order while downloading either of them. I had my Downloader.new calls using the wrong argument order. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Fix #1933 - Inconsistent resource evaluation order in subsequent evaluation runsBrice Figureau2009-02-141-1/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | While evaluating the AST, catalog vertices are not always ordered the same way on different run, leading to some tags (which should have been applied in evaluation order) to not be associated with some underlying resources. This changeset change all accesses to resources inside the compiler to always use an ordered (in evaluation order) list of added resources. Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* | Merge branch '0.24.x'Luke Kanies2009-02-138-26/+529
|\| | | | | | | | | | | | | | | | | | | Conflicts: lib/puppet/indirector/facts/facter.rb lib/puppet/provider/augeas/augeas.rb lib/puppet/util/filetype.rb spec/unit/indirector/facts/facter.rb spec/unit/provider/augeas/augeas.rb test/util/filetype.rb
| * Fixed #1831 - Added sprintf functionJames Turnbull2009-02-141-0/+42
| |
| * Fixed #1830 - Added regsubst functionJames Turnbull2009-02-141-0/+88
| |
| * Bug 1948: Add logic and testing for the command parsing logicBryan Kearney2009-02-141-4/+13
| |
| * Fixing #1964 - Facts get loaded from pluginsLuke Kanies2009-02-121-2/+20
| | | | | | | | | | | | | | | | Applying slightly modified patch. Also added tests. Signed-off-by: Luke Kanies <luke@madstop.com>
| * Adding tests for Puppet::Indirector::Facts::Facter.loadfactsLuke Kanies2009-02-121-2/+9
| | | | | | | | | | | | | | I just copied the tests from the master branch, changed as necessary. Signed-off-by: Luke Kanies <luke@madstop.com>
| * Fixing #1541 - ParsedFile only backs up files once per transactionLuke Kanies2009-02-132-6/+36
| | | | | | | | | | | | | | | | This moves responsibility for backups from the filetype to the consumer of the filetype, but only ParsedFile actually uses filetypes. Signed-off-by: Luke Kanies <luke@madstop.com>
| * Migrated FileType tests to spec, and fleshed them out a bit.Luke Kanies2009-02-131-0/+116
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
| * Bug #1948: Added patch by jab to support the correct ins syntax. Updated the ↵Bryan Kearney2009-02-131-12/+13
| | | | | | | | test cases as well
| * Fixing #961 - closing the http connection after every xmlrpc callLuke Kanies2009-02-121-0/+22
| | | | | | | | | | | | | | | | There were apparently some circumstances that resulted in the connection not being closed; this just closes it every time if it's still open after the rpc call is complete. Signed-off-by: Luke Kanies <luke@madstop.com>
| * Refactoring the XMLRPC::Client error-handlingLuke Kanies2009-02-121-7/+143
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I split it all into smaller, manageable chunks, and used methods for each step, instead of having one huge call. Note that I made all of the tests first, then refactored the code, so I'm confident there's no behavior change. I don't know that this is actually a lot cleaner, but it seems that way to me. I'm open to skipping this, but I think it makes the whole thing a lot cleaner. Signed-off-by: Luke Kanies <luke@madstop.com>