the_comments 1.1.0 → 2.0.0

data/spec/dummy_app/db/schema.rb

@@ -0,0 +1,74 @@
+# encoding: UTF-8
+# This file is auto-generated from the current state of the database. Instead
+# of editing this file, please use the migrations feature of Active Record to
+# incrementally modify your database, and then regenerate this schema definition.
+#
+# Note that this schema.rb definition is the authoritative source for your
+# database schema. If you need to create the application database on another
+# system, you should be using db:schema:load, not running all the migrations
+# from scratch. The latter is a flawed and unsustainable approach (the more migrations
+# you'll amass, the slower it'll run and the greater likelihood for issues).
+#
+# It's strongly recommended that you check this file into your version control system.
+
+ActiveRecord::Schema.define(version: 20131027185334) do
+
+  create_table "comments", force: true do |t|
+    t.integer  "user_id"
+    t.integer  "holder_id"
+    t.integer  "commentable_id"
+    t.string   "commentable_type"
+    t.string   "commentable_url"
+    t.string   "commentable_title"
+    t.string   "commentable_state"
+    t.string   "anchor"
+    t.string   "title"
+    t.string   "contacts"
+    t.text     "raw_content"
+    t.text     "content"
+    t.string   "view_token"
+    t.string   "state",             default: "draft"
+    t.string   "ip",                default: "undefined"
+    t.string   "referer",           default: "undefined"
+    t.string   "user_agent",        default: "undefined"
+    t.integer  "tolerance_time"
+    t.boolean  "spam",              default: false
+    t.integer  "parent_id"
+    t.integer  "lft"
+    t.integer  "rgt"
+    t.integer  "depth",             default: 0
+    t.datetime "created_at"
+    t.datetime "updated_at"
+  end
+
+  create_table "posts", force: true do |t|
+    t.integer  "user_id"
+    t.string   "title"
+    t.text     "content"
+    t.datetime "created_at"
+    t.datetime "updated_at"
+    t.integer  "draft_comments_count",     default: 0
+    t.integer  "published_comments_count", default: 0
+    t.integer  "deleted_comments_count",   default: 0
+  end
+
+  create_table "users", force: true do |t|
+    t.string   "username",                                null: false
+    t.string   "email"
+    t.string   "crypted_password"
+    t.string   "salt"
+    t.datetime "created_at"
+    t.datetime "updated_at"
+    t.integer  "my_draft_comments_count",     default: 0
+    t.integer  "my_published_comments_count", default: 0
+    t.integer  "my_comments_count",           default: 0
+    t.integer  "draft_comcoms_count",         default: 0
+    t.integer  "published_comcoms_count",     default: 0
+    t.integer  "deleted_comcoms_count",       default: 0
+    t.integer  "spam_comcoms_count",          default: 0
+    t.integer  "draft_comments_count",        default: 0
+    t.integer  "published_comments_count",    default: 0
+    t.integer  "deleted_comments_count",      default: 0
+  end
+
+end

data/spec/dummy_app/db/seeds.rb

@@ -0,0 +1,42 @@
+puts 'Users'
+
+3.times do
+  User.create!(username: Faker::Name.name)
+  print '.'
+end
+
+puts
+puts 'Posts'
+
+User.all.each do |user|
+  3.times do
+    user.posts.create!(
+      title:   Faker::Lorem.sentence,
+      content: Faker::Lorem.paragraphs(3).join
+    )
+    print '.'
+  end
+end
+
+puts
+puts 'Comments'
+
+users = User.all
+posts = Post.all
+
+20.times do
+  user = users.sample
+  post = posts.sample
+
+  Comment.create!(
+    user: user,
+    commentable: post,
+    title: Faker::Lorem.sentence,
+    raw_content: Faker::Lorem.paragraphs(3).join,
+    state: :published
+  )
+
+  print '.'
+end
+
+puts

data/spec/dummy_app/lib/tasks/app_bootstrap.rake

@@ -0,0 +1,15 @@
+namespace :db do
+  desc "DB: drop, create, migrate"
+  task bootstrap: :environment do
+    Rake::Task["db:drop"].invoke
+    Rake::Task["db:create"].invoke
+    Rake::Task["db:migrate"].invoke
+  end
+
+  # rake db:bootstrap_and_seed
+  desc "Reset DB and seed"
+  task bootstrap_and_seed: :environment do
+    Rake::Task["db:bootstrap"].invoke
+    Rake::Task["db:seed"].invoke
+  end
+end

data/spec/dummy_app/public/404.html

@@ -0,0 +1,58 @@
+<!DOCTYPE html>
+<html>
+<head>
+  <title>The page you were looking for doesn't exist (404)</title>
+  <style>
+  body {
+    background-color: #EFEFEF;
+    color: #2E2F30;
+    text-align: center;
+    font-family: arial, sans-serif;
+  }
+
+  div.dialog {
+    width: 25em;
+    margin: 4em auto 0 auto;
+    border: 1px solid #CCC;
+    border-right-color: #999;
+    border-left-color: #999;
+    border-bottom-color: #BBB;
+    border-top: #B00100 solid 4px;
+    border-top-left-radius: 9px;
+    border-top-right-radius: 9px;
+    background-color: white;
+    padding: 7px 4em 0 4em;
+  }
+
+  h1 {
+    font-size: 100%;
+    color: #730E15;
+    line-height: 1.5em;
+  }
+
+  body > p {
+    width: 33em;
+    margin: 0 auto 1em;
+    padding: 1em 0;
+    background-color: #F7F7F7;
+    border: 1px solid #CCC;
+    border-right-color: #999;
+    border-bottom-color: #999;
+    border-bottom-left-radius: 4px;
+    border-bottom-right-radius: 4px;
+    border-top-color: #DADADA;
+    color: #666;
+    box-shadow:0 3px 8px rgba(50, 50, 50, 0.17);
+  }
+  </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>
+  <p>If you are the application owner check the logs for more information.</p>
+</body>
+</html>

data/spec/dummy_app/public/422.html

@@ -0,0 +1,58 @@
+<!DOCTYPE html>
+<html>
+<head>
+  <title>The change you wanted was rejected (422)</title>
+  <style>
+  body {
+    background-color: #EFEFEF;
+    color: #2E2F30;
+    text-align: center;
+    font-family: arial, sans-serif;
+  }
+
+  div.dialog {
+    width: 25em;
+    margin: 4em auto 0 auto;
+    border: 1px solid #CCC;
+    border-right-color: #999;
+    border-left-color: #999;
+    border-bottom-color: #BBB;
+    border-top: #B00100 solid 4px;
+    border-top-left-radius: 9px;
+    border-top-right-radius: 9px;
+    background-color: white;
+    padding: 7px 4em 0 4em;
+  }
+
+  h1 {
+    font-size: 100%;
+    color: #730E15;
+    line-height: 1.5em;
+  }
+
+  body > p {
+    width: 33em;
+    margin: 0 auto 1em;
+    padding: 1em 0;
+    background-color: #F7F7F7;
+    border: 1px solid #CCC;
+    border-right-color: #999;
+    border-bottom-color: #999;
+    border-bottom-left-radius: 4px;
+    border-bottom-right-radius: 4px;
+    border-top-color: #DADADA;
+    color: #666;
+    box-shadow:0 3px 8px rgba(50, 50, 50, 0.17);
+  }
+  </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>
+  <p>If you are the application owner check the logs for more information.</p>
+</body>
+</html>

data/spec/dummy_app/public/500.html

@@ -0,0 +1,57 @@
+<!DOCTYPE html>
+<html>
+<head>
+  <title>We're sorry, but something went wrong (500)</title>
+  <style>
+  body {
+    background-color: #EFEFEF;
+    color: #2E2F30;
+    text-align: center;
+    font-family: arial, sans-serif;
+  }
+
+  div.dialog {
+    width: 25em;
+    margin: 4em auto 0 auto;
+    border: 1px solid #CCC;
+    border-right-color: #999;
+    border-left-color: #999;
+    border-bottom-color: #BBB;
+    border-top: #B00100 solid 4px;
+    border-top-left-radius: 9px;
+    border-top-right-radius: 9px;
+    background-color: white;
+    padding: 7px 4em 0 4em;
+  }
+
+  h1 {
+    font-size: 100%;
+    color: #730E15;
+    line-height: 1.5em;
+  }
+
+  body > p {
+    width: 33em;
+    margin: 0 auto 1em;
+    padding: 1em 0;
+    background-color: #F7F7F7;
+    border: 1px solid #CCC;
+    border-right-color: #999;
+    border-bottom-color: #999;
+    border-bottom-left-radius: 4px;
+    border-bottom-right-radius: 4px;
+    border-top-color: #DADADA;
+    color: #666;
+    box-shadow:0 3px 8px rgba(50, 50, 50, 0.17);
+  }
+  </style>
+</head>
+
+<body>
+  <!-- This file lives in public/500.html -->
+  <div class="dialog">
+    <h1>We're sorry, but something went wrong.</h1>
+  </div>
+  <p>If you are the application owner check the logs for more information.</p>
+</body>
+</html>

data/spec/dummy_app/public/robots.txt

@@ -0,0 +1,5 @@
+# See http://www.robotstxt.org/wc/norobots.html for documentation on how to use the robots.txt file
+#
+# To ban all spiders from the entire site uncomment the next two lines:
+# User-agent: *
+# Disallow: /

data/spec/dummy_app/spec/factories/post.rb

@@ -0,0 +1,6 @@
+FactoryGirl.define do
+  factory :post, class: Post do
+    sequence(:title)   { Faker::Lorem.sentence }
+    sequence(:content) { Faker::Lorem.sentence }
+  end
+end

data/spec/dummy_app/spec/factories/user.rb

@@ -0,0 +1,6 @@
+FactoryGirl.define do
+  factory :user, class: User do
+    sequence(:username) { Faker::Name.name }
+    sequence(:email)    { Faker::Internet.email }
+  end
+end

data/spec/dummy_app/spec/models/user_counters_spec.rb

@@ -0,0 +1,339 @@
+require 'spec_helper'
+
+def destroy_all
+  User.destroy_all
+  3.times{ begin; Comment.destroy_all; rescue; end; }
+end
+
+# --------------------------------------
+# Helpers
+# --------------------------------------
+def my_comments_count_assert user, count
+  user.my_comments.active.count.should eq count
+  user.my_comments_count.should eq count
+end
+
+def comments_count_assert(obj, values)
+  obj.comments.with_state(:draft).count.should     eq values[0]
+  obj.comments.with_state(:published).count.should eq values[1]
+  obj.comments.with_state(:deleted).count.should   eq values[2]
+end
+
+def comments_counters_assert(obj, values)
+  obj.draft_comments_count.should     eq values[0]
+  obj.published_comments_count.should eq values[1]
+  obj.deleted_comments_count.should   eq values[2]
+end
+
+def comcoms_count_assert(obj, values)
+  obj.comcoms.with_state(:draft).count.should     eq values[0]
+  obj.comcoms.with_state(:published).count.should eq values[1]
+  obj.comcoms.with_state(:deleted).count.should   eq values[2]
+end
+
+def comcoms_counters_assert(obj, values)
+  obj.draft_comcoms_count.should     eq values[0]
+  obj.published_comcoms_count.should eq values[1]
+  obj.deleted_comcoms_count.should   eq values[2]
+end
+
+# --------------------------------------
+# init functions
+# --------------------------------------
+def create_users_and_post
+  @user        = FactoryGirl.create(:user)
+  @post_holder = FactoryGirl.create(:user)
+  @post        = FactoryGirl.create(:post, user: @post_holder)
+end
+
+def base_test_situation
+  create_users_and_post
+
+  3.times do
+    @comment = Comment.create!(
+      user: @user,
+      commentable: @post,
+      title: Faker::Lorem.sentence,
+      raw_content: Faker::Lorem.paragraphs(3).join
+    )
+  end
+
+  @user.reload
+  @post_holder.reload
+end
+
+def tree_test_situation
+  create_users_and_post
+
+  # LEVEL 1
+  3.times do
+    Comment.create!(
+      user: @user,
+      commentable: @post,
+      title: Faker::Lorem.sentence,
+      raw_content: Faker::Lorem.paragraphs(3).join,
+      state: :published
+    )
+  end
+    # LEVEL 2
+    parent_comment = Comment.first
+    3.times do
+      Comment.create!(
+        user: @user,
+        commentable: @post,
+        parent_id: parent_comment.id,
+        title: Faker::Lorem.sentence,
+        raw_content: Faker::Lorem.paragraphs(3).join,
+        state: :published
+      )
+    end
+      # LEVEL 3
+      parent_comment = Comment.first.children.first
+      3.times do
+        Comment.create!(
+          user: @user,
+          commentable: @post,
+          parent_id: parent_comment.id,
+          title: Faker::Lorem.sentence,
+          raw_content: Faker::Lorem.paragraphs(3).join,
+          state: :published
+        )
+      end
+
+  @user.reload
+  @post_holder.reload
+end
+# --------------------------------------
+# ~ init functions
+# --------------------------------------
+
+describe User do
+  context 'User leave comment to the post' do
+    after(:all){ destroy_all }
+
+    before(:all){ create_users_and_post }
+
+    it 'should be User' do
+      @user.should        be_instance_of User
+      @post_holder.should be_instance_of User
+    end
+
+    it 'Post should has owner' do
+      @post.user.should eq @post_holder
+    end
+
+    it 'should be Post' do
+      @post.should be_instance_of Post
+    end
+
+    it 'Create new comment' do
+      @comment = Comment.create!(
+        user: @user,
+        commentable: @post,
+        title: Faker::Lorem.sentence,
+        raw_content: Faker::Lorem.paragraphs(3).join
+      )
+
+      @comment.user.should   eq @user
+      @comment.holder.should eq @post_holder
+    end
+  end
+
+  context "Written by me counters" do
+    after(:all) { destroy_all }
+    before(:all){ create_users_and_post }
+    it 'should has correct My counters values' do 
+      @comment = Comment.create!(
+        user: @user,
+        commentable: @post,
+        title: Faker::Lorem.sentence,
+        raw_content: Faker::Lorem.paragraphs(3).join
+      )
+      @user.reload
+      @user.my_comments_count.should           eq 1
+      @user.my_draft_comments_count.should     eq 1
+      @user.my_published_comments_count.should eq 0
+    end
+  end
+
+  context "Commentable Denormalization" do
+    after(:all) { destroy_all }
+    before(:all) do
+      base_test_situation
+    end
+
+    it 'should have denormalized fields' do
+      title = "New Title!"
+      @post.update_attribute(:title, title)
+      @post.title.should eq title
+      @comment = @post.comments.first
+      @comment.commentable_title.should eq title
+    end
+  end
+
+  context 'User leave 3 comments and Instances has expectable counter values' do
+    after(:all) { destroy_all }
+
+    before(:all){ base_test_situation }
+
+    describe 'User counters' do
+      it 'my_comments counter' do
+        my_comments_count_assert(@user, 3)
+      end
+
+      it 'Comcoms counters' do
+        @user.comcoms.count.should eq 0
+
+        comcoms_counters_assert(@user, [0,0,0])
+      end
+    end
+
+    describe 'Holder counters' do
+      it 'my_comments counter' do
+        my_comments_count_assert(@post_holder, 0)
+      end
+
+      it 'Comcoms counters' do
+        @post_holder.comcoms.count.should eq 3
+        comcoms_counters_assert(@post_holder, [3,0,0])
+      end
+    end
+  end
+
+  context 'User leave 3 comments, 1 Comment DRAFT => PUBLISHED' do
+    after(:all) { destroy_all }
+
+    before(:all) do
+      base_test_situation
+      @comment = Comment.first
+      @comment.to_published
+
+      @user.reload
+      @post.reload
+      @comment.reload
+      @post_holder.reload
+    end
+
+    describe 'User counters' do
+      it 'has expectable values' do
+        comments_counters_assert @user, [0,0,0]
+        comments_count_assert    @user, [0,0,0]
+        comcoms_counters_assert  @user, [0,0,0]
+        comcoms_count_assert     @user, [0,0,0]
+        my_comments_count_assert @user, 3
+      end
+    end
+
+    describe 'Holder counters' do
+      it 'has expectable values' do
+        @post_holder.comcoms.count.should eq 3
+
+        my_comments_count_assert(@post_holder, 0)
+        comcoms_count_assert    @post_holder, [2,1,0]
+        comcoms_counters_assert @post_holder, [2,1,0]
+      end
+    end
+  end
+
+  context '3 comments, 1 Comment DRAFT => DELETE' do
+    after(:all) { destroy_all }
+
+    before(:all) do
+      base_test_situation
+      @comment = Comment.first
+      @comment.to_published
+      @comment.to_deleted
+
+      @user.reload
+      @post.reload
+      @comment.reload
+      @post_holder.reload
+    end
+
+    it 'has correct counters values' do
+      comments_counters_assert @user, [0,0,0]
+      comments_count_assert    @user, [0,0,0]
+      comcoms_count_assert     @user, [0,0,0]
+      my_comments_count_assert @user, 2
+      @user.my_comments.count.should eq 3
+
+      comments_counters_assert @post_holder, [0,0,0]
+      comments_count_assert    @post_holder, [0,0,0]
+      comcoms_count_assert     @post_holder, [2,0,1]
+      my_comments_count_assert @post_holder, 0
+      @post_holder.my_comments.count.should eq 0
+
+      comments_count_assert    @post, [2,0,1]
+      comments_counters_assert @post, [2,0,1]
+    end
+  end
+
+  context 'Comments tree, 1 branch to deleted' do
+    after(:each) { destroy_all }
+    before(:each){ tree_test_situation }
+
+    it 'has correct counters values before deleting' do
+      Comment.count.should eq 9
+
+      my_comments_count_assert @user, 9
+      comments_count_assert    @user, [0,0,0]
+
+      my_comments_count_assert @post_holder, 0
+      comments_count_assert    @post_holder, [0,0,0]
+      
+      comcoms_count_assert    @post_holder, [0,9,0]
+      comcoms_counters_assert @post_holder, [0,9,0]
+
+      comments_count_assert    @post, [0,9,0]
+      comments_counters_assert @post, [0,9,0]
+    end
+
+    it 'has correct counters values after branch deleting' do
+      @comment = Comment.first.children.first
+      @comment.to_deleted
+
+      @user.reload
+      @post.reload
+      @post_holder.reload
+
+      Comment.with_state(:published).count.should eq 5
+      Comment.with_state(:deleted).count.should   eq 4
+
+      my_comments_count_assert @user, 5
+      comments_count_assert    @user, [0,0,0]
+
+      my_comments_count_assert @post_holder, 0
+      comments_count_assert    @post_holder, [0,0,0]
+      
+      comcoms_count_assert    @post_holder, [0,5,4]
+      comcoms_counters_assert @post_holder, [0,5,4]
+
+      comments_count_assert    @post, [0,5,4]
+      comments_counters_assert @post, [0,5,4]
+    end
+
+    # This code helps to me to catch logic bugs
+    # there is no any asserts
+    # context "callbacks catch" do
+    #   after(:all) { destroy_all }
+    #   it "before validation callback with increment! must be call manually" do
+    #     @user        = FactoryGirl.create(:user)
+    #     @post_holder = FactoryGirl.create(:user)
+    #     @post        = FactoryGirl.create(:post, user: @post_holder)
+
+    #     Comment.create!(
+    #       user: @user,
+    #       commentable: @post,
+    #       title: Faker::Lorem.sentence,
+    #       raw_content: Faker::Lorem.paragraphs(3).join,
+    #       state: :deleted
+    #     )
+    #     Comment.first.to_published
+    #   end
+    # end
+
+    # it 'has correct counters after comment destroy' do 
+    #   pending("has correct counters after comment destroy")
+    # end
+  end
+end