summaryrefslogtreecommitdiffstats
path: root/test
Commit message (Collapse)AuthorAgeFilesLines
* Resources now return the 'should' value for properties fromLuke Kanies2008-04-111-37/+0
| | | | | | | the [] accessor method (they previously threw an exception when this method was used with properties). This shouldn't have any affect functionally; it just makes the method equivalent to 'should' for properties, but it works for all attribute types now.
* Modified the 'master' handler to use the Catalog class toLuke Kanies2008-04-112-204/+9
| | | | | | | | | | compile node configurations, rather than using the Configuration handler, which was never used directly. I removed the Configuration handler as a result. Modified the 'master' handler (responsible for sending configurations to clients) to always return Time.now as its compile date, so configurations will always get recompiled.
* Fixed #1184 -- definitions now autoload correctly all of the time.Luke Kanies2008-04-101-1/+8
|
* Removed the code from the client that tries to avoid recompilingLuke Kanies2008-04-101-110/+0
| | | | | | the catalog. The client will now always recompile, assuming it can reach the server. It will still use the cached config if there's a failure.
* Adding validation to the user type to confirm that theLuke Kanies2008-04-081-0/+6
| | | | | group list does not contain any commas. This seems to be a common problem.
* Fixes #1148 - replaces #!/usr/bin/ruby with #!/usr/bin/env ruby.Paul Lathrop2008-03-281-1/+1
|
* Fixing #571 -- provider suitability is now checked at resourceLuke Kanies2008-03-241-5/+21
| | | | | | | evaluation time, rather than resource instantiation time. This means that you don't catch your "errors" as early, but it also means you should be able to realistically configure a whole host in one run.
* Fixing a couple of tests.Luke Kanies2008-03-243-36/+35
|
* Found all instances of methods where split() is used withoutLuke Kanies2008-03-211-1/+0
| | | | | | | | | | any local variables and added a local variable -- see http://snurl.com/21zf8. My own testing showed that this caused memory growth to level off at a reasonable level. Note that the link above says the problem is only with class methods, but my own testing showed that it's any method that meets these criteria. This is not a functional change, but should hopefully be the last nail in the coffin of #1131.
* Found an array that leaked pretty quickly between reparsingLuke Kanies2008-03-211-0/+1
| | | | | | | | | | files, thanks to work by Adam Jacob and Arjuna Christenson (the finding, not the leak). I'm going to act like this fixes #1131, at least for now, but I doubt it does, since that shows general memory growth over time, whereas the leak here should go away as soon as files are reparsed (because the parser is holding the reference to the leaking array).
* tweak the (already applied) patch in ↵Sam Quigley2008-03-201-0/+4
| | | | 388cf7c3df7ce26e953949ed6fe63d76cbbb3691 to resolve #1137; also, add tests which detect the problem.
* fixing another failing testLuke Kanies2008-03-191-0/+2
|
* Fixing some tests that only failed under certainLuke Kanies2008-03-183-3/+6
| | | | | circumstances (mostly, when loaded with other files, or when loaded from rake or autotest rather than separately).
* Regression in :node_name functionalityJohn Ferlito2008-03-141-2/+9
| | | | When :node_name="cert" is specified the 'hostname' fact should be set to the SSL certificate common name instead of the results from facter. I've extended this to also set 'domain' and 'fqdn' since that makes a lot of sense to me. This fixes a regression introduced in SVN#1673
* Fixing #1132 -- host names can now have dashes anywhere.Luke Kanies2008-03-131-0/+44
| | | | (Patch by freiheit.)
* Fixing #1118 -- downloading plugins and facts now ignores noop.Luke Kanies2008-03-131-16/+0
| | | | | | Note that this changes the behaviour a bit -- the resource's noop setting always beats the global setting (previously, whichever was true would win).
* Always duplicating resource defaults in the parser, so thatLuke Kanies2008-03-061-159/+0
| | | | | | stacked metaparameter values do not result in all resources that receive a given default also getting those stacked values.
* Fixing a few testsLuke Kanies2008-02-292-6/+1
|
* Fixing tests that did not work with Rails 2.Luke Kanies2008-02-292-0/+3
|
* Fixing a test that changed the environment for all later tests,Luke Kanies2008-02-291-9/+1
| | | | thus breaking some of them.
* renaming ral/types to ral/type in the testsLuke Kanies2008-02-2821-0/+0
|
* Removed the loglevels from the valid values for 'logoutput'Luke Kanies2008-02-271-2/+2
| | | | | | | in the Exec resource type -- the log levels are specified using the 'loglevel' parameter, not 'logoutput'. This never worked, or at least hasn't for ages, and now the docs are just correct.
* Merge commit 'turnbull/0.24.x' into 0.24.xLuke Kanies2008-02-231-0/+18
|\
| * Fixes #1078 and includes new testJames Turnbull2008-02-231-0/+18
| |
* | Somewhat refactored fileserving so that it no longer cachesLuke Kanies2008-02-2311-163/+92
|/ | | | | | | | | | any objects, nor does it use Puppet's RAL resources. In the process, I fixed #894 (you can now copy links) and refactored other classes as necessary. Mostly it was fixing tests. This is a squashed commit of a temporary branch, fwiw, and it also includes any fixes to the tests that were necessary to get all tests passing again.
* Quashed commit of my fixes for #1010.Luke Kanies2008-02-211-11/+28
|
* Fixing comment -- ticket #1027 instead of #1064Luke Kanies2008-02-131-1/+1
|
* Fixing #1064 -- providers et al are now autoloadedLuke Kanies2008-02-131-5/+5
| | | | from modules even when Autoload#loadall is used.
* Fixed #1043 -- autoloading now searches the plugins directoryLuke Kanies2008-02-132-0/+21
| | | | | | in each module, in addition to the lib directory. The 'lib' directory is also deprecated, but supported for now to give people a chance to convert.
* Fixed #968 again, this time with tests -- parseonly works,Luke Kanies2008-02-131-0/+17
| | | | | including not compiling the configurations, and also storeconfigs is no longer required during parse-testing.
* Renaming the 'pfile' and 'pfilebucket' files to plainLuke Kanies2008-02-131-1/+0
| | | | | 'file' and 'filebucket'. This should have been done years ago.
* Loading the mocha gem from the puppettest.rb file.Luke Kanies2008-02-121-1/+10
|
* Trying to upgrade rspec, but not having much luck.Luke Kanies2008-02-1236-1680/+0
|
* Fixed #997 -- virtual defined types are no longer evaluated.Luke Kanies2008-02-124-38/+34
| | | | | | | NOTE: This introduces a behaviour change, in that you previously could realize a resource within a virtual defined resource, and now you must realize the entire defined resource, rather than just the contained resource.
* Moving the ast node tests to rspec (which I could haveLuke Kanies2008-02-122-234/+0
| | | | | | | *sworn* I did this weekend). In the process, I fixed a couple of bugs related to differentiating between nodes and classes, and then cleaned up quite a few error messages.
* Removing the last remaining vestiges of GRATR --Luke Kanies2008-02-114-9/+9
| | | | removing the bangs from 'add_vertex!' and 'add_edge!'.
* Changing the name of the Compile class to Compiler,Luke Kanies2008-02-1110-40/+40
| | | | | since it's stupid to have a class named after a verb.
* Changing some methods in the Compile class toLuke Kanies2008-02-112-3/+3
| | | | | be more internally consistent (switched store_resource to add_resource, and store_override to add_override).
* Fixed #1030 - class and definition evaluation has been significantlyLuke Kanies2008-02-111-184/+0
| | | | | | refactored, fixing this problem and making the whole interplay between the classes, definitions, and nodes, and the Compile class much cleaner.
* Converting the Compile class to use a Node::Catalog instanceLuke Kanies2008-02-114-574/+1
| | | | | as its resource container, instead of having its own behaviour around resource uniqueness.
* Moving all of the tests for Puppet::Parser::Compile toLuke Kanies2008-02-111-33/+0
| | | | | | rspec, so I can refactor the class to more heavily rely on a Node::Catalog instead of doing its own resource container management.
* More AST refactoring -- each of the code wrapping classesLuke Kanies2008-02-085-63/+81
| | | | | | | | | | | just returns a resource from its evaluate() method, and all of the work is done in the evaluate_code method. This makes the code cleaner, because it means 1) evaluate() has the same prototype as all of the other AST classes, 2) evaluate() is no longer called indirectly through the Parser Resource class, and 3) the classes themselves are responsible for creating the resources, rather than it being done in the Compile class.
* Refactoring the AST classes just a bit. I realized thatLuke Kanies2008-02-0812-63/+62
| | | | | | all of the evaluate() methods only ever accepted a scope, and sometimes one other option, so I switched them all to use named arguments instead of a hash.
* Stubbing Facter during the snippet tests, so they are faster and work with ↵Luke Kanies2008-02-081-0/+2
| | | | no network
* Added :env parameter for backwards-compatibility, with warning about ↵Paul Lathrop2008-02-051-0/+40
| | | | deprecation. :env parameter sets new :environment parameter. Changed instances of :env to :environment for consistency with other types. Added tests for new parameters. This cimmit fixes ticket 1007.
* Merge branch '0.24.x' of git://wormwood.lovedthanlost.net/puppet into 0.24.xLuke Kanies2008-02-051-1/+8
|\
| * Added hostname test for hosts typeJames Turnbull2008-02-061-1/+8
| |
* | Merge commit 'turnbull/0.24.x' into 0.24.xLuke Kanies2008-02-041-0/+9
|\|
| * Updated fix for ticket #151 and added a testJames Turnbull2008-02-051-0/+9
| |
* | Fixing #1017 -- environment-specific modulepath is noLuke Kanies2008-02-042-2/+14
|/ | | | longer ignored. (Cherry-picked from master.)