activity_notification 0.0.8 → 0.0.9

data/spec/models/dummy/dummy_notifier_spec.rb

@@ -0,0 +1,6 @@
+describe Dummy::DummyNotifier, type: :model do
+
+  it_behaves_like :notifier
+  it_behaves_like :common
+
+end

data/spec/models/dummy/dummy_target_spec.rb

@@ -0,0 +1,6 @@
+describe Dummy::DummyTarget, type: :model do
+
+  it_behaves_like :target
+  it_behaves_like :common
+
+end

data/spec/models/notification_spec.rb

@@ -1,10 +1,11 @@
 describe ActivityNotification::Notification, type: :model do
 
   it_behaves_like :notification_api
+  it_behaves_like :renderable
 
   describe "with association" do
     it "belongs to target" do
-      target = create(:user)
+      target = create(:confirmed_user)
       notification = create(:notification, target: target)
       expect(notification.reload.target).to eq(target)
     end
@@ -34,7 +35,7 @@ describe ActivityNotification::Notification, type: :model do
     end
 
     it "belongs to notifier" do
-      notifier = create(:user)
+      notifier = create(:confirmed_user)
       notification = create(:notification, notifier: notifier)
       expect(notification.reload.notifier).to eq(notifier)
     end
@@ -168,8 +169,8 @@ describe ActivityNotification::Notification, type: :model do
     context "to filter by association" do
       before do
         ActivityNotification::Notification.delete_all
-        @target_1, @notifiable_1, @group_1, @key_1 = create(:user), create(:article), nil, "key.1"
-        @target_2, @notifiable_2, @group_2, @key_2 = create(:user), create(:comment), @notifiable_1, "key.2"
+        @target_1, @notifiable_1, @group_1, @key_1 = create(:confirmed_user), create(:article), nil, "key.1"
+        @target_2, @notifiable_2, @group_2, @key_2 = create(:confirmed_user), create(:comment), @notifiable_1, "key.2"
         @notification_1 = create(:notification, target: @target_1, notifiable: @notifiable_1, group: @group_1, key: @key_1)
         @notification_2 = create(:notification, target: @target_2, notifiable: @notifiable_2, group: @group_2, key: @key_2)
       end

data/spec/rails_app/app/assets/javascripts/application.js

@@ -0,0 +1,2 @@
+//= require jquery
+//= require jquery_ujs

data/spec/rails_app/app/controllers/articles_controller.rb

@@ -0,0 +1,62 @@
+class ArticlesController < ApplicationController
+  before_action :authenticate_user!, except: [:index, :show]
+  before_action :set_article, only: [:show, :edit, :update, :destroy]
+
+  # GET /articles
+  def index
+    @articles = Article.all.includes(:user)
+  end
+
+  # GET /articles/1
+  def show
+    @comment = Comment.new
+  end
+
+  # GET /articles/new
+  def new
+    @article = Article.new
+  end
+
+  # GET /articles/1/edit
+  def edit
+  end
+
+  # POST /articles
+  def create
+    @article = Article.new(article_params)
+    @article.user = current_user
+
+    if @article.save
+      @article.notify :users
+      redirect_to @article, notice: 'Article was successfully created.'
+    else
+      render :new
+    end
+  end
+
+  # PATCH/PUT /articles/1
+  def update
+    if @article.update(article_params)
+      redirect_to @article, notice: 'Article was successfully updated.'
+    else
+      render :edit
+    end
+  end
+
+  # DELETE /articles/1
+  def destroy
+    @article.destroy
+    redirect_to articles_url, notice: 'Article was successfully destroyed.'
+  end
+
+  private
+    # Use callbacks to share common setup or constraints between actions.
+    def set_article
+      @article = Article.find(params[:id])
+    end
+
+    # Only allow a trusted parameter "white list" through.
+    def article_params
+      params.require(:article).permit(:title, :body)
+    end
+end

data/spec/rails_app/app/controllers/comments_controller.rb

@@ -0,0 +1,34 @@
+class CommentsController < ApplicationController
+  before_action :set_comment, only: [:destroy]
+
+  # POST /comments
+  def create
+    @comment = Comment.new(comment_params)
+    @comment.user = current_user
+
+    if @comment.save
+      @comment.notify :users
+      redirect_to @comment.article, notice: 'Comment was successfully created.'
+    else
+      redirect_to @comment.article
+    end
+  end
+
+  # DELETE /comments/1
+  def destroy
+    article = @comment.article
+    @comment.destroy
+    redirect_to article, notice: 'Comment was successfully destroyed.'
+  end
+
+  private
+    # Use callbacks to share common setup or constraints between actions.
+    def set_comment
+      @comment = Comment.find(params[:id])
+    end
+
+    # Only allow a trusted parameter "white list" through.
+    def comment_params
+      params.require(:comment).permit(:article_id, :body)
+    end
+end

data/spec/rails_app/app/models/admin.rb

@@ -0,0 +1,8 @@
+class Admin < ActiveRecord::Base
+  belongs_to :user
+  validates :user, presence: true
+
+  acts_as_notification_target email: :email,
+    email_allowed: ->(admin, key) { admin.user.confirmed_at.present? },
+    devise_resource: :user
+end

data/spec/rails_app/app/models/article.rb

@@ -1,12 +1,12 @@
 class Article < ActiveRecord::Base
-  include ActivityNotification::Notifiable
+  belongs_to :user
+  has_many :comments, dependent: :delete_all
+  has_many :commented_users, through: :comments, source: :user
+  validates :user, presence: true
+
   acts_as_notifiable :users,
-    targets: ->(article, key) { [article.user] },
+    targets: ->(article, key) { User.all.to_a - [article.user] },
     notifier: :user,
     email_allowed: true#, 
     #notifiable_path: ->(article) { concept_issue_path(issue.concept, issue) }
-
-  belongs_to :user
-  has_many :comments, dependent: :delete_all
-  has_many :commented_users, through: :comments, source: :user
 end

data/spec/rails_app/app/models/comment.rb

@@ -1,8 +1,9 @@
 class Comment < ActiveRecord::Base
   belongs_to :article
   belongs_to :user
+  validates :article, presence: true
+  validates :user, presence: true
 
-  include ActivityNotification::Notifiable
   acts_as_notifiable :users,
     targets: ->(comment, key) { (comment.article.commented_users.to_a - [comment.user] + [comment.article.user]).uniq },
     group: :article,
@@ -14,5 +15,4 @@ class Comment < ActiveRecord::Base
   def custom_notification_email_to_users_allowed?(user, key)
     true
   end
-
 end

data/spec/rails_app/app/models/dummy/dummy_base.rb

@@ -0,0 +1,2 @@
+class Dummy::DummyBase < ActiveRecord::Base
+end

data/spec/rails_app/app/models/dummy/dummy_notifiable.rb

@@ -0,0 +1,4 @@
+class Dummy::DummyNotifiable < ActiveRecord::Base
+  self.table_name = :articles
+  include ActivityNotification::Notifiable
+end

data/spec/rails_app/app/models/dummy/dummy_notifier.rb

@@ -0,0 +1,4 @@
+class Dummy::DummyNotifier < ActiveRecord::Base
+  self.table_name = :users
+  include ActivityNotification::Notifier
+end

data/spec/rails_app/app/models/dummy/dummy_target.rb

@@ -0,0 +1,4 @@
+class Dummy::DummyTarget < ActiveRecord::Base
+  self.table_name = :users
+  include ActivityNotification::Target
+end

data/spec/rails_app/app/models/user.rb

@@ -1,8 +1,8 @@
 class User < ActiveRecord::Base
-  # devise :database_authenticatable, :registerable, :confirmable
-
-  include ActivityNotification::Target
-  acts_as_target email: :email, email_allowed: :confirmed_at
-
+  devise :database_authenticatable, :registerable, :confirmable
   validates :email, presence: true
+  has_many :articles, dependent: :delete_all
+  has_many :comments, through: :articles, dependent: :delete_all
+
+  acts_as_notification_target email: :email, email_allowed: :confirmed_at
 end

data/spec/rails_app/app/views/activity_notification/notifications/default/custom/_path_test.html.erb

@@ -0,0 +1 @@
+Custom template root for path test: <%= notification.id %>

data/spec/rails_app/app/views/activity_notification/notifications/default/custom/_test.html.erb

@@ -0,0 +1 @@
+Custom template root for default target: <%= notification.id %>

data/spec/rails_app/app/views/activity_notification/notifications/users/_custom_index.html.erb

@@ -0,0 +1 @@
+Custom index: <%= yield :notification_index %>

data/spec/rails_app/app/views/activity_notification/notifications/users/custom/_test.html.erb

@@ -0,0 +1 @@
+Custom template root for user target: <%= notification.id %>

data/spec/rails_app/app/views/articles/_form.html.erb

@@ -0,0 +1,20 @@
+<%= form_for(@article) do |f| %>
+  <% if @article.errors.any? %>
+    <div id="error_explanation">
+      <h2><%= pluralize(@article.errors.count, "error") %> prohibited this article from being saved:</h2>
+
+      <ul>
+      <% @article.errors.full_messages.each do |message| %>
+        <li><%= message %></li>
+      <% end %>
+      </ul>
+    </div>
+  <% end %>
+
+  <%= f.text_field :title %>
+  <%= f.text_area :body %>
+
+  <div class="actions">
+    <%= f.submit %>
+  </div>
+<% end %>

data/spec/rails_app/app/views/articles/edit.html.erb

@@ -0,0 +1,6 @@
+<h1>Editing Article</h1>
+
+<%= render 'form' %>
+
+<%= link_to 'Show', @article %> |
+<%= link_to 'Back', articles_path %>

data/spec/rails_app/app/views/articles/index.html.erb

@@ -0,0 +1,67 @@
+<p id="notice"><%= notice %></p>
+
+<h1>Listing Users</h1>
+
+<table>
+  <thead>
+    <tr>
+      <th colspan="3"></th>
+    </tr>
+  </thead>
+
+  <tbody>
+    <% User.all.each do |user| %>
+      <tr>
+        <td><%= user.email %></td>
+        <td><%= user.name %></td>
+        <td><%= link_to 'Notifications', user_notifications_path(user) %></td>
+      </tr>
+    <% end %>
+  </tbody>
+</table>
+
+<h1>Listing Admins</h1>
+
+<table>
+  <thead>
+    <tr>
+      <th colspan="3"></th>
+    </tr>
+  </thead>
+
+  <tbody>
+    <% Admin.all.each do |admin| %>
+      <tr>
+        <td><%= admin.user.email %></td>
+        <td><%= admin.user.name %></td>
+        <td><%= link_to 'Notifications', admin_notifications_path(admin) %></td>
+      </tr>
+    <% end %>
+  </tbody>
+</table>
+
+<h1>Listing Articles</h1>
+
+<table>
+  <thead>
+    <tr>
+      <th colspan="5"></th>
+    </tr>
+  </thead>
+
+  <tbody>
+    <% @articles.each do |article| %>
+      <tr>
+        <td><%= article.user.name %></td>
+        <td><%= article.title %></td>
+        <td><%= link_to 'Show', article %></td>
+        <td><%= link_to 'Edit', edit_article_path(article) if user_signed_in? and current_user == article.user %></td>
+        <td><%= link_to 'Destroy', article, method: :delete, data: { confirm: 'Are you sure?' } %></td>
+      </tr>
+    <% end %>
+  </tbody>
+</table>
+
+<br>
+
+<%= link_to 'New Article', new_article_path %>

data/spec/rails_app/app/views/articles/new.html.erb

@@ -0,0 +1,5 @@
+<h1>New Article</h1>
+
+<%= render 'form' %>
+
+<%= link_to 'Back', articles_path %>

data/spec/rails_app/app/views/articles/show.html.erb

@@ -0,0 +1,38 @@
+<p id="notice"><%= notice %></p>
+
+<h1><%= @article.title %></h1>
+<p><%= @article.user.name %></p>
+<p><%= @article.body %></p>
+<%= link_to 'Edit', edit_article_path(@article) %> |
+<%= link_to 'Destroy', @article, method: :delete, data: { confirm: 'Are you sure?' } %>
+<%= link_to 'Back', articles_path %>
+
+<h2>Listing Comments</h2>
+
+<table>
+  <thead>
+    <tr>
+      <th colspan="3"></th>
+    </tr>
+  </thead>
+
+  <tbody>
+    <% @article.comments.each do |comment| %>
+      <tr>
+        <td><%= comment.user.name %></td>
+        <td><%= comment.body %></td>
+        <td><%= link_to 'Destroy', comment, method: :delete, data: { confirm: 'Are you sure?' } %></td>
+      </tr>
+    <% end %>
+  </tbody>
+</table>
+
+<h2>New Comment</h2>
+<%= form_for(@comment) do |f| %>
+  <%= f.hidden_field :article_id, value: @article.id %>
+  <%= f.text_area :body %>
+
+  <div class="actions">
+    <%= f.submit %>
+  </div>
+<% end %>

data/spec/rails_app/app/views/layouts/_header.html.erb

@@ -0,0 +1,8 @@
+<%= link_to 'Root', root_path %>
+<% if user_signed_in? %>
+  <%= "Login as #{current_user.name}" %>
+  <%= link_to 'Logout', destroy_user_session_path, method: :delete %>
+  <%= render_notifications_of current_user, fallback: :default, index_content: :with_attributes %>
+<% else %>
+  <%= link_to 'Login', new_user_session_path %>
+<% end %>

data/spec/rails_app/app/views/layouts/application.html.erb

@@ -2,13 +2,12 @@
 <html>
 <head>
   <title>Dummy</title>
-  <%= stylesheet_link_tag    'application', media: 'all', 'data-turbolinks-track' => true %>
-  <%= javascript_include_tag 'application', 'data-turbolinks-track' => true %>
+  <%#= stylesheet_link_tag    'application', media: 'all', 'data-turbolinks-track': true %>
+  <%= javascript_include_tag 'application', 'data-turbolinks-track': true %>
   <%= csrf_meta_tags %>
 </head>
 <body>
-
+<%= render 'layouts/header' %>
 <%= yield %>
-
 </body>
 </html>

data/spec/rails_app/config/initializers/activity_notification.rb

@@ -1,9 +1,17 @@
 ActivityNotification.configure do |config|
 
-  # Table name to store notification data
+  # Configure if all activity notifications are enabled
+  # Set false when you want to turn off activity notifications
+  config.enabled = true
+
+  # Configure table name to store notification data
   config.table_name = "notifications"
 
-  # ==> Mailer Configuration
+  # Configure if email notification is enabled as default
+  # Note that you can configure them for each model by acts_as roles.
+  # Set true when you want to turn on email notifications as default
+  config.email_enabled = false
+
   # Configure the e-mail address which will be shown in ActivityNotification::Mailer,
   # note that it will be overwritten if you use your own mailer class with default "from" parameter.
   config.mailer_sender = 'please-change-me-at-config-initializers-activity_notification@example.com'
@@ -13,6 +21,8 @@ ActivityNotification.configure do |config|
 
   # Configure the parent class responsible to send e-mails.
   # config.parent_mailer = 'ActionMailer::Base'
-    
+
+  # Configure default limit number of opened notifications you can get from opened* scope
   config.opened_limit = 10
+
 end