boxgrinder-build 0.10.0 → 0.10.1

data/spec/util/permissions/fs-observer-spec.rb

@@ -0,0 +1,141 @@
+#
+# Copyright 2012 Red Hat, Inc.
+#
+# This is free software; you can redistribute it and/or modify it
+# under the terms of the GNU Lesser General Public License as
+# published by the Free Software Foundation; either version 3 of
+# the License, or (at your option) any later version.
+#
+# This software is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+# Lesser General Public License for more details.
+#
+# You should have received a copy of the GNU Lesser General Public
+# License along with this software; if not, write to the Free
+# Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA
+# 02110-1301 USA, or see the FSF site: http://www.fsf.org.
+
+require 'boxgrinder-build/util/permissions/fs-observer'
+
+module BoxGrinder
+  describe FSObserver do
+    let(:empty_set){ Set.new }
+    let(:init_filterset){ Set.new([%r(^/(etc|dev|sys|bin|sbin|etc|lib|lib64|boot|run|proc|selinux|tmp)(/|$))]) }
+ 
+    before(:each) do
+      @fs_observer = FSObserver.new('some-usr', 'some-grp')
+    end
+      
+    subject{ @fs_observer }
+    
+    its(:path_set){ should be_empty }
+    its(:filter_set){ should eq init_filterset }
+
+    context "#initialize" do
+      it "should merge extra :path" do
+        fso = FSObserver.new('j', 'u', :paths => '/a/b/c')
+        fso.path_set.should eql(mkset('/a/b/c'))
+      end
+
+      it "should merge extra :path array" do
+        fso = FSObserver.new('i', 'c', :paths => ['a/b/c', '/d/e/f'])
+        fso.path_set.should eql(mkset('a/b/c', '/d/e/f'))
+      end
+    end
+
+    context "#update" do
+      context ":command=:add_path" do
+        let(:simple_update) do 
+          path_update(subject, '/the/great/escape')
+        end
+
+        it "should add the path to the path_set", :if => RSpec::Expectations::Version::STRING >= "2.7.0" do
+          expect{ simple_update }.to change(subject, :path_set).
+            from(empty_set).to(mkset('/the/great/escape'))
+        end
+        
+        it "should add regex for all children of new path to the filter_set", :if => RSpec::Expectations::Version::STRING >= "2.7.0" do
+          expect_set = init_filterset + mkset(%r[^/the/great/escape/])
+ 
+          expect{ simple_update }.to change(subject, :filter_set).
+            from(init_filterset).to(expect_set)
+        end
+
+        context "rejecting blacklisted and filtered paths" do
+          before(:each) do
+            simple_update
+          end
+          
+          it "should not add a path if it is a duplicate" do
+            expect{ simple_update }.to_not change(subject, :path_set)
+
+            expect{ simple_update }.to_not change(subject, :filter_set)
+          end
+
+          it "should not add a path if it is pre-filtered" do
+            expect{ path_update(subject, "/etc/sneaky/file") }.
+              to_not change(subject, :path_set) 
+
+            expect{ path_update(subject, "/etc/sneaky/file") }.
+              to_not change(subject, :filter_set)
+          end
+
+          it "should not add a new path if it is a child of existing path" do
+            expect{ path_update(subject, "/the/great/escape/900") }.
+              to_not change(subject, :path_set)
+
+            expect{ path_update(subject, "/the/great/escape/900") }.
+              to_not change(subject, :filter_set)
+          end
+        end
+      end
+      
+      context ":command=:stop_capture and :command=:chown" do
+        let(:update_a){ path_update(subject, '/a/path') }  
+        let(:update_b){ path_update(subject, '/b/path') }        
+
+        before(:each) do
+          update_a
+          update_b
+        end
+
+        context "all files exist" do
+          before(:each){ File.stub(:exist?).and_return(true) }
+
+          it "should change ownership of captured paths" do
+            subject.stub(:change_user)
+
+            FileUtils.should_receive(:chown_R).
+              with('some-usr', 'some-grp', '/a/path', :force => true).once
+
+            FileUtils.should_receive(:chown_R).
+              with('some-usr', 'some-grp', '/b/path', :force => true).once
+
+            subject.update({ :command => :stop_capture })
+          end
+        end
+
+        it "should ignore any paths that no longer exist" do
+          subject.stub(:change_user)
+
+          File.stub(:exist?).and_return(false)
+          
+          FileUtils.should_not_receive(:chown_R)
+
+          update_a
+          
+          subject.update({ :command => :stop_capture })
+        end
+      end
+    end
+
+    def path_update(subject, path)
+      subject.update({ :command => :add_path, :data => path })
+    end
+
+    def mkset(*vars)
+      Set.new(Array(vars))
+    end
+  end
+end

data/spec/util/permissions/user-switcher-spec.rb

@@ -0,0 +1,69 @@
+#
+# Copyright 2012 Red Hat, Inc.
+#
+# This is free software; you can redistribute it and/or modify it
+# under the terms of the GNU Lesser General Public License as
+# published by the Free Software Foundation; either version 3 of
+# the License, or (at your option) any later version.
+#
+# This software is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+# Lesser General Public License for more details.
+#
+# You should have received a copy of the GNU Lesser General Public
+# License along with this software; if not, write to the Free
+# Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA
+# 02110-1301 USA, or see the FSF site: http://www.fsf.org.
+
+require 'boxgrinder-build/util/permissions/user-switcher'
+
+module BoxGrinder          
+  describe UserSwitcher do
+    context "#change_user" do
+
+      before(:each) do
+        FileUtils.stub(:rm_rf)
+      end
+      
+      it "should change from the current user to specified, then revert after the block" do
+        Process.stub(:uid).and_return(1, 1, 2)
+        Process.stub(:gid).and_return(3, 3, 4)
+       
+        # First switch
+        Process.should_receive(:uid=).with(2)
+        Process.should_receive(:euid=).with(2)
+        Process.should_receive(:gid=).with(4)
+        Process.should_receive(:egid=).with(4)
+
+
+        # Change back
+        Process.should_receive(:uid=).with(1)
+        Process.should_receive(:euid=).with(1)
+        Process.should_receive(:gid=).with(3)
+        Process.should_receive(:egid=).with(3)
+        
+        UserSwitcher.change_user(2, 4){}
+      end
+
+      it "should not change user if the uid and gid already match" do 
+        Process.stub(:uid).and_return(1)
+        Process.stub(:gid).and_return(2)
+        
+        Process.should_not_receive(:uid=)
+        Process.should_not_receive(:euid=)
+        Process.should_not_receive(:gid=)
+        Process.should_not_receive(:egid=)
+
+        UserSwitcher.change_user(1, 2){}
+      end
+    end
+
+    def stub_process_ids(u, g, eu=u, eg=g)
+      Process.stub(:uid => u)
+      Process.stub(:gid => g)
+      Process.stub(:euid => eu)
+      Process.stub(:egid => eg)
+    end
+  end
+end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: boxgrinder-build
 version: !ruby/object:Gem::Version 
-  hash: 55
+  hash: 53
   prerelease: 
   segments: 
   - 0
   - 10
-  - 0
-  version: 0.10.0
+  - 1
+  version: 0.10.1
 platform: ruby
 authors: 
 - Marek Goldmann
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-12-19 00:00:00 Z
+date: 2012-03-19 00:00:00 Z
 dependencies: 
 - !ruby/object:Gem::Dependency 
   name: boxgrinder-core
@@ -192,13 +192,19 @@ extra_rdoc_files:
 - lib/boxgrinder-build/plugins/platform/vmware/src/base.vmdk
 - lib/boxgrinder-build/plugins/platform/vmware/src/base.vmx
 - lib/boxgrinder-build/plugins/platform/vmware/vmware-plugin.rb
+- lib/boxgrinder-build/util/concurrent/get_set.rb
+- lib/boxgrinder-build/util/permissions/fs-monitor.rb
+- lib/boxgrinder-build/util/permissions/fs-observer.rb
+- lib/boxgrinder-build/util/permissions/user-switcher.rb
 files: 
 - CHANGELOG
 - LICENSE
 - Manifest
 - README.md
 - Rakefile
+- bash_completion
 - bin/boxgrinder-build
+- boxgrinder-build.gemspec
 - integ/appliances/gnome-fedora.appl
 - integ/appliances/jeos-centos5-files.appl
 - integ/appliances/jeos-centos5.appl
@@ -272,6 +278,10 @@ files:
 - lib/boxgrinder-build/plugins/platform/vmware/src/base.vmdk
 - lib/boxgrinder-build/plugins/platform/vmware/src/base.vmx
 - lib/boxgrinder-build/plugins/platform/vmware/vmware-plugin.rb
+- lib/boxgrinder-build/util/concurrent/get_set.rb
+- lib/boxgrinder-build/util/permissions/fs-monitor.rb
+- lib/boxgrinder-build/util/permissions/fs-observer.rb
+- lib/boxgrinder-build/util/permissions/user-switcher.rb
 - rubygem-boxgrinder-build.spec
 - spec/appliance-spec.rb
 - spec/helpers/augeas-helper-spec.rb
@@ -309,8 +319,13 @@ files:
 - spec/plugins/platform/virtualbox/virtualbox-plugin-spec.rb
 - spec/plugins/platform/virtualpc/virtualpc-plugin-spec.rb
 - spec/plugins/platform/vmware/vmware-plugin-spec.rb
+- spec/rcov_helper.rb
 - spec/rspec-plugin-helper.rb
-- boxgrinder-build.gemspec
+- spec/spec_helper.rb
+- spec/util/concurrent/get-set-spec.rb
+- spec/util/permissions/fs-monitor-spec.rb
+- spec/util/permissions/fs-observer-spec.rb
+- spec/util/permissions/user-switcher-spec.rb
 homepage: http://boxgrinder.org/
 licenses: []