RubyGems - commontator - Versions diffs - 0.1.46 - Mend

commontator 0.1.46

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 (156) hide show

data/MIT-LICENSE +20 -0
data/README.md +35 -0
data/Rakefile +40 -0
data/app/assets/images/commontator/downvote.png +0 -0
data/app/assets/images/commontator/downvote_hover.png +0 -0
data/app/assets/images/commontator/upvote.png +0 -0
data/app/assets/images/commontator/upvote_hover.png +0 -0
data/app/assets/stylesheets/commontator/application.css +13 -0
data/app/assets/stylesheets/commontator/application.css~ +88 -0
data/app/assets/stylesheets/commontator/comments.css +86 -0
data/app/assets/stylesheets/commontator/comments.css~ +90 -0
data/app/assets/stylesheets/commontator/threads.css +13 -0
data/app/assets/stylesheets/commontator/threads.css~ +13 -0
data/app/controllers/commontator/application_controller.rb +18 -0
data/app/controllers/commontator/application_controller.rb~ +16 -0
data/app/controllers/commontator/comments_controller.rb +143 -0
data/app/controllers/commontator/comments_controller.rb~ +143 -0
data/app/controllers/commontator/subscriptions_controller.rb +42 -0
data/app/controllers/commontator/subscriptions_controller.rb~ +40 -0
data/app/controllers/commontator/threads_controller.rb +42 -0
data/app/controllers/commontator/threads_controller.rb~ +43 -0
data/app/helpers/commontator/application_helper.rb +11 -0
data/app/helpers/commontator/application_helper.rb~ +11 -0
data/app/helpers/commontator/comments_helper.rb +47 -0
data/app/helpers/commontator/comments_helper.rb~ +58 -0
data/app/helpers/commontator/commontable_helper.rb~ +9 -0
data/app/helpers/commontator/commontator_helper.rb +8 -0
data/app/helpers/commontator/commontator_helper.rb~ +8 -0
data/app/helpers/commontator/subscriptions_helper.rb +8 -0
data/app/helpers/commontator/subscriptions_helper.rb~ +8 -0
data/app/helpers/commontator/threads_helper.rb +21 -0
data/app/helpers/commontator/threads_helper.rb~ +32 -0
data/app/mailers/commontator/subscription_mailer.rb~ +38 -0
data/app/mailers/commontator/subscriptions_mailer.rb +51 -0
data/app/mailers/commontator/subscriptions_mailer.rb~ +51 -0
data/app/models/commontator/comment.rb +84 -0
data/app/models/commontator/comment.rb~ +84 -0
data/app/models/commontator/subscription.rb +21 -0
data/app/models/commontator/subscription.rb~ +21 -0
data/app/models/commontator/thread.rb +138 -0
data/app/models/commontator/thread.rb~ +141 -0
data/app/views/commontator/comments/_actions.html.erb +32 -0
data/app/views/commontator/comments/_actions.html.erb~ +33 -0
data/app/views/commontator/comments/_body.html.erb +7 -0
data/app/views/commontator/comments/_body.html.erb~ +8 -0
data/app/views/commontator/comments/_form.html.erb +20 -0
data/app/views/commontator/comments/_form.html.erb~ +21 -0
data/app/views/commontator/comments/_show.html.erb +41 -0
data/app/views/commontator/comments/_show.html.erb~ +45 -0
data/app/views/commontator/comments/_votes.html.erb +60 -0
data/app/views/commontator/comments/_votes.html.erb~ +62 -0
data/app/views/commontator/comments/create.js.erb +15 -0
data/app/views/commontator/comments/create.js.erb~ +15 -0
data/app/views/commontator/comments/delete.js.erb +15 -0
data/app/views/commontator/comments/delete.js.erb~ +15 -0
data/app/views/commontator/comments/destroy.js.erb~ +1 -0
data/app/views/commontator/comments/edit.html.erb +6 -0
data/app/views/commontator/comments/edit.html.erb~ +6 -0
data/app/views/commontator/comments/edit.js.erb +5 -0
data/app/views/commontator/comments/edit.js.erb~ +5 -0
data/app/views/commontator/comments/new.html.erb +8 -0
data/app/views/commontator/comments/new.html.erb~ +8 -0
data/app/views/commontator/comments/new.js.erb +12 -0
data/app/views/commontator/comments/new.js.erb~ +12 -0
data/app/views/commontator/comments/update.js.erb +7 -0
data/app/views/commontator/comments/update.js.erb~ +7 -0
data/app/views/commontator/comments/vote.js.erb +5 -0
data/app/views/commontator/comments/vote.js.erb~ +5 -0
data/app/views/commontator/commontator/_thread.html.erb +14 -0
data/app/views/commontator/commontator/_thread.html.erb~ +15 -0
data/app/views/commontator/subscriptions/_subscription_link.html.erb +12 -0
data/app/views/commontator/subscriptions/_subscription_link.html.erb~ +12 -0
data/app/views/commontator/subscriptions/create.js.erb +5 -0
data/app/views/commontator/subscriptions/create.js.erb~ +3 -0
data/app/views/commontator/subscriptions/destroy.js.erb +5 -0
data/app/views/commontator/subscriptions/destroy.js.erb~ +3 -0
data/app/views/commontator/subscriptions/index.html.erb +16 -0
data/app/views/commontator/subscriptions/index.html.erb~ +16 -0
data/app/views/commontator/subscriptions_mailer/comment_created_email.html.erb +14 -0
data/app/views/commontator/subscriptions_mailer/comment_created_email.html.erb~ +14 -0
data/app/views/commontator/threads/_show.html.erb +26 -0
data/app/views/commontator/threads/_show.html.erb~ +26 -0
data/app/views/commontator/threads/show.html.erb +3 -0
data/app/views/commontator/threads/show.js.erb +6 -0
data/app/views/layouts/commontator/application.html.erb +14 -0
data/config/initializers/commontator.rb +7 -0
data/config/routes.rb +24 -0
data/config/routes.rb~ +22 -0
data/db/migrate/0_install_commontator.rb +40 -0
data/db/migrate/0_install_commontator.rb~ +40 -0
data/lib/commontator.rb +66 -0
data/lib/commontator.rb~ +66 -0
data/lib/commontator/acts_as_commontable.rb +39 -0
data/lib/commontator/acts_as_commontable.rb~ +42 -0
data/lib/commontator/acts_as_commontator.rb +28 -0
data/lib/commontator/acts_as_commontator.rb~ +37 -0
data/lib/commontator/commontable_config.rb +13 -0
data/lib/commontator/commontator_config.rb +13 -0
data/lib/commontator/engine.rb +5 -0
data/lib/commontator/routes.rb~ +37 -0
data/lib/commontator/version.rb +3 -0
data/lib/commontator/version.rb~ +3 -0
data/test/commontator_test.rb +7 -0
data/test/dummy/README.rdoc +261 -0
data/test/dummy/Rakefile +7 -0
data/test/dummy/app/assets/javascripts/application.js +15 -0
data/test/dummy/app/assets/stylesheets/application.css +13 -0
data/test/dummy/app/controllers/application_controller.rb +3 -0
data/test/dummy/app/helpers/application_helper.rb +2 -0
data/test/dummy/app/views/layouts/application.html.erb +14 -0
data/test/dummy/config.ru +4 -0
data/test/dummy/config/application.rb +59 -0
data/test/dummy/config/boot.rb +10 -0
data/test/dummy/config/database.yml +25 -0
data/test/dummy/config/environment.rb +5 -0
data/test/dummy/config/environments/development.rb +37 -0
data/test/dummy/config/environments/production.rb +67 -0
data/test/dummy/config/environments/test.rb +37 -0
data/test/dummy/config/initializers/backtrace_silencers.rb +7 -0
data/test/dummy/config/initializers/inflections.rb +15 -0
data/test/dummy/config/initializers/mime_types.rb +5 -0
data/test/dummy/config/initializers/secret_token.rb +7 -0
data/test/dummy/config/initializers/session_store.rb +8 -0
data/test/dummy/config/initializers/wrap_parameters.rb +14 -0
data/test/dummy/config/locales/en.yml +5 -0
data/test/dummy/config/routes.rb +4 -0
data/test/dummy/db/development.sqlite3 +0 -0
data/test/dummy/db/schema.rb +16 -0
data/test/dummy/db/test.sqlite3 +0 -0
data/test/dummy/log/development.log +204 -0
data/test/dummy/public/404.html +26 -0
data/test/dummy/public/422.html +26 -0
data/test/dummy/public/500.html +25 -0
data/test/dummy/public/favicon.ico +0 -0
data/test/dummy/script/rails +6 -0
data/test/functional/comment_thread_subscriptions_controller_test.rb~ +125 -0
data/test/functional/comments_controller_test.rb +181 -0
data/test/functional/comments_controller_test.rb~ +184 -0
data/test/functional/subscriptions_controller_test.rb +122 -0
data/test/functional/subscriptions_controller_test.rb~ +122 -0
data/test/integration/navigation_test.rb +10 -0
data/test/test_helper.rb +15 -0
data/test/unit/comment_test.rb +35 -0
data/test/unit/comment_test.rb~ +38 -0
data/test/unit/comment_thread_subscription_test.rb~ +18 -0
data/test/unit/comment_thread_test.rb~ +60 -0
data/test/unit/helpers/comment_thread_subscriptions_helper_test.rb~ +7 -0
data/test/unit/helpers/comments_helper_test.rb +4 -0
data/test/unit/helpers/comments_helper_test.rb~ +7 -0
data/test/unit/helpers/subscriptions_helper_test.rb +4 -0
data/test/unit/helpers/subscriptions_helper_test.rb~ +4 -0
data/test/unit/subscription_test.rb +15 -0
data/test/unit/subscription_test.rb~ +15 -0
data/test/unit/thread_test.rb +57 -0
data/test/unit/thread_test.rb~ +57 -0
metadata +301 -0

data/test/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/test/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/test/dummy/public/500.html ADDED Viewed

@@ -0,0 +1,25 @@
+<!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/public/favicon.ico ADDED Viewed

File without changes

data/test/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/test/functional/comment_thread_subscriptions_controller_test.rb~ ADDED Viewed

@@ -0,0 +1,125 @@
+# Copyright 2011-2012 Rice University. Licensed under the Affero General Public
+# License version 3 or later.  See the COPYRIGHT file for details.
+require 'test_helper'
+class CommentThreadSubscriptionsControllerTest < 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 comment_thread_subscription not logged in" do
+    assert_difference('CommentThreadSubscription.count', 0) do
+      get :create, :question_id => @question.to_param
+    end
+    assert_redirected_to login_path
+  end
+  test "should not create comment_thread_subscription not authorized" do
+    user_login
+    assert_difference('CommentThreadSubscription.count', 0) do
+      get :create, :question_id => @question.to_param
+    end
+    assert_response(403)
+  end
+  test "should not create comment_thread_subscription already subscribed" do
+    sign_in @user
+    FactoryGirl.create(:comment_thread_subscription,
+                   :user => @user, :comment_thread => @question.comment_thread)
+    assert_difference('CommentThreadSubscription.count', 0) do
+      get :create, :question_id => @question.to_param
+    end
+    assert_redirected_to question_comments_path(@question.to_param)
+  end
+  test "should create comment_thread_subscription" do
+    sign_in @user
+    assert_difference('CommentThreadSubscription.count') do
+      get :create, :question_id => @question.to_param
+    end
+    assert_redirected_to question_comments_path(@question.to_param)
+  end
+  test "should create comment_thread_subscription published question" do
+    user_login
+    assert_difference('CommentThreadSubscription.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/functional/comments_controller_test.rb ADDED Viewed

@@ -0,0 +1,181 @@
+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/comments_controller_test.rb~ ADDED Viewed

@@ -0,0 +1,184 @@
+# Copyright 2011-2012 Rice University. Licensed under the Affero General Public
+# License version 3 or later.  See the COPYRIGHT file for details.
+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