typo 3.99.0

data/test/unit/assumptions_tests.rb

@@ -0,0 +1,54 @@
+require File.dirname(__FILE__) + '/../test_helper'
+
+require 'content'
+require 'article'
+
+# Assumptions test? What's that then?
+#
+# Here's where we test assumptions about way Rails behaves that we
+# rely on in the codebase. That way, if the Rails behaviour changes,
+# we can catch the underlying cause here, which can save a good deal
+# of bug chasing.
+
+class Article
+  def got_saved
+    @saved
+  end
+
+  protected
+
+  before_save :set_saved_var
+
+  def set_saved_var
+    @saved = true
+  end
+end
+
+class AssumptionsTest < Test::Unit::TestCase
+
+  # This only works in the case where the article caches the comment
+  # counter. Because the counter_cache is 'invisible' to the model, it
+  # could be argued that incrementing it shouldn't go triggering the
+  # 'visible' save/update/whatever hooks in Activerecord. (Which would
+  # be a shame, because we rely on it). So, we'll continue to check
+  # that Rails does what we expect.
+
+  def test_adding_a_comment_saves_the_article
+    a = Article.new(:title => 'A title', :body => 'Some body')
+
+    assert !a.got_saved
+    assert  a.save
+    assert  a.got_saved
+
+    a = nil
+
+    a = Article.find(:first)
+
+    assert !a.got_saved
+
+    c = Comment.new( :author => 'Piers Cawley', :body => 'A boring comment')
+
+    a.comments << c
+    assert a.got_saved
+  end
+end

data/test/unit/audioscrobbler_test.rb

@@ -0,0 +1,132 @@
+require File.dirname(__FILE__) + '/../test_helper'
+
+class AudioscrobblerTest < Test::Unit::TestCase
+
+  def setup
+    @audioscrobbler = Audioscrobbler.new("", false)
+    @audioscrobbler.send(:parse, fixture)
+  end
+
+  def test_parser
+    assert_equal 3, @audioscrobbler.items.size
+  end
+
+  def test_fields
+    assert_equal "Audioscrobbler Musical Profile: benjackson", @audioscrobbler.title
+    assert_equal "http://www.audioscrobbler.com/user/benjackson/", @audioscrobbler.link
+  end
+
+  def test_items
+    assert_equal "Badly Drawn Boy", @audioscrobbler.items[0].artist
+    assert_equal "Life Turned Upside Down", @audioscrobbler.items[0].title
+    assert_equal "http://www.audioscrobbler.com/music/Badly+Drawn+Boy/_/Life+Turned+Upside+Down", @audioscrobbler.items[0].link
+  end
+
+  private
+
+  def fixture
+    %{    <rdf:RDF>
+
+    	<channel rdf:about="http://ws.audioscrobbler.com/rdf/history/benjackson">
+    <title>Audioscrobbler Musical Profile: benjackson</title>
+    <link>http://www.audioscrobbler.com/user/benjackson/</link>
+
+    	<description>
+    benjackson's last played tracks, as recorded by Audioscrobbler.com
+    </description>
+    <cc:license rdf:resource="http://www.creativecommons.org/licenses/by-nc-sa/1.0/"/>
+    <dc:creator>benjackson</dc:creator>
+    <dc:date>2005-08-03T2:33:30+00:00</dc:date>
+    <admin:generatorAgent rdf:resource="http://www.audioscrobbler.com/"/>
+    <admin:errorReportsTo rdf:resource="mailto:support@audioscrobbler.com"/>
+
+    	<items>
+
+    	<rdf:Seq>
+    <rdf:li rdf:resource="http://mm.musicbrainz.org/track/MBIDHERE"/>
+    <rdf:li rdf:resource="http://mm.musicbrainz.org/track/MBIDHERE"/>
+    <rdf:li rdf:resource="http://mm.musicbrainz.org/track/MBIDHERE"/>
+    <rdf:li rdf:resource="http://mm.musicbrainz.org/track/MBIDHERE"/>
+    <rdf:li rdf:resource="http://mm.musicbrainz.org/track/MBIDHERE"/>
+    <rdf:li rdf:resource="http://mm.musicbrainz.org/track/MBIDHERE"/>
+    <rdf:li rdf:resource="http://mm.musicbrainz.org/track/MBIDHERE"/>
+    <rdf:li rdf:resource="http://mm.musicbrainz.org/track/MBIDHERE"/>
+    <rdf:li rdf:resource="http://mm.musicbrainz.org/track/MBIDHERE"/>
+    <rdf:li rdf:resource="http://mm.musicbrainz.org/track/MBIDHERE"/>
+    </rdf:Seq>
+    </items>
+    </channel>
+
+    	<item rdf:about="http://mm.musicbrainz.org/track/MBIDHERE">
+
+    	<link>http://www.audioscrobbler.com/music/Badly+Drawn+Boy/_/Life+Turned+Upside+Down</link>
+    <description>Badly Drawn Boy - Life Turned Upside Down</description>
+    <guid isPermaLink="false">8394@www.audioscrobbler.com/user/benjackson/</guid>
+    <dc:date>2005-08-03T02:31:18+00:00</dc:date>
+    <dc:title>Life Turned Upside Down</dc:title>
+
+    	<mm:Artist rdf:about="http://mm.musicbrainz.org/artist/0881daf1-20df-4a3e-a84f-6476a84bb172">
+
+    	<dc:creator>
+    <dc:title>Badly Drawn Boy</dc:title>
+    </dc:creator>
+    </mm:Artist>
+
+    	<mm:albumlist>
+
+    	<mm:Album rdf:about="http://mm.musicbrainz.org/album/MBIDHERE">
+    <dc:title>One Plus One Is One</dc:title>
+    </mm:Album>
+    </mm:albumlist>
+    </item>
+
+    	<item rdf:about="http://mm.musicbrainz.org/track/MBIDHERE">
+
+    	<link>http://www.audioscrobbler.com/music/Badly+Drawn+Boy/_/Logic+of+a+Friend</link>
+    <description>Badly Drawn Boy - Logic of a Friend</description>
+    <guid isPermaLink="false">8393@www.audioscrobbler.com/user/benjackson/</guid>
+    <dc:date>2005-08-03T02:26:39+00:00</dc:date>
+    <dc:title>Logic of a Friend</dc:title>
+
+    	<mm:Artist rdf:about="http://mm.musicbrainz.org/artist/0881daf1-20df-4a3e-a84f-6476a84bb172">
+
+    	<dc:creator>
+    <dc:title>Badly Drawn Boy</dc:title>
+    </dc:creator>
+    </mm:Artist>
+
+    	<mm:albumlist>
+
+    	<mm:Album rdf:about="http://mm.musicbrainz.org/album/MBIDHERE">
+    <dc:title>One Plus One Is One</dc:title>
+    </mm:Album>
+    </mm:albumlist>
+    </item>
+
+    	<item rdf:about="http://mm.musicbrainz.org/track/MBIDHERE">
+
+    	<link>http://www.audioscrobbler.com/music/Badly+Drawn+Boy/_/Four+Leaf+Clover</link>
+    <description>Badly Drawn Boy - Four Leaf Clover</description>
+    <guid isPermaLink="false">8392@www.audioscrobbler.com/user/benjackson/</guid>
+    <dc:date>2005-08-03T02:22:17+00:00</dc:date>
+    <dc:title>Four Leaf Clover</dc:title>
+
+    	<mm:Artist rdf:about="http://mm.musicbrainz.org/artist/0881daf1-20df-4a3e-a84f-6476a84bb172">
+
+    	<dc:creator>
+    <dc:title>Badly Drawn Boy</dc:title>
+    </dc:creator>
+    </mm:Artist>
+
+    	<mm:albumlist>
+
+    	<mm:Album rdf:about="http://mm.musicbrainz.org/album/MBIDHERE">
+    <dc:title>One Plus One Is One</dc:title>
+    </mm:Album>
+    </mm:albumlist>
+    </item>
+
+    </rdf:RDF>
+    }
+  end
+end

data/test/unit/blacklist_pattern_test.rb

@@ -0,0 +1,14 @@
+require File.dirname(__FILE__) + '/../test_helper'
+
+class BlacklistPatternTest < Test::Unit::TestCase
+  fixtures :blacklist_patterns
+
+  def setup
+    @blacklist_pattern = BlacklistPattern.find(1)
+  end
+
+  # Replace this with your real tests.
+  def test_truth
+    assert_kind_of BlacklistPattern,  @blacklist_pattern
+  end
+end

data/test/unit/blog_test.rb

@@ -0,0 +1,83 @@
+require File.dirname(__FILE__) + '/../test_helper'
+
+class BlogTest < Test::Unit::TestCase
+  fixtures :blogs, :contents
+
+  def setup
+    @blog = Blog.find(:first)
+  end
+
+  def test_blog_name
+    assert_equal "test blog", @blog.blog_name
+  end
+
+  def test_default_allow_pings
+    assert ! @blog.default_allow_pings?
+  end
+
+  def test_setting_booleans_with_integers
+    @blog.sp_global = 1
+    assert  @blog.sp_global
+    @blog.sp_global = 0
+    assert !@blog.sp_global
+  end
+
+  def test_setting_booleans_with_strings
+    {"0 but true" => true, "" => false,
+     "false" => false, 1 => true, 0 => false,
+     nil => false, 'f' => false }.each do |value, expected|
+      @blog.sp_global = value
+      assert_equal expected, @blog.sp_global
+    end
+  end
+
+  def test_find_already_published
+    assert articles = @blog.find_already_published(:articles)
+    assert_kind_of Array, articles
+    assert articles.all? { |a| a.is_a?(Article) }
+    assert_equal 6, articles.size
+
+    assert comments = @blog.find_already_published(:comments)
+    assert_kind_of Array, comments
+    assert comments.all? { |c| c.is_a?(Comment) }
+  end
+
+  def test_current_theme_path
+    assert_equal Theme.themes_root + "/azure", @blog.current_theme_path
+  end
+
+  def test_current_theme
+    assert_equal "azure", @blog.current_theme.name
+  end
+
+  def test_url_for
+    assert_equal('/articles/read/1',
+                 @blog.url_for(:controller => 'articles',
+                               :action     => 'read',
+                               :id         => 1))
+  end
+
+  def test_url_for_article
+    assert_equal('/articles/2004/06/01/article-3',
+                 @blog.url_for(contents(:article3)))
+    assert_equal('/articles/2004/06/01/article-3#foo',
+                 @blog.url_for(contents(:article3), 'foo'))
+  end
+
+  def test_url_for_page
+    assert_equal('/pages/page_one',
+                 @blog.url_for(contents(:first_page)))
+  end
+
+  def test_url_for_comment
+    comment = contents(:comment2)
+    assert_equal(@blog.url_for(comment.article, "comment-#{comment.id}"),
+                 @blog.url_for(comment))
+  end
+
+  def test_url_for_trackback
+    tb = contents(:trackback1)
+    assert_equal(@blog.url_for(tb.article, "trackback-#{tb.id}"),
+                 @blog.url_for(tb))
+  end
+end

data/test/unit/category_test.rb

@@ -0,0 +1,37 @@
+require File.dirname(__FILE__) + '/../test_helper'
+
+class CategoryTest < Test::Unit::TestCase
+  fixtures :contents, :categories, :articles_categories
+
+  def setup
+    @category = Category.find(1)
+  end
+
+  def test_truth
+    assert_kind_of Category,  @category
+  end
+
+  def test_find_all_with_article_counters
+    c = Category.find_all_with_article_counters
+
+    assert_equal categories(:software), c[0]
+    assert_equal categories(:hardware), c[1]
+    assert_equal categories(:personal), c[2]
+
+    assert_equal 1, c[0].article_counter
+    assert_equal 1, c[1].article_counter
+    assert_equal 3, c[2].article_counter
+  end
+
+  def test_reorder
+    assert_equal categories(:software), Category.find(:first, :order => :position)
+    Category.reorder([categories(:personal).id, categories(:hardware).id, categories(:software).id])
+    assert_equal categories(:personal), Category.find(:first, :order => :position)
+  end
+
+  def test_reorder_alpha
+    assert_equal categories(:software), Category.find(:first, :order => :position)
+    Category.reorder_alpha
+    assert_equal categories(:hardware), Category.find(:first, :order => :position)
+  end
+end

data/test/unit/comment_test.rb

@@ -0,0 +1,117 @@
+require File.dirname(__FILE__) + '/../test_helper'
+
+require 'dns_mock'
+
+class CommentTest < Test::Unit::TestCase
+  fixtures :contents, :blacklist_patterns, :text_filters, :blogs
+
+  def test_save_regular
+    assert contents(:comment2).save
+    assert_equal "http://www.google.com", contents(:comment2).url
+  end
+
+  def test_save_spam
+    assert contents(:spam_comment).save
+    assert_equal "http://fakeurl.com", contents(:spam_comment).url
+  end
+
+  def test_create_comment
+    c = Comment.new
+    c.author = 'Bob'
+    c.article_id = 1
+    c.body = 'nice post'
+    c.ip = '1.2.3.4'
+
+    assert c.save
+    assert c.guid.size > 15
+  end
+
+  def test_reject_spam_rbl
+    c = Comment.new
+    c.author = "Spammer"
+    c.body = %{This is just some random text. &lt;a href="http://chinaaircatering.com"&gt;without any senses.&lt;/a&gt;. Please disregard.}
+    c.url = "http://buy-computer.us"
+    c.ip = "212.42.230.206"
+
+    assert ! c.save
+    assert c.errors.invalid?('body')
+    assert c.errors.invalid?('url')
+    assert c.errors.invalid?('ip')
+  end
+
+  def test_not_spam_but_rbl_lookup_succeeds
+    c        = Comment.new
+    c.author = "Not a Spammer"
+    c.body   = "Useful commentary!"
+    c.url    = "http://www.bofh.org.uk"
+    c.ip     = "10.10.10.10"
+
+    assert c.save
+  end
+
+  def test_reject_spam_pattern
+    c = Comment.new
+    c.author = "Another Spammer"
+    c.body = "Texas hold-em poker crap"
+    c.url = "http://texas.hold-em.us"
+
+    assert ! c.save
+    assert c.errors.invalid?('body')
+  end
+
+  def test_reject_spam_uri_limit
+    c = Comment.new
+    c.author = "Yet Another Spammer"
+    c.body = %{ <a href="http://www.one.com/">one</a> <a href="http://www.two.com/">two</a> <a href="http://www.three.com/">three</a> <a href="http://www.four.com/">four</a> }
+    c.url = "http://www.uri-limit.com"
+    c.ip = "123.123.123.123"
+
+    assert ! c.save
+    assert c.errors.invalid?('body')
+  end
+
+  def test_reject_article_age
+    c = Comment.new
+    c.author = "Old Spammer"
+    c.body = "Old trackback body"
+    c.article = contents(:inactive_article)
+
+    assert ! c.save
+    assert c.errors.invalid?('article_id')
+
+    c.article = @article1
+
+    assert c.save
+    assert c.errors.empty?
+  end
+
+  def test_modify_old_comment
+    c = contents(:inactive_article).comments.first
+    c.body = 'Comment body <em>italic</em> <strong>bold</strong>'
+    assert c.save
+    assert c.errors.empty?
+  end
+
+  def test_article_relation
+    assert_equal true, contents(:comment2).has_article?
+    assert_equal 1, contents(:comment2).article.id
+  end
+
+  def test_xss_rejection
+    c = Comment.new
+    c.body = "Test foo <script>do_evil();</script>"
+    c.author = 'Bob'
+    c.article_id = 1
+
+    # Test each filter to make sure that we don't allow scripts through.
+    # Yes, this is ugly.
+    ['','textile','markdown','smartypants','markdown smartypants'].each do |filter|
+      this_blog.comment_text_filter = filter
+
+      assert c.save
+      assert c.errors.empty?
+
+      assert c.body_html !~ /<script>/
+    end
+  end
+end

data/test/unit/configuration_test.rb

@@ -0,0 +1,21 @@
+require File.dirname(__FILE__) + '/../test_helper'
+
+class ConfigurationTest < Test::Unit::TestCase
+  fixtures :blogs
+
+  def test_booleans
+    assert  this_blog.sp_global?
+    assert  this_blog.sp_allow_non_ajax_comments?
+    assert  this_blog.default_allow_comments?
+    assert !this_blog.default_allow_pings?
+
+    assert TrueClass === this_blog.sp_global
+    assert String    === this_blog.blog_name
+    assert Fixnum    === this_blog.limit_rss_display
+  end
+
+  def test_is_ok
+    assert this_blog.is_ok?
+  end
+
+end