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

commontator 0.5.14 → 1.0.0

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

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