Download as
Browse Code »

Commit 628002e31d386a6ce6f91e7ed9d61c3683d3d127

Authored by Antonio Terceiro
1 parent 0cd4c377
Exists in master and in 28 other branches 1.2+SPB2, 2_split_by_role, add_fields_in_api, api_fixies, api_pagination, api_private_token, api_tasks, article_hit_count, blog_page_bug, checkbox_to_user_can_edit_page, communities_ratings, communities_ratings_moderation, community_ratings_rebase, fix_cloned_article_parent, fix_gallery_image_url, fix_javascript_categories, fix_optional_fields, fix_suggest_article, fix_user_email_validation, highlight_block_fix, notification_plugin, notification_plugin_rebase, script_to_update, split_by_role, stable, temp_ratings, tests, tests_fixies

Testing documentation ordering 

(ActionItem1487)
Showing 2 changed files with 14 additions and 3 deletions   Show diff stats
app/models/doc_topic.rb
  Diff comments   View file @628002e
... ... @@ -22,8 +22,11 @@ class DocTopic < DocItem
22 22 end
23 23  
24 24 def self.order(line)
25   - line =~ /<h1 class="order-(.*)">.*<\/h1>/
26   - $1
  25 + if line =~ /<h1 class="order-(.*)">.*<\/h1>/
  26 + $1.to_i
  27 + else
  28 + 0
  29 + end
27 30 end
28 31  
29 32 end
... ...
test/unit/doc_topic_test.rb
  Diff comments   View file @628002e
1   -require 'test_helper'
  1 +require File.dirname(__FILE__) + '/../test_helper'
2 2  
3 3 class DocTopicTest < ActiveSupport::TestCase
4 4 should 'be a DocItem' do
... ... @@ -25,4 +25,12 @@ class DocTopicTest &lt; ActiveSupport::TestCase
25 25 end
26 26 end
27 27  
  28 + should 'read order from HTML' do
  29 + assert_equal 1, DocTopic.order('<h1 class="order-1">Some topic</h1>')
  30 + end
  31 +
  32 + should 'use 0 as order by default' do
  33 + assert_equal 0, DocTopic.order('<h1>Some topic</h1>')
  34 + end
  35 +
28 36 end
... ...