olelo 0.9.0

data/test/hooks_test.rb

@@ -0,0 +1,85 @@
+require 'olelo/extensions'
+require 'olelo/hooks'
+
+describe 'Olelo::Hooks' do
+  before do
+    @hooks_test = Class.new do
+      include Olelo::Hooks
+      has_hooks :ping
+      has_around_hooks :action
+    end
+  end
+
+  after do
+    @hooks_test = nil
+  end
+
+  it 'should check for hook existence' do
+    lambda { @hooks_test.hook(:action) {} }.should.raise RuntimeError
+    lambda { @hooks_test.before(:ping) {} }.should.raise RuntimeError
+    lambda { @hooks_test.after(:ping) {} }.should.raise RuntimeError
+    lambda { @hooks_test.new.invoke_hook(:action) {} }.should.raise RuntimeError
+    lambda { @hooks_test.new.with_hooks(:ping) {} }.should.raise RuntimeError
+    @hooks_test.hook(:ping) {}
+    @hooks_test.before(:action) {}
+    @hooks_test.after(:action) {}
+    @hooks_test.new.invoke_hook(:ping)
+    @hooks_test.new.with_hooks(:action) {}
+  end
+
+  it 'should provide #hook' do
+    @hooks_test.should.respond_to :hook
+  end
+
+  it 'should provide #before' do
+    @hooks_test.should.respond_to :before
+  end
+
+  it 'should provide #after' do
+    @hooks_test.should.respond_to :after
+  end
+
+  it 'should invoke hooks' do
+    hooks_test = @hooks_test
+    @hooks_test.hook(:ping) do |a, b|
+      self.should.be.instance_of hooks_test
+      a.should.equal 1
+      b.should.equal 2
+      :hook1
+    end
+    @hooks_test.hook(:ping) do |a, b|
+      :hook2
+    end
+    result = @hooks_test.new.invoke_hook(:ping, 1, 2)
+    result.should.be.instance_of Array
+    result.should.equal [:hook1, :hook2]
+  end
+
+  it 'should invoke before and after hooks' do
+    hooks_test = @hooks_test
+    @hooks_test.before(:action) do |a, b|
+      self.should.be.instance_of hooks_test
+      a.should.equal 1
+      b.should.equal 2
+      :action_before1
+    end
+    @hooks_test.before(:action) do |a, b|
+      :action_before2
+    end
+    @hooks_test.after(:action) do |a, b|
+      :action_after
+    end
+    result = @hooks_test.new.with_hooks(:action, 1, 2) do
+      :action
+    end
+    result.should.be.instance_of Array
+    result.should.equal [:action_before1, :action_before2, :action, :action_after]
+  end
+
+  it 'should have hook priority' do
+    @hooks_test.hook(:ping, 2) { :hook1 }
+    @hooks_test.hook(:ping, 1) { :hook2 }
+    @hooks_test.hook(:ping, 3) { :hook3 }
+    @hooks_test.new.invoke_hook(:ping).should.equal [:hook2, :hook1, :hook3]
+  end
+end

data/test/object_extensions_test.rb

@@ -0,0 +1,20 @@
+require 'olelo/extensions'
+require 'olelo/util'
+
+describe 'Object extensions' do
+  it 'should have #blank?' do
+    ''.should.be.blank
+    {}.should.be.blank
+    [].should.be.blank
+    nil.should.be.blank
+    'foo'.should.not.be.blank
+    !{42=>'answer'}.should.not.be.blank
+    [42].should.not.be.blank
+    42.should.not.be.blank
+  end
+
+  it 'should have #try' do
+    nil.try(:succ).should.be.nil
+    1.try(:succ).should.equal 2
+  end
+end

data/test/page_test.rb

@@ -0,0 +1,168 @@
+require 'helper'
+
+describe 'Olelo::Page' do
+  before { create_repository }
+  after { destroy_repository }
+
+  it 'check for path validity' do
+    lambda do
+      Olelo::Page.check_path(' spaces ')
+    end.should.raise RuntimeError
+    Olelo::Page.check_path('spaces in the path')
+  end
+
+  it 'should have no root before the first page is created' do
+    Olelo::Page.find('').should.equal nil
+    create_page('Home')
+    Olelo::Page.find('').should.be.an.instance_of Olelo::Page
+  end
+
+  it 'find or raise' do
+    lambda { Olelo::Page.find!('') }.should.raise Olelo::NotFound
+    create_page('Home')
+    Olelo::Page.find('').should.be.an.instance_of Olelo::Page
+    Olelo::Page.find('Home').should.be.an.instance_of Olelo::Page
+  end
+
+  it 'should be new' do
+    create_page('Home')
+    Olelo::Page.find('Home').should.not.be.new
+    Olelo::Page.find('').should.not.be.new
+    Olelo::Page.new('new').should.be.new
+    Olelo::Page.new('new').should.be.new
+  end
+
+  it 'has name' do
+    Olelo::Page.new('/path/name.ext').name.should.equal 'name.ext'
+  end
+
+  it 'has path' do
+    Olelo::Page.new('/path/name.ext').path.should.equal 'path/name.ext'
+  end
+
+  it 'should have correct path' do
+    Olelo::Page.new('/path/name.ext').path.should.equal 'path/name.ext'
+  end
+
+  it 'should have correct extension' do
+    Olelo::Page.new('/path/name.tar.gz').extension.should.equal 'tar.gz'
+  end
+
+  it 'write content' do
+    page = Olelo::Page.new('test')
+    page.should.be.new
+    page.should.not.be.modified
+    page.content.should.equal ''
+    Olelo::Page.transaction do
+      page.content = 'old content'
+      page.save
+      Olelo::Page.commit "comment1\ntext"
+    end
+
+    page.content.should.equal 'old content'
+    page.should.not.be.new
+    page.should.not.be.modified
+
+    page.tree_version.comment.should.equal "comment1\ntext"
+
+    page.content = 'new content'
+    page.should.not.be.new
+    page.should.be.modified
+    page.content.should.equal 'new content'
+    Olelo::Page.transaction do
+      page.content = 'new content'
+      page.save
+      Olelo::Page.commit 'comment2'
+    end
+
+    page.tree_version.comment.should.equal 'comment2'
+
+    page = Olelo::Page.find!('test')
+    page.should.not.be.new
+    page.content.should.equal 'new content'
+
+    page.tree_version.comment.should.equal 'comment2'
+  end
+
+  it 'fail on duplicates' do
+    Olelo::Page.transaction do
+      page = Olelo::Page.new('test')
+      page.content = 'content'
+      page.save
+      Olelo::Page.commit 'comment'
+    end
+
+    lambda do
+      Olelo::Page.transaction do
+        page = Olelo::Page.new('test')
+        page.content = 'content'
+        page.save
+        Olelo::Page.commit 'comment'
+      end
+    end.should.raise RuntimeError
+  end
+
+  it 'should have children' do
+    create_page('page1')
+    create_page('page2')
+    create_page('page3')
+
+    tree = Olelo::Page.find!('/')
+    tree.tree_version.should.be.head
+
+    tree.tree_version.should.equal tree.children[0].tree_version
+    tree.tree_version.should.equal tree.children[1].tree_version
+    tree.tree_version.should.equal tree.children[2].tree_version
+    tree.children[0].tree_version.should.be.head
+    tree.children[1].tree_version.should.be.head
+    tree.children[2].tree_version.should.be.head
+
+    old_tree = Olelo::Page.find('/', tree.previous_version)
+    old_tree.tree_version.should.not.be.head
+    tree.previous_version.should.equal old_tree.tree_version
+    old_tree.tree_version.should.equal old_tree.children[0].tree_version
+    old_tree.tree_version.should.equal old_tree.children[1].tree_version
+    old_tree.tree_version.should.equal old_tree.children[2].tree_version
+    old_tree.children[0].tree_version.should.not.be.head
+  end
+
+  it 'has working children' do
+    create_page('tree1/subpage1')
+    create_page('tree1/subpage2')
+    create_page('tree1/subpage3')
+    create_page('tree2/subpage1')
+    create_page('tree2/subpage2')
+    create_page('tree2/subpage3')
+    create_page('tree3/subpage1')
+    create_page('tree3/subpage2')
+    create_page('tree3/subpage3')
+    create_page('tree1/subtree1/subsubpage1')
+    create_page('tree1/subtree1/subsubpage2')
+    create_page('tree1/subtree1/subsubpage3')
+    create_page('page1')
+    create_page('page2')
+    create_page('page3')
+
+    tree = Olelo::Page.find!('/')
+    tree.children.size.should.equal 6
+    tree.children[0].path.should.equal 'page1'
+    tree.children[1].path.should.equal 'page2'
+    tree.children[2].path.should.equal 'page3'
+    tree.children[3].path.should.equal 'tree1'
+    tree.children[4].path.should.equal 'tree2'
+    tree.children[5].path.should.equal 'tree3'
+
+    tree = Olelo::Page.find!('/tree1')
+    tree.children.size.should.equal 4
+    tree.children[0].path.should.equal 'tree1/subpage1'
+    tree.children[1].path.should.equal 'tree1/subpage2'
+    tree.children[2].path.should.equal 'tree1/subpage3'
+    tree.children[3].path.should.equal 'tree1/subtree1'
+
+    tree = Olelo::Page.find!('/tree1/subtree1')
+    tree.children.size.should.equal 3
+    tree.children[0].path.should.equal 'tree1/subtree1/subsubpage1'
+    tree.children[1].path.should.equal 'tree1/subtree1/subsubpage2'
+    tree.children[2].path.should.equal 'tree1/subtree1/subsubpage3'
+  end
+end

data/test/request_test.rb

@@ -0,0 +1,166 @@
+require 'helper'
+require 'olelo/middleware/flash'
+require 'olelo/middleware/force_encoding'
+require 'rack/session/pool'
+
+Rack::MockRequest::DEFAULT_ENV['REMOTE_ADDR'] = 'localhorst'
+
+class Bacon::Context
+  include Rack::Test::Methods
+  include Olelo::Util
+
+  attr_reader :app
+end
+
+describe 'requests' do
+  before do
+    @test_path = File.expand_path(File.join(File.dirname(__FILE__), '.test'))
+    @app_path = File.expand_path(File.join(File.dirname(__FILE__), '..'))
+
+    default_config = {
+      :title             => 'Olelo',
+      :app_path          => @app_path,
+      :plugins_path      => File.join(@app_path, 'plugins'),
+      :config_path       => File.join(@app_path, 'config'),
+      :initializers_path => File.join(@app_path, 'config', 'initializers'),
+      :views_path        => File.join(@app_path, 'views'),
+      :themes_path       => File.join(@app_path, 'static', 'themes'),
+      :theme             => 'atlantis',
+      :cache_store       => {
+        :type => 'file',
+        :file => {
+          :root => File.join(@test_path, 'cache')
+        }
+      },
+      :base_path         => '/',
+      :production        => true,
+      :locale	         => 'en_US',
+      :sidebar_page      => 'Sidebar',
+      :authentication => {
+        :service  => :yamlfile,
+        :enable_signup => true,
+        :yamlfile => {
+          :store  => File.join(@test_path, 'users.yml'),
+        },
+      },
+      :mime => [
+                'extension',
+                'content',
+                'text/x-creole',
+               ],
+      :mime_suggestions => [],
+      :disabled_plugins => [
+                            'security/readonly_wiki',
+			    'security/private_wiki',
+                            'editor/antispam',
+                           ],
+      :repository => {
+        :type  => 'git',
+        :git => {
+          :path => File.join(@test_path, 'repository'),
+          :bare => true,
+        },
+      }
+    }
+
+    Olelo::Config.instance.update(default_config)
+    Olelo::Repository.instance = nil
+
+    logger = Logger.new(File.join(@app_path, 'test.log'))
+
+    @app = Rack::Builder.new do
+      use Olelo::Middleware::ForceEncoding
+      use Rack::Session::Pool
+      use Olelo::Middleware::Flash, :set_accessors => %w(error warn info)
+      Olelo::Initializer.initialize(logger)
+      run Olelo::Application.new
+    end
+  end
+
+  after do
+    FileUtils.rm_rf(@test_path)
+  end
+
+  it 'should have empty repository' do
+    get '/'
+    last_response.should.be.redirect
+    last_response.location.should.equal '/new'
+  end
+
+  it 'should show login page' do
+    get '/login'
+    last_response.should.be.ok
+    last_response.body.should.include '<form action="&#47;signup" method="post">'
+    last_response.body.should.include '<form action="&#47;login" method="post">'
+  end
+
+  it 'should show to /new' do
+    get '/new/not-existing'
+    last_response.should.be.ok
+  end
+
+  it 'should redirect to /new' do
+    get '/not-existing'
+    last_response.should.be.redirect
+    last_response.location.should.equal '/new/not-existing'
+  end
+
+  it 'should not redirect to /new' do
+    get '/edit/not-existing'
+    last_response.should.be.not_found
+
+    get '/history/not-existing'
+    last_response.should.be.not_found
+  end
+
+  it 'should create page' do
+    data = {
+      'action' => 'edit-close',
+      'content' => 'Content of the Testpage',
+      'comment' => 'My Comment'
+    }
+    post '/Testfolder/Testpage', data
+
+    last_response.should.be.redirect
+    last_response.location.should.equal '/Testfolder/Testpage'
+
+    get '/'
+    last_response.should.be.ok
+
+    get '/Testfolder/Testpage'
+    last_response.should.be.ok
+    last_response.body.should.include '<h1>Testpage</h1>'
+    last_response.body.should.include 'Content of the Testpage'
+
+    get '/history/Testfolder/Testpage'
+    last_response.should.be.ok
+    last_response.body.should.include 'My Comment'
+
+    get '/edit/Testfolder/Testpage'
+    last_response.should.be.ok
+  end
+
+  it 'should create page with special characters' do
+    data = {
+      'action' => 'edit-close',
+      'content' => "\343\201\231\343\201\277\343\201\276\343\201\233\343\202\223\343\202\217\343\201\213\343\202\212\343\201\276\343\201\233\343\202\223",
+      'comment' => "\346\265\213\350\257\225",
+    }
+    post '/%E5%AD%90%E4%BE%9B%E3%82%92%E5%85%AC%E5%9C%92/%E4%B8%AD%E6%96%87', data
+    last_response.should.be.redirect
+
+    loc1, loc2 = '', ''
+    last_response.location.each_byte {|c| loc1 << c}
+    unescape('/%E5%AD%90%E4%BE%9B%E3%82%92%E5%85%AC%E5%9C%92/%E4%B8%AD%E6%96%87').each_byte {|c| loc2 << c }
+    loc1.should.equal loc2
+
+    get '/%E5%AD%90%E4%BE%9B%E3%82%92%E5%85%AC%E5%9C%92/%E4%B8%AD%E6%96%87'
+    last_response.should.be.ok
+
+    get '/history/%E5%AD%90%E4%BE%9B%E3%82%92%E5%85%AC%E5%9C%92/%E4%B8%AD%E6%96%87'
+    last_response.should.be.ok
+
+    get '/edit/%E5%AD%90%E4%BE%9B%E3%82%92%E5%85%AC%E5%9C%92/%E4%B8%AD%E6%96%87'
+    last_response.should.be.ok
+  end
+end

data/test/string_extensions_test.rb

@@ -0,0 +1,32 @@
+require 'olelo/extensions'
+
+describe 'String extensions' do
+  it 'should have unindent' do
+    %{a
+      b
+      c}.unindent.should.equal "a\nb\nc"
+  end
+
+  it 'should have #starts_with?' do
+    '123456789'.starts_with?('12').should.equal true
+    '123456789'.starts_with?('23').should.not.equal true
+  end
+
+  it 'should have #ends_with?' do
+    '123456789'.ends_with?('89').should.equal true
+    '123456789'.ends_with?('98').should.not.equal true
+  end
+
+  it 'should have #cleanpath' do
+    '/'.cleanpath.should.equal ''
+    '/a/b/c/../'.cleanpath.should.equal 'a/b'
+    '/a/./b/../c/../d/./'.cleanpath.should.equal 'a/d'
+    '1///2'.cleanpath.should.equal '1/2'
+  end
+
+  it 'should have #/' do
+    (''/'').should.equal ''
+    ('//a/b///'/'').should.equal 'a/b'
+    ('a'/'x'/'..'/'b'/'c'/'.').should.equal 'a/b/c'
+  end
+end