RubyGems - rails_blog_engine - Versions diffs - 0.0.1 → 0.0.2 - Mend

rails_blog_engine 0.0.1 → 0.0.2

Files changed (79) hide show

data/.gitignore +12 -0
data/.rspec +1 -0
data/Gemfile +24 -0
data/Guardfile +21 -0
data/Procfile +1 -0
data/TODO.txt +31 -0
data/app/assets/images/rails_blog_engine/.gitkeep +0 -0
data/app/controllers/rails_blog_engine/comments_controller.rb +1 -1
data/app/controllers/rails_blog_engine/posts_controller.rb +1 -1
data/lib/rails_blog_engine.rb +2 -0
data/lib/rails_blog_engine/version.rb +1 -1
data/rails_blog_engine.gemspec +64 -0
data/script/rails +6 -0
data/spec/acceptance/acceptance_helper.rb +4 -0
data/spec/acceptance/rails_blog_engine/posts_admin_spec.rb +33 -0
data/spec/acceptance/rails_blog_engine/posts_spec.rb +74 -0
data/spec/acceptance/rails_blog_engine/spam_filtering_spec.rb +110 -0
data/spec/acceptance/support/helpers.rb +18 -0
data/spec/acceptance/support/paths.rb +5 -0
data/spec/controllers/rails_blog_engine/comments_controller_spec.rb +22 -0
data/spec/controllers/rails_blog_engine/posts_controller_spec.rb +81 -0
data/spec/dummy/Rakefile +7 -0
data/spec/dummy/app/assets/javascripts/application.js +10 -0
data/spec/dummy/app/assets/stylesheets/application.css +8 -0
data/spec/dummy/app/controllers/application_controller.rb +3 -0
data/spec/dummy/app/controllers/home_controller.rb +4 -0
data/spec/dummy/app/helpers/application_helper.rb +2 -0
data/spec/dummy/app/mailers/.gitkeep +0 -0
data/spec/dummy/app/models/.gitkeep +0 -0
data/spec/dummy/app/models/ability.rb +11 -0
data/spec/dummy/app/models/user.rb +9 -0
data/spec/dummy/app/views/home/index.html.haml +2 -0
data/spec/dummy/app/views/layouts/application.html.erb +15 -0
data/spec/dummy/config.ru +4 -0
data/spec/dummy/config/application.rb +54 -0
data/spec/dummy/config/boot.rb +10 -0
data/spec/dummy/config/database.yml +25 -0
data/spec/dummy/config/environment.rb +5 -0
data/spec/dummy/config/environments/development.rb +30 -0
data/spec/dummy/config/environments/production.rb +60 -0
data/spec/dummy/config/environments/test.rb +42 -0
data/spec/dummy/config/initializers/backtrace_silencers.rb +7 -0
data/spec/dummy/config/initializers/devise.rb +211 -0
data/spec/dummy/config/initializers/inflections.rb +10 -0
data/spec/dummy/config/initializers/mime_types.rb +5 -0
data/spec/dummy/config/initializers/rails_blog_engine.rb +13 -0
data/spec/dummy/config/initializers/secret_token.rb +7 -0
data/spec/dummy/config/initializers/session_store.rb +8 -0
data/spec/dummy/config/initializers/wrap_parameters.rb +14 -0
data/spec/dummy/config/locales/devise.en.yml +58 -0
data/spec/dummy/config/locales/en.yml +5 -0
data/spec/dummy/config/routes.rb +6 -0
data/spec/dummy/db/migrate/20110913113004_devise_create_users.rb +28 -0
data/spec/dummy/db/schema.rb +67 -0
data/spec/dummy/lib/assets/.gitkeep +0 -0
data/spec/dummy/log/.gitkeep +0 -0
data/spec/dummy/public/404.html +26 -0
data/spec/dummy/public/422.html +26 -0
data/spec/dummy/public/500.html +26 -0
data/spec/dummy/public/favicon.ico +0 -0
data/spec/dummy/script/rails +6 -0
data/spec/helpers/rails_blog_engine/application_helper_spec.rb +56 -0
data/spec/helpers/rails_blog_engine/comments_helper_spec.rb +14 -0
data/spec/helpers/rails_blog_engine/posts_helper_spec.rb +14 -0
data/spec/lib/generators/rails_blog_engine/install_generator_spec.rb +78 -0
data/spec/lib/rails_blog_engine/filters/base_spec.rb +13 -0
data/spec/lib/rails_blog_engine/filters/code_spec.rb +16 -0
data/spec/lib/rails_blog_engine/filters_spec.rb +58 -0
data/spec/models/rails_blog_engine/comment_spec.rb +84 -0
data/spec/models/rails_blog_engine/post_spec.rb +121 -0
data/spec/spec_helper.rb +36 -0
data/spec/support/blueprints.rb +25 -0
data/spec/support/javascript.rb +27 -0
data/spec/support/vcr.rb +8 -0
data/spec/vcr_cassettes/rakismet-ham.yml +30 -0
data/spec/vcr_cassettes/rakismet-spam.yml +30 -0
data/spec/vcr_cassettes/rakismet-train-as-ham.yml +32 -0
data/spec/vcr_cassettes/rakismet-train-as-spam.yml +32 -0
metadata +175 -101

data/spec/dummy/lib/assets/.gitkeep ADDED Viewed

File without changes

data/spec/dummy/log/.gitkeep ADDED Viewed

File without changes

data/spec/dummy/public/404.html ADDED Viewed

@@ -0,0 +1,26 @@
+<!DOCTYPE html>
+<html>
+<head>
+  <title>The page you were looking for doesn't exist (404)</title>
+  <style type="text/css">
+    body { background-color: #fff; color: #666; text-align: center; font-family: arial, sans-serif; }
+    div.dialog {
+      width: 25em;
+      padding: 0 4em;
+      margin: 4em auto 0 auto;
+      border: 1px solid #ccc;
+      border-right-color: #999;
+      border-bottom-color: #999;
+    }
+    h1 { font-size: 100%; color: #f00; line-height: 1.5em; }
+  </style>
+</head>
+<body>
+  <!-- This file lives in public/404.html -->
+  <div class="dialog">
+    <h1>The page you were looking for doesn't exist.</h1>
+    <p>You may have mistyped the address or the page may have moved.</p>
+  </div>
+</body>
+</html>

data/spec/dummy/public/422.html ADDED Viewed

@@ -0,0 +1,26 @@
+<!DOCTYPE html>
+<html>
+<head>
+  <title>The change you wanted was rejected (422)</title>
+  <style type="text/css">
+    body { background-color: #fff; color: #666; text-align: center; font-family: arial, sans-serif; }
+    div.dialog {
+      width: 25em;
+      padding: 0 4em;
+      margin: 4em auto 0 auto;
+      border: 1px solid #ccc;
+      border-right-color: #999;
+      border-bottom-color: #999;
+    }
+    h1 { font-size: 100%; color: #f00; line-height: 1.5em; }
+  </style>
+</head>
+<body>
+  <!-- This file lives in public/422.html -->
+  <div class="dialog">
+    <h1>The change you wanted was rejected.</h1>
+    <p>Maybe you tried to change something you didn't have access to.</p>
+  </div>
+</body>
+</html>

data/spec/dummy/public/500.html ADDED Viewed

@@ -0,0 +1,26 @@
+<!DOCTYPE html>
+<html>
+<head>
+  <title>We're sorry, but something went wrong (500)</title>
+  <style type="text/css">
+    body { background-color: #fff; color: #666; text-align: center; font-family: arial, sans-serif; }
+    div.dialog {
+      width: 25em;
+      padding: 0 4em;
+      margin: 4em auto 0 auto;
+      border: 1px solid #ccc;
+      border-right-color: #999;
+      border-bottom-color: #999;
+    }
+    h1 { font-size: 100%; color: #f00; line-height: 1.5em; }
+  </style>
+</head>
+<body>
+  <!-- This file lives in public/500.html -->
+  <div class="dialog">
+    <h1>We're sorry, but something went wrong.</h1>
+    <p>We've been notified about this issue and we'll take a look at it shortly.</p>
+  </div>
+</body>
+</html>

data/spec/dummy/public/favicon.ico ADDED Viewed

File without changes

data/spec/dummy/script/rails ADDED Viewed

@@ -0,0 +1,6 @@
+#!/usr/bin/env ruby
+# This command will automatically be run when you run "rails" with Rails 3 gems installed from the root of your application.
+APP_PATH = File.expand_path('../../config/application',  __FILE__)
+require File.expand_path('../../config/boot',  __FILE__)
+require 'rails/commands'

data/spec/helpers/rails_blog_engine/application_helper_spec.rb ADDED Viewed

@@ -0,0 +1,56 @@
+require 'spec_helper'
+describe RailsBlogEngine::ApplicationHelper do
+  class SillyFilter < RailsBlogEngine::Filters::Base
+    register_filter :silly
+    def process(text, options)
+      "Whee"
+    end
+  end
+  describe ".markdown" do
+    it "converts markdown text to HTML" do
+      helper.markdown("_foo_ bar").should match(/<em>foo<\/em> bar/)
+    end
+    it "does not pass scripts" do
+      helper.markdown("<script>foo</script>").
+        should_not match(/script/)
+    end
+    it "applies filters" do
+      helper.markdown("<filter:silly/>").should match(/Whee/)
+    end
+    context "for trusted users" do
+      it "allows images" do
+        helper.markdown("<img src='foo.png'>", :trusted? => true).
+          should match(/<img/)
+      end
+      it "does not add nofollow to links" do
+        helper.markdown("<a href='foo.html'>", :trusted? => true).
+          should_not match(/nofollow/)
+      end
+      it "allows code highlighting" do
+        formatted = helper.markdown(<<EOD, :trusted? => true)
+<div class='foo'><span class='bar'></span></div>
+EOD
+        formatted.should match(/foo/)
+        formatted.should match(/bar/)
+      end
+    end
+    context "for untrusted users" do
+      it "does not allow images" do
+        helper.markdown("<img src='foo.png'>").should_not match(/<img/)
+      end
+      it "adds nofollow to links" do
+        helper.markdown("<a href='foo.html'>").should match(/nofollow/)
+      end
+    end
+  end
+end

data/spec/helpers/rails_blog_engine/comments_helper_spec.rb ADDED Viewed

@@ -0,0 +1,14 @@
+require 'spec_helper'
+# Specs in this file have access to a helper object that includes
+# the CommentsHelper. For example:
+#
+# describe CommentsHelper do
+#   describe "string concat" do
+#     it "concats two strings with spaces" do
+#       helper.concat_strings("this","that").should == "this that"
+#     end
+#   end
+# end
+describe RailsBlogEngine::CommentsHelper do
+end

data/spec/helpers/rails_blog_engine/posts_helper_spec.rb ADDED Viewed

@@ -0,0 +1,14 @@
+require 'spec_helper'
+# Specs in this file have access to a helper object that includes
+# the PostsHelper. For example:
+#
+# describe PostsHelper do
+#   describe "string concat" do
+#     it "concats two strings with spaces" do
+#       helper.concat_strings("this","that").should == "this that"
+#     end
+#   end
+# end
+describe RailsBlogEngine::PostsHelper do
+end

data/spec/lib/generators/rails_blog_engine/install_generator_spec.rb ADDED Viewed

@@ -0,0 +1,78 @@
+require "spec_helper.rb"
+require "generator_spec/test_case"
+require "generators/rails_blog_engine/install/install_generator"
+describe RailsBlogEngine::InstallGenerator do
+  include GeneratorSpec::TestCase
+  destination File.expand_path("../../../../tmp", __FILE__)
+  # Create a file with the specified contents in our fake application tree.
+  # This is faster than generating a real application every time we test.
+  def create_file(path, contents)
+    full_path = ::File.join(destination_root, path)
+    mkdir_p(::File.dirname(full_path))
+    ::File.open(full_path, 'w') {|f| f.write(contents) }
+  end
+  def prepare_destination
+    super
+    create_file('config/routes.rb', <<EOD)
+Dummy::Application.routes.draw do
+end
+EOD
+    create_file('app/assets/stylesheets/application.css', <<EOD)
+/*
+ *= require_self
+ *= require_tree .
+*/
+EOD
+    create_file('app/assets/javascripts/application.js', <<EOD)
+//= require jquery
+//= require jquery_ujs
+//= require_tree .
+EOD
+  end
+  before do
+    prepare_destination
+    run_generator
+  end
+  it "generates the expected files" do
+    destination_root.should have_structure {
+      directory "app" do
+        directory "assets" do
+          directory "javascripts" do
+            file "application.js" do
+              contains "//= require rails_blog_engine"
+            end
+          end
+          directory "stylesheets" do
+            file "application.css" do
+              contains " *= require rails_blog_engine"
+            end
+          end
+        end
+      end
+      directory "config" do
+        file "routes.rb" do
+          contains 'mount RailsBlogEngine::Engine => "/blog"'
+        end
+        directory "initializers" do
+          file "rails_blog_engine.rb"
+        end
+        directory "locales" do
+          file "rails_blog_engine.en.yml"
+        end
+      end
+      directory "db" do
+        directory "migrate" do
+          # We don't need to test every single one of these--just make sure
+          # a couple are copied over, and the rest should be OK.
+          file "20110912153527_create_rails_blog_engine_posts.rb"
+          file "20110913190319_add_fields_to_rails_blog_engine_post.rb"
+        end
+      end
+    }
+  end
+end

data/spec/lib/rails_blog_engine/filters/base_spec.rb ADDED Viewed

@@ -0,0 +1,13 @@
+require 'spec_helper'
+describe RailsBlogEngine::Filters::Base do
+  let(:filter) { RailsBlogEngine::Filters::Base.new }
+  describe "#process" do
+    it "raises an error if not overridden" do
+      lambda do
+        filter.process("text", {})
+      end.should raise_error(/override/)
+    end
+  end
+end

data/spec/lib/rails_blog_engine/filters/code_spec.rb ADDED Viewed

@@ -0,0 +1,16 @@
+require 'spec_helper'
+describe RailsBlogEngine::Filters::Code do
+  let(:filter) { RailsBlogEngine::Filters::Code.new }
+  describe ".process" do
+    it "applies syntax highlighting to code blocks" do
+      output = filter.process(<<END_OF_CODE, :lang => 'ruby')
+def foo
+  42
+end
+END_OF_CODE
+      output.should match(/foo/)
+    end
+  end
+end

data/spec/lib/rails_blog_engine/filters_spec.rb ADDED Viewed

@@ -0,0 +1,58 @@
+require 'spec_helper'
+describe RailsBlogEngine::Filters do
+  # A sample filter.
+  class HelloFilter < RailsBlogEngine::Filters::Base
+    register_filter :hello
+    def process(text, options)
+      attrs =
+        if options[:class] then " class=\"#{options[:class]}\"" else "" end
+      name = if text then ", #{text}" else "" end
+      "<p#{attrs}>Hello#{name}!</p>"
+    end
+  end
+  describe ".find" do
+    it "returns the filter registered for a name" do
+      RailsBlogEngine::Filters.find(:hello).should be_kind_of(HelloFilter)
+    end
+  end
+  describe ".apply_all_to" do
+    %w(filter macro typo).each do |tag|
+      it "applies registered filters to empty '#{tag}' tags" do
+        RailsBlogEngine::Filters.apply_all_to(<<"END_OF_INPUT").should == <<END_OF_OUTPUT
+<#{tag}:hello/>
+<#{tag}:hello class="example" />
+END_OF_INPUT
+<p>Hello!</p>
+<p class="example">Hello!</p>
+END_OF_OUTPUT
+      end
+      it "applies registered filters to '#{tag}' tags with content" do
+        RailsBlogEngine::Filters.apply_all_to(<<"END_OF_INPUT").should == <<END_OF_OUTPUT
+<#{tag}:hello>Judy</#{tag}:hello>
+<#{tag}:hello class='example' extra="" >Mike
+Smith</#{tag}:hello>
+END_OF_INPUT
+<p>Hello, Judy!</p>
+<p class="example">Hello, Mike
+Smith!</p>
+END_OF_OUTPUT
+      end
+    end
+    it "reports errors inline" do
+        RailsBlogEngine::Filters.apply_all_to(<<END_OF_INPUT).should == <<END_OF_OUTPUT
+<filter:invalid/>
+<filter:hello class= >Mike</filter:hello>
+END_OF_INPUT
+<p><strong>Text filter not installed: invalid</strong></p>
+<p><strong>Can't parse filter arguments: {{ class= }}</strong></p>
+END_OF_OUTPUT
+    end
+  end
+end

data/spec/models/rails_blog_engine/comment_spec.rb ADDED Viewed

@@ -0,0 +1,84 @@
+require 'spec_helper'
+describe RailsBlogEngine::Comment do
+  it { should belong_to(:post) }
+  it { should_not allow_value('').for(:author_byline) }
+  it { should_not allow_value('').for(:body) }
+  describe ".visible" do
+    it "includes unfiltered and ham messages, but not spam" do
+      @unfiltered = RailsBlogEngine::Comment.make!
+      @ham = RailsBlogEngine::Comment.make!(:state => 'filtered_as_ham')
+      @ham2 = RailsBlogEngine::Comment.make!(:state => 'marked_as_ham')
+      @spam = RailsBlogEngine::Comment.make!(:state => 'filtered_as_spam')
+      @spam2 = RailsBlogEngine::Comment.make!(:state => 'marked_as_spam')
+      RailsBlogEngine::Comment.visible.should include(@unfiltered)
+      RailsBlogEngine::Comment.visible.should include(@ham)
+      RailsBlogEngine::Comment.visible.should include(@ham2)
+      RailsBlogEngine::Comment.visible.should_not include(@spam)
+      RailsBlogEngine::Comment.visible.should_not include(@spam2)
+    end
+  end
+  describe "#state" do
+    subject { RailsBlogEngine::Comment.make! }
+    def enable_spam_filter
+      Rakismet.key = "fakekey"
+    end
+    def disable_spam_filter
+      Rakismet.key = nil
+    end
+    it "begins in state unfiltered" do
+      subject.should be_unfiltered
+    end
+    it "transitions to filtered_as_ham if rakismet likes it" do
+      enable_spam_filter
+      subject.stub(:spam?) { false }
+      subject.run_spam_filter
+      subject.should be_filtered_as_ham
+    end
+    it "transitions to filtered_as_spam if rakismet doesn't like it" do
+      enable_spam_filter
+      subject.stub(:spam?) { true }
+      subject.run_spam_filter
+      subject.should be_filtered_as_spam
+    end
+    it "remains unfiltered if the rakismet is not configured" do
+      disable_spam_filter
+      subject.run_spam_filter
+      subject.should be_unfiltered
+    end
+    it "supports manually marking a filtered post as ham" do
+      enable_spam_filter
+      subject.filter_as_spam
+      subject.can_mark_as_ham?.should == true
+      subject.should_receive(:ham!)
+      subject.mark_as_ham
+      subject.should be_marked_as_ham
+    end
+    it "supports manually marking a filtered post as spam" do
+      enable_spam_filter
+      subject.filter_as_ham
+      subject.can_mark_as_spam?.should == true
+      subject.should_receive(:spam!)
+      subject.mark_as_spam
+      subject.should be_marked_as_spam
+    end
+    it "supports manually marking an unfiltered post as spam" do
+      disable_spam_filter
+      subject.can_mark_as_spam?.should == true
+      subject.mark_as_spam
+      subject.should be_marked_as_spam
+    end
+  end
+end

data/spec/models/rails_blog_engine/post_spec.rb ADDED Viewed

@@ -0,0 +1,121 @@
+require 'spec_helper'
+describe RailsBlogEngine::Post do
+  Post = RailsBlogEngine::Post
+  it { should have_many(:comments) }
+  describe "validations" do
+    before { Post.make! }
+    it { should allow_value("Title").for(:title) }
+    it { should_not allow_value("").for(:title) }
+    it { should allow_value("Body").for(:body) }
+    it { should_not allow_value("").for(:body) }
+    %w(unpublished published).each do |state|
+      it { should allow_value(state).for(:state) }
+    end
+    it { should_not allow_value("invalid").for(:state) }
+    it { should validate_presence_of(:state) }
+    it { should allow_value("abc-def").for(:permalink) }
+    it { should validate_presence_of(:permalink) }
+    it { should validate_uniqueness_of(:permalink) }
+    it { should validate_presence_of(:author) }
+  end
+  describe ".recently_published" do
+    before do
+      base = Time.now
+      @posts = (0...3).map do |i|
+        Post.make!(:published, :published_at => base + i.seconds)
+      end
+      @posts[1].published_at = base + 10.seconds
+      @posts[1].save!
+      @unpublished = Post.make!
+    end
+    it "does not include unpublished posts" do
+      Post.recently_published.should_not include(@unpublished)
+    end
+    it "returns in order of descending published_at" do
+      Post.recently_published.should == [@posts[1], @posts[2], @posts[0]]
+    end
+    it "excludes posts which have been explicitly unpublished" do
+      @posts[1].unpublish!
+      Post.recently_published.should == [@posts[2], @posts[0]]
+    end
+  end
+  describe "state machine" do
+    let(:post) { Post.make }
+    it "begins as unpublished" do
+      post.should be_unpublished
+    end
+    it "can be published and unpublished" do
+      post.publish!
+      post.should be_published
+      lambda do
+        post.publish!
+      end.should raise_error(StateMachine::InvalidTransition)
+      post.unpublish!
+      post.should be_unpublished
+      lambda do
+        post.unpublish!
+      end.should raise_error(StateMachine::InvalidTransition)
+    end
+  end
+  describe "published_at" do
+    it "is set automatically when post is first published" do
+      first_publication = Time.utc(1980, 1, 1)
+      second_publication = Time.utc(1980, 1, 2)
+      post = Post.make
+      post.published_at.should be_nil
+      Time.stub(:now) { first_publication }
+      post.publish
+      post.published_at.should == first_publication
+      Time.stub(:now) { second_publication }
+      post.publish
+      post.published_at.should == first_publication
+    end
+  end
+  describe "#author_byline" do
+    it "is set from the #author method before save" do
+      author = User.make(:email => 'jane@example.com')
+      Post.make!(:author => author).author_byline.
+        should == 'jane'
+    end
+  end
+  describe ".author_byline" do
+    it "returns .byline if present" do
+      author = mock('author', :byline => 'Jane Smith')
+      Post.author_byline(author).should == 'Jane Smith'
+    end
+    it "returns .email without domain if present" do
+      author = mock('author', :email => 'jane@example.com')
+      Post.author_byline(author).should == 'jane'
+    end
+    it "returns 'unknown' otherwise" do
+      author = mock('author')
+      Post.author_byline(author).should == 'unknown'
+    end
+  end
+end