summaryrefslogtreecommitdiffstats
path: root/spec
Commit message (Collapse)AuthorAgeFilesLines
...
* | Fixing tests broken by fileserving and other refactoring.Luke Kanies2008-11-081-5/+0
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* | Fixing a test broken by previous refactoringLuke Kanies2008-11-071-7/+4
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* | Finishing the refactoring of the resource generation interface.Luke Kanies2008-11-073-185/+62
| | | | | | | | | | | | | | All of the code works, and there are integration tests all around to prove it. I think. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Refactoring and clarifying the resource generation methods.Luke Kanies2008-11-071-0/+247
| | | | | | | | | | | | It now works with the new Catalog#add_resource method. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Refactoring Catalog#add_resource to correctly handle implicit resources.Luke Kanies2008-11-071-71/+101
| | | | | | | | | | | | | | | | | | | | | | | | | | | | It now ignores or removes implicit resources that conflict, and it yields all resources that are valid. This makes it simple for calling classes to pass in a list of resources but only perform a chunk of work for valid (i.e., non-conflicting) resources. This refactor is entirely meant as a way of cleaning up the Transaction#generate interface to the catalog. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Changing the catalog's relationship graph into a normal graph.Luke Kanies2008-11-061-67/+74
| | | | | | | | | | | | | | It was previously another catalog instance, but I was only ever actually using the graphing abilities. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Removing the PGraph class and subsuming it into SimpleGraph.Luke Kanies2008-11-063-444/+431
| | | | | | | | | | | | | | This class is a holdover from when I was using GRATR, and it's obsolete now. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Moving Catalog#write_graph to SimpleGraph, where it belongs.Luke Kanies2008-11-061-0/+25
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* | Converting the file 'source' property to a parameter.Luke Kanies2008-11-053-153/+119
| | | | | | | | | | | | | | | | | | | | | | | | | | | | 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>
* | Removing mid-transaction resources from the catalog.Luke Kanies2008-11-051-0/+14
| | | | | | | | | | | | This basically just cleans up dynamic resources. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Adding a method to Checksums to extract the sum typeLuke Kanies2008-11-052-0/+177
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* | Aliasing "must_not" just like we alias "must"Luke Kanies2008-11-051-0/+1
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* | Deduplicating slashes in the fileserving codeLuke Kanies2008-11-042-1/+6
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* | Fixing the augeas type tests to work when augeas is missingLuke Kanies2008-11-041-88/+98
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* | Switched all value management in props/params to internal classes.Luke Kanies2008-11-042-22/+609
| | | | | | | | | | | | | | | | | | | | | | 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>
* | Fixing #1677 - fixing the selinux tests in master.Luke Kanies2008-11-041-0/+6
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* | Changing the meaning of the unused Puppet::Type#parameter method to return ↵Luke Kanies2008-11-041-1/+16
| | | | | | | | | | | | | | | | | | | | an instance rather than a value. This parallels and largely obviates the 'property' method. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Moving a file purging test to rspecLuke Kanies2008-11-041-0/+49
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* | Fixing #1641 - file recursion now only passes original parameters to child ↵Luke Kanies2008-11-041-20/+25
| | | | | | | | | | | | | | | | | | resources. Previously, parameter values provided by remote sources or default values were all passed to children, which provided strange state maintenance. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Making it so (once again) files with sources set can still be deletedLuke Kanies2008-11-042-0/+45
| | | | | | | | | | | | (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-043-82/+294
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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>
* | Adding a starting point for spec tests for tidy.Luke Kanies2008-11-041-7/+12
| | | | | | | | | | | | | | | | | | | | So far it just validates that lstat is used instead of stat. Signed-off-by: Luke Kanies <luke@madstop.com> Conflicts: spec/unit/type/tidy.rb
* | Retrieving the CA certificate before the client certificate.Luke Kanies2008-11-031-8/+27
| | | | | | | | | | | | | | | | | | | | | | We have to have a CA cert first, because the host will start using the client cert as soon as it's available, but it's not functional without a CA cert. Also removing extra stupid stuff from wait_for_cert -- the connection is now always recycled, which is much simpler. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Handling the case where a symbol (e.g., :ca) is used for a certificate name.Luke Kanies2008-11-031-2/+2
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* | Fixing :bindaddress setting to work with the new server subsystem.Luke Kanies2008-11-031-0/+20
| | | | | | | | | | | | | | It now automatically switches values depending on server type, while still allowing overriding. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Fixing CertificateRequest#save to accept arguments.Luke Kanies2008-11-031-0/+7
| | | | | | | | | | | | This happens when called over REST. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Merge branch '0.24.x'Luke Kanies2008-10-319-47/+565
|\| | | | | | | | | | | Conflicts: CHANGELOG
| * Added unit tests for the augeas type and providerBryan Kearney2008-10-302-0/+341
| |
| * Fix #1402 - Allow multiline commentsBrice Figureau2008-10-291-1/+29
| | | | | | | | Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
| * Fix #857 - Multiple class of the same name don't append codeBrice Figureau2008-10-291-0/+28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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>
| * Fixing #1669 - The dump parameter can now be changed on mounts.Luke Kanies2008-10-291-0/+29
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
| * Fix #1682 - ASTArray should flatten product of evaluation of its childrenBrice Figureau2008-10-291-0/+66
| | | | | | | | | | If the ASTArray contains children that evaluate to arrays themselves, they aren't flattened.
| * Fixing #1660 - Adding specifically supported values for tidy recursion.Luke Kanies2008-10-281-0/+60
| | | | | | | | | | | | Basically just copied the code for the recurse parameter from 'file'. Signed-off-by: Luke Kanies <luke@madstop.com>
| * Fixing #1698 - all logs again show up in the report.Luke Kanies2008-10-281-46/+0
| |
| * Fixed #1104 - Classes and nodes should set $name variablesBrice Figureau2008-10-281-0/+13
| | | | | | | | Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* | Merge branch '0.24.x'Luke Kanies2008-10-228-18/+409
|\| | | | | | | | | | | Conflicts: lib/puppet/type/user.rb
| * Fixes #1663 - added Symbol check and additional testSean E. Millichamp2008-10-231-0/+9
| |
| * Fix SELinux test to succeed when Puppet debug mode is enabledSean E. Millichamp2008-10-221-0/+1
| |
| * Fix regression caused by switch to Puppet's execute() functionsSean E. Millichamp2008-10-221-1/+1
| |
| * Solaris RBAC AttributesAndrew Shafer2008-10-224-15/+345
| | | | | | | | | | | | | | | | | | | | | | Adding support to user type for: profiles auths project key/value pairs Refactored useradd.addcmd so I could override how properties get added in the subclass Added keyvalue property to manage generic keyvalues
| * Fix #936 - Allow trailing comma in array definitionBrice Figureau2008-10-211-0/+8
| | | | | | | | Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
| * Fix #1115 - part2 - fix tests and add all_tagsBrice Figureau2008-10-211-1/+7
| | | | | | | | | | | | | | | | Up until this patch, TemplateWrapper.tags was returning all the tags defined in the catalog. I think this is wrong and tags shoul only return the defined tags in the current scope. Hence, I defined a all_tags method that returns the list of tags defined in the whole catalog.
| * Fixed #1663 - Regression relating to facter fact naming from 0.24.5James Turnbull2008-10-211-1/+14
| |
| * Fix #636 - Allow extraneous comma in function argument listBrice Figureau2008-10-211-0/+24
| | | | | | | | 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-1746-146/+2915
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | 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 tests I broke when trying to fix the Providers reference.Luke Kanies2008-10-171-5/+15
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
| * Use fully qualified paths when calling binaries, adjust chcon call to use ↵Sean E. Millichamp2008-10-171-8/+8
| | | | | | | | Puppet's execute() function.
| * Setting SELinux contexts with chcon should not dereference symbolic linksSean E. Millichamp2008-10-171-5/+5
| |
| * Fix #1115 - Allow checking tags/classes from ERb templatesBrice Figureau2008-10-141-0/+14
| | | | | | | | | | | | | | | | Add to predefined variables to the ERB template space. "classes" and "tags" which provide arrays of all the defined classes and tags for a node. Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
| * Fixing #1647 - puppetdoc's 'providers' report works again.Luke Kanies2008-10-141-0/+17
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>