summaryrefslogtreecommitdiffstats
path: root/test
Commit message (Collapse)AuthorAgeFilesLines
...
* | 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>
| * Adding warnings when example groups are skipped.Luke Kanies2008-10-081-0/+2
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
| * Moving some test/ package tests to rspec integration testsLuke Kanies2008-10-081-31/+0
| | | | | | | | | | | | and disabling a gem test that hangs forever for me. Signed-off-by: Luke Kanies <luke@madstop.com>
| * Fix #1109 - allow empty if or else branchesBrice Figureau2008-10-071-0/+9
| | | | | | | | | | | | | | | | | | | | | | | | This changesets allow empty if or else branches: if true { } else { } It works by emitting on the parser stack an AST node that doesn't do anything (a no-op). This allows the less intrusive code as no part of the if evaluation code has been touched.
| * Fix #1530: Correctly parse ssh type 1 keysFrancois Deppierraz2008-10-071-0/+3
| | | | | | | | | | This doesn't implies that puppet can managed SSH type 1 keys, it only ignores them.
| * Fixed additional environments testsJames Turnbull2008-10-051-2/+1
| |
| * Rspec Tests for #381.Brice Figureau2008-10-051-28/+0
| | | | | | | | Moved part of the old resource reference tests to rspec.
| * Minor test fix for #1614James Turnbull2008-10-041-1/+1
| |
| * Adding rspec tests for the Puppet::Util::Log class.Luke Kanies2008-10-041-59/+0
| | | | | | | | | | | | | | Also using Puppet::Util::Tagging to handle the tagging, rather than custom methods. Signed-off-by: Luke Kanies <luke@madstop.com>
| * Fixing #947 - pluginsync no longer fails poorly when no plugins existLuke Kanies2008-10-031-0/+8
| | | | | | | | | | | | Note that it still fails -- it's just a more reasonable failure. Signed-off-by: Luke Kanies <luke@madstop.com>
| * Fixing package provider tests to use the new Transaction::Change interfaceLuke Kanies2008-10-022-4/+3
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
| * Migrating the apt and dpkg tests to rspec.Luke Kanies2008-10-022-233/+0
| | | | | | | | | | | | | | I left the aptitude and aptrpm tests as an exercise for the reader. Signed-off-by: Luke Kanies <luke@madstop.com>
| * Fixed #791 - You should now be able to create and find a user/group in one ↵Luke Kanies2008-10-021-182/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | transaction. The real problem was that the 'gid' and 'uid' methods didn't handle the case where 'get_posix_field' didn't return a value, and the subsequent 'get_posix_field' calls couldn't handle that. This commit moves the tests for Posix to spec, and fixes the specific bug. Signed-off-by: Luke Kanies <luke@madstop.com>
| * Add parser for arbitrary expressionsBrice Figureau2008-10-012-0/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The expressions can be used in if 'test' and in the right side of assignements. The expressions can contain any number of sub-expressions combined by either arithmetic operators, comparison operators, or boolean operators. Random Usage Examples: $result = ((( $two + 2) / $one) + 4 * 5.45) - (6 << 7) + (0x800 + -9) or if ($a < 10) and ($a + 10 != 200) { ... }
* | Merge branch '0.24.x'Luke Kanies2008-09-232-0/+21
|\| | | | | | | | | | | | | | | | | | | Conflicts: lib/puppet/metatype/container.rb lib/puppet/metatype/instances.rb lib/puppet/metatype/metaparams.rb lib/puppet/metatype/relationships.rb lib/puppet/metatype/schedules.rb
| * Add rspec unit test for the append operatorBrice Figureau2008-09-211-0/+11
| | | | | | | | Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
| * Fixes tests broken by 95aa085James Turnbull2008-08-271-0/+10
| |
* | Merge commit 'turnbull/0.24.x'Luke Kanies2008-08-261-0/+8
|\|
| * Fix leaking LoadedFile when adding templates to be watchedAndrew Shafer2008-08-231-0/+8
| |
* | Merge branch '0.24.x'Luke Kanies2008-08-184-14/+140
|\| | | | | | | | | | | | | Conflicts: CHANGELOG test/util/posixtest.rb
| * Fixed $1456 - add proxy configuration to yum repoJames Turnbull2008-08-131-1/+7
| |
| * Add testing for the changes to resolve redmine #1427, where Kernel methods ↵Daniel Pittman2008-08-012-11/+108
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | shadow variables that the puppet template should otherwise see. Specific changes: * Added testing of the Scope#to_hash method, which returns a hash of name and value pairs in a scope or, optionally, in the scope and enclosing scopes. * Use member variables rather than methods in the function tests. * Fix up tests that fail once we move over to instance variables rather than methods: Puppet can no longer detect an undefined variable reference, so we end up failing any test that expected to get a parser error. * Several tests have manual checks introduced to simulate an end user manually writing the checks that used to be automatic, and others drop the validation that parsing fails when a variable is not in scope. * Added tests for legacy variable lookup and that the shadowing of local variables by Kernel methods is still in effect. Signed-off-by: Daniel Pittman <daniel@rimspace.net> (cherry picked from commit 5c5f315cceadc52203e53883b77bc01c1d7a2e7f)
| * Feature #1241 : Improve performance of group lookupsJames Turnbull2008-07-291-2/+15
| |
* | Merge branch '0.24.x' into mergingLuke Kanies2008-07-295-78/+164
|\| | | | | | | | | | | Conflicts: test/ral/type/filesources.rb
| * Fixing a cert test to pass on Darwin.Luke Kanies2008-07-211-17/+26
| | | | | | | | | | | | | | Darwin has a case-insensitive FS, so the test was failing because it was specifically testing case sensitivity. Signed-off-by: Luke Kanies <luke@madstop.com>
| * 'Fix' broken tests related to missing source raisingAndrew Shafer2008-07-201-18/+2
| | | | | | | | | | | | Issue 1437 In two cases, I removed the assertion that caused the failure. In one case, I changed the assertion to expect an exception.
| * You can now select the encoding format when transferring the catalog,Luke Kanies2008-07-181-0/+34
| | | | | | | | | | | | | | | | | | | | with 'yaml' still being the default but 'marshal' being an option. This is because testing has shown drastic performance differences between the two, with up to 70% of compile time being spent in YAML code. Use the 'catalog_format' setting to choose your format, and the setting must be set on the client. Signed-off-by: Luke Kanies <luke@madstop.com>