summaryrefslogtreecommitdiffstats
path: root/test
Commit message (Collapse)AuthorAgeFilesLines
* Fix #636 - Allow extraneous comma in function argument listBrice Figureau2008-10-212-0/+12
| | | | Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* 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) { ... }
* 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
|
* Fix leaking LoadedFile when adding templates to be watchedAndrew Shafer2008-08-231-0/+8
|
* 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
|
* 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>
* Fixed #1431 - Provider confines must now specify similar tests in one call.Luke Kanies2008-07-171-45/+0
| | | | | | | | | | I.e., you can't do confine :operatingsystem => %w{a b} and then confine :operatingsystem => %w{b c}; you'd need to do them in one command. This now-obsolete behaviour does not seem to be used anywhere. The fix for #1431 is actually just removing the tests that exposed this change; the change happened when I refactored how confines work. Signed-off-by: Luke Kanies <luke@madstop.com>
* The mongrel-related tests now run without mongrel.Luke Kanies2008-07-171-0/+3
| | | | | | | | | | | Here were the main changes necessary: * Fixed the class loader so it only loads mongrel if it's available. * Fixed the test runner to skip example groups contained in non-runnable example groups. * Fixed the Mongrel tests to use quoted class names instead of constants, since the constants themselves would be absent. Signed-off-by: Luke Kanies <luke@madstop.com>
* Merge branch 'refactor/0.24.x/transaction_changes' of ↵James Turnbull2008-07-053-147/+6
|\ | | | | | | git://github.com/lak/puppet into 0.24.x
| * Refactoring the Transaction::Event class.Luke Kanies2008-07-042-4/+3
| | | | | | | | | | | | | | | | | | | | | | The class had a 'transaction' accessor that was assigned but never used, and it is simple enough that it needed direct arguments rather than named arguments. The rest of the code is changing the other classes that use Events. Signed-off-by: Luke Kanies <luke@madstop.com>
| * Adding tests to the Transaction::Change class.Luke Kanies2008-07-041-140/+0
| | | | | | | | | | | | | | There's a small amount of refactoring here, mostly removing code that appears to not be used at all. Signed-off-by: Luke Kanies <luke@madstop.com>
| * Renaming Puppet::Event to Puppet::Transaction::EventLuke Kanies2008-07-032-5/+5
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
| * Renaming the Puppet::PropertyChange class to Puppet::Transaction::Change.Luke Kanies2008-07-031-2/+2
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* | Updated test/ral/type/sshkey.rb testJames Turnbull2008-07-041-1/+0
|/
* Fixing #1382 - existing uppercase certs, keys, et al will be renamed.Luke Kanies2008-07-021-1/+19
| | | | This correctly renames the files and they still get read in.
* Fixed #1114 - Facts in plugin directories should now be autoloaded,Luke Kanies2008-07-011-6/+10
| | | | as long as you're using Facter 1.5.
* Fixed #707 - special '@reboot'-style cron jobs work again.Luke Kanies2008-06-171-4/+5
|
* Removed test/util/loadedfile.rb tests which fixes #1370James Turnbull2008-06-171-121/+0
|
* Fixes for install.rb running of tests that fixes #1267James Turnbull2008-05-271-0/+3
|
* Merge branch 'tickets/1265' of git://github.com/lak/puppet into 0.24.xJames Turnbull2008-05-271-0/+1
|\
| * Fixing #1265 -- the ca/client tests now all pass again.Luke Kanies2008-05-261-0/+1
| |
* | Always using the cert name to store yaml files, which fixes #1178.Luke Kanies2008-05-262-63/+37
|/ | | | | | | | | | | | | | | | | | | The Master handler previously provided the support for the :node_name setting, and that functionality has now been moved into the Node class. At the same time, the names to search through have been changed somewhat: Previously, the certificate name and the hostname were both used for searching, but now, the cert name is always searched first (unless node_name == facter), but only the Facter hostname, domain, and fqdn are used otherwise. We no longer split the cert name, only the hostname/domain/fqdn. In the general case, this provides no behaviour change, because people's hostname is the same as their certname. This only results in a change in behaviour if you specify a certificate name that is a normal node name, and you want to look that node up by something other than the full name in the certificate. Signed-off-by: Luke Kanies <luke@madstop.com>
* Fixing transaction support for prefetching generated resources.Luke Kanies2008-05-211-9/+48
| | | | | | Previously, we prefetched then generated, which caused generated resources that needed prefetching not to work. This just reorders the calls, so generated resources now get prefetched.
* Crontab provider: fix a parse error when a line begins with a space characterJames Turnbull2008-05-161-0/+28
| | | | | | Tests for Bug #1216 Updated CHANGELOG
* Moving all confine code out of the Provider class, and fixing #1197.Luke Kanies2008-05-151-1/+1
| | | | | | | | | I created a Confiner module for the Provider class methods, and then I enhanced the interface between it and the Confine class to make sure binary paths are searched for fresh each time. This fixes #1197, which was a result of binary paths being searched for at startup, rather than at execution.