win32-file-security 1.0.6 → 1.0.7

data/test/test_win32_file_security_ffi.rb

@@ -1,33 +1,33 @@
-#############################################################################
-# test_win32_file_security_ffi.rb
-#
-# Tests to ensure that the FFI functions are private
-#############################################################################
-require 'test-unit'
-require 'win32/file/security'
-
-class TC_Win32_File_Security_FFI < Test::Unit::TestCase
-  def setup
-    @singleton_methods = File.methods.map{ |m| m.to_s }
-    @instance_methods  = File.instance_methods.map{ |m| m.to_s }
-  end
-
-  test "internal ffi functions are not public as singleton methods" do
-    assert_false(@singleton_methods.include?('AddAce'))
-    assert_false(@singleton_methods.include?('CloseHandle'))
-    assert_false(@singleton_methods.include?('GetFileSecurityW'))
-    assert_false(@singleton_methods.include?('PathIsRootW'))
-  end
-
-  test "internal ffi functions are not public as instance methods" do
-    assert_false(@instance_methods.include?('AddAce'))
-    assert_false(@instance_methods.include?('CloseHandle'))
-    assert_false(@instance_methods.include?('GetFileSecurityW'))
-    assert_false(@instance_methods.include?('PathIsRootW'))
-  end
-
-  def teardown
-    @singleton_methods = nil
-    @instance_methods = nil
-  end
-end
+#############################################################################
+# test_win32_file_security_ffi.rb
+#
+# Tests to ensure that the FFI functions are private
+#############################################################################
+require 'test-unit'
+require 'win32/file/security'
+
+class TC_Win32_File_Security_FFI < Test::Unit::TestCase
+  def setup
+    @singleton_methods = File.methods.map{ |m| m.to_s }
+    @instance_methods  = File.instance_methods.map{ |m| m.to_s }
+  end
+
+  test "internal ffi functions are not public as singleton methods" do
+    assert_false(@singleton_methods.include?('AddAce'))
+    assert_false(@singleton_methods.include?('CloseHandle'))
+    assert_false(@singleton_methods.include?('GetFileSecurityW'))
+    assert_false(@singleton_methods.include?('PathIsRootW'))
+  end
+
+  test "internal ffi functions are not public as instance methods" do
+    assert_false(@instance_methods.include?('AddAce'))
+    assert_false(@instance_methods.include?('CloseHandle'))
+    assert_false(@instance_methods.include?('GetFileSecurityW'))
+    assert_false(@instance_methods.include?('PathIsRootW'))
+  end
+
+  def teardown
+    @singleton_methods = nil
+    @instance_methods = nil
+  end
+end

data/test/test_win32_file_security_ownership.rb

@@ -1,174 +1,174 @@
-#############################################################################
-# test_win32_file_ownership.rb
-#
-# Test case for the file ownership related methods
-#############################################################################
-require 'etc'
-require 'socket'
-require 'sys/admin'
-require 'test-unit'
-require 'win32/security'
-require 'win32/file/security'
-require 'pathname'
-
-class TC_Win32_File_Security_Ownership < Test::Unit::TestCase
-  extend FFI::Library
-  ffi_lib :netapi32
-  attach_function :NetGetDCName, [:pointer, :pointer, :buffer_out], :int
-  attach_function :NetApiBufferFree, [:pointer], :int
-
-  # Helper method to determine if we're on a domain controller
-  def self.in_domain?
-    bool = true
-    buf = (0.chr * 256).encode('UTF-16LE')
-    rv = NetGetDCName(nil, nil, buf)
-    bool = false if rv != 0
-    NetApiBufferFree(buf)
-    bool
-  end
-
-  def self.startup
-    Dir.chdir(File.dirname(File.expand_path(File.basename(__FILE__))))
-    @@file = File.join(Dir.pwd, 'ownership_test.txt')
-
-    @@host  = Socket.gethostname
-    @@temp  = "Temp"
-    @@login = Etc.getlogin
-    @@domain = in_domain?
-
-    if Win32::Security.elevated_security?
-      Sys::Admin.add_user(:name => @@temp, :description => "Delete Me")
-    end
-  end
-
-  def setup
-    @elevated = Win32::Security.elevated_security?
-    File.open(@@file, 'w'){ |fh| fh.puts "This is an ownership test." }
-  end
-
-  test "owned? method basic functionality" do
-    assert_respond_to(File, :owned?)
-    assert_nothing_raised{ File.owned?(@@file) }
-    assert_boolean(File.owned?(@@file))
-  end
-
-  test "owned? method returns expected result" do
-    if Win32::Security.elevated_security?
-      assert_false(File.owned?(@@file))
-    else
-      assert_true(File.owned?(@@file))
-    end
-    assert_false(File.owned?("C:\\Windows\\regedit.exe"))
-  end
-
-  test "owned? requires a single argument" do
-    assert_raise(ArgumentError){ File.owned? }
-    assert_raise(ArgumentError){ File.owned?(@@file, @@file) }
-  end
-
-  test "owner method basic functionality" do
-    assert_respond_to(File, :owner)
-    assert_nothing_raised{ File.owner(@@file) }
-    assert_kind_of(String, File.owner(@@file))
-  end
-
-  test "owner method returns the expected value" do
-    if Win32::Security.elevated_security?
-      expected = "BUILTIN\\Administrators"
-    else
-      expected = @@host + "\\" + @@login
-    end
-    assert_equal(expected, File.owner(@@file))
-  end
-
-  test "owner method requires a single argument" do
-    assert_raise(ArgumentError){ File.owner }
-    assert_raise(ArgumentError){ File.owner(@@file, @@file) }
-  end
-
-  test "owner allows a pathname object" do
-    assert_nothing_raised{ File.owner(Pathname.new(@@file)) }
-  end
-
-  test "grpowned? method basic functionality" do
-    assert_respond_to(File, :grpowned?)
-    assert_nothing_raised{ File.grpowned?(@@file) }
-    assert_boolean(File.grpowned?(@@file))
-  end
-
-  test "grpowned? method returns expected result" do
-    if Win32::Security.elevated_security? && @@domain
-      assert_false(File.grpowned?(@@file))
-    else
-      assert_true(File.grpowned?(@@file))
-    end
-    assert_false(File.grpowned?("C:\\Windows\\regedit.exe"))
-  end
-
-  test "grpowned? requires a single argument" do
-    assert_raise(ArgumentError){ File.grpowned? }
-    assert_raise(ArgumentError){ File.grpowned?(@@file, @@file) }
-  end
-
-  test "group method basic functionality" do
-    assert_respond_to(File, :group)
-    assert_nothing_raised{ File.group(@@file) }
-    assert_kind_of(String, File.group(@@file))
-  end
-
-  test "group method returns the expected value" do
-    if Win32::Security.elevated_security? && @@domain
-      expected = "BUILTIN\\Administrators"
-    else
-      expected = @@host + "\\None"
-    end
-    assert_equal(expected, File.group(@@file))
-  end
-
-  test "group method allows a pathname object" do
-    assert_nothing_raised{ File.group(Pathname.new(@@file)) }
-  end
-
-  test "group method requires a stringy argument" do
-    assert_raise(TypeError){ File.group(nil) }
-    assert_raise(TypeError){ File.group([]) }
-  end
-
-  test "chown method basic functionality" do
-    assert_respond_to(File, :chown)
-  end
-
-  test "chown works as expected" do
-    omit_unless(@elevated)
-    original_owner = File.owner(@@file)
-    expected_owner = @@host + "\\" + @@temp
-
-    assert_nothing_raised{ File.chown(@@temp, nil, @@file) }
-    assert_equal(expected_owner, File.owner(@@file))
-    assert_nothing_raised{ File.chown(original_owner, nil, @@file) }
-    assert_equal(original_owner, File.owner(@@file))
-  end
-
-  test "chown returns the number of files processed" do
-    omit_unless(@elevated)
-    assert_equal(1, File.chown(@@temp, nil, @@file))
-  end
-
-  test "chown requires at least two arguments" do
-    assert_raise(ArgumentError){ File.chown }
-    assert_raise(ArgumentError){ File.chown(@@temp) }
-  end
-
-  def teardown
-    @elevated = nil
-    File.delete(@@file) if File.exist?(@@file)
-  end
-
-  def self.shutdown
-    Sys::Admin.delete_user(@@temp) if Win32::Security.elevated_security?
-    @@file = nil
-    @@login = nil
-    @@host = nil
-    @@domain = nil
-  end
-end
+#############################################################################
+# test_win32_file_ownership.rb
+#
+# Test case for the file ownership related methods
+#############################################################################
+require 'etc'
+require 'socket'
+require 'sys/admin'
+require 'test-unit'
+require 'win32/security'
+require 'win32/file/security'
+require 'pathname'
+
+class TC_Win32_File_Security_Ownership < Test::Unit::TestCase
+  extend FFI::Library
+  ffi_lib :netapi32
+  attach_function :NetGetDCName, [:pointer, :pointer, :buffer_out], :int
+  attach_function :NetApiBufferFree, [:pointer], :int
+
+  # Helper method to determine if we're on a domain controller
+  def self.in_domain?
+    bool = true
+    buf = (0.chr * 256).encode('UTF-16LE')
+    rv = NetGetDCName(nil, nil, buf)
+    bool = false if rv != 0
+    NetApiBufferFree(buf) if bool
+    bool
+  end
+
+  def self.startup
+    Dir.chdir(File.dirname(File.expand_path(File.basename(__FILE__))))
+    @@file = File.join(Dir.pwd, 'ownership_test.txt')
+
+    @@host  = Socket.gethostname
+    @@temp  = "Temp"
+    @@login = Etc.getlogin
+    @@domain = in_domain?
+
+    if Win32::Security.elevated_security?
+      Sys::Admin.add_user(:name => @@temp, :description => "Delete Me")
+    end
+  end
+
+  def setup
+    @elevated = Win32::Security.elevated_security?
+    File.open(@@file, 'w'){ |fh| fh.puts "This is an ownership test." }
+  end
+
+  test "owned? method basic functionality" do
+    assert_respond_to(File, :owned?)
+    assert_nothing_raised{ File.owned?(@@file) }
+    assert_boolean(File.owned?(@@file))
+  end
+
+  test "owned? method returns expected result" do
+    if Win32::Security.elevated_security?
+      assert_false(File.owned?(@@file))
+    else
+      assert_true(File.owned?(@@file))
+    end
+    assert_false(File.owned?("C:\\Windows\\regedit.exe"))
+  end
+
+  test "owned? requires a single argument" do
+    assert_raise(ArgumentError){ File.owned? }
+    assert_raise(ArgumentError){ File.owned?(@@file, @@file) }
+  end
+
+  test "owner method basic functionality" do
+    assert_respond_to(File, :owner)
+    assert_nothing_raised{ File.owner(@@file) }
+    assert_kind_of(String, File.owner(@@file))
+  end
+
+  test "owner method returns the expected value" do
+    if Win32::Security.elevated_security?
+      expected = "BUILTIN\\Administrators"
+    else
+      expected = @@host + "\\" + @@login
+    end
+    assert_equal(expected, File.owner(@@file))
+  end
+
+  test "owner method requires a single argument" do
+    assert_raise(ArgumentError){ File.owner }
+    assert_raise(ArgumentError){ File.owner(@@file, @@file) }
+  end
+
+  test "owner allows a pathname object" do
+    assert_nothing_raised{ File.owner(Pathname.new(@@file)) }
+  end
+
+  test "grpowned? method basic functionality" do
+    assert_respond_to(File, :grpowned?)
+    assert_nothing_raised{ File.grpowned?(@@file) }
+    assert_boolean(File.grpowned?(@@file))
+  end
+
+  test "grpowned? method returns expected result" do
+    if Win32::Security.elevated_security? && @@domain
+      assert_false(File.grpowned?(@@file))
+    else
+      assert_true(File.grpowned?(@@file))
+    end
+    assert_false(File.grpowned?("C:\\Windows\\regedit.exe"))
+  end
+
+  test "grpowned? requires a single argument" do
+    assert_raise(ArgumentError){ File.grpowned? }
+    assert_raise(ArgumentError){ File.grpowned?(@@file, @@file) }
+  end
+
+  test "group method basic functionality" do
+    assert_respond_to(File, :group)
+    assert_nothing_raised{ File.group(@@file) }
+    assert_kind_of(String, File.group(@@file))
+  end
+
+  test "group method returns the expected value" do
+    if Win32::Security.elevated_security? && @@domain
+      expected = "BUILTIN\\Administrators"
+    else
+      expected = @@host + "\\None"
+    end
+    assert_equal(expected.downcase, File.group(@@file).downcase)
+  end
+
+  test "group method allows a pathname object" do
+    assert_nothing_raised{ File.group(Pathname.new(@@file)) }
+  end
+
+  test "group method requires a stringy argument" do
+    assert_raise(TypeError){ File.group(nil) }
+    assert_raise(TypeError){ File.group([]) }
+  end
+
+  test "chown method basic functionality" do
+    assert_respond_to(File, :chown)
+  end
+
+  test "chown works as expected" do
+    omit_unless(@elevated)
+    original_owner = File.owner(@@file).downcase
+    expected_owner = (@@host + "\\" + @@temp).downcase
+
+    assert_nothing_raised{ File.chown(@@temp, nil, @@file) }
+    assert_equal(expected_owner, File.owner(@@file).downcase)
+    assert_nothing_raised{ File.chown(original_owner, nil, @@file) }
+    assert_equal(original_owner, File.owner(@@file).downcase)
+  end
+
+  test "chown returns the number of files processed" do
+    omit_unless(@elevated)
+    assert_equal(1, File.chown(@@temp, nil, @@file))
+  end
+
+  test "chown requires at least two arguments" do
+    assert_raise(ArgumentError){ File.chown }
+    assert_raise(ArgumentError){ File.chown(@@temp) }
+  end
+
+  def teardown
+    @elevated = nil
+    File.delete(@@file) if File.exist?(@@file)
+  end
+
+  def self.shutdown
+    Sys::Admin.delete_user(@@temp) if Win32::Security.elevated_security?
+    @@file = nil
+    @@login = nil
+    @@host = nil
+    @@domain = nil
+  end
+end

data/test/test_win32_file_security_permissions.rb

@@ -1,87 +1,88 @@
-##############################################################################
-# test_win32_file_permissions.rb
-#
-# Test case for permission related methods of win32-file-security. You should
-# use the 'rake test' or 'rake test:perms' task to run this.
-##############################################################################
-require 'test-unit'
-require 'win32/file/security'
-require 'socket'
-require 'etc'
-
-class TC_Win32_File_Security_Permissions < Test::Unit::TestCase
-  def self.startup
-    @@user = Etc.getlogin
-    @@host = Socket.gethostname
-    @@file = File.join(Dir.pwd, 'security_test.txt')
-
-    Dir.chdir(File.dirname(File.expand_path(File.basename(__FILE__))))
-    File.open(@@file, 'w'){ |fh| fh.puts "This is a security test." }
-  end
-
-  def setup
-    @perms = nil
-  end
-
-  test "get_permissions basic functionality" do
-    assert_respond_to(File, :get_permissions)
-    assert_nothing_raised{ File.get_permissions(@@file) }
-  end
-
-  test "get_permissions returns a hash" do
-    assert_kind_of(Hash, File.get_permissions(@@file))
-  end
-
-  test "get_permissions accepts an optional hostname argument" do
-    assert_nothing_raised{ File.get_permissions(@@file, @@host) }
-  end
-
-  test "get_permissions requires at least one argument" do
-    assert_raise(ArgumentError){ File.get_permissions }
-  end
-
-  test "set_permissions basic functionality" do
-    assert_respond_to(File, :set_permissions)
-  end
-
-  test "set_permissions works as expected" do
-    assert_nothing_raised{ @perms = File.get_permissions(@@file) }
-    assert_nothing_raised{ File.set_permissions(@@file, @perms) }
-    assert_equal(@perms, File.get_permissions(@@file))
-  end
-
-  test "set_permissions works if host is specified" do
-    @perms = {"#{@@host}\\#{@@user}" => File::GENERIC_ALL}
-    assert_nothing_raised{ File.set_permissions(@@file, @perms) }
-    assert_equal(@perms, File.get_permissions(@@file))
-  end
-
-  test "securities method basic functionality" do
-    assert_respond_to(File, :securities)
-  end
-
-  test "securities method works as expected" do
-    @perms = File.get_permissions(@@file)
-
-    @perms.each{ |acct, mask|
-      assert_nothing_raised{ File.securities(mask) }
-      assert_kind_of(Array, File.securities(mask))
-    }
-  end
-
-  test "securities method accepts a single argument only" do
-    assert_raise(ArgumentError){ File.securities }
-    assert_raise(ArgumentError){ File.securities({}, {}) }
-  end
-
-  def teardown
-    @perms = nil
-  end
-
-  def self.shutdown
-    File.delete(@@file) if File.exist?(@@file)
-    @@file = nil
-    @@host = nil
-    @@user = nil
-  end
-end
+##############################################################################
+# test_win32_file_permissions.rb
+#
+# Test case for permission related methods of win32-file-security. You should
+# use the 'rake test' or 'rake test:perms' task to run this.
+##############################################################################
+require 'test-unit'
+require 'win32/file/security'
+require 'socket'
+require 'etc'
+
+class TC_Win32_File_Security_Permissions < Test::Unit::TestCase
+  def self.startup
+    @@user = Etc.getlogin
+    @@host = Socket.gethostname
+    @@file = File.join(Dir.pwd, 'security_test.txt')
+
+    Dir.chdir(File.dirname(File.expand_path(File.basename(__FILE__))))
+    File.open(@@file, 'w'){ |fh| fh.puts "This is a security test." }
+  end
+
+  def setup
+    @perms = nil
+  end
+
+  test "get_permissions basic functionality" do
+    assert_respond_to(File, :get_permissions)
+    assert_nothing_raised{ File.get_permissions(@@file) }
+  end
+
+  test "get_permissions returns a hash" do
+    assert_kind_of(Hash, File.get_permissions(@@file))
+  end
+
+  test "get_permissions accepts an optional hostname argument" do
+    assert_nothing_raised{ File.get_permissions(@@file, @@host) }
+  end
+
+  test "get_permissions requires at least one argument" do
+    assert_raise(ArgumentError){ File.get_permissions }
+  end
+
+  test "set_permissions basic functionality" do
+    assert_respond_to(File, :set_permissions)
+  end
+
+  test "set_permissions works as expected" do
+    assert_nothing_raised{ @perms = File.get_permissions(@@file) }
+    assert_nothing_raised{ File.set_permissions(@@file, @perms) }
+    assert_equal(@perms, File.get_permissions(@@file))
+  end
+
+  test "set_permissions works if host is specified" do
+    @perms = {"#{@@host}\\#{@@user}" => File::GENERIC_ALL}
+    assert_nothing_raised{ File.set_permissions(@@file, @perms) }
+    assert_equal(@perms.keys.map(&:downcase), File.get_permissions(@@file).keys.map(&:downcase))
+    assert_equal(@perms.values, File.get_permissions(@@file).values)
+  end
+
+  test "securities method basic functionality" do
+    assert_respond_to(File, :securities)
+  end
+
+  test "securities method works as expected" do
+    @perms = File.get_permissions(@@file)
+
+    @perms.each{ |acct, mask|
+      assert_nothing_raised{ File.securities(mask) }
+      assert_kind_of(Array, File.securities(mask))
+    }
+  end
+
+  test "securities method accepts a single argument only" do
+    assert_raise(ArgumentError){ File.securities }
+    assert_raise(ArgumentError){ File.securities({}, {}) }
+  end
+
+  def teardown
+    @perms = nil
+  end
+
+  def self.shutdown
+    File.delete(@@file) if File.exist?(@@file)
+    @@file = nil
+    @@host = nil
+    @@user = nil
+  end
+end