Merge pull request #334 from timsutton/fix-osxpkg-spec-skip

OS X spec tests, properly skip on non-OS X

spec/fpm/package/osxpkg_spec.rb

@@ -2,13 +2,13 @@
 require "fpm" # local
 require "fpm/package/osxpkg" # local
 
-describe FPM::Package::OSXpkg do
-
-  if %x{uname -s}.chomp != "Darwin"
-    Cabin::Channel.get("rspec").warn("Skipping OS X tests because " \
-      "this system is #{%x{uname -s}.chomp}, Darwin required")
-  end
+platform_is_darwin = (%x{uname -s}.chomp == "Darwin")
+if !platform_is_darwin
+  Cabin::Channel.get("rspec").warn("Skipping OS X pkg tests requiring 'pkgbuild', " \
+      "which requires a Darwin platform.")
+end
 
+describe FPM::Package::OSXpkg do
   describe "#identifier" do
     it "should be of the form reverse.domain.pkgname" do
       subject.name = "name"
@@ -34,7 +34,7 @@
     end
   end
 
-  describe "#output" do 
+  describe "#output", :if => platform_is_darwin do 
     before :all do
       # output a package, use it as the input, set the subject to that input
       # package. This helps ensure that we can write and read packages
@@ -69,5 +69,4 @@
       end
     end # package attributes
   end # #output
-
 end # describe FPM::Package:OSXpkg