aslakjo-aslakjo-comatose 2.0.5.1

data/test/test_helper.rb

@@ -0,0 +1,55 @@
+ENV["RAILS_ENV"] = 'test'
+
+require File.expand_path(File.join(File.dirname(__FILE__), '../../../../config/environment.rb'))
+
+require 'test/unit'
+require 'test_help'
+
+class Test::Unit::TestCase
+
+  self.fixture_path = File.expand_path( File.join(File.dirname(__FILE__), 'fixtures') )
+
+  self.use_transactional_fixtures = true
+  self.use_instantiated_fixtures  = false
+  
+  def setup
+    Comatose.configure do |config|
+      config.default_filter      = :textile
+      config.default_processor   = :liquid
+      config.authorization       = Proc.new { true }
+      config.admin_authorization = Proc.new { true }
+      config.admin_get_author    = Proc.new { request.env['REMOTE_ADDR'] }
+      config.admin_get_root_page = Proc.new { ComatosePage.root }
+    end
+    TextFilters.default_filter = "Textile"
+  end
+
+  def create_page(options={})
+    ComatosePage.create({ :title => 'Comatose Page', :author=>'test', :parent_id=>1 }.merge(options))
+  end
+  
+  def comatose_page(sym)
+    ComatosePage.find_by_slug(sym.to_s.dasherize)
+  end
+  
+  def assert_difference(object, method = nil, difference = 1)
+    initial_value = object.send(method)
+    yield
+    assert_equal initial_value + difference, object.send(method), "#{object}##{method}"
+  end
+
+  def assert_no_difference(object, method, &block)
+    assert_difference object, method, 0, &block
+  end
+
+  class << self
+    def should(behave,&block)
+      method_name = "test_should_#{behave.gsub(' ', '_')}"
+      if block
+        define_method method_name, &block
+      else
+        puts ">>> Untested: #{name.sub(/Test$/,'')} should #{behave}"
+      end
+    end
+  end
+end

data/test/unit/class_options_test.rb

@@ -0,0 +1,52 @@
+require File.dirname(__FILE__) + '/../test_helper'
+require 'support/class_options'
+
+class ClassOptionsTest < Test::Unit::TestCase
+
+  class Opts
+    define_option :nothing, nil
+    define_option :nada
+    define_option :name, "Matt"
+    define_option :age, 30
+    define_option :mode, :test
+    define_option :listing, ['one', 'two', 'three']
+    define_option :opts_1, {'one'=>'ONE', 'two'=>'TWO'}
+    define_option :opts_2, {:one=>'ONE', :two=>'TWO'}
+    define_option :should_work, true
+    define_option :would_surprise, false
+  end
+
+  should "allow nil as a default" do
+    assert_equal nil, Opts.nothing
+    assert_equal nil, Opts.nada
+  end
+  
+  should "allow boolean defaults" do
+    assert_equal true, Opts.should_work
+    assert_equal false, Opts.would_surprise
+  end
+
+  should "allow string defaults" do
+    assert_equal 'Matt', Opts.name
+  end
+
+  should "allow numeric defaults" do
+    assert_equal 30, Opts.age
+  end
+
+  should "allow symbolic defaults" do
+    assert_equal :test, Opts.mode
+  end
+
+  should "allow array literals as defaults" do
+    assert_equal ['one', 'two', 'three'], Opts.listing
+  end
+
+  should "allow hash literals as defaults" do
+    h1 = {'one'=>'ONE', 'two'=>'TWO'}
+    h2 = {:one=>'ONE', :two=>'TWO'}
+    assert_equal h1, Opts.opts_1
+    assert_equal h2, Opts.opts_2
+  end
+  
+end

data/test/unit/comatose_page_test.rb

@@ -0,0 +1,136 @@
+require File.dirname(__FILE__) + '/../test_helper'
+
+class ComatosePageTest < Test::Unit::TestCase
+
+  fixtures :comatose_pages
+
+  should "create page" do
+    assert_difference ComatosePage, :count do
+      page = create_page :title=>'New Page Name'
+      assert !page.new_record?, "#{page.errors.full_messages.to_sentence}"
+    end
+  end
+  
+  should "create a new version of an updated page" do
+    page = create_page
+    assert_difference page, :version do
+      page.update_attribute :body, "I'm the new content!"
+      assert_equal "<p>I&#8217;m the new content!</p>", page.to_html
+    end
+  end
+  
+  should "render content through textile and liquid processors" do
+    page = create_page :title=>'Title Here', :body=>'h1. {{page.title}}'
+    assert_equal "<h1>Title Here</h1>", page.to_html
+  end
+  
+  should "not allow creation of page when missing a title" do
+    assert_no_difference ComatosePage, :count do
+      p = create_page(:title => nil)
+      assert p.errors.on(:title)
+    end
+  end
+  
+  should "have good fixtures for this to work out" do
+    assert_equal 'Home Page', root_page.title
+    assert_equal 'home-page', root_page.slug
+    assert_equal 'Comatose',  root_page.author
+    assert_equal "",          root_page.full_path
+    assert_equal 'faq',       faq_page.full_path
+  end
+  
+  should "generate slugs correctly" do
+    assert_equal 'hello-how-are-you',     new_page_slug( "Hello, How Are You?" )
+    assert_equal 'i-have-too-much-space', new_page_slug( "I    have  too   much space" )
+    assert_equal 'i-have-leading-and-trailing-space', new_page_slug( "   I have leading and trailing space  " )
+    assert_equal 'what-about-dashes',     new_page_slug( "What about - dashes?" )
+    assert_equal 'a-bizarre-title',       new_page_slug( 'A !@!@#$%^<>&*()_+{} Bizarre TiTle!' )
+    assert_equal '001-numbers-too',       new_page_slug( "001 Numbers too" )
+  end
+   
+  should "generate page paths correctly" do
+    products = root_page.children.create( :title=>'Products' )
+    assert_equal 'products', products.full_path
+   
+    books = products.children.create( :title=>'Books' )
+    assert_equal 'products/books', books.full_path
+    
+    novels = books.children.create( :title=>'Novels' )
+    assert_equal 'products/books/novels', novels.full_path
+   
+    comics = books.children.create( :title=>'Comics' )
+    assert_equal 'products/books/comics', comics.full_path
+  end
+  
+  should "update page paths when pages are moved" do
+    page = comatose_page :params
+    assert_equal 'params', page.full_path
+    
+    q1pg = comatose_page :question_one
+    page.parent_id = q1pg.id
+    assert page.save, "Page.save"
+    
+    page.reload
+    assert_equal 'faq/question-one/params', page.full_path
+  
+    q1pg.reload
+    q1pg.slug = "q-1"
+    assert q1pg.save, "Page.save"
+    assert_equal "faq/q-1", q1pg.full_path
+  
+    page.reload
+    assert_equal 'faq/q-1/params', page.full_path
+  end
+  
+  should "set an AR error with processor syntax error info" do
+    page = create_page :title=>'Title Here', :body=>'h1. {% crap %}'
+    assert !page.save, page.errors.full_messages.to_sentence
+  end
+  
+  should "render body text accurately" do
+    assert_equal "<h1>Home Page</h1>\n<p>This is your <strong>home page</strong>.</p>", root_page.to_html
+    assert_equal "<h1>Frequently Asked Questions</h1>\n<h2><a href=\"/faq/question-one\">Question One?</a></h2>\n<p>Content for <strong>question one</strong>.</p>\n<h2><a href=\"/faq/question-two\">Question Two?</a></h2>\n<p>Content for <strong>question two</strong>.</p>", faq_page.to_html
+  end
+   
+  should "render data from parameterized calls too" do
+    assert_equal "<p>I&#8217;m</p>", param_driven_page.to_html
+    assert_equal "<p>I&#8217;m From the Params Hash</p>", param_driven_page.to_html(:extra=>'From the Params Hash')
+  end
+  
+  should "render data from a Drop" do
+    Comatose.define_drop "app" do
+      def test
+        "From Drop"
+      end
+    end
+    p = create_page(:title=>'Test Drop', :body=>'{{ app.test  }}')
+    assert_equal "<p>From Drop</p>", p.to_html
+  end
+
+  # Test illustrates broken partial, sort of
+  # <%= render :comatose => 'faq/question-one' %> should work, but does not
+  should "render inline partial" do
+    Comatose.config.default_processor = :erb
+    page = create_page :body => "Included: <%= ComatosePage.find_by_path('faq/question-one').to_html %>", :filter_type => :none
+    assert_equal "Included: <p>Content for <strong>question one</strong>.</p>", page.to_html
+  end
+
+  protected
+
+    def new_page_slug(title)
+      create_page( :title=>title ).slug
+    end
+    
+    def root_page
+      ComatosePage.root
+    end
+    
+    def faq_page
+      comatose_page :faq
+    end
+    
+    def param_driven_page
+      comatose_page :params
+    end
+    
+end

data/test/unit/processing_context_test.rb

@@ -0,0 +1,108 @@
+require File.dirname(__FILE__) + '/../test_helper'
+require 'comatose'
+
+class ProcessingContextTest < Test::Unit::TestCase
+
+  fixtures :comatose_pages
+  
+  def setup
+    @root = comatose_page( :home_page ) 
+    @binding = Comatose::ProcessingContext.new(@root)
+  end
+
+  # should "process liquid tags with no filters correctly" do
+  #   result = TextFilters.transform('{{ page.title }}', @binding, :none, :liquid)
+  #   assert_equal 'Home Page', result
+  # end
+  # 
+  should "process erb tags correctly" do
+    result = TextFilters.transform('<%= page.title %>', @binding, :none, :erb)
+    assert_equal 'Home Page', result
+  end
+
+  should "support text translation and processing with ERB" do
+    src = 'The title is *<%= title %>*'
+    if TextFilters.all.keys.include? 'None'
+      assert_equal "The title is *Home Page*", TextFilters.transform(src, @binding, :none, :erb)
+    end    
+    if TextFilters.all.keys.include? 'Textile'
+      assert_equal "<p>The title is <strong>Home Page</strong></p>", TextFilters.transform(src, @binding, :textile, :erb)
+    end    
+    if TextFilters.all.keys.include? 'Markdown'
+      assert_equal "<p>The title is <em>Home Page</em></p>", TextFilters.transform(src, @binding, 'Markdown', :erb)
+    end    
+    if TextFilters.all.keys.include? 'RDoc'
+      assert_equal "<p>\nThe title is *Home Page*\n</p>\n", TextFilters.transform(src, @binding, :rdoc, :erb)
+    end
+  end
+  
+  should "support text translation and processing with Liquid" do
+    src = 'The title is *{{ page.title }}*'
+    assert_equal "The title is *Home Page*", TextFilters.transform(src, @binding, :none, :liquid)
+    if TextFilters.all_titles.include? 'Textile'
+      assert_equal "<p>The title is <strong>Home Page</strong></p>", TextFilters.transform(src, @binding, :textile, :liquid)
+    end    
+    if TextFilters.all_titles.include? 'Markdown'
+      assert_equal "<p>The title is <em>Home Page</em></p>", TextFilters.transform(src, @binding, :markdown, :liquid)
+    end    
+    if TextFilters.all_titles.include? 'RDoc'
+      assert_equal "<p>\nThe title is *Home Page*\n</p>\n", TextFilters.transform(src, @binding, :rdoc, :liquid)
+    end
+  end
+  
+  UNSAFE_PROPS = %w(version position)
+  SAFE_PROPS = %w(id full_path uri slug keywords title to_html filter_type author updated_on created_on)
+  
+  should "allow access to safe properties and methods when processing with ERB" do
+    binding = Comatose::ProcessingContext.new( comatose_page(:faq) )
+    SAFE_PROPS.each do |prop|
+      assert_not_equal '', TextFilters.transform("<%= page.#{prop} %>", binding, :none, :liquid), "on page.#{prop}"
+    end
+  end
+  
+  should "prevent access to protected properties and methods when processing with ERB" do
+    binding = Comatose::ProcessingContext.new( comatose_page(:faq) )
+    UNSAFE_PROPS.each do |prop|
+      assert_equal "<%= page.#{prop} %>", TextFilters.transform("<%= page.#{prop} %>", binding, :none, :liquid), "on page.#{prop}"
+    end    
+  end
+  
+  should "allow access to safe properties and methods when processing with Liquid" do
+    binding = Comatose::ProcessingContext.new( comatose_page(:faq) )
+    SAFE_PROPS.each do |prop|
+      assert_not_equal '', TextFilters.transform("{{ page.#{prop} }}", binding, :none, :liquid), "on page.#{prop}"
+    end
+  end
+  
+  should "prevent access to protected properties and methods when processing with Liquid" do
+    binding = Comatose::ProcessingContext.new( comatose_page(:faq) )
+    UNSAFE_PROPS.each do |prop|
+      assert_equal '', TextFilters.transform("{{ page.#{prop} }}", binding, :none, :liquid), "on page.#{prop}"
+    end    
+  end
+  
+  should "allow referenceing of defined ComatoseDrops" do
+    Comatose.define_drop "app" do
+      def test
+        "TEST"
+      end
+    end
+    
+    result = TextFilters.transform('{{ app.test }}', @binding, :none, :liquid)
+    assert_equal 'TEST', result
+    
+    result2 = TextFilters.transform('<%= app.test %>', @binding, :none, :erb)
+    assert_equal 'TEST', result2
+  end
+  
+  should "let ComatoseDrop errors bubble upward" do
+    Comatose.define_drop "broken" do
+      def test
+        "TEST #{crap}"
+      end
+    end
+    
+    assert_raise(RuntimeError) { TextFilters.transform('{{ broken.test }}', @binding, :none, :liquid) }
+  end
+
+end

data/test/unit/text_filters_test.rb

@@ -0,0 +1,52 @@
+require File.dirname(__FILE__) + '/../test_helper'
+
+require 'erb'
+require 'liquid'
+
+require 'text_filters'
+# require 'comatose/page'
+# require 'comatose/processing_context'
+
+class TextFiltersTest < Test::Unit::TestCase
+  # fixtures :comatose_pages
+  # self.use_instantiated_fixtures  = true
+
+  should "not alter output when using filter :none" do
+    assert_equal "line one\nline two", TextFilters.render_text("line one\nline two", :none)
+  end
+
+  should "convert newlines into <br/>s when using :simple filter" do
+    assert_equal "line one<br/>line two", TextFilters.render_text("line one\nline two", :simple)
+  end
+
+  should "support Textile, if it's available, using :textile or 'Textile' as a key" do
+    if TextFilters.all.keys.include? 'Textile'
+      assert_equal "<p>testing <strong>bold</strong></p>", TextFilters.render_text("testing *bold*", :textile)
+      assert_equal "<p>testing <strong>bold</strong></p>", TextFilters.render_text("testing *bold*", 'Textile')
+    end
+  end
+  
+  should "support Markdown, if it's available, using :markdown or 'Markdown' as a key" do
+    if TextFilters.all.keys.include? 'Markdown'
+      assert_equal "<p>testing <em>bold</em></p>", TextFilters.render_text("testing *bold*", :markdown)
+    end
+  end
+
+  should "support RDoc, if it's available, using :rdoc or 'RDoc' as a key" do
+    if TextFilters.all.keys.include? 'RDoc'
+      assert_equal "<p>\ntesting <b>bold</b>\n</p>\n", TextFilters.render_text("testing *bold*", :rdoc)
+      assert_equal "<p>\ntesting <b>bold</b>\n</p>\n", TextFilters.render_text("testing *bold*", 'RDoc')
+    end
+  end
+
+  should "support transformation of parameters via ERB" do
+    src = 'Hello, <%= name %>'
+    assert_equal "Hello, Matt", TextFilters.transform(src, {'name'=>'Matt'}, :none, :erb)
+  end
+
+  should "support transformation of parameters via Liquid" do
+    src = 'Hello, {{ name }}'
+    assert_equal "Hello, Matt", TextFilters.transform(src, {'name'=>'Matt'}, :none, :liquid)
+  end
+  
+end

data/views/comatose_admin/_form.html.erb

@@ -0,0 +1,96 @@
+<%= error_messages_for :page %>
+
+<% form_for :page, @page do |f| %>
+<table class="page-form" cellpadding="0" cellspacing="0">
+
+  <tr>
+    <td class="label"><label for="page_title">Title</label></td>
+    <td class="field"><%= f.text_field :title, :tabindex=>1, :maxlength=>255, :size=>50 %> <%= link_to_function "More...", "ComatoseEditForm.toggle_extra_fields(this)", :id=>'more-options' %></td>
+  </tr>
+
+  <tr id="slug_row" class="meta-info">
+    <td class="label"><label for="page_slug">Slug</label></td>
+    <td class="field"><%= f.text_field :slug, :tabindex=>6, :maxlength=>255, :size=>50, :disabled=>@root_pages.include?(@page) %></td>
+  </tr>
+
+<% if show_field? 'keywords' %>
+  <tr id="keywords_row" class="meta-info">
+    <td class="label"><label for="page_keywords">Keywords</label></td>
+    <td class="field"><%= f.text_field :keywords, :tabindex=>7, :maxlength=>1000, :size=>50 %></td>
+  </tr>
+<% end %>
+
+<% if show_field? 'parent' %>
+  <tr id="parent_row" class="meta-info">
+    <td class="label"><label for="page_parent">Parent</label></td>
+    <td class="field">
+      <% if mode != :new and @root_pages.include? @page %>
+      <select id="page_parent" disabled="true"><option><%= @page.title %> is a root page</option></select>
+      <% else %>
+      <select id="page_parent" name="page[parent_id]" tabindex="8"><%= tree_select_box @root_pages, @page.parent_id, @page.id %></select>
+       <% end %>
+    </td>
+  </tr>
+<% end %>
+
+  <tr>
+    <td class="label body"><label for="page_body">Body</label></td>
+    <td class="field"><%= f.text_area :body, :rows=>20, :tabindex=>2 %></td>
+  </tr>
+
+<% if show_field? 'filter' %>
+  <tr id="filter_row" class="meta-info">
+    <td class="label"><label for="page_filter_type">Filter</label></td>
+    <td class="field">
+      <%= select_tag 'page[filter_type]', options_for_select(TextFilters.all_titles.sort, @page.filter_type || Comatose.config.default_filter), :tabindex=>3, :id=>'page_filter_type' %>
+      <span class="field-help">Converts plain text into html</span>
+    </td>
+  </tr>
+<% end %>
+
+<% if show_field? 'created_on' %>
+  <tr id="created_row" class="meta-info">
+    <td class="label"><label for="page_created_on">Created</label></td>
+    <td class="field">
+      <%= f.datetime_select :created_on %>
+    </td>
+  </tr>
+<% end %>
+
+</table>
+
+<div id="button-group">
+<% unless @page.updated_on.nil? %>
+  <div class="last-update">
+    <label>
+      Updated <%= time_ago_in_words @page.updated_on, true %> ago by
+      <%= @page.author %>.
+      <%= link_to(pluralize(@page.versions.length, 'revision', 'revisions'), :action=>'versions', :id=>@page) if @page.versions.length > 0 %>
+    </label>
+  </div>
+<% end %>
+<%= image_tag 'comatose/spinner.gif', :id=>'spinner', :align=>'absmiddle', :style=>'display:none;' %>
+  <%= button_to_function('Preview', "ComatoseEditForm.preview_content('#{url_for :controller=>controller.controller_name, :action=>'preview', :id=>@page}')", :tabindex=>3, :id=>'preview-btn' ) if show_field? 'preview' %> &nbsp;
+  <%= submit_tag ((mode == :edit) ? 'Save Changes' : 'Create Page'), :tabindex=>4 %>
+  <% if @page.versions.length > 0 %>
+  &nbsp; <%= link_to "Revert", :action=>'versions', :id=>@page %>
+  <% end %>
+  or
+  <a href="<%= url_for :controller=>controller.controller_name, :action=>'index' %>" onlick="ComatoseEditForm.cancel(this.href); return false;" tabindex="5">Cancel</a>
+</div>
+<% end %>
+
+<div id="preview-area">
+  <fieldset>
+    <legend>Page Preview</legend>
+    <div class="preview-body">
+      <div id="preview-panel"><span style='color:blue;'>Loading Preview...</span></div>
+      <div style="clear:both"></div>
+    </div>
+  </fieldset>
+  <div class="commands">
+    <a href="#" onclick="$('preview-area').hide();">Close Preview</a> or <a href="#">Back to top</a>
+  </div>
+</div>
+
+<%= javascript_tag "ComatoseEditForm.init('#{mode.to_s}');" %>