ruote-kit 2.1.10 → 2.1.11

data/spec/spec_helper.rb

@@ -1,3 +1,5 @@
+$:.unshift(File.join(File.dirname(__FILE__), '..'))
+
 ENV['RACK_ENV'] = 'test'
 
 require 'rubygems'
@@ -10,25 +12,33 @@ require 'rack/test'
 
 require 'webrat'
 
-require 'json'
+begin
+  require 'yajl'
+rescue LoadError
+  begin
+    require 'json'
+  rescue LoadError
+    puts 'Please specify "gem {yajl-ruby|json_pure|json}" in the Gemfile'
+    exit
+  end
+end
 
 Test::Unit::TestCase.send :include, Rack::Test::Methods
 
-require File.dirname(__FILE__) + '/../vendor/gems/environment' if File.exists?( File.dirname(__FILE__) + '/../vendor/gems/environment.rb' )
-require File.dirname(__FILE__) + '/../lib/ruote-kit'
-
-require 'ruote/part/storage_participant'
-
-RuoteKit.configure do |config|
-
-  # In memory is perfect for tests
-  config.mode = :transient
+begin
+  require File.join(File.dirname(__FILE__), '/../vendor/gems/environment')
+rescue LoadError
 end
+require File.join(File.dirname(__FILE__), '/../lib/ruote-kit')
 
 require 'ruote-kit/spec/ruote_helpers'
+require 'spec/it_has_an_engine'
+
 require 'ruote/log/test_logger'
 
+
 Spec::Runner.configure do |config|
+
   # == Mock Framework
   #
   # RSpec uses it's own mocking framework by default. If you prefer to
@@ -42,40 +52,15 @@ Spec::Runner.configure do |config|
   config.include Webrat::Matchers, :type => :views
   config.include RuoteKit::Spec::RuoteHelpers
 
-  config.before(:each, :type => :with_engine) do
-    RuoteKit.configure do |conf|
-      conf.register do
-        catchall Ruote::StorageParticipant
-      end
-    end
-    RuoteKit.run_engine!
-
-    @tracer = Tracer.new
-    RuoteKit.engine.add_service( 'tracer', @tracer )
-
-    # Specs use their own worker since we need the trace
-    @_spec_worker = Ruote::Worker.new( RuoteKit.engine.storage )
-    @_spec_worker.context.add_service( 'tracer', @tracer )
-    @_spec_worker.run_in_thread
+  def it_has_an_engine
+    it_should_behave_like 'it has an engine'
   end
-
-  # Purge the engine after every run
-  config.after(:each, :type => :with_engine) do
-    @_spec_worker.shutdown
-
-    # Seems in some rubies this block gets called multiple times
-    unless RuoteKit.engine.nil?
-      RuoteKit.storage_participant.purge!
-      RuoteKit.engine.storage.purge! unless RuoteKit.engine.storage.nil?
-
-      RuoteKit.shutdown_engine( true )
-    end
+  def it_has_an_engine_with_no_participants
+    it_should_behave_like 'it has an engine with no participants'
   end
 
   RuoteKit::Application.included_modules.each do |klass|
-    if klass.name =~ /RuoteKit::Helpers::\w+Helpers/
-      config.include klass
-    end
+    config.include(klass) if klass.name =~ /RuoteKit::Helpers::\w+Helpers/
   end
 end
 
@@ -83,44 +68,16 @@ def app
   RuoteKit::Application
 end
 
-# Sets the local variables that will be accessible in the HAML
-# template
-def assigns
-  @assigns ||= { }
-end
-
-# Renders the supplied template with Haml::Engine and assigns the
-# @response instance variable
-def render(template_path)
-  template = File.read("#{app.views}/#{template_path.sub(/^\//, '')}")
+def render(template, scope=nil, locals={}, &block)
+  template = File.read(File.join(app.views, template.to_s))
   engine = Haml::Engine.new(template)
-  @response = engine.render(self, assigns_for_template)
-end
-
-# Convenience method to access the @response instance variable set in
-# the render call
-def response
-  @response
-end
-
-# Sets the local variables that will be accessible in the HAML
-# template
-def assigns
-  @assigns ||= { }
-end
-
-# Prepends the assigns keywords with an "@" so that they will be
-# instance variables when the template is rendered.
-def assigns_for_template
-  assigns.inject({}) do |memo, kv|
-    memo["@#{kv[0].to_s}".to_sym] = kv[1]
-    memo
-  end
+  engine.render(scope, locals, &block)
 end
 
 class Rack::MockResponse
+
   def json_body
-    Rufus::Json.decode( body )
+    Rufus::Json.decode(body)
   end
 
   def json?
@@ -133,25 +90,7 @@ class Rack::MockResponse
   end
 
   def html?
-    !json?
+    ! json?
   end
 end
 
-
-class Tracer
-  def initialize
-    @trace = ''
-  end
-  def to_s
-    @trace.to_s.strip
-  end
-  def << s
-    @trace << s
-  end
-  def clear
-    @trace = ''
-  end
-  def puts s
-    @trace << "#{s}\n"
-  end
-end

data/spec/webapp_helpers_spec.rb

@@ -0,0 +1,309 @@
+
+require File.join(File.dirname(__FILE__), 'spec_helper.rb')
+
+
+describe RuoteKit::Helpers::RenderHelpers do
+
+  describe 'alink()' do
+
+    it 'link to top resources' do
+
+      alink(:processes).should ==
+        '<a href="/_ruote/processes" rel="http://ruote.rubyforge.org/rels.html#processes" title="/_ruote/processes">/_ruote/processes</a>'
+    end
+
+    it 'should link to identified resources' do
+
+      alink(:processes, '20120808-tokushima').should ==
+        '<a href="/_ruote/processes/20120808-tokushima" rel="http://ruote.rubyforge.org/rels.html#process" title="/_ruote/processes/20120808-tokushima">/_ruote/processes/20120808-tokushima</a>'
+    end
+
+    it 'should accept a custom :text' do
+
+      alink(:processes, :text => 'processes').should ==
+        '<a href="/_ruote/processes" rel="http://ruote.rubyforge.org/rels.html#processes" title="/_ruote/processes">processes</a>'
+    end
+
+    it 'should aggregate non-:text options into the query string' do
+
+      alink(:processes, :skip => 3, :limit => 4).should ==
+        '<a href="/_ruote/processes?limit=4&skip=3" rel="http://ruote.rubyforge.org/rels.html#processes" title="/_ruote/processes?limit=4&skip=3">/_ruote/processes?limit=4&skip=3</a>'
+    end
+
+    it 'should accept a custom :rel' do
+
+      alink(:processes, :rel => 'last').should ==
+        '<a href="/_ruote/processes" rel="last" title="/_ruote/processes">/_ruote/processes</a>'
+    end
+
+    it 'should accept a :head id for processes, errors, schedules and workitems' do
+
+      self.instance_eval do
+        def settings; OpenStruct.new(:limit => 100); end
+      end
+        # tricking the helper...
+
+      alink(:processes, :head, :text => 'processes').should ==
+        '<a href="/_ruote/processes?limit=100&skip=0" rel="http://ruote.rubyforge.org/rels.html#processes" title="/_ruote/processes?limit=100&skip=0">processes</a>'
+    end
+  end
+
+  describe 'links(resource) (JSON)' do
+
+    before(:each) do
+
+      @resource = Object.new
+      class << @resource
+        include RuoteKit::Helpers::LinkHelpers
+        include RuoteKit::Helpers::JsonHelpers
+        attr_accessor :processes, :count, :skip, :limit, :request
+        def settings
+          OpenStruct.new(:limit => @limit)
+        end
+      end
+      @resource.request = OpenStruct.new(:fullpath => '/_ruote/processes')
+    end
+
+    it 'should paginate correctly' do
+
+      @resource.processes = (1..201).to_a
+      @resource.count = @resource.processes.size
+
+      @resource.skip = 5
+      @resource.limit = 5
+
+      @resource.links(:processes)[8..-1].should == [
+        { 'href' => '/_ruote/processes',
+          'rel' => 'all' },
+        { 'href' => '/_ruote/processes?limit=5&skip=0',
+          'rel' => 'first' },
+        { 'href' => '/_ruote/processes?limit=5&skip=200',
+          'rel' => 'last' },
+        { 'href' => '/_ruote/processes?limit=5&skip=0',
+          'rel' => 'previous' },
+        { 'href' => '/_ruote/processes?limit=5&skip=10',
+          'rel' => 'next' }
+      ]
+    end
+  end
+
+  describe '_pagination.html.haml (HTML)' do
+
+    before(:each) do
+      @resource = Object.new
+      class << @resource
+        attr_accessor :count, :skip, :limit, :request
+        def to_html
+          '<div>' + render('_pagination.html.haml', self) + '</div>'
+        end
+        def settings
+          OpenStruct.new(:limit => @limit)
+        end
+      end
+      @resource.request = OpenStruct.new(:path => '/_ruote/processes')
+    end
+
+    it 'should paginate correctly (1st page)' do
+
+      @resource.count = 201
+      @resource.skip = 0
+      @resource.limit = 7
+
+      html = @resource.to_html
+
+      html.index("  1\n  to\n  7\n  of\n  201\n  processes").should > 0
+
+      html.should have_selector(
+        'a', :href => '/_ruote/processes', :rel => 'all')
+      #html.should have_selector(
+      #  'a', :href => '/_ruote/processes?limit=7&skip=0', :rel => 'first')
+      #html.should have_selector(
+      #  'a', :href => '/_ruote/processes?limit=7&skip=0', :rel => 'previous')
+      html.should have_selector(
+        'a', :href => '/_ruote/processes?limit=7&skip=7', :rel => 'next')
+      html.should have_selector(
+        'a', :href => '/_ruote/processes?limit=7&skip=196', :rel => 'last')
+    end
+
+    it 'should paginate correctly (2nd page)' do
+
+      @resource.count = 201
+      @resource.skip = 7
+      @resource.limit = 7
+
+      html = @resource.to_html
+
+      html.index("  8\n  to\n  14\n  of\n  201\n  processes").should_not be(nil)
+
+      html.should have_selector(
+        'a', :href => '/_ruote/processes', :rel => 'all')
+      html.should have_selector(
+        'a', :href => '/_ruote/processes?limit=7&skip=0', :rel => 'first')
+      #html.should have_selector(
+      #  'a', :href => '/_ruote/processes?limit=7&skip=0', :rel => 'previous')
+      html.should have_selector(
+        'a', :href => '/_ruote/processes?limit=7&skip=14', :rel => 'next')
+      html.should have_selector(
+        'a', :href => '/_ruote/processes?limit=7&skip=196', :rel => 'last')
+    end
+
+    it 'should paginate correctly (3rd page)' do
+
+      @resource.count = 201
+      @resource.skip = 14
+      @resource.limit = 7
+
+      html = @resource.to_html
+
+      html.index("  15\n  to\n  21\n  of\n  201\n  processes").should_not be(nil)
+
+      html.should have_selector(
+        'a', :href => '/_ruote/processes', :rel => 'all')
+      html.should have_selector(
+        'a', :href => '/_ruote/processes?limit=7&skip=0', :rel => 'first')
+      html.should have_selector(
+        'a', :href => '/_ruote/processes?limit=7&skip=7', :rel => 'previous')
+      html.should have_selector(
+        'a', :href => '/_ruote/processes?limit=7&skip=21', :rel => 'next')
+      html.should have_selector(
+        'a', :href => '/_ruote/processes?limit=7&skip=196', :rel => 'last')
+    end
+
+    it 'should paginate correctly (right before last page)' do
+
+      @resource.count = 201
+      @resource.skip = 189
+      @resource.limit = 7
+
+      html = @resource.to_html
+
+      html.index("  190\n  to\n  196\n  of\n  201\n  processes").should_not be(nil)
+
+      html.should have_selector(
+        'a', :href => '/_ruote/processes', :rel => 'all')
+      html.should have_selector(
+        'a', :href => '/_ruote/processes?limit=7&skip=0', :rel => 'first')
+      html.should have_selector(
+        'a', :href => '/_ruote/processes?limit=7&skip=182', :rel => 'previous')
+      #html.should have_selector(
+      #  'a', :href => '/_ruote/processes?limit=7&skip=21', :rel => 'next')
+      html.should have_selector(
+        'a', :href => '/_ruote/processes?limit=7&skip=196', :rel => 'last')
+    end
+
+    it 'should paginate correctly (last page)' do
+
+      @resource.count = 201
+      @resource.skip = 196
+      @resource.limit = 7
+
+      html = @resource.to_html
+
+      html.index("  197\n  to\n  201\n  of\n  201\n  processes").should_not be(nil)
+
+      html.should have_selector(
+        'a', :href => '/_ruote/processes', :rel => 'all')
+      html.should have_selector(
+        'a', :href => '/_ruote/processes?limit=7&skip=0', :rel => 'first')
+      html.should have_selector(
+        'a', :href => '/_ruote/processes?limit=7&skip=189', :rel => 'previous')
+      #html.should have_selector(
+      #  'a', :href => '/_ruote/processes?limit=7&skip=21', :rel => 'next')
+      #html.should have_selector(
+      #  'a', :href => '/_ruote/processes?limit=7&skip=196', :rel => 'last')
+    end
+
+    it 'should paginate correctly (corner case 3 items, limit 3)' do
+
+      @resource.count = 3
+      @resource.skip  = 0
+      @resource.limit = 3
+
+      html = @resource.to_html
+
+      html.index("  1\n  to\n  3\n  of\n  3\n  processes").should_not be(nil)
+
+      html.should have_selector(
+        'a', :href => '/_ruote/processes', :rel => 'all')
+      html.should_not have_selector(
+        'a', :rel => 'first')
+      html.should_not have_selector(
+        'a', :rel => 'previous')
+      html.should_not have_selector(
+        'a', :rel => 'next')
+      html.should_not have_selector(
+        'a', :rel => 'last')
+
+    end
+
+    it 'should paginate correctly (corner case 30 items, limit 10, skip 20)' do
+
+      @resource.count = 30
+      @resource.skip  = 20
+      @resource.limit = 10
+
+      html = @resource.to_html
+
+      html.should_not have_selector(
+        'a', :rel => 'next')
+      html.should_not have_selector(
+        'a', :rel => 'last')
+
+    end
+
+    it 'should not show there is an item if there is none' do
+
+      @resource.count = 0
+      @resource.skip  = 0
+      @resource.limit = 100
+
+      html = @resource.to_html
+
+      html.index("  0\n  to\n  0\n  of\n  0\n  processes").should_not be(nil)
+
+      html.should have_selector(
+        'a', :href => '/_ruote/processes', :rel => 'all')
+      html.should_not have_selector(
+        'a', :rel => 'first')
+      html.should_not have_selector(
+        'a', :rel => 'previous')
+      html.should_not have_selector(
+        'a', :rel => 'next')
+      html.should_not have_selector(
+        'a', :rel => 'last')
+
+    end
+
+    it 'should ignore negative values for skip param' do
+
+      @resource.count = 10
+      @resource.skip  = -5
+      @resource.limit = 100
+
+      html = @resource.to_html
+
+      html.index("  1\n  to\n  10\n  of\n  10\n  processes").should_not be(nil)
+
+      # perhaps a redirection to the same url but with a sane param would be
+      # more appropriate?
+
+    end
+
+    it 'should ignore values for the skip param larger than the number of items' do
+
+      @resource.count = 10
+      @resource.skip  = 20
+      @resource.limit = 5
+
+      html = @resource.to_html
+
+      html.index("  6\n  to\n  10\n  of\n  10\n  processes").should_not be(nil)
+
+      # perhaps a redirection to the same url but with a sane param would be
+      # more appropriate?
+
+    end
+
+  end
+end
+