commontator 4.11.0 → 4.11.1

data/spec/controllers/commontator/subscriptions_controller_spec.rb~

@@ -1,101 +0,0 @@
-require 'rails_helper'
-
-module Commontator
-  RSpec.describe SubscriptionsController, type: :controller do
-    routes { Commontator::Engine.routes }
-
-    before(:each) do
-      setup_controller_spec
-    end
-    
-    it "won't subscribe unless authorized" do
-      put :subscribe, :id => @thread.id
-      expect(response).to have_http_status(:forbidden)
-      expect(@thread.subscription_for(nil)).to be_nil
-      expect(@thread.subscription_for(@user)).to be_nil
-      
-      sign_in @user
-      put :subscribe, :id => @thread.id
-      expect(response).to have_http_status(:forbidden)
-      expect(@thread.subscription_for(@user)).to be_nil
-      
-      @thread.subscribe(@user)
-      @user.can_read = true
-      put :subscribe, :id => @thread.id
-      expect(response).to redirect_to(@thread)
-      expect(assigns(:thread).errors).not_to be_empty
-    end
-    
-    it 'must subscribe if authorized' do
-      sign_in @user
-      
-      @user.can_read = true
-      put :subscribe, :id => @thread.id
-      expect(response).to redirect_to(@thread)
-      expect(assigns(:thread).errors).to be_empty
-      expect(@thread.subscription_for(@user)).not_to be_nil
-      
-      @thread.unsubscribe(@user)
-      @user.can_read = false
-      @user.can_edit = true
-      put :subscribe, :id => @thread.id
-      expect(response).to redirect_to(@thread)
-      expect(assigns(:thread).errors).to be_empty
-      expect(@thread.subscription_for(@user)).not_to be_nil
-      
-      @thread.unsubscribe(@user)
-      @user.can_edit = false
-      @user.is_admin = true
-      put :subscribe, :id => @thread.id
-      expect(response).to redirect_to(@thread)
-      expect(assigns(:thread).errors).to be_empty
-      expect(@thread.subscription_for(@user)).not_to be_nil
-    end
-    
-    it "won't unsubscribe unless authorized" do
-      @thread.subscribe(@user)
-      put :unsubscribe, :id => @thread.id
-      expect(response).to have_http_status(:forbidden)
-      expect(@thread.subscription_for(nil)).to be_nil
-      expect(@thread.subscription_for(@user)).not_to be_nil
-      
-      sign_in @user
-      put :unsubscribe, :id => @thread.id
-      expect(response).to have_http_status(:forbidden)
-      expect(@thread.subscription_for(@user)).not_to be_nil
-      
-      @thread.unsubscribe(@user)
-      @user.can_read = true
-      put :unsubscribe, :id => @thread.id
-      expect(response).to redirect_to(@thread)
-      expect(assigns(:thread).errors).not_to be_empty
-    end
-    
-    it 'must unsubscribe if authorized' do
-      sign_in @user
-      
-      @thread.subscribe(@user)
-      @user.can_read = true
-      put :unsubscribe, :id => @thread.id
-      expect(response).to redirect_to(@thread)
-      expect(assigns(:thread).errors).to be_empty
-      expect(@thread.subscription_for(@user)).to be_nil
-      
-      @thread.subscribe(@user)
-      @user.can_read = false
-      @user.can_edit = true
-      put :unsubscribe, :id => @thread.id
-      expect(response).to redirect_to(@thread)
-      expect(assigns(:thread).errors).to be_empty
-      expect(@thread.subscription_for(@user)).to be_nil
-      
-      @thread.subscribe(@user)
-      @user.can_edit = false
-      @user.is_admin = true
-      put :unsubscribe, :id => @thread.id
-      expect(response).to redirect_to(@thread)
-      expect(assigns(:thread).errors).to be_empty
-      expect(@thread.subscription_for(@user)).to be_nil
-    end
-  end
-end

data/spec/controllers/commontator/threads_controller_spec.rb~

@@ -1,218 +0,0 @@
-require 'rails_helper'
-
-module Commontator
-  RSpec.describe ThreadsController, type: :controller do
-    routes { Commontator::Engine.routes }
-
-    before(:each) do
-      setup_controller_spec
-    end
-    
-    it "won't show unless authorized" do
-      get :show, :id => @thread.id
-      expect(response).to have_http_status(:forbidden)
-      
-      sign_in @user
-      get :show, :id => @thread.id
-      expect(response).to have_http_status(:forbidden)
-    end
-    
-    it 'must show if authorized' do
-      commontable_path = Rails.application.routes.url_helpers.dummy_model_path(@commontable)
-      sign_in @user
-      
-      @user.can_read = true
-      get :show, :id => @thread.id
-      expect(response).to redirect_to(commontable_path)
-      
-      @user.can_read = false
-      @user.can_edit = true
-      get :show, :id => @thread.id
-      expect(response).to redirect_to(commontable_path)
-      
-      @user.can_edit = false
-      @user.is_admin = true
-      get :show, :id => @thread.id
-      expect(response).to redirect_to(commontable_path)
-    end
-    
-    it "won't close unless authorized and open" do
-      put :close, :id => @thread.id
-      expect(response).to have_http_status(:forbidden)
-      @thread.reload
-      expect(@thread.is_closed?).to eq false
-      
-      sign_in @user
-      put :close, :id => @thread.id
-      expect(response).to have_http_status(:forbidden)
-      @thread.reload
-      expect(@thread.is_closed?).to eq false
-      
-      @user.can_read = true
-      put :close, :id => @thread.id
-      expect(response).to have_http_status(:forbidden)
-      @thread.reload
-      expect(@thread.is_closed?).to eq false
-      
-      @user.can_edit = true
-      expect(@thread.close).to eq true
-      put :close, :id => @thread.id
-      expect(response).to redirect_to(@thread)
-      expect(assigns(:thread).errors).not_to be_empty
-    end
-    
-    it 'must close if authorized and open' do
-      sign_in @user
-      
-      @user.can_edit = true
-      put :close, :id => @thread.id
-      expect(response).to redirect_to(@thread)
-      expect(assigns(:thread).errors).to be_empty
-      expect(assigns(:thread).is_closed?).to eq true
-      expect(assigns(:thread).closer).to eq @user
-      
-      expect(assigns(:thread).reopen).to eq true
-      @user.can_edit = false
-      @user.is_admin = true
-      put :close, :id => @thread.id
-      expect(response).to redirect_to(@thread)
-      expect(assigns(:thread).errors).to be_empty
-      expect(assigns(:thread).is_closed?).to eq true
-      expect(assigns(:thread).closer).to eq @user
-    end
-    
-    it "won't reopen unless authorized and closed" do
-      expect(@thread.close).to eq true
-      put :reopen, :id => @thread.id
-      expect(response).to have_http_status(:forbidden)
-      @thread.reload
-      expect(@thread.is_closed?).to eq true
-      
-      sign_in @user
-      put :reopen, :id => @thread.id
-      expect(response).to have_http_status(:forbidden)
-      @thread.reload
-      expect(@thread.is_closed?).to eq true
-      
-      @user.can_read = true
-      put :reopen, :id => @thread.id
-      expect(response).to have_http_status(:forbidden)
-      @thread.reload
-      expect(@thread.is_closed?).to eq true
-      
-      expect(@thread.reopen).to eq true
-      @user.can_edit = true
-      put :reopen, :id => @thread.id
-      expect(response).to redirect_to(@thread)
-      expect(assigns(:thread).errors).not_to be_empty
-    end
-    
-    it 'must reopen if authorized and closed' do
-      sign_in @user
-      
-      expect(@thread.close).to eq true
-      @user.can_edit = true
-      put :reopen, :id => @thread.id
-      expect(response).to redirect_to(@thread)
-      expect(assigns(:thread).errors).to be_empty
-      expect(assigns(:thread).is_closed?).to eq false
-      
-      expect(assigns(:thread).close).to eq true
-      @user.can_edit = false
-      @user.is_admin = true
-      put :reopen, :id => @thread.id
-      expect(response).to redirect_to(@thread)
-      expect(assigns(:thread).errors).to be_empty
-      expect(assigns(:thread).is_closed?).to eq false
-    end
-    
-    context '#mentions' do
-      let(:search_phrase) { nil }
-      let(:call_request)  { get :mentions, id: @thread.id, format: :json, q: search_phrase }
-
-      let!(:other_user)   { DummyUser.create }
-
-      context 'mentions disabled' do
-        before(:all) { Commontator.mentions_enabled = false }
-        after(:all)  { Commontator.mentions_enabled = true }
-
-        it 'returns 403 Forbidden' do
-          call_request
-          expect(response).to have_http_status(:forbidden)
-        end
-      end
-
-      context 'mentions enabled' do
-        context 'anonymous user' do
-          it 'returns 403 Forbidden' do
-            call_request
-            expect(response).to have_http_status(:forbidden)
-          end
-        end
-
-        context 'unauthorized user' do
-          before { sign_in @user }
-
-          it 'returns 403 Forbidden' do
-            call_request
-            expect(response).to have_http_status(:forbidden)
-          end
-        end
-
-        context 'authorized user' do
-          before do
-            @user.can_read = true
-            sign_in @user
-          end
-
-          context 'query is blank' do
-            it 'returns a JSON error message' do
-              call_request
-              expect(response).to have_http_status(:unprocessable_entity)
-              expect(JSON.parse(response.body)['errors']).to(
-                include('Query string is too short (minimum 3 characters)')
-              )
-            end
-          end
-
-          context 'query is too short' do
-            let(:search_phrase) { 'Us' }
-
-            it 'returns a JSON error message' do
-              call_request
-              expect(response).to have_http_status(:unprocessable_entity)
-              expect(JSON.parse(response.body)['errors']).to(
-                include('Query string is too short (minimum 3 characters)')
-              )
-            end
-          end
-
-          context 'query is 3 characters or more' do
-            let(:search_phrase) { 'User' }
-
-            let(:valid_result) { [@user] }
-            let(:valid_response) do
-              { 'mentions' => valid_result.map do |user|
-                { 'id' => user.id, 'name' => user.name, 'type' => 'user' }
-              end }
-            end
-
-            it 'calls the user_mentions_proc and returns the result' do
-              expect(Commontator.user_mentions_proc).to(
-                receive(:call).with(@user, search_phrase).and_return(valid_result)
-              )
-
-              call_request
-              expect(response).to have_http_status(:success)
-
-              response_body = JSON.parse(response.body)
-              expect(response_body['errors']).to be_nil
-              expect(response_body).to eq valid_response
-            end
-          end
-        end
-      end
-    end
-  end
-end
-

data/spec/dummy/README.md~

@@ -1,3 +0,0 @@
-## Dummy
-
-A dummy application used to test the Commontator engine.

data/spec/dummy/Rakefile~

@@ -1,6 +0,0 @@
-# Add your own tasks in files placed in lib/tasks ending in .rake,
-# for example lib/tasks/capistrano.rake, and they will automatically be available to Rake.
-
-require File.expand_path('../config/application', __FILE__)
-
-Dummy::Application.load_tasks

data/spec/dummy/app/assets/javascripts/application.js~

@@ -1,16 +0,0 @@
-// This is a manifest file that'll be compiled into application.js, which will include all the files
-// listed below.
-//
-// Any JavaScript/Coffee file within this directory, lib/assets/javascripts, vendor/assets/javascripts,
-// or vendor/assets/javascripts of plugins, if any, can be referenced here using a relative path.
-//
-// It's not advisable to add code directly here, but if you do, it'll appear at the bottom of the
-// the compiled file.
-//
-// WARNING: THE FIRST BLANK LINE MARKS THE END OF WHAT'S TO BE PROCESSED, ANY BLANK LINE SHOULD
-// GO AFTER THE REQUIRES BELOW.
-//
-//= require jquery
-//= require jquery_ujs
-//= require_tree .
-

data/spec/dummy/app/assets/stylesheets/application.css~

@@ -1,14 +0,0 @@
-/*
- * This is a manifest file that'll be compiled into application.css, which will include all the files
- * listed below.
- *
- * Any CSS and SCSS file within this directory, lib/assets/stylesheets, vendor/assets/stylesheets,
- * or vendor/assets/stylesheets of plugins, if any, can be referenced here using a relative path.
- *
- * You're free to add application-wide styles to this file and they'll appear at the top of the
- * compiled file, but it's generally better to create a new file per style scope.
- *
- *= require_self
- *= require_tree .
- */
-

data/spec/dummy/app/controllers/application_controller.rb~

@@ -1,4 +0,0 @@
-class ApplicationController < ActionController::Base
-  protect_from_forgery
-end
-

data/spec/dummy/app/controllers/dummy_models_controller.rb~

@@ -1,23 +0,0 @@
-class DummyModelsController < ApplicationController
-  before_filter :get_dummy
-
-  def show
-    #commontator_thread_show(@dummy_model)
-  end
-
-  def hide
-    render :show
-  end
-
-  def url_options
-    return Hash.new if request.nil?
-    super
-  end
-
-  protected
-
-  def get_dummy
-    @dummy_model = DummyModel.find(params[:id])
-  end
-end
-

data/spec/dummy/app/helpers/application_helper.rb~

@@ -1,15 +0,0 @@
-module ApplicationHelper
-  @@user = nil
-  
-  def current_user
-    @@user
-  end
-  
-  def sign_in(user)
-    @@user = user
-  end
-  
-  def sign_out
-    @@user = nil
-  end
-end

data/spec/dummy/app/models/dummy_model.rb~

@@ -1,3 +0,0 @@
-class DummyModel < ActiveRecord::Base
-  acts_as_commontable
-end

data/spec/dummy/app/models/dummy_user.rb~

@@ -1,13 +0,0 @@
-class DummyUser < ActiveRecord::Base
-  acts_as_commontator
-  
-  attr_accessor :is_admin, :can_edit, :can_read
-
-  def email
-    "dummy_user#{id}@example.com"
-  end
-
-  def name
-    "Dummy User ##{id}"
-  end
-end

data/spec/dummy/app/views/dummy_models/index.html.erb~

@@ -1,6 +0,0 @@
-<h3>Dummy Models</h3>
-
-<% @dummy_models.each do |model| %>
-  <li><%= 'Show', dummy_model_path(model) %></li>
-<% end %>
-

data/spec/dummy/app/views/dummy_models/show.html.erb~

@@ -1,2 +0,0 @@
-<%= commontator_thread(@dummy_model) %>
-HI

data/spec/dummy/app/views/layouts/application.html.erb~

@@ -1,14 +0,0 @@
-<!DOCTYPE html>
-<html>
-<head>
-  <title>Dummy</title>
-  <%= stylesheet_link_tag    "application", :media => "all" %>
-  <%= javascript_include_tag "application" %>
-  <%= csrf_meta_tags %>
-</head>
-<body>
-
-<%= yield %>
-
-</body>
-</html>