radiant-comments-extension 0.0.6

data/spec/models/simple_spam_filter_spec.rb

@@ -0,0 +1,44 @@
+require File.dirname(__FILE__) + '/../spec_helper'
+
+describe SimpleSpamFilter do
+  dataset :comments
+
+  before :each do
+    @comment = comments(:first)
+    Radiant::Config['comments.simple_spam_filter_required?'] = true
+  end
+
+  it "should always approve comments (they passed validation already)" do
+    SimpleSpamFilter.should be_approved(@comment)
+  end
+
+  it "should validate the comment when the challenge is not defined and not required" do
+    Radiant::Config['comments.simple_spam_filter_required?'] = false
+    @comment.valid_spam_answer.should be_nil
+    @comment.spam_answer.should be_nil
+    SimpleSpamFilter.valid?(@comment).should be_true
+  end
+
+  it "should not validate the comment when the response does not match the challenge" do
+    @comment.valid_spam_answer = 'TRUE'
+    @comment.spam_answer = 'FALSE'
+    SimpleSpamFilter.valid?(@comment).should be_false
+    @comment.errors.full_messages.to_sentence.should =~ /Spam answer/
+  end
+
+  it "should validate the comment when the response matches the challenge" do
+    correct_answer = "that's   THE    way it ought to be!".to_slug
+    hashed_answer = Digest::MD5.hexdigest(correct_answer)
+    @comment.valid_spam_answer = hashed_answer
+    @comment.spam_answer = correct_answer
+    SimpleSpamFilter.valid?(@comment).should be_true
+  end
+
+  it "should allow differing capitalization and punctuation in the response" do
+    correct_answer = "that's   THE    way it ought to be!".to_slug
+    hashed_answer = Digest::MD5.hexdigest(correct_answer)
+    @comment.valid_spam_answer = hashed_answer
+    @comment.spam_answer = "That's the way it ought to be!"
+    SimpleSpamFilter.valid?(@comment).should be_true
+  end
+end

data/spec/models/spam_filter_spec.rb

@@ -0,0 +1,38 @@
+require File.dirname(__FILE__) + '/../spec_helper'
+
+describe SpamFilter do
+  dataset :comments
+  
+  it "should be a Simpleton" do
+    SpamFilter.included_modules.should include(Simpleton)
+  end
+  
+  it "should require subclasses to implement the approved? method" do
+    lambda { SpamFilter.approved?(comments(:first)) }.should raise_error(NotImplementedError)
+  end
+  
+  it "should accept a comment as spam and do nothing" do
+    lambda { SpamFilter.spam!(comments(:first)) }.should_not raise_error
+  end
+  
+  it "should not be configured by default" do
+    SpamFilter.should_not be_configured
+  end
+  
+  describe "selecting an appropriate subclass" do
+    it "should select the Simple filter if no other filters are configured" do
+      SpamFilter.descendants.without(SimpleSpamFilter).each do |f|
+        f.stub!(:configured?).and_return(false)
+      end
+      SpamFilter.select.should == SimpleSpamFilter
+    end
+    
+    it "should select the first properly configured filter" do
+      MollomSpamFilter.stub!(:configured?).and_return(true)
+      SpamFilter.descendants.without(MollomSpamFilter).each do |f|
+        f.stub!(:configured?).and_return(false)
+      end
+      SpamFilter.select.should == MollomSpamFilter
+    end
+  end
+end

data/spec/spec.opts

@@ -0,0 +1,6 @@
+--colour
+--format
+progress
+--loadby
+mtime
+--reverse

data/spec/spec_helper.rb

@@ -0,0 +1,36 @@
+unless defined? RADIANT_ROOT
+  ENV["RAILS_ENV"] = "test"
+  case
+  when ENV["RADIANT_ENV_FILE"]
+    require ENV["RADIANT_ENV_FILE"]
+  when File.dirname(__FILE__) =~ %r{vendor/radiant/vendor/extensions}
+    require "#{File.expand_path(File.dirname(__FILE__) + "/../../../../../../")}/config/environment"
+  else
+    require "#{File.expand_path(File.dirname(__FILE__) + "/../../../../")}/config/environment"
+  end
+end
+require "#{RADIANT_ROOT}/spec/spec_helper"
+
+Dataset::Resolver.default << (File.dirname(__FILE__) + "/datasets")
+
+if File.directory?(File.dirname(__FILE__) + "/matchers")
+  Dir[File.dirname(__FILE__) + "/matchers/*.rb"].each {|file| require file }
+end
+
+Spec::Runner.configure do |config|
+  # config.use_transactional_fixtures = true
+  # config.use_instantiated_fixtures  = false
+  # config.fixture_path = RAILS_ROOT + '/spec/fixtures'
+
+  # You can declare fixtures for each behaviour like this:
+  #   describe "...." do
+  #     fixtures :table_a, :table_b
+  #
+  # Alternatively, if you prefer to declare them only once, you can
+  # do so here, like so ...
+  #
+  #   config.global_fixtures = :table_a, :table_b
+  #
+  # If you declare global fixtures, be aware that they will be declared
+  # for all of your examples, even those that don't use them.
+end

data/test/fixtures/users.yml

@@ -0,0 +1,6 @@
+admin:
+  id: 3
+  name: Admin User
+  login: admin
+  password: <%= User.new.sha1 'password' %>
+  admin: true

data/test/integration/comment_enabling_test.rb

@@ -0,0 +1,18 @@
+require File.dirname(__FILE__) + '/../test_helper'
+
+class CommentEnablingTest < ActionController::IntegrationTest
+  fixtures :users
+
+  def test_should_enable_comments
+    post '/admin/welcome/login', :user => {:login => 'admin', :password => 'password'}
+    assert_redirected_to '/admin/welcome'
+
+    page = Page.create!(:title => "FOO", :slug => "foo", :breadcrumb => "FOO", :class_name => "Page")
+    assert !page.enable_comments
+    
+    post "/admin/pages/#{page.id}/comments/enable"
+    assert_redirected_to '/admin/pages'
+    
+    assert page.reload.enable_comments    
+  end
+end

data/test/test_helper.rb

@@ -0,0 +1,24 @@
+require 'test/unit'
+# Load the environment
+unless defined? RADIANT_ROOT
+  ENV["RAILS_ENV"] = "test"
+  case
+  when ENV["RADIANT_ENV_FILE"]
+    require ENV["RADIANT_ENV_FILE"]
+  when File.dirname(__FILE__) =~ %r{vendor/radiant/vendor/extensions}
+    require "#{File.expand_path(File.dirname(__FILE__) + "/../../../../../../")}/config/environment"
+  else
+    require "#{File.expand_path(File.dirname(__FILE__) + "/../../../../")}/config/environment"
+  end
+end
+require "#{RADIANT_ROOT}/test/test_helper"
+
+class Test::Unit::TestCase
+  self.use_transactional_fixtures = true
+  self.use_instantiated_fixtures = false
+  self.fixture_path << File.expand_path(File.dirname(__FILE__)) + '/fixtures'
+end
+
+class ActionController::IntegrationTest < Test::Unit::TestCase
+  self.fixture_path = File.expand_path(File.dirname(__FILE__)) + '/fixtures'
+end

data/test/unit/comment_test.rb

@@ -0,0 +1,52 @@
+require 'test/unit'
+require File.dirname(__FILE__) + '/../test_helper'
+
+class CommentTest < Test::Unit::TestCase
+  def test_valid_comment
+    comment = Comment.new do |c| 
+      c.author = "Foo Bar"
+      c.author_email = "foo@bar.com"
+      c.author_url = "http://www.test.com/"
+      c.content = "This is a comment"
+    end
+    
+    assert_valid comment
+    
+    comment.save
+    
+    comment_stored = Comment.find_by_author("Foo Bar")
+    
+    assert_not_nil(comment)
+    assert_equal(comment.author, comment_stored.author)
+    assert_equal(comment.author_email, comment_stored.author_email)
+    assert_equal(comment.author_url, comment_stored.author_url)
+    assert_equal(comment.content, comment_stored.content)
+  end
+  
+  def test_download_csv_routes
+    assert_routing "admin/comments/all",
+      {:controller => "admin/comments", :action => "index", :status => "all"}
+    assert_routing "admin/comments/all.csv",
+      {:controller => "admin/comments", :action => "index", :status => "all", :format => 'csv'}
+
+    assert_routing "admin/pages/6/comments/all.csv",
+      {:controller => "admin/comments", :action => "index", :status => "all", :format => 'csv', :page_id => "6"}
+    assert_generates "admin/pages/6/comments/all.csv",
+      {:controller => "admin/comments", :action => "index", :format => 'csv', :page_id => "6"}
+  end
+  
+  def test_not_allowing_update_of_protected_attribs
+    @comment = Comment.create(
+      :author       => "Evil Approve",
+      :author_email  => "foo@bar.com",
+      :author_url  => "http://www.test.com/",
+      :content     => "Comment approved?",
+      :approved_at => Time.now,
+      :approved_by => 1
+      );
+    @comment = Comment.find_by_author('Evil Approve')
+    assert_nil(@comment.approved_at)
+    assert_nil(@comment.approved_by)
+  end
+  
+end

metadata

@@ -0,0 +1,177 @@
+--- !ruby/object:Gem::Specification 
+name: radiant-comments-extension
+version: !ruby/object:Gem::Version 
+  prerelease: false
+  segments: 
+  - 0
+  - 0
+  - 6
+  version: 0.0.6
+platform: ruby
+authors: 
+- Jim Gay
+- Ryan Heneise
+- Sean Cribbs
+- John Muhl
+- Sven Schwyn
+- Gerrit Kaiser
+- Stephen Lombardo
+- Benny Degezelle
+- Frank Louwers
+- Michael Hale
+- Nathaniel Talbott
+- John Croisant
+- Jon Leighton
+- Witter Cheng
+- Keith Bingman
+autorequire: 
+bindir: bin
+cert_chain: []
+
+date: 2010-04-01 00:00:00 -04:00
+default_executable: 
+dependencies: 
+- !ruby/object:Gem::Dependency 
+  name: radiant
+  prerelease: false
+  requirement: &id001 !ruby/object:Gem::Requirement 
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        segments: 
+        - 0
+        version: "0"
+  type: :development
+  version_requirements: *id001
+description: Adds blog-like comment functionality to Radiant.
+email: jim@saturnflyer.com
+executables: []
+
+extensions: []
+
+extra_rdoc_files: 
+- README.rdoc
+- TODO
+files: 
+- .gitignore
+- CHANGELOG
+- HELP_admin.markdown
+- HELP_designer.markdown
+- MIT-LICENSE
+- README.rdoc
+- Rakefile
+- TODO
+- VERSION
+- app/controllers/admin/comments_controller.rb
+- app/controllers/comments_controller.rb
+- app/helpers/admin/comments_helper.rb
+- app/models/akismet_spam_filter.rb
+- app/models/comment.rb
+- app/models/comment_mailer.rb
+- app/models/mollom_spam_filter.rb
+- app/models/simple_spam_filter.rb
+- app/models/spam_filter.rb
+- app/views/admin/comments/_comment.rhtml
+- app/views/admin/comments/_form.rhtml
+- app/views/admin/comments/edit.rhtml
+- app/views/admin/comments/index.rhtml
+- app/views/admin/pages/_comments.rhtml
+- app/views/admin/pages/_edit_comments_enabled.rhtml
+- app/views/admin/pages/_index_head_view_comments.rhtml
+- app/views/admin/pages/_index_view_comments.rhtml
+- app/views/comment_mailer/comment_notification.rhtml
+- app/views/comments/_comment.rhtml
+- app/views/comments/_form.rhtml
+- app/views/comments/_new.rhtml
+- autotest/discover.rb
+- comments_extension.rb
+- cucumber.yml
+- db/migrate/001_create_comments.rb
+- db/migrate/002_create_snippets.rb
+- db/migrate/003_change_filter_id_from_integer_to_string.rb
+- db/migrate/004_add_approval_columns.rb
+- db/migrate/005_add_mollomid_column.rb
+- db/migrate/006_move_config_to_migrations.rb
+- db/migrate/007_add_preference_for_simple_spamcheck.rb
+- features/support/env.rb
+- features/support/paths.rb
+- lib/akismet.rb
+- lib/comment_page_extensions.rb
+- lib/comment_tags.rb
+- lib/mollom.rb
+- lib/radiant-comments-extension.rb
+- lib/tasks/comments_extension_tasks.rake
+- public/images/admin/accept.png
+- public/images/admin/comment_edit.png
+- public/images/admin/comments.png
+- public/images/admin/comments_delete.png
+- public/images/admin/delete.png
+- public/images/admin/email.png
+- public/images/admin/error.png
+- public/images/admin/link.png
+- public/images/admin/page_white_edit.png
+- public/images/admin/table_save.png
+- public/images/admin/tick.png
+- public/stylesheets/admin/comments.css
+- radiant-comments-extension.gemspec
+- spec/controllers/admin/comments_controller_spec.rb
+- spec/controllers/admin/comments_routing_spec.rb
+- spec/controllers/page_postback_spec.rb
+- spec/datasets/comments_dataset.rb
+- spec/models/akismet_spam_filter_spec.rb
+- spec/models/comment_spec.rb
+- spec/models/comment_tags_spec.rb
+- spec/models/mollom_spam_filter_spec.rb
+- spec/models/simple_spam_filter_spec.rb
+- spec/models/spam_filter_spec.rb
+- spec/spec.opts
+- spec/spec_helper.rb
+- test/fixtures/users.yml
+- test/integration/comment_enabling_test.rb
+- test/test_helper.rb
+- test/unit/comment_test.rb
+has_rdoc: true
+homepage: http://github.com/saturnflyer/radiant-comments-extension
+licenses: []
+
+post_install_message: 
+rdoc_options: 
+- --charset=UTF-8
+require_paths: 
+- lib
+required_ruby_version: !ruby/object:Gem::Requirement 
+  requirements: 
+  - - ">="
+    - !ruby/object:Gem::Version 
+      segments: 
+      - 0
+      version: "0"
+required_rubygems_version: !ruby/object:Gem::Requirement 
+  requirements: 
+  - - ">="
+    - !ruby/object:Gem::Version 
+      segments: 
+      - 0
+      version: "0"
+requirements: []
+
+rubyforge_project: 
+rubygems_version: 1.3.6
+signing_key: 
+specification_version: 3
+summary: Comments Extension for Radiant CMS
+test_files: 
+- spec/controllers/admin/comments_controller_spec.rb
+- spec/controllers/admin/comments_routing_spec.rb
+- spec/controllers/page_postback_spec.rb
+- spec/datasets/comments_dataset.rb
+- spec/models/akismet_spam_filter_spec.rb
+- spec/models/comment_spec.rb
+- spec/models/comment_tags_spec.rb
+- spec/models/mollom_spam_filter_spec.rb
+- spec/models/simple_spam_filter_spec.rb
+- spec/models/spam_filter_spec.rb
+- spec/spec_helper.rb
+- test/integration/comment_enabling_test.rb
+- test/test_helper.rb
+- test/unit/comment_test.rb