RubyGems - commontator - Versions diffs - 0.5.14 → 1.0.0 - Mend

commontator 0.5.14 → 1.0.0

Files changed (83) hide show

data/README.md +47 -10
data/Rakefile +6 -5
data/app/controllers/commontator/application_controller.rb +3 -1
data/app/controllers/commontator/application_controller.rb~ +5 -2
data/app/controllers/commontator/comments_controller.rb +9 -10
data/app/controllers/commontator/comments_controller.rb~ +9 -10
data/app/controllers/commontator/commontator_controller.rb~ +22 -0
data/app/controllers/commontator/subscriptions_controller.rb +2 -2
data/app/controllers/commontator/subscriptions_controller.rb~ +3 -3
data/app/controllers/commontator/threads_controller.rb +4 -3
data/app/controllers/commontator/threads_controller.rb~ +5 -3
data/app/helpers/commontator/commontator_helper.rb~ +7 -0
data/app/mailers/commontator/subscriptions_mailer.rb +2 -1
data/app/mailers/commontator/subscriptions_mailer.rb~ +3 -1
data/app/models/commontator/comment.rb +13 -9
data/app/models/commontator/comment.rb~ +14 -10
data/app/models/commontator/subscription.rb +3 -3
data/app/models/commontator/subscription.rb~ +19 -0
data/app/models/commontator/thread.rb +17 -41
data/app/models/commontator/thread.rb~ +119 -0
data/app/views/commontator/shared/_thread.html.erb +4 -1
data/app/views/commontator/shared/_thread.html.erb~ +5 -2
data/app/views/commontator/subscriptions_mailer/comment_created_email.html.erb +1 -1
data/app/views/commontator/subscriptions_mailer/comment_created_email.html.erb~ +6 -0
data/config/initializers/commontator.rb +42 -64
data/{test/dummy/config/initializers/commontator.rb → config/initializers/commontator.rb~} +42 -64
data/db/migrate/0_install_commontator.rb +1 -1
data/{test/dummy/db/migrate/1_install_commontator.commontator.rb → db/migrate/0_install_commontator.rb~} +1 -2
data/lib/commontator.rb +7 -12
data/lib/commontator.rb~ +7 -12
data/lib/commontator/commontable_config.rb +1 -1
data/lib/commontator/commontable_config.rb~ +13 -0
data/lib/commontator/commontator_config.rb +1 -1
data/lib/commontator/commontator_config.rb~ +13 -0
data/lib/commontator/controller_includes.rb +1 -0
data/lib/commontator/controller_includes.rb~ +1 -1
data/lib/commontator/security_transgression.rb +3 -0
data/{test/dummy/public/favicon.ico → lib/commontator/security_transgression.rb~} +0 -0
data/lib/commontator/shared_helper.rb +1 -1
data/lib/commontator/shared_helper.rb~ +2 -2
data/lib/commontator/version.rb +1 -1
data/lib/commontator/version.rb~ +1 -1
metadata +46 -89
data/test/commontator_test.rb +0 -7
data/test/dummy/README.rdoc +0 -261
data/test/dummy/Rakefile +0 -7
data/test/dummy/app/assets/javascripts/application.js +0 -15
data/test/dummy/app/assets/stylesheets/application.css +0 -13
data/test/dummy/app/controllers/application_controller.rb +0 -3
data/test/dummy/app/helpers/application_helper.rb +0 -2
data/test/dummy/app/views/layouts/application.html.erb +0 -14
data/test/dummy/config.ru +0 -4
data/test/dummy/config/application.rb +0 -59
data/test/dummy/config/boot.rb +0 -10
data/test/dummy/config/database.yml +0 -25
data/test/dummy/config/environment.rb +0 -5
data/test/dummy/config/environments/development.rb +0 -37
data/test/dummy/config/environments/production.rb +0 -67
data/test/dummy/config/environments/test.rb +0 -37
data/test/dummy/config/initializers/backtrace_silencers.rb +0 -7
data/test/dummy/config/initializers/inflections.rb +0 -15
data/test/dummy/config/initializers/mime_types.rb +0 -5
data/test/dummy/config/initializers/secret_token.rb +0 -7
data/test/dummy/config/initializers/session_store.rb +0 -8
data/test/dummy/config/initializers/wrap_parameters.rb +0 -14
data/test/dummy/config/locales/en.yml +0 -5
data/test/dummy/config/routes.rb +0 -4
data/test/dummy/db/development.sqlite3 +0 -0
data/test/dummy/db/schema.rb +0 -61
data/test/dummy/log/development.log +0 -414
data/test/dummy/public/404.html +0 -26
data/test/dummy/public/422.html +0 -26
data/test/dummy/public/500.html +0 -25
data/test/dummy/script/rails +0 -6
data/test/functional/comments_controller_test.rb +0 -181
data/test/functional/subscriptions_controller_test.rb +0 -122
data/test/integration/navigation_test.rb +0 -10
data/test/test_helper.rb +0 -15
data/test/unit/comment_test.rb +0 -35
data/test/unit/helpers/comments_helper_test.rb +0 -4
data/test/unit/helpers/subscriptions_helper_test.rb +0 -4
data/test/unit/subscription_test.rb +0 -15
data/test/unit/thread_test.rb +0 -57

data/test/dummy/public/404.html DELETED Viewed

@@ -1,26 +0,0 @@
-<!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/test/dummy/public/422.html DELETED Viewed

@@ -1,26 +0,0 @@
-<!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/test/dummy/public/500.html DELETED Viewed

@@ -1,25 +0,0 @@
-<!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>
-  </div>
-</body>
-</html>

data/test/dummy/script/rails DELETED Viewed

@@ -1,6 +0,0 @@
-#!/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/test/functional/comments_controller_test.rb DELETED Viewed

@@ -1,181 +0,0 @@
-require 'test_helper'
-class CommentsControllerTest < ActionController::TestCase
-  setup do
-    @comment = FactoryGirl.create(:comment)
-    @question = @comment.comment_thread.commentable
-    @user = @comment.creator
-    FactoryGirl.create(:project_question, :question => @question, :project => Project.default_for_user!(@user))
-    @published_question = make_simple_question(:method => :create, :published => true)
-    @published_comment = FactoryGirl.create(:comment,
-                                        :comment_thread => @published_question.comment_thread)
-  end
-  test "should not get index not logged in" do
-    get :index, :question_id => @question.to_param
-    assert_redirected_to login_path
-  end
-  test "should not get index not authorized" do
-    user_login
-    get :index, :question_id => @question.to_param
-    assert_response(403)
-  end
-  test "should get index" do
-    sign_in @user
-    get :index, :question_id => @question.to_param
-    assert_response :success
-  end
-  test "should get index published question" do
-    user_login
-    get :index, :question_id => @published_question.to_param
-    assert_response :success
-  end
-  test "should not get new not logged in" do
-    get :new, :question_id => @question.to_param
-    assert_redirected_to login_path
-  end
-  test "should not get new not authorized" do
-    user_login
-    get :new, :question_id => @question.to_param
-    assert_response(403)
-  end
-  test "should get new" do
-    sign_in @user
-    get :new, :question_id => @question.to_param
-    assert_response :success
-    assert_not_nil assigns(:comments)
-  end
-  test "should get new published question" do
-    user_login
-    get :new, :question_id => @published_question.to_param
-    assert_response :success
-    assert_not_nil assigns(:comments)
-  end
-  test "should not create comment not logged in" do
-    assert_difference('Comment.count', 0) do
-      post :create, :question_id => @question.to_param, :comment => @comment.attributes
-    end
-    assert_redirected_to login_path
-  end
-  test "should not create comment not authorized" do
-    user_login
-    assert_difference('Comment.count', 0) do
-      post :create, :question_id => @question.to_param, :comment => @comment.attributes
-    end
-    assert_response(403)
-  end
-  test "should create comment" do
-    sign_in @user
-    assert_difference('Comment.count') do
-      post :create, :question_id => @question.to_param, :comment => @comment.attributes
-    end
-    assert_redirected_to question_comments_path(@question.to_param)
-  end
-  test "should create comment published question" do
-    user_login
-    assert_difference('Comment.count') do
-      post :create, :question_id => @published_question.to_param, :comment => @comment.attributes
-    end
-    assert_redirected_to question_comments_path(@published_question.to_param)
-  end
-  test "should not show comment not logged in" do
-    get :show, :id => @comment.to_param
-    assert_redirected_to login_path
-  end
-  test "should not show comment not authorized" do
-    user_login
-    get :show, :id => @comment.to_param
-    assert_response(403)
-  end
-  test "should show comment" do
-    sign_in @user
-    get :show, :id => @comment.to_param
-    assert_response :success
-  end
-  test "should show comment published question" do
-    user_login
-    get :show, :id => @published_comment.to_param
-    assert_response :success
-  end
-  test "should not get edit not logged in" do
-    get :edit, :id => @comment.to_param
-    assert_redirected_to login_path
-  end
-  test "should not get edit not authorized" do
-    user_login
-    get :edit, :id => @comment.to_param
-    assert_response(403)
-  end
-  test "should get edit" do
-    sign_in @user
-    get :edit, :id => @comment.to_param
-    assert_response :success
-  end
-  test "should not update comment not logged in" do
-    put :update, :id => @comment.to_param
-    assert_redirected_to login_path
-  end
-  test "should not update comment not authorized" do
-    user_login
-    put :update, :id => @comment.to_param
-    assert_response(403)
-  end
-  test "should update comment" do
-    sign_in @user
-    put :update, :id => @comment.to_param
-    assert_redirected_to question_comments_path(@question.to_param)
-  end
-  test "should not destroy comment not logged in" do
-    assert_difference('Comment.count', 0) do
-      delete :destroy, :id => @comment.to_param
-    end
-    assert_redirected_to login_path
-  end
-  test "should not destroy comment not authorized" do
-    user_login
-    assert_difference('Comment.count', 0) do
-      delete :destroy, :id => @comment.to_param
-    end
-    assert_response(403)
-  end
-  test "should destroy comment" do
-    sign_in @user
-    assert_difference('Comment.count', -1) do
-      delete :destroy, :id => @comment.to_param
-    end
-    assert_redirected_to question_comments_path(@question.to_param)
-  end
-end

data/test/functional/subscriptions_controller_test.rb DELETED Viewed

@@ -1,122 +0,0 @@
-require 'test_helper'
-class SubscriptionsControllerTest < ActionController::TestCase
-  setup do
-    @question = FactoryGirl.create(:simple_question)
-    @user = FactoryGirl.create(:user)
-    FactoryGirl.create(:project_question, :question => @question, :project => Project.default_for_user!(@user))
-    @published_question = make_simple_question(:method => :create, :published => true)
-    @user2 = FactoryGirl.create(:user)
-  end
-  test "should not get index not logged in" do
-    get :index
-    assert_redirected_to login_path
-  end
-  test "should get index" do
-    user_login
-    get :index
-    assert_response :success
-  end
-  test "should not create subscription not logged in" do
-    assert_difference('Subscription.count', 0) do
-      get :create, :question_id => @question.to_param
-    end
-    assert_redirected_to login_path
-  end
-  test "should not create subscription not authorized" do
-    user_login
-    assert_difference('Subscription.count', 0) do
-      get :create, :question_id => @question.to_param
-    end
-    assert_response(403)
-  end
-  test "should not create subscription already subscribed" do
-    sign_in @user
-    FactoryGirl.create(:subscription,
-                   :user => @user, :thread => @question.thread)
-    assert_difference('Subscription.count', 0) do
-      get :create, :question_id => @question.to_param
-    end
-    assert_redirected_to question_comments_path(@question.to_param)
-  end
-  test "should create subscription" do
-    sign_in @user
-    assert_difference('Subscription.count') do
-      get :create, :question_id => @question.to_param
-    end
-    assert_redirected_to question_comments_path(@question.to_param)
-  end
-  test "should create subscription published question" do
-    user_login
-    assert_difference('Subscription.count') do
-      get :create, :question_id => @published_question.to_param
-    end
-    assert_redirected_to question_comments_path(@published_question.to_param)
-  end
-  test "should not destroy comment_thread_subscription not logged in" do
-    FactoryGirl.create(:comment_thread_subscription,
-                   :user => @user, :comment_thread => @question.comment_thread)
-    assert_difference('CommentThreadSubscription.count', 0) do
-      get :destroy, :question_id => @question.to_param
-    end
-    assert_redirected_to login_path
-  end
-  test "should not destroy comment_thread_subscription not authorized" do
-    user_login
-    FactoryGirl.create(:comment_thread_subscription,
-                   :user => @user, :comment_thread => @question.comment_thread)
-    assert_difference('CommentThreadSubscription.count', 0) do
-      get :destroy, :question_id => @question.to_param
-    end
-    assert_response(403)
-  end
-  test "should not destroy comment_thread_subscription no subscription" do
-    sign_in @user
-    assert_difference('CommentThreadSubscription.count', 0) do
-      get :destroy, :question_id => @question.to_param
-    end
-    assert_redirected_to question_comments_path(@question.to_param)
-  end
-  test "should destroy comment_thread_subscription" do
-    sign_in @user
-    FactoryGirl.create(:comment_thread_subscription,
-                   :user => @user, :comment_thread => @question.comment_thread)
-    assert_difference('CommentThreadSubscription.count', -1) do
-      get :destroy, :question_id => @question.to_param
-    end
-    assert_redirected_to question_comments_path(@question.to_param)
-  end
-  test "should destroy comment_thread_subscription published question" do
-    sign_in @user2
-    FactoryGirl.create(:comment_thread_subscription,
-                   :user => @user2, :comment_thread => @published_question.comment_thread)
-    assert_difference('CommentThreadSubscription.count', -1) do
-      get :destroy, :question_id => @published_question.to_param
-    end
-    assert_redirected_to question_comments_path(@published_question.to_param)
-  end
-end

data/test/integration/navigation_test.rb DELETED Viewed

@@ -1,10 +0,0 @@
-require 'test_helper'
-class NavigationTest < ActionDispatch::IntegrationTest
-  fixtures :all
-  # test "the truth" do
-  #   assert true
-  # end
-end

data/test/test_helper.rb DELETED Viewed

@@ -1,15 +0,0 @@
-# Configure Rails Environment
-ENV["RAILS_ENV"] = "test"
-require File.expand_path("../dummy/config/environment.rb",  __FILE__)
-require "rails/test_help"
-Rails.backtrace_cleaner.remove_silencers!
-# Load support files
-Dir["#{File.dirname(__FILE__)}/support/**/*.rb"].each { |f| require f }
-# Load fixtures from the engine
-if ActiveSupport::TestCase.method_defined?(:fixture_path=)
-  ActiveSupport::TestCase.fixture_path = File.expand_path("../fixtures", __FILE__)
-end

data/test/unit/comment_test.rb DELETED Viewed

@@ -1,35 +0,0 @@
-require 'test_helper'
-class CommentTest < ActiveSupport::TestCase
-  test "cannot mass-assign comment_thread, creator" do
-    ct = FactoryGirl.create(:comment_thread)
-    u = FactoryGirl.create(:user)
-    c = Comment.new(:comment_thread => ct, :creator => u)
-    assert c.comment_thread != ct
-    assert c.creator != u
-  end
-  test "is_modified" do
-    c = FactoryGirl.create(:comment)
-    assert !c.is_modified?
-    c.message = 'Another message'
-    c.save!
-    assert c.is_modified?
-  end
-  test "must have comment_thread and creator" do
-    ct = FactoryGirl.create(:comment_thread)
-    u = FactoryGirl.create(:user)
-    c = Comment.new(:message => "Some message")
-    assert !c.save
-    c.comment_thread = ct
-    assert !c.save
-    c.comment_thread = nil
-    c.creator = u
-    assert !c.save
-    c.comment_thread = ct
-    c.save!
-  end
-end