summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJeff McCune <jeff@puppetlabs.com>2011-10-03 11:51:05 -0700
committerJeff McCune <jeff@puppetlabs.com>2011-10-03 11:51:05 -0700
commit11c18377456bd32fc91af78300e4d4dad2c42a02 (patch)
treec774302c46efde850662da3c3588d93e21e6f570
parent05018f68199470f5f43e68bd54c7a39c9e8b3a85 (diff)
downloadpuppet-stdlib-11c18377456bd32fc91af78300e4d4dad2c42a02.tar.gz
puppet-stdlib-11c18377456bd32fc91af78300e4d4dad2c42a02.tar.bz2
(#9859) Add root_home fact and tests
Without this patch applied, the stdlib module does not provide a root_home fact. This fact is necessary to easily determine the root account home directory on platforms Puppet is supported on. The major variations this fact address are: --- solaris: / linux: /root macosx: /var/root Spec tests using rspec have been provided as well to cover these three general cases. Windows tests are marked as pending.
-rw-r--r--lib/facter/root_home.rb17
-rw-r--r--spec/spec_helper.rb1
-rw-r--r--spec/unit/facter/root_home_spec.rb42
3 files changed, 60 insertions, 0 deletions
diff --git a/lib/facter/root_home.rb b/lib/facter/root_home.rb
new file mode 100644
index 0000000..61fcf39
--- /dev/null
+++ b/lib/facter/root_home.rb
@@ -0,0 +1,17 @@
+# A facter fact to determine the root home directory.
+# This varies on PE supported platforms and may be
+# reconfigured by the end user.
+
+module Facter::Util::RootHome
+ class << self
+ def get_root_home
+ root_ent = Facter::Util::Resolution.exec("getent passwd root")
+ # The home directory is the sixth element in the passwd entry
+ root_ent.split(":")[5]
+ end
+ end
+end
+
+Facter.add(:root_home) do
+ setcode { Facter::Util::RootHome.get_root_home }
+end
diff --git a/spec/spec_helper.rb b/spec/spec_helper.rb
index fdc04bc..fc19cef 100644
--- a/spec/spec_helper.rb
+++ b/spec/spec_helper.rb
@@ -7,6 +7,7 @@ p dir
ARGV.clear
require 'puppet'
+require 'facter'
require 'mocha'
gem 'rspec', '>=2.0.0'
require 'rspec/expectations'
diff --git a/spec/unit/facter/root_home_spec.rb b/spec/unit/facter/root_home_spec.rb
new file mode 100644
index 0000000..8946d9d
--- /dev/null
+++ b/spec/unit/facter/root_home_spec.rb
@@ -0,0 +1,42 @@
+require 'spec_helper'
+require 'facter/root_home'
+
+describe Facter::Util::RootHome do
+ context "solaris" do
+ let(:root_ent) { "root:x:0:0:Super-User:/:/sbin/sh" }
+ let(:expected_root_home) { "/" }
+
+ it "should return /" do
+ Facter::Util::Resolution.expects(:exec).with("getent passwd root").returns(root_ent)
+ Facter::Util::RootHome.get_root_home.should == expected_root_home
+ end
+ end
+ context "linux" do
+ let(:root_ent) { "root:x:0:0:root:/root:/bin/bash" }
+ let(:expected_root_home) { "/root" }
+
+ it "should return /root" do
+ Facter::Util::Resolution.expects(:exec).with("getent passwd root").returns(root_ent)
+ Facter::Util::RootHome.get_root_home.should == expected_root_home
+ end
+ end
+ context "macosx" do
+ let(:root_ent) { "root:*:0:0:System Administrator:/var/root:/bin/sh" }
+ let(:expected_root_home) { "/var/root" }
+
+ it "should return /var/root" do
+ Facter::Util::Resolution.expects(:exec).with("getent passwd root").returns(root_ent)
+ Facter::Util::RootHome.get_root_home.should == expected_root_home
+ end
+ end
+ context "windows" do
+ let(:root_ent) { "FIXME TBD on Windows" }
+ let(:expected_root_home) { "FIXME TBD on Windows" }
+
+ it "should return FIXME TBD on windows" do
+ pending "FIXME: TBD on windows"
+ Facter::Util::Resolution.expects(:exec).with("getent passwd root").returns(root_ent)
+ Facter::Util::RootHome.get_root_home.should == expected_root_home
+ end
+ end
+end