ruber 0.0.5 → 0.0.7

data/spec/plugin_specification_reader_spec.rb

@@ -53,7 +53,7 @@ module Ruber
         @reader.process_pdf({}).should eql(@info)
       end
       
-      it 'should require all files obtained by joining the PluginSpecification\'s directory with the files returned by the read_required method, before calling the read_required method' do
+      it 'requires all files obtained by joining the PluginSpecification\'s directory with the files returned by the read_required method, before calling the read_required method' do
         @info.directory = '/xyz'
         flexmock(@reader).should_receive(:read_required).and_return %w[a b]
         flexmock(@reader).should_receive(:require).globally.ordered.once.with '/xyz/a'
@@ -61,6 +61,15 @@ module Ruber
         flexmock(@reader).should_receive(:read_ui_file).globally.ordered.once.and_return 'testui.rc'
         @reader.process_pdf({})
       end
+      
+      it 'loads files returned by the read_required method rather than require them if they end in .rb' do
+        @info.directory = '/xyz'
+        flexmock(@reader).should_receive(:read_required).and_return %w[a b.rb]
+        flexmock(@reader).should_receive(:require).globally.ordered.once.with '/xyz/a'
+        flexmock(@reader).should_receive(:load).globally.ordered.once.with '/xyz/b.rb'
+        flexmock(@reader).should_receive(:read_ui_file).globally.ordered.once.and_return 'testui.rc'
+        @reader.process_pdf({})
+      end
     
     end
     
@@ -1651,6 +1660,33 @@ describe Ruber::PluginSpecificationReader do
     
     end
     
+    context ', when reading the place' do
+    
+      it 'puts the contents of the place entry of the argument in the :place entry of the returned hash' do
+        @reader.send(:read_rules, {:place => [:local]})[:place].should == [:local]
+        @reader.send(:read_rules, {'place' => [:remote]})[:place].should == [:remote]
+      end
+      
+      it 'converts the values :all and all to [:local, :remote]' do
+        @reader.send(:read_rules, {:place => ['all']})[:place].should == [:local, :remote]
+        @reader.send(:read_rules, {'place' => [:all]})[:place].should == [:local, :remote]
+      end
+      
+      it 'encloses the place entry of the argument in a array, unless it\'s already an array' do
+        @reader.send(:read_rules, {:place => :local})[:place].should == [:local]
+      end
+      
+      it 'converts each entry of the place array to a symbol' do
+        @reader.send(:read_rules, {:place => %w[local remote]})[:place].should == [:local, :remote]
+      end
+      
+      it 'uses [:local] as default value' do
+        @reader.send(:read_rules, {})[:place].should == [:local]
+      end
+    
+    end
+
+    
     describe ', when reading the mimetype' do
       
       it 'stores the contents of the mimetype entry of the argument in the returned value' do

data/spec/project_files_list_spec.rb

@@ -131,49 +131,49 @@ describe 'Ruber::ProjectFilesList' do
       FileUtils.rm_f @dir
     end
     
-    it 'should update the cache if it\'s not up to date' do
+    it 'updates the cache if it\'s not up to date' do
       @list.instance_variable_set :@up_to_date, false
       flexmock(@list).should_receive(:refresh).once
       @list.project_files
     end
     
-    it 'should not update the cache if it\'s already up to date' do
+    it 'doesn\'t update the cache if it\'s already up to date' do
       @list.instance_variable_set :@up_to_date, true
       flexmock(@list).should_receive(:refresh).never
       @list.project_files
     end
     
-    it 'should return an array containing all the files corresponding to include rules of type path unless they also correspond to an exclude rule of type path' do
+    it 'returns an array containing all the files corresponding to include rules of type path unless they also correspond to an exclude rule of type path' do
       @prj[:general, :project_files] = {:include => ['README', 'COPYING', 'f1.rb', 'd2/CHANGELOG', 'd2/f21.rb'], :exclude => ['COPYING', 'd2/f21.rb'], :extensions => []}
       @list.project_files(false).should =~ %w[README f1.rb d2/CHANGELOG]
     end
     
-    it 'should not include in the returned array files which don\'t exist' do
+    it 'doesn\'t include in the returned array files which don\'t exist' do
       @prj[:general, :project_files] = {:include => ['README1', 'COPYING', 'f1.rb', 'd2/CHANGELOG', 'd2/f211.rb'], :exclude => ['COPYING'], :extensions => []}
       @list.project_files(false).should =~ %w[f1.rb d2/CHANGELOG]
     end
     
-    it 'should skip include rules of type path corresponding to directories' do
+    it 'skips include rules of type path corresponding to directories' do
       @prj[:general, :project_files] = {:include => ['README', 'COPYING', 'f1.rb', 'd2/CHANGELOG', 'd2/f21.rb', 'd3'], :exclude => ['COPYING', 'd2/f21.rb'], :extensions => []}
       @list.project_files(false).should =~ %w[README f1.rb d2/CHANGELOG]
     end
     
-    it 'should include all the files in the project directory which correspond to one of the extensions in the returned array, unless they match one of the exclude rule' do
+    it 'includes all the files in the project directory which correspond to one of the extensions in the returned array, unless they match one of the exclude rule' do
       @prj[:general, :project_files] = {:include => [], :exclude => ['COPYING', 'd2/f21.rb'], :extensions => %w[*.rb]}
       @list.project_files(false).should =~ %w[f1.rb d2/f22.rb d3/f31.rb d3/f32.rb]
     end
     
-    it 'should include all the files in the project directory which match one of the include rules of type regexp and don\'t match any exclude rule' do
+    it 'includes all the files in the project directory which match one of the include rules of type regexp and don\'t match any exclude rule' do
       @prj[:general, :project_files] = {:include => [%r{d3/.*}, %r{.*\.yaml}], :exclude => [%r{d3/.*2.*}, %r{.*1.*\.rb}], :extensions => []}
       @list.project_files(false).should =~ %w[d2/f22.yaml f1.yaml d2/f21.yaml]
     end
     
-    it 'should not include duplicate elements' do
+    it 'doesn\'t include duplicate elements' do
       @prj[:general, :project_files] = {:include => [%r{d2/.*}, 'f1.rb'], :exclude => [], :extensions => ['*.rb']}
       @list.project_files(false).should =~ %w[f1.rb d2/f21.rb d2/f22.rb d2/f21.yaml d2/f22.yaml d2/CHANGELOG d3/f31.rb d3/f32.rb]
     end
     
-    it 'should return a deep copy of the cache object' do
+    it 'returns a deep copy of the cache object' do
       @prj[:general, :project_files] = {:include => [%r{d2/.*}, 'f1.rb'], :exclude => [], :extensions => ['*.rb']}
       old_cache = @list.instance_variable_get(:@project_files).deep_copy
       res = @list.project_files(false)
@@ -182,12 +182,12 @@ describe 'Ruber::ProjectFilesList' do
       @list.instance_variable_get(:@project_files).should == old_cache
     end
     
-    it 'should return full paths if the argument is true' do
+    it 'returns full paths if the argument is true' do
       @prj[:general, :project_files] = {:include => [%r{d2/.*}, 'f1.rb'], :exclude => [], :extensions => ['*.rb']}
       @list.project_files(true).should =~ %w[f1.rb d2/f21.rb d2/f22.rb d2/f21.yaml d2/f22.yaml d2/CHANGELOG d3/f31.rb d3/f32.rb].map{|i| File.join @dir, i}
     end
     
-    it 'should return the correct list if a file or directory is added or deleted after the file watcher has been created' do
+    it 'returns the correct list if a file or directory is added or deleted after the file watcher has been created' do
       @prj[:general, :project_files] = {:include => [], :exclude => [], :extensions => ['*.rb']}
       # Experiments show that KDE::DirWatch needs an event loop to work. Since we don't have one running, we'll have to make it emit the 'dirty' signal manually
       watcher = @list.instance_variable_get(:@watcher)
@@ -213,7 +213,7 @@ describe 'Ruber::ProjectFilesList' do
       watcher.instance_eval{emit dirty( @dir)}
       @list.project_files(false).should_not include("f1.rb")
     end
-    
+   
   end
   
   describe 'Ruber::ProjectFilesList#each' do
@@ -344,41 +344,41 @@ describe 'Ruber::ProjectFilesList' do
         @prj.add_extension :file_lister, @list
       end
       
-      it 'should return true if the argument matches one of the exact include rules' do
+      it 'returns true if the argument matches one of the exact include rules' do
         @prj[:general, :project_files] = {:include => %w[./xyz ./abc ./123], :exclude => [], :extensions => ['*.rb']}
         @list.file_in_project?(File.join(@prj.project_dir, 'abc')).should be_true
       end
       
-      it 'should return true if the argument matches one of the regexp include rules' do
+      it 'returns true if the argument matches one of the regexp include rules' do
         @prj[:general, :project_files] = {:include => [/a/, /x/], :exclude => [], :extensions => ['*.rb']}
         @list.file_in_project?(File.join(@prj.project_dir, 'xyz')).should be_true
       end
       
-      it 'should return true if the argument matches one of the include extensions' do
+      it 'returns true if the argument matches one of the include extensions' do
         @prj[:general, :project_files] = {:include => [], :exclude => [], :extensions => ['*.rb', '*.yaml']}
         @list.file_in_project?(File.join(@prj.project_dir, 'abc.rb')).should be_true
       end
       
-      it 'should return false if the file isn\'t in the project directory' do
+      it 'returns false if the file isn\'t in the project directory' do
         @prj[:general, :project_files] = {:include => %w[xyz abc 123], :exclude => [], :extensions => ['*.rb']}
         @list.file_in_project?('/usr/abc').should be_false
       end
       
-      it 'should treat the argument as a path relative to the project directory if it isn\'t an absolute path' do
+      it 'treats the argument as a path relative to the project directory if it isn\'t an absolute path' do
         @prj[:general, :project_files] = {:include => %w[xyz abc 123], :exclude => [], :extensions => ['*.rb']}
         @list.file_in_project?('abc').should be_true
       end
       
-      it 'should return false if the file doesn\'t match any include rule' do
+      it 'returns false if the file doesn\'t match any include rule' do
         @list.file_in_project?(File.join(@prj.project_dir, 'abc.rb')).should be_false
       end
       
-      it 'should return false if the file matches one of the file exclude rules' do
+      it 'returns false if the file matches one of the file exclude rules' do
         @prj[:general, :project_files] = {:include => %w[xyz abc 123], :exclude => ['abc'], :extensions => ['*.rb']}
         @list.file_in_project?(File.join(@prj.project_dir, 'abc')).should be_false
       end
       
-      it 'should return false if the file matches one of the exclude regexps' do
+      it 'returns false if the file matches one of the exclude regexps' do
         @prj[:general, :project_files] = {:include => [/a/], :exclude => [/a/], :extensions => []}
         @list.file_in_project?(File.join(@prj.project_dir, 'abc')).should be_false
       end
@@ -394,6 +394,16 @@ describe 'Ruber::ProjectFilesList' do
         @list.file_in_project?('xyz/').should be_nil
       end
       
+      it 'returns false if the path represents a remote URL' do
+        @prj[:general, :project_files] = {:include => [], :exclude => [], :extensions => ['*.rb']}
+        @list.file_in_project?("http://#{@dir}/abc.xyz.rb").should be_false
+      end
+      
+      it 'works normally if the path represents a local url' do
+        @prj[:general, :project_files] = {:include => [], :exclude => [], :extensions => ['*.rb']}
+        @list.file_in_project?("file://#{@dir}/abc.xyz.rb").should be_true
+      end
+      
     end
     
   end

data/spec/qt_sugar_spec.rb

@@ -326,3 +326,272 @@ describe Qt::Variant, '#to_bool' do
   end
   
 end
+
+describe Qt::Layout do
+  
+  it 'includes QtEnumerable' do
+    Qt::Layout.ancestors.should include(QtEnumerable)
+  end
+  
+  describe '#each' do
+    
+    it 'passes to the block each widget in turn if called with a block' do
+      #Since Qt::Layout is an abstract class, we fake calls to count and item_at,
+      #which are pure virtual, with mocks
+      w = Qt::Widget.new
+      layout = Qt::Layout.new w
+      w.layout = layout
+      widgets = [Qt::PushButton, Qt::VBoxLayout, Qt::CheckBox].map{|c| c.new}
+      res = []
+      flexmock(layout).should_receive(:count).once.and_return(3)
+      flexmock(layout).should_receive(:item_at).with(0).once.and_return(Qt::WidgetItem.new(widgets[0]))
+      flexmock(layout).should_receive(:item_at).with(1).once.and_return(widgets[1])
+      flexmock(layout).should_receive(:item_at).with(2).once.and_return(Qt::WidgetItem.new(widgets[2]))
+      layout.each{|w| res << w}
+      res.should == widgets
+    end
+    
+    it 'returns an enumerator if no block is given' do
+      w = Qt::Widget.new
+      layout = Qt::Layout.new w
+      w.layout = layout
+      widgets = [Qt::PushButton, Qt::VBoxLayout, Qt::CheckBox].map{|c| c.new}
+      res = []
+      flexmock(layout).should_receive(:count).once.and_return(3)
+      flexmock(layout).should_receive(:item_at).with(0).once.and_return(Qt::WidgetItem.new(widgets[0]))
+      flexmock(layout).should_receive(:item_at).with(1).once.and_return(widgets[1])
+      flexmock(layout).should_receive(:item_at).with(2).once.and_return(Qt::WidgetItem.new(widgets[2]))
+      e = layout.each
+      e.each{|w| res << w}
+      res.should == widgets
+    end
+    
+  end
+  
+end
+
+shared_examples_for 'any box layout' do
+  
+  it 'includes QtEnumerable' do
+    @layout.class.ancestors.should include(QtEnumerable)
+  end
+  
+  it 'has an each method which passes to the block each widget in turn if called with a block' do
+    w = Qt::Widget.new
+    w.layout = @layout
+    widgets = [Qt::PushButton, Qt::VBoxLayout, Qt::CheckBox].map{|c| c.new}
+    widgets.each{|w| @layout.send w.is_a?(Qt::Widget) ? :add_widget : :add_layout, w}
+    res = []
+    @layout.each{|w| res << w}
+    res.should == widgets
+  end
+  
+  it 'has an each method which returns an enumerator if no block is given' do
+    w = Qt::Widget.new
+    w.layout = @layout
+    widgets = [Qt::PushButton, Qt::VBoxLayout, Qt::CheckBox].map{|c| c.new}
+    res = []
+    widgets.each{|w| @layout.send w.is_a?(Qt::Widget) ? :add_widget : :add_layout, w}
+    e = @layout.each
+    e.each{|w| res << w}
+    res.should == widgets
+  end
+    
+end
+
+describe Qt::BoxLayout do
+  
+  before do
+    @layout = Qt::BoxLayout.new(Qt::Horizontal)
+  end
+  
+  it_behaves_like 'any box layout'
+  
+end
+
+describe Qt::VBoxLayout do
+  
+  before do
+    @layout = Qt::VBoxLayout.new
+  end
+  
+  it_behaves_like 'any box layout'
+  
+end
+
+describe Qt::HBoxLayout do
+  
+  before do
+    @layout = Qt::HBoxLayout.new
+  end
+  
+  it_behaves_like 'any box layout'
+  
+end
+
+describe Qt::StackedLayout do
+
+  before do
+    @layout = Qt::StackedLayout.new
+  end
+  
+  it 'includes QtEnumerable' do
+    @layout.class.ancestors.should include(QtEnumerable)
+  end
+  
+  it 'has an each method which passes to the block each widget in turn if called with a block' do
+    w = Qt::Widget.new
+    w.layout = @layout
+    widgets = [Qt::PushButton, Qt::LineEdit, Qt::CheckBox].map{|c| c.new}
+    widgets.each{|w| @layout.add_widget w}
+    res = []
+    @layout.each{|w| res << w}
+    res.should == widgets
+  end
+  
+  it 'has an each method which returns an enumerator if no block is given' do
+    w = Qt::Widget.new
+    w.layout = @layout
+    widgets = [Qt::PushButton, Qt::LineEdit, Qt::CheckBox].map{|c| c.new}
+    res = []
+    widgets.each{|w| @layout.add_widget w}
+    e = @layout.each
+    e.each{|w| res << w}
+    res.should == widgets
+  end
+  
+end
+
+describe Qt::FormLayout do
+  
+  before do
+    @layout = Qt::FormLayout.new
+  end
+  
+  it 'includes QtEnumerable' do
+    @layout.class.ancestors.should include(QtEnumerable)
+  end
+  
+  it 'has an each method which passes to the block each widget in turn if called with a block' do
+    w = Qt::Widget.new
+    w.layout = @layout
+    widgets = [['l1', Qt::PushButton], ['l2', Qt::LineEdit], ['l3', Qt::CheckBox]].map{|c| [Qt::Label.new(c[0]), c[1].new]}
+    widgets.each{|w| @layout.add_row w[0], w[1]}
+    res = []
+    @layout.each{|w| res << w}
+    res.should == widgets.flatten
+  end
+  
+  it 'has an each method which returns an enumerator if no block is given' do
+    w = Qt::Widget.new
+    w.layout = @layout
+    widgets = [['l1', Qt::PushButton], ['l2', Qt::LineEdit], ['l3', Qt::CheckBox]].map{|c| [Qt::Label.new(c[0]), c[1].new]}
+    widgets.each{|w| @layout.add_row w[0], w[1]}
+    res = []
+    e = @layout.each
+    e.each{|w| res << w}
+    res.should == widgets.flatten
+  end
+  
+end
+
+describe Qt::GridLayout do
+  
+  before do
+    @layout = Qt::GridLayout.new
+  end
+  
+  it 'includes QtEnumerable' do
+    @layout.class.ancestors.should include(QtEnumerable)
+  end
+  
+  it 'has an each method which passes to the block each widget in turn if called with a block' do
+    w = Qt::Widget.new
+    w.layout = @layout
+    widgets = [[Qt::TextEdit, Qt::HBoxLayout], [Qt::Label, Qt::LineEdit], [Qt::VBoxLayout, Qt::CheckBox]].map{|c1, c2| [c1.new, c2.new]}
+    add = lambda{|w, r, c|w.is_a?(Qt::Widget) ? @layout.add_widget(w, r, c) : @layout.add_layout(w, r, c) }
+    widgets.each_with_index do |w, r| 
+      add.call w[0], r, 0
+      add.call w[1], r, 1
+    end
+    res = []
+    @layout.each{|w| res << w}
+    res.should == widgets.flatten
+  end
+  
+  it 'has an each method which returns an enumerator if no block is given' do
+    w = Qt::Widget.new
+    w.layout = @layout
+    widgets = [[Qt::TextEdit, Qt::HBoxLayout], [Qt::Label, Qt::LineEdit], [Qt::VBoxLayout, Qt::CheckBox]].map{|c1, c2| [c1.new, c2.new]}
+    add = lambda{|w, r, c|w.is_a?(Qt::Widget) ? @layout.add_widget(w, r, c) : @layout.add_layout(w, r, c) }
+    widgets.each_with_index do |w, r| 
+      add.call w[0], r, 0
+      add.call w[1], r, 1
+    end
+    res = []
+    e = @layout.each
+    e.each{|w| res << w}
+    res.should == widgets.flatten
+  end
+  
+  it 'passes widgets spanning more than one row or column only once' do
+    w = Qt::Widget.new
+    w.layout = @layout
+    widgets = [[Qt::TextEdit, Qt::HBoxLayout], [Qt::LineEdit], [Qt::VBoxLayout, Qt::CheckBox]].map do |c1, c2| 
+      c2 ? [c1.new, c2.new] : [c1.new]
+    end
+    add = lambda{|w, r, c, r1, c1|w.is_a?(Qt::Widget) ? @layout.add_widget(w, r, c, r1, c1) : @layout.add_layout(w, r, c, r1, c1) }
+    widgets.each_with_index do |w, r| 
+      if w.size == 1
+        add.call w[0], r, 0, r, 1
+      else
+        add.call w[0], r, 0, r, 0
+        add.call w[1], r, 1, r, 1
+      end
+    end
+    res = []
+    @layout.each{|w| res << w}
+    res.should == widgets.flatten
+  end
+  
+end
+
+describe Qt::Splitter do
+
+  it 'includes the QtEnumerable module' do
+    Qt::Splitter.ancestors.should include(QtEnumerable)
+  end
+  
+  describe '#each' do
+    
+    before do
+      @splitter = Qt::Splitter.new Qt::Vertical
+    end
+    
+    it 'calls the block each widget in the splitter' do
+      widgets = [Qt::LineEdit, Qt::PushButton, Qt::Label].map{|cls| cls.new}
+      widgets.reverse_each{|w| @splitter.add_widget w}
+      res = []
+      @splitter.each{|w| res << w}
+      res.should == widgets.reverse
+    end
+    
+    it 'returns an enumerator if no block is given' do
+      widgets = [Qt::LineEdit, Qt::PushButton, Qt::Label].map{|cls| cls.new}
+      widgets.reverse_each{|w| @splitter.add_widget w}
+      res = []
+      enum = @splitter.each
+      enum.should be_an(Enumerator)
+      enum.each{|w| res << w}
+      res.should == widgets.reverse
+    end
+    
+    it 'returns self if a block has been given' do
+      widgets = [Qt::LineEdit, Qt::PushButton, Qt::Label].map{|cls| cls.new}
+      widgets.reverse_each{|w| @splitter.add_widget w}
+      @splitter.each{}.should == @splitter
+    end
+    
+  end
+  
+end