diff options
| author | Josh Partlow <joshua.partlow@puppetlabs.com> | 2014-09-16 18:11:34 -0700 |
|---|---|---|
| committer | Josh Partlow <joshua.partlow@puppetlabs.com> | 2014-09-16 18:11:34 -0700 |
| commit | 3f8bb6b892047a4b1cffc80c246b8b81670a857c (patch) | |
| tree | 0db9c1b723c73b5b156bac22ef0abf554096cb9c | |
| parent | 1d3fc2724c65c6fce455bc35a87b24b6c361205a (diff) | |
| download | puppet-3f8bb6b892047a4b1cffc80c246b8b81670a857c.tar.gz | |
(maint) Remove the resource/host/should_query test
This test was attempting to verify that the host provider will show an
entry as present in a target host file. But in fact, because Puppet has
just the single resource representing a host entry of the given name,
what this test is doing is ensuring that the 'localhost' host entry is
found in the given target file. On vms with two localhost entries (say
127.0.0.1 and ::1) this ends up moving the ::1 entry to the target file,
and the system continues to work. But on a system with a single
localhost entry in its /etc/hosts, this ends up moving that to the
target tmp file and crippling localhost name resolution for subsequent
tests. See PUP-1479 for more info.
| -rwxr-xr-x | acceptance/tests/resource/host/should_query.rb | 17 |
1 files changed, 0 insertions, 17 deletions
diff --git a/acceptance/tests/resource/host/should_query.rb b/acceptance/tests/resource/host/should_query.rb deleted file mode 100755 index a01889a9a..000000000 --- a/acceptance/tests/resource/host/should_query.rb +++ /dev/null @@ -1,17 +0,0 @@ -test_name "should query hosts out of a hosts file" - -agents.each do |agent| - file = agent.tmpfile('host-query') - - step "set up the system for the test" - on agent, "printf '127.0.0.1 localhost.local localhost\n' > #{file}" - - step "fetch the list of hosts from puppet" - on(agent, puppet_resource('host', 'localhost', "target=#{file}")) do - found = stdout.scan('present').length - fail_test "found #{found} hosts, not 1" if found != 1 - end - - step "clean up the system" - on agent, "rm -f #{file}" -end |