do not confine smartd fact to linux

samuraiii · Apr 11, 2014 · 57af86d · 57af86d
1 parent 8cbfc2b
commit 57af86d
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 16 deletions.
diff --git a/lib/facter/smartd.rb b/lib/facter/smartd.rb
@@ -1,6 +1,4 @@
 Facter.add(:smartd) do
-  confine :kernel => :linux
-
   setcode do
     Facter::Util::Resolution.which('smartd')
   end

diff --git a/spec/unit/facts/smartd_spec.rb b/spec/unit/facts/smartd_spec.rb
@@ -3,23 +3,18 @@
 describe 'smartd', :type => :fact do
   before(:each) { Facter.clear }
 
-  context 'on linux' do
-    context 'not in path' do
-      it do
-        Facter.fact(:kernel).stubs(:value).returns('Linux') 
-        Facter::Util::Resolution.stubs(:which).with('smartd').returns(nil)
-        Facter.fact(:smartd).value.should be_nil
-      end
+  context 'not in path' do
+    it do
+      Facter::Util::Resolution.stubs(:which).with('smartd').returns(nil)
+      Facter.fact(:smartd).value.should be_nil
     end
+  end
 
-    context 'in path' do
-      it do
-        Facter.fact(:kernel).stubs(:value).returns('Linux') 
-        Facter::Util::Resolution.stubs(:which).with('smartd').returns('/usr/sbin/smartd')
-        Facter.fact(:smartd).value.should == '/usr/sbin/smartd'
-      end
+  context 'in path' do
+    it do
+      Facter::Util::Resolution.stubs(:which).with('smartd').returns('/usr/sbin/smartd')
+      Facter.fact(:smartd).value.should == '/usr/sbin/smartd'
     end
   end
-
 end