Merge branch 'ticket/master/8663-windows-spec-tests'

* ticket/master/8663-windows-spec-tests:
  (#8663) Disable spec tests for unsupported functionality on Windows

Author: joshcooper

spec/integration/parser/compiler_spec.rb

@@ -13,7 +13,7 @@
     Puppet.settings.clear
   end
 
-  it "should be able to determine the configuration version from a local version control repository", :fails_on_windows => true do
+  it "should be able to determine the configuration version from a local version control repository" do
     # This should always work, because we should always be
     # in the puppet repo when we run this.
     version = %x{git rev-parse HEAD}.chomp

spec/integration/provider/mount_spec.rb

@@ -2,7 +2,7 @@
 
 require 'puppet/file_bucket/dipper'
 
-describe "mount provider (integration)", :fails_on_windows => true do
+describe "mount provider (integration)", :unless => Puppet.features.microsoft_windows? do
   include PuppetSpec::Files
 
   def create_fake_fstab(initially_contains_entry)

spec/integration/provider/ssh_authorized_key_spec.rb

@@ -3,7 +3,7 @@
 require 'spec_helper'
 require 'puppet/file_bucket/dipper'
 
-describe "ssh_authorized_key provider (integration)", :fails_on_windows => true do
+describe "ssh_authorized_key provider (integration)", :unless => Puppet.features.microsoft_windows? do
   include PuppetSpec::Files
 
   before :each do

spec/unit/provider/exec/shell_spec.rb

@@ -3,7 +3,7 @@
 
 provider_class = Puppet::Type.type(:exec).provider(:shell)
 
-describe provider_class, :fails_on_windows => true do
+describe provider_class, :unless => Puppet.features.microsoft_windows? do
   before :each do
     @resource = Puppet::Resource.new(:exec, 'foo')
     @provider = provider_class.new(@resource)

spec/unit/provider/ssh_authorized_key/parsed_spec.rb

@@ -5,7 +5,7 @@
 
 provider_class = Puppet::Type.type(:ssh_authorized_key).provider(:parsed)
 
-describe provider_class do
+describe provider_class, :unless => Puppet.features.microsoft_windows? do
   include PuppetSpec::Files
 
   before :each do
@@ -77,7 +77,7 @@ def genkey(key)
   end
 end
 
-describe provider_class do
+describe provider_class, :unless => Puppet.features.microsoft_windows? do
   before :each do
     @resource = Puppet::Type.type(:ssh_authorized_key).new(:name => "foo", :user => "random_bob")
 
@@ -128,7 +128,7 @@ def genkey(key)
       end
     end
 
-    describe "and a user has been specified with no target", :fails_on_windows => true do
+    describe "and a user has been specified with no target" do
       before :each do
         @resource[:user] = "nobody"
         #
@@ -189,7 +189,7 @@ def genkey(key)
       end
     end
 
-    describe "and a invalid user has been specified with no target", :fails_on_windows => true do
+    describe "and a invalid user has been specified with no target" do
       it "should catch an exception and raise a Puppet error" do
         @resource[:user] = "thisusershouldnotexist"
 

spec/unit/type/ssh_authorized_key_spec.rb

@@ -3,7 +3,7 @@
 
 ssh_authorized_key = Puppet::Type.type(:ssh_authorized_key)
 
-describe ssh_authorized_key do
+describe ssh_authorized_key, :unless => Puppet.features.microsoft_windows? do
   include PuppetSpec::Files
 
   before do
@@ -182,7 +182,7 @@
         proc { @class.new(:name => "whev", :type => :rsa, :target => "/tmp/here") }.should_not raise_error
       end
 
-      it "should use the user's path if not explicitly specified", :fails_on_windows => true do
+      it "should use the user's path if not explicitly specified" do
         @class.new(:name => "whev", :user => 'root').should(:target).should == File.expand_path("~root/.ssh/authorized_keys")
       end
 
@@ -228,7 +228,7 @@
   end
 
 
-  describe "when user is specified", :unless => Puppet.features.microsoft_windows? do
+  describe "when user is specified" do
 
     it "should determine target" do
       resource = @class.create(