summaryrefslogtreecommitdiffstats
path: root/test
Commit message (Collapse)AuthorAgeFilesLines
...
* Applying fix to xmlrpc client tests by Matt PalmerLuke Kanies2007-11-271-0/+1
|
* Fixing #796 -- the fileserver can now start with noLuke Kanies2007-11-271-2/+23
| | | | | | configuration file (it creates both default mount points if it does) and puppetmasterd no longer requires the configuration file to exist.
* Fixing the asuser method in Puppet::Util::SUIDManagerLuke Kanies2007-11-271-9/+27
| | | | | | | | so that it correctly just yields if you're not root. It also no longer tries to set :uid or :gid; just :euid and :egid, and it once again sets :egid before it sets :euid, which is important because you usually can't change your group after you've changed your user id.
* Fixing #931 by keeping track in configurations ofLuke Kanies2007-11-272-6/+7
| | | | | | | | | | what transportable resources get converted to, so different names don't throw it off. I also got rid of the Puppet::Type#merge method, which has been deprecated for ages but was still in there. I had to fix a few tests that weren't cleaning up after themselves as a result.
* Fixing #927 -- rewriting the test to actually test what it'sLuke Kanies2007-11-271-4/+9
| | | | | | | | | supposed to be doing and to skip the whole network connect thing. FTR, the reason this is happening is because the keepalive patch causes http instances to start immediately, which wasn't previously the case. I have been noticing that not all failures that should result from this happen consistently.
* Fixing the persistent and periodic schedule test failuresLuke Kanies2007-11-271-356/+0
| | | | by rewriting the schedule tests entirely.
* Fixing the breakage that I caused when I added the 'declared_feature?'Luke Kanies2007-11-261-3/+9
| | | | method to provider features.
* Fixing #920 -- I have replaced the existing mount test with anLuke Kanies2007-11-261-361/+0
| | | | | rspec version. It's not perfect, in that it only tests the :ensure state, but that's where 90% of the behaviour is.
* Fixing #919 -- installed packages used for testing are just ignored,Luke Kanies2007-11-261-0/+2
| | | | rather than throwing a failure.
* Rewriting the tests for the package resource type, fixing #930.Luke Kanies2007-11-262-159/+1
|
* Fixing #921, mostly by just deleting the existing test. I hadLuke Kanies2007-11-261-240/+0
| | | | | | already migrated all of the tests into rspec but forgot about these tests -- they were only in the rails/ subdir because people kept not running the parser/ tests after modifying the Rails code.
* Merge branch 'plugins_mount', fixing #891.Luke Kanies2007-11-242-1/+1
|\ | | | | | | | | | | Conflicts: CHANGELOG
| * Integrating Matt Palmer's patch to provide a 'plugins'Luke Kanies2007-11-242-1/+1
| | | | | | | | | | | | | | | | | | | | mount, fixing #891. The patch was ported to the current code by David Schmitt, I applied the rest of Matt's patches, and I then fixed all of the code so that the tests passed. The primary change I had to make to the patch was reenabling host expansion in paths -- his patch had disabled it.
* | Incorporating patch ↵Luke Kanies2007-11-241-178/+0
| | | | | | | | | | | | | | | | | | | | 20071030034736-6856b-6004090b3968cdbf7d366a03ee1c44e2160a3fe0.patch from womble, and rewriting and significantly enhancing the unit tests for the Puppet::Parser::Collector class; it should have full coverage now. There are no integration tests for it, so there's still no guarantee that it works at all, but hey, we're a lot better off than we were.
* | Integrating most of Matt Palmer's fromLuke Kanies2007-11-243-44/+22
|/ | | | | | | http://theshed.hezmatt.org/mattshacks/puppet/_patches/puppet-0.23.2/. There are still a few that haven't made it in, notably those related to the plugins module, which I'm planning on integrating separately.
* Fixing a couple of tests that were failing because of the environment changes.Luke Kanies2007-11-232-0/+2
|
* Adding some unit tests for the interface type before i go messing around with itLuke Kanies2007-11-231-39/+0
|
* Removing the manual ssldir setting by David in ↵Luke Kanies2007-11-191-1/+0
| | | | 59626cb3907d36e4fd762277daa76f523faf0908
* Merge commit 'davids-bugfixes/tests/fix-test-client-ca'Luke Kanies2007-11-192-0/+2
|\
| * fix failing CA test, when testing with incomplete setup (no ssldir, no DNS)David Schmitt2007-11-112-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 3) Failure: test_webxmlmix(TestBucket) [./network/handler/bucket.rb:222:in `test_webxmlmix' /home/david/Work/puppet/misspiggy/puppet-luke/test/lib/mocha/test_case_adapter.rb:19:in `__send__' /home/david/Work/puppet/misspiggy/puppet-luke/test/lib/mocha/test_case_adapter.rb:19:in `run']: Exception raised: Class: <OpenSSL::SSL::SSLError> Message: <"hostname was not match with the server certificate"> ---Backtrace--- /usr/lib/ruby/1.8/openssl/ssl.rb:91:in `post_connection_check' /usr/lib/ruby/1.8/net/http.rb:588:in `connect' /usr/lib/ruby/1.8/net/http.rb:553:in `do_start' /usr/lib/ruby/1.8/net/http.rb:548:in `start' /home/david/Work/puppet/misspiggy/puppet-luke/lib/puppet/network/xmlrpc/client.rb:141:in `start' /home/david/Work/puppet/misspiggy/puppet-luke/lib/puppet/network/client.rb:95:in `initialize' /home/david/Work/puppet/misspiggy/puppet-luke/lib/puppet/network/client/dipper.rb:16:in `initialize' ./network/handler/bucket.rb:223:in `new' ./network/handler/bucket.rb:223:in `test_webxmlmix' ./network/handler/bucket.rb:222:in `test_webxmlmix' /home/david/Work/puppet/misspiggy/puppet-luke/test/lib/mocha/test_case_adapter.rb:19:in `__send__' /home/david/Work/puppet/misspiggy/puppet-luke/test/lib/mocha/test_case_adapter.rb:19:in `run' ---------------
* | Fixing a failing test from my fix for #446 -- I had changedLuke Kanies2007-11-191-14/+0
| | | | | | | | | | | | the behaviour of Resource#override_parameter unintentionally. I've corrected the comments so it's clear why the original behaviour was there.
* | Fixing a few test suites that passed when run asLuke Kanies2007-11-199-0/+18
| | | | | | | | | | | | part of the suite, but failed when run individually. The problem was that I changed lib/puppettest/support/utils to have a separate module and I didn't test the individual files.
* | Merge commit 'davids-bugfixes/tests/fix-cron-tests'Luke Kanies2007-11-191-2/+2
|\ \
| * | fix crontests depending on ENV[USER] by using Etc.getpwuid(Process.uid) insteadDavid Schmitt2007-11-111-2/+2
| |/
* | Reverting the changes I'd made toward removing the globalLuke Kanies2007-11-1924-203/+328
| | | | | | | | | | | | | | | | resources. These are commits: c19835ce9f8a5138b30a1a32ca741c996b0916d2 9290cc89a2206fb5204578f8e91208857a48b147 ffb4c2dbc7314b364d25e4f7be599ef05b767b44
* | Merge commit 'davids-bugfixes/rest/fix-815-by-using-main-explicitly'Luke Kanies2007-11-181-2/+2
|\ \
| * | fix #815: add :main to all use() for :reporting and :metricsDavid Schmitt2007-11-091-2/+2
| |/
* | Fixed most failing tests, but there are still over thirty failing.Luke Kanies2007-11-1824-328/+203
| | | | | | | | | | | | At this point, I'm holding the experiment until after the release, so I'm committing this for now and will take it back up after 0.24.0 is out.
* | Fixing a parser test -- really, just removing testsLuke Kanies2007-11-131-17/+3
| | | | | | | | | | | | | | | | that belong with the AST classes rather than in the parser. Yeah, these tests need to be rewritten. Committed on an airplane. :)
* | Fixing #800 by refactoring how configurations are retrievedLuke Kanies2007-11-1214-264/+146
| | | | | | | | | | | | | | | | | | from the server. The real problem was getting all of the validation done before any caching, which required a good bit more refactoring than I expected. In actuality, this commit is relatively small even though it covers many files; most of the changes just make the code clearer or shorter.
* | Moving some compile tests to the spec/ directory, andLuke Kanies2007-11-121-73/+0
| | | | | | | | | | | | | | | | | | | | | | | | switching the node scope to no longer be lazy evaluation, just like I switched 'main'. When I made all of these classes and nodes lazy evaluated, I should have decoupled my real goal (using resources to evaluate them) from the idea of lazy-evaluating them, and this basically does that. I also changed the scope heirarchy slightly so that scopes will tend to be below the node scope, altho this was already generally the case.
* | Changing the 'main' class to no longer be lazy-evaluated.Luke Kanies2007-11-121-5/+8
|/ | | | | It was getting evaluated after node classes, which caused even stranger ordering issues.
* Merge branch 'master' of ssh://reductivelabs.com/opt/rl/git/puppetLuke Kanies2007-11-083-52/+6
|\
| * Switching the class resource evaluation to only happenLuke Kanies2007-11-081-1/+1
| | | | | | | | | | when using :include, not (for example) when evaluating node classes.
| * Classes once again get evaluated immediately when theLuke Kanies2007-11-083-52/+6
| | | | | | | | | | | | | | | | | | | | 'include' function is used, instead of being lazy-evaluated. Previous work caused resources to get created to model these classes, but in the process, I removed the fact that the classes were evaluated immediately. This meant that you couldn't guarantee that a class was evaluated before you went to use its variables.
* | Fixing the 'mount' tests so that they no longerLuke Kanies2007-11-084-270/+19
|/ | | | | modify the local system and they run fine as non-root users.
* Switching the base class for the Relationship class.Luke Kanies2007-11-082-74/+3
| | | | | | | | | | | | | | It was previously using the GRATR::Edge class, which had wonky overrides that dramatically slowed down sorting (its hash mechanism hashed the source and target so that edges with the same source/target got the same hash, which we actually don't want any more). This shouldn't change any functionality, just performance. I didn't retain all functionality from the Edge class, but a lot of that functionality was, um, horrible, like Edge[] being equivalent to Edge.new.
* Splitting the puppetd tests into two tests. It is still not a very good ↵Luke Kanies2007-11-081-27/+22
| | | | test, but I do not know of a good way to test this, really.
* Switching the graph base class from GRATR::DigraphLuke Kanies2007-11-071-31/+0
| | | | | | to Puppet::SimpleGraph, which should dramatically enhance performance. It should be largely functionally equivalent, with the only difference being that edges are no longer deduplicated.
* Changing the test/ classes so that they work from the mainLuke Kanies2007-10-26139-140/+142
| | | | | | test/ dir or from their own working dir, like the specs do. This was just a question of changing how their libraries are loaded.
* Fixing failing tests, including making the debian serviceLuke Kanies2007-10-132-19/+5
| | | | provider test work on non-Debian platforms.
* Translating the report handler to an indirected model.Luke Kanies2007-10-132-108/+8
| | | | | | | | | | | I've provided backward compatibility with the old handler. The only terminus type that currently exists for reports is the 'code' terminus, which is used to process reports in the style of the old handler. At some point, we should likely switch at least some of these report types (e.g., 'store') to terminus types.
* Merge branch 'master' of git://michaelobrien.info/puppet into michaelLuke Kanies2007-10-091-1/+10
|\
| * Merge branch 'parser-absent' of http://marcin.owsiany.pl/tmp/puppet into ↵Michael V. O'Brien2007-10-091-1/+10
| |\ | | | | | | | | | porridge
| | * Make it apparent that absent fields in a record have a value of :absent, ↵Marcin Owsiany2007-10-061-1/+10
| | | | | | | | | | | | which is different from what appears in a line.
* | | Merge branch 'master' of git://michaelobrien.info/puppet into michaelLuke Kanies2007-10-0915-148/+139
|\| |
| * | Merge branch 'master' of git://michaelobrien.info/puppetLuke Kanies2007-10-091-0/+71
| |\ \
| | * | Fixed #822. Applied patch provided by DavidS.Michael V. O'Brien2007-10-081-0/+71
| | | |
| * | | All tests should now pass again.Luke Kanies2007-10-087-54/+28
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is the first real pass towards using caching. The `puppet` executable actually uses the indirection work, instead of handlers and such (and man! is it cleaner). Most of this work was a result of trying to get the client-side story working, with correct yaml caching of configurations, which means this commit also covers converting configurations to yaml, which was a much bigger PITA than it needed to be. I still need to write integration tests, and I also need to cover the server-side story of a normal configuration retrieval.
| * | Another round of test-fixing around the changes I madeLuke Kanies2007-10-054-28/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | to the configuration system. 'puppet' itself still works, even with -e, but I expect that puppetd and puppetmasterd are broken, and there are still quite a few broken tests because the default fact store can't write but that's the default behaviour for a networked configuration master.