summaryrefslogtreecommitdiffstats
path: root/spec/unit/provider
Commit message (Collapse)AuthorAgeFilesLines
...
* Fix for #2736, target doesn't work for ssh_authorized_keysMarkus Roberts2009-10-211-51/+28
| | | | | | | | There were a number of problems here (duplicated code, meaningless tests, etc.) but the core was that the last definition of target ignored the provided value if there was also a user specified. Signed-off-by: Markus Roberts <Markus@reality.com>
* Fix #2642 - Runit provider reworkBrice Figureau2009-09-221-3/+13
| | | | | | | | | | | The runit provider was left broken after some work on daemontools on which runit is based, and #2640 didn't override the restart command, so daemontools once was called. This patch aims to fix this provider and bring it on par with daemontools. Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* Fix #2640 - Daemontools and Runit were not creating the enable symlinkBrice Figureau2009-09-162-3/+21
| | | | | | | | Due to an incorrect tests, those providers weren't enabling themselves when starting, thus failing to create the symlink necessary for them to run. Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* Fix for ticket #2618 (stubbing so redhat tests run under debian)Markus Roberts2009-09-091-0/+2
| | | | | | | | This adds stubs so that the service script used by redhat appears to exist while the tests are running, regardless of the operating system being used to test. Signed-off-by: Markus Roberts <Markus@reality.com>
* Updated permissions on test filesJames Turnbull2009-09-055-0/+0
|
* Refactoring tests to reduce code size, increase coverage, and makeMarkus Roberts2009-09-052-111/+49
| | | | Luke happy.
* This further normalizes the handling of init-style services (includingMarkus Roberts2009-09-052-0/+262
| | | | | | | | | the redhat "service" wrapper script). Removes special case handling of non-zero exit code in redhat (base already did this) and centralizes scattered @resource[:has_____] checks. Tests that proper versions of each are called and one level of fallbacks. Signed-off-by: Markus Roberts <Markus@reality.com>
* Fixed #2589 - Renamed zfs delete to destroy and added testsJames Turnbull2009-09-051-2/+14
|
* Fixes #2581. Use new 10.6 global launchd overrides file for service ↵Nigel Kersten2009-09-031-0/+58
| | | | status/enabled
* Fixes #2513. debian service provider now uses invoke-rc.d to determine ↵Nigel Kersten2009-08-181-0/+89
| | | | enabled? status
* Fixing typo in two tests which caused them to always passMarkus Roberts2009-07-311-2/+2
|
* Fix #2448 - fix the broken runit provider and testsBrice Figureau2009-07-291-1/+13
| | | | | | | The runit provider was broken when the daemontools provider was enhanced.This patch aims to fix this. Signed-off-by: Brice Figureau <brice-puppet@daysofwonder.com>
* Fixing #2445 - fixing the mount test mockLuke Kanies2009-07-241-1/+2
| | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* Fixing #2197 - daemontools tests now passLuke Kanies2009-06-261-13/+38
| | | | | | | This actually involved a bit of rewriting of the code, but the code's simpler now, too. Signed-off-by: Luke Kanies <luke@madstop.com>
* * provider/augeas: strip whitespace and ignore blank linesDavid Lutterkort2009-06-121-0/+13
| | | | Signed-off-by: James Turnbull <james@lovedthanlost.net>
* #2332: Remove trailing slashes from path commands in the pluginBryan Kearney2009-06-091-0/+6
|
* Removed extra whitespace from end of linesIan Taylor2009-06-0613-76/+76
|
* * provider/augeas: minor spec test cleanupDavid Lutterkort2009-06-031-29/+30
| | | | | | - rename some tests to make their purpose clearer - add a test for nested predicates - remove trailing whitespace
* * provider/augeas: allow escaped whitespace and brackets in pathsDavid Lutterkort2009-06-031-0/+7
|
* * provider/augeas: match comparison uses '==' and '!=' againDavid Lutterkort2009-06-031-4/+4
| | | | | | Change the syntax for match in an onlyif back to using '==' for equality comparison instead of 'eq' (this was the behavior before 9a2642) and make operator for inequality '!='.
* Brought in lutters parse_commands patch and integrated it into the type.David Lutterkort2009-06-031-109/+105
| | | | | | | | | | | | | | | | | | | | | This includes reworking the get and match commands as well. This change introduces a few small changes. These are: 1) There can be no ' or " characters around path elements. 2) The context is now only prepended to relative paths. So, if a path is specified as /foo/bar the context will not be appended. It will only be appended if it is foo/bar 3) The syntax for array matching in the onlyif is now eq or noteq. It was == before. 4) The get and set commnands used to concatenate all items at the end of a string so "set path to some value" would be interpreted as "set" "path" "to some value". This is no longer supported. The caller must put ' or " around the "to some value" for it to work" Cherry-pick of 9a2642 from 0.24.x Conflicts: lib/puppet/provider/augeas/augeas.rb
* Fix #2142 - Convert pkgdmg provider to use plists instead of string scanning ↵Nigel Kersten2009-04-221-0/+73
| | | | | | | | | | | | | | for future proofing update pkgdmg patch with feedback from dev-list initial checking of pkgdmg package provider tests clean up fail conditions to raise Puppet::Error instead Finalized tests for pkgdmg provider remove duplicate facter/util/plist require
* Fixing all tests that were apparently broken in the 0.24.x merge.Luke Kanies2009-04-021-0/+1
| | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* Merge branch '0.24.x'Luke Kanies2009-04-023-39/+121
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: bin/ralsh lib/puppet/executables/client/certhandler.rb lib/puppet/parser/functions/versioncmp.rb lib/puppet/parser/resource/reference.rb lib/puppet/provider/augeas/augeas.rb lib/puppet/provider/nameservice/directoryservice.rb lib/puppet/provider/ssh_authorized_key/parsed.rb lib/puppet/type.rb lib/puppet/type/file/checksum.rb spec/integration/defaults.rb spec/integration/transaction/report.rb spec/unit/executables/client/certhandler.rb spec/unit/indirector/ssl_rsa/file.rb spec/unit/node/catalog.rb spec/unit/provider/augeas/augeas.rb spec/unit/rails.rb spec/unit/type/ssh_authorized_key.rb spec/unit/type/tidy.rb test/executables/filebucket.rb test/executables/puppetbin.rb
| * Fixed #2004 - ssh_authorized_key fails if no target is definedFrancois Deppierraz2009-03-241-22/+2
| | | | | | | | | | | | | | This commit depends on 7f291afdacf59f762c3b78481f5420ec8919e46d (fixing #1629) which was cherry-picked from master. Signed-off-by: Francois Deppierraz <francois@ctrlaltdel.ch>
| * Fix Bug #1629Francois Deppierraz2009-03-241-0/+64
| | | | | | | | | | A refactoring of ssh_authorized_key parsed provider was needed and tests were improved. flush method has been split for clarity.
| * Added a force option to ensure the change is always applied, and call augeas ↵Bryan Kearney2009-03-041-91/+121
| | | | | | | | twice to reduce the chance that data is lost
| * Backport the fix for #1835Bryan Kearney2009-03-041-0/+80
| |
| * First cut at the not running if augeas does not change any of the ↵Bryan Kearney2009-03-041-9/+16
| | | | | | | | underlieing files
| * Fixing every failing test I can find on the build server.Luke Kanies2009-02-281-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | All but one of these tests is fixed by: * Stubbing Puppet.settings.use * Adding /usr/sbin to PATH The only other one was the package integration test, which stupidly assumed a default was specified in the test. The fix here is twofold: Remove that assumption (the test is now 'pending' if no default is available), and add a default for Ubuntu. (The default is in the test, not the default package provider - that is, it's testing the default package provider, thus it can't rely on that information.) Signed-off-by: Luke Kanies <luke@madstop.com>
* | Fixed #1849 - Ruby 1.9 portability: `when' doesn't like colons, replace with ↵James Turnbull2009-02-261-1/+1
| | | | | | | | semicolons
* | 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-142-4/+21
|\| | | | | | | | | 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.
| * 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>
* | Merge branch '0.24.x'Luke Kanies2009-02-132-10/+56
|\| | | | | | | | | | | | | | | | | | | 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
| * Bug 1948: Add logic and testing for the command parsing logicBryan Kearney2009-02-141-4/+13
| |
| * Fixing #1541 - ParsedFile only backs up files once per transactionLuke Kanies2009-02-131-0/+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>
| * Bug #1948: Added patch by jab to support the correct ins syntax. Updated the ↵Bryan Kearney2009-02-131-12/+13
| | | | | | | | test cases as well
* | Merge branch '0.24.x'Luke Kanies2009-02-112-0/+108
|\| | | | | | | | | | | Conflicts: CHANGELOG spec/unit/type/file/selinux.rb
| * Prefetching, and thus purging, Nagios resources now worksLuke Kanies2009-02-112-0/+58
| | | | | | | | | | | | | | | | | | | | | | *only* if you use the default configuration file locations. In the end, this was a relatively minor change; most of the actual diff centers around making the code more readable so I could think my way into the fix, and adding tests for cases that were either untested or refactored slightly. Signed-off-by: Luke Kanies <luke@madstop.com>
| * Adding some basic tests for the Naginator provider base classLuke Kanies2009-02-111-0/+50
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* | Fix Bug #1629Francois Deppierraz2009-01-131-0/+64
| | | | | | | | | | A refactoring of ssh_authorized_key parsed provider was needed and tests were improved. flush method has been split for clarity.
* | Fix #1835 : Add whitespace/quote parsing toBryan Kearney2009-01-131-76/+156
| |
* | Merge branch '0.24.x'James Turnbull2008-12-271-3/+24
|\| | | | | | | | | Conflicts: CHANGELOG
| * Issue 1804 VDev with the same devices should be in syncAndrew Shafer2008-12-151-3/+24
| | | | | | | | | | Added VDev and MultiVDev properties to the ZPool type to handle logic Vdevs with the same devices are now in sync even if the order changes
* | Deprecating the Puppet::Type.create.Luke Kanies2008-12-181-1/+1
| | | | | | | | | | | | | | This method is no longer necessary; you can use the normal 'new' class method. Signed-off-by: Luke Kanies <luke@madstop.com>
* | Deprecating 'Puppet.type'; replacing all instances with Puppet::Type.typeLuke Kanies2008-12-182-2/+2
| | | | | | | | Signed-off-by: Luke Kanies <luke@madstop.com>
* | Merge branch '0.24.x'Luke Kanies2008-12-093-3/+150
|\| | | | | | | | | | | | | | | | | Conflicts: CHANGELOG lib/puppet/type/tidy.rb spec/unit/type/file/ensure.rb spec/unit/type/tidy.rb