message_train 0.1.7 → 0.2.0

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

@@ -1,4 +1,7 @@
 class ApplicationController < ActionController::Base
+
+  include MessageTrain::MessageTrainSupport
+
   # Prevent CSRF attacks by raising an exception.
   # For APIs, you may want to use :null_session instead.
   protect_from_forgery with: :exception

data/spec/dummy/app/models/group.rb

@@ -3,11 +3,18 @@ class Group < ActiveRecord::Base
   resourcify
 
   # MessageTrain Gem
-  message_train only: :recipient, valid_senders: :owners, name_column: :title, slug_column: :slug
+  message_train only: :recipient,
+                valid_senders: :owners,
+                name_column: :title,
+                slug_column: :slug,
+                collectives_for_recipient: :membered_by,
+                valid_recipients: :recipients
 
   # Callbacks
   before_create :set_slug
 
+  scope :membered_by, ->(user) { with_roles([:member, :owner], user).uniq }
+
   def set_slug
     # Manually generate slug instead of using friendly id, for testing.
     self.slug = title.downcase.gsub(/[^a-z0-9]+/, '-')
@@ -16,4 +23,12 @@ class Group < ActiveRecord::Base
   def owners
     User.with_role(:owner, self)
   end
+
+  def recipients
+    User.with_role(:member, self)
+  end
+
+  def self.fallback_address_book(user)
+    Group.with_role(:owner, user)
+  end
 end

data/spec/dummy/app/models/role.rb

@@ -7,4 +7,26 @@ class Role < ActiveRecord::Base
             :allow_nil => true
 
   scopify
+
+  # MessageTrain Gem
+  message_train except: :sender,
+                valid_senders: :superadmins,
+                name_column: :capitalized_name,
+                slug_column: :name,
+                collectives_for_recipient: :sendable_roles,
+                valid_recipients: :recipients
+
+  scope :sendable_roles, ->(user) { where(resource: nil).where.not(name: 'superadmin') }
+
+  def recipients
+    User.with_role(self.name.to_sym)
+  end
+
+  def superadmins
+    User.with_role(:superadmin)
+  end
+
+  def capitalized_name
+    name.capitalize
+  end
 end

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

@@ -1,4 +1,4 @@
-require 'group' #TODO This is a hack to get the group model to load in development. Works, but for how long?
+# require 'group' #TODO This is a hack to get the group model to load in development. Works, but for how long?
 
 class User < ActiveRecord::Base
   # Rolify Gem

data/spec/dummy/app/views/layouts/_top_navigation.html.haml

@@ -1,5 +1,7 @@
 = navbar container_mode: :inside, class: 'navbar-static-top' do
   = nav_list class: 'navbar-nav navbar-right' do
     - if user_signed_in?
-      = boxes_dropdown_list
-    = nav_item 'Sign Out', '/users/sign_out', method: :delete
+      = collective_boxes_dropdown_list(current_user)
+      = boxes_dropdown_list(current_user)
+      = nav_item 'Manage Notifications', '/unsubscribes'
+      = nav_item 'Sign Out', '/users/sign_out', method: :delete

data/spec/dummy/app/views/layouts/application.html.haml

@@ -16,10 +16,9 @@
   .container
     = render_hero_unit
     - if user_signed_in?
-      - add_widget do
-        = boxes_widget
+      = message_train_widget
     = render_widgets 'md', 3
-    .col-md-9
+    .col-md-9.content
       = render_h1         # For the h1 tag containing the title alone
       #alert_area
         = alert_flash_messages

data/spec/dummy/app/views/pages/index.html.haml

@@ -0,0 +1,4 @@
+- if user_signed_in?
+  = link_to 'Message Train Application', message_train.box_path(:in)
+- else
+  = link_to 'Sign In', new_user_session_path

data/spec/dummy/config/application.rb

@@ -35,6 +35,12 @@ module Dummy
       g.stylesheets     false
       g.javascripts     false
     end
+
+    config.time_zone = 'America/Denver'
+    config.active_record.default_timezone = :local
   end
 end
 
+FactoryGirl.definition_file_paths << MessageTrain::Engine.root.join('spec/factories')
+FactoryGirl.reload
+

data/spec/dummy/config/environments/development.rb

@@ -15,6 +15,7 @@ Rails.application.configure do
 
   # Don't care if the mailer can't send.
   config.action_mailer.raise_delivery_errors = false
+  config.action_mailer.default_url_options = { :host => 'localhost:3000' }
 
   # Print deprecation notices to the Rails logger.
   config.active_support.deprecation = :log

data/spec/dummy/config/environments/test.rb

@@ -30,6 +30,7 @@ Rails.application.configure do
   # The :test delivery method accumulates sent emails in the
   # ActionMailer::Base.deliveries array.
   config.action_mailer.delivery_method = :test
+  config.action_mailer.default_url_options = { :host => 'localhost:3000' }
 
   # Randomize the order test cases are executed.
   config.active_support.test_order = :random

data/spec/dummy/config/initializers/high_voltage.rb

@@ -0,0 +1,3 @@
+HighVoltage.configure do |config|
+  config.home_page = 'index'
+end

data/spec/dummy/config/initializers/message_train.rb

@@ -5,4 +5,9 @@ MessageTrain.configure do |config|
   # config.user_sign_in_path = '/user/sign_in'
   # config.user_route_authentication_method = :user
   # config.address_book_methods[:users] = :address_book
-end
+  config.address_book_method = :fallback_address_book # "Groups" will pick this up, having no address book method.
+  config.from_email = 'do-not-reply@gemvein.com'
+  config.site_name = 'MessageTrain Test Site'
+end
+
+Rails.application.config.eager_load = true #FIXME: This is a weird place to put this. What would be better?

data/spec/dummy/config/initializers/paperclip.rb

@@ -1,2 +1,2 @@
-Paperclip::Attachment.default_options[:path] = ':rails_root/public/system/:rails_env/:class/:attachment/:id_partition/:filename'
-Paperclip::Attachment.default_options[:url] = '/system/:rails_env/:class/:attachment/:id_partition/:filename'
+Paperclip::Attachment.default_options[:path] = ':rails_root/public/system/:rails_env/:class/:attachment/:id_partition/:style/:filename'
+Paperclip::Attachment.default_options[:url] = '/system/:rails_env/:class/:attachment/:id_partition/:style/:filename'

data/spec/dummy/config/routes.rb

@@ -1,5 +1,5 @@
 Rails.application.routes.draw do
-
-	mount MessageTrain::Engine => '/', :as => 'message_train'
+  
+  mount MessageTrain::Engine => '/', :as => 'message_train'
   devise_for :users
 end

data/spec/dummy/config/settings.yml

@@ -0,0 +1,9 @@
+mailer:
+  from: "MessageTrain Mailer <webmaster@gemvein.com>"
+  address: "smtp.gmail.com"
+  port: 587
+  domain: "gmail.com"
+  user_name: <%= ENV["MAILER_USERNAME"] %>
+  password: <%= ENV["MAILER_PASSWORD"] %>
+  authentication: :login
+  enable_starttls_auto: true

data/spec/dummy/db/migrate/20150901183634_add_received_through_to_message_train_receipts.message_train.rb

@@ -0,0 +1,7 @@
+# This migration comes from message_train (originally 20150901183458)
+class AddReceivedThroughToMessageTrainReceipts < ActiveRecord::Migration
+  def change
+    add_reference :message_train_receipts, :received_through, polymorphic: true
+    add_index :message_train_receipts, [:received_through_type, :received_through_id], name: :index_message_train_receipts_on_received_through
+  end
+end

data/spec/dummy/db/migrate/20151004184519_add_unique_index_to_receipts.message_train.rb

@@ -0,0 +1,6 @@
+# This migration comes from message_train (originally 20151004184347)
+class AddUniqueIndexToReceipts < ActiveRecord::Migration
+  def change
+    add_index :message_train_receipts, [:message_id, :recipient_type, :recipient_id], name: :message_recipient, unique: true
+  end
+end

data/spec/dummy/db/migrate/20151124001417_create_message_train_unsubscribes.message_train.rb

@@ -0,0 +1,15 @@
+# This migration comes from message_train (originally 20151124000820)
+class CreateMessageTrainUnsubscribes < ActiveRecord::Migration
+  def change
+    create_table :message_train_unsubscribes do |t|
+      t.references :recipient, polymorphic: true
+      t.references :from, polymorphic: true
+
+      t.timestamps null: false
+    end
+
+    add_index :message_train_unsubscribes, [:recipient_type, :recipient_id], name: :unsubscribe_recipient
+    add_index :message_train_unsubscribes, [:from_type, :from_id], name: :unsubscribe_from
+    add_index :message_train_unsubscribes, [:recipient_type, :recipient_id, :from_type, :from_id], name: :unsubscribes, unique: true
+  end
+end

data/spec/dummy/db/schema.rb

@@ -11,7 +11,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 20150724142850) do
+ActiveRecord::Schema.define(version: 20151124001417) do
 
   create_table "groups", force: :cascade do |t|
     t.string   "title"
@@ -69,17 +69,34 @@ ActiveRecord::Schema.define(version: 20150724142850) do
     t.integer  "recipient_id"
     t.string   "recipient_type"
     t.integer  "message_id"
-    t.boolean  "marked_read",    default: false
-    t.boolean  "marked_trash",   default: false
-    t.boolean  "marked_deleted", default: false
-    t.boolean  "sender",         default: false
-    t.datetime "created_at",                     null: false
-    t.datetime "updated_at",                     null: false
+    t.boolean  "marked_read",           default: false
+    t.boolean  "marked_trash",          default: false
+    t.boolean  "marked_deleted",        default: false
+    t.boolean  "sender",                default: false
+    t.datetime "created_at",                            null: false
+    t.datetime "updated_at",                            null: false
+    t.integer  "received_through_id"
+    t.string   "received_through_type"
   end
 
+  add_index "message_train_receipts", ["message_id", "recipient_type", "recipient_id"], name: "message_recipient", unique: true
   add_index "message_train_receipts", ["message_id"], name: "index_message_train_receipts_on_message_id"
+  add_index "message_train_receipts", ["received_through_type", "received_through_id"], name: "index_message_train_receipts_on_received_through"
   add_index "message_train_receipts", ["recipient_type", "recipient_id"], name: "index_message_train_receipts_on_recipient"
 
+  create_table "message_train_unsubscribes", force: :cascade do |t|
+    t.integer  "recipient_id"
+    t.string   "recipient_type"
+    t.integer  "from_id"
+    t.string   "from_type"
+    t.datetime "created_at",     null: false
+    t.datetime "updated_at",     null: false
+  end
+
+  add_index "message_train_unsubscribes", ["from_type", "from_id"], name: "unsubscribe_from"
+  add_index "message_train_unsubscribes", ["recipient_type", "recipient_id", "from_type", "from_id"], name: "unsubscribes", unique: true
+  add_index "message_train_unsubscribes", ["recipient_type", "recipient_id"], name: "unsubscribe_recipient"
+
   create_table "roles", force: :cascade do |t|
     t.string   "name"
     t.integer  "resource_id"

data/spec/dummy/db/seeds/conversations.seeds.rb

@@ -2,8 +2,11 @@ after :users, :groups do
   first_user = User.friendly.find('first-user')
   second_user = User.friendly.find('second-user')
   third_user = User.friendly.find('third-user')
+  fourth_user = User.friendly.find('fourth-user')
+  superadmin_user = User.friendly.find('superadmin-user')
 
   first_group = Group.find_by_slug('first-group')
+  membered_group = Group.find_by_slug('membered-group')
 
   sent_message = FactoryGirl.create(
       :simple_message,
@@ -19,6 +22,13 @@ after :users, :groups do
       subject: 'Unread Conversation'
   )
 
+  to_many_message = FactoryGirl.create(
+      :simple_message,
+      sender: first_user,
+      recipients_to_save: { 'users' => [second_user.slug, third_user.slug, fourth_user.slug].join(', ') },
+      subject: 'To Many Conversation'
+  )
+
   ignored_message = FactoryGirl.create(
       :simple_message,
       sender: second_user,
@@ -43,6 +53,15 @@ after :users, :groups do
   )
   read_message.mark_read_for(first_user)
 
+
+  attachment_message = FactoryGirl.create(
+      :simple_message,
+      sender: second_user,
+      recipients_to_save: { 'users' => first_user.slug },
+      subject: 'Attachment Conversation',
+      generate_attachment?: true
+  )
+
   deleted_message = FactoryGirl.create(
       :simple_message,
       sender: second_user,
@@ -55,7 +74,77 @@ after :users, :groups do
       :simple_message,
       sender: first_user,
       recipients_to_save: { 'groups' => first_group.slug },
-      subject: 'Group Message'
+      subject: 'Group Announcement'
+  )
+
+  membered_group_message = FactoryGirl.create(
+      :simple_message,
+      sender: second_user,
+      recipients_to_save: { 'groups' => membered_group.slug },
+      subject: 'Membered Group Announcement'
+  )
+
+  membered_group_trashed_message = FactoryGirl.create(
+      :simple_message,
+      sender: second_user,
+      recipients_to_save: { 'groups' => membered_group.slug },
+      subject: 'Membered Group Trashed Conversation'
+  )
+  membered_group_trashed_message.mark(:trash, first_user)
+
+  membered_group_read_message = FactoryGirl.create(
+      :simple_message,
+      sender: second_user,
+      recipients_to_save: { 'groups' => membered_group.slug },
+      subject: 'Membered Group Read Conversation'
+  )
+  membered_group_read_message.mark(:read, first_user)
+
+  membered_group_ignored_message = FactoryGirl.create(
+      :simple_message,
+      sender: second_user,
+      recipients_to_save: { 'groups' => membered_group.slug },
+      subject: 'Membered Group Ignored Conversation'
+  )
+  membered_group_ignored_message.conversation.set_ignored(first_user)
+
+  membered_group_deleted_message = FactoryGirl.create(
+      :simple_message,
+      sender: second_user,
+      recipients_to_save: { 'groups' => membered_group.slug },
+      subject: 'Membered Group Deleted Conversation'
+  )
+  membered_group_deleted_message.mark(:deleted, first_user)
+
+  membered_group_draft = FactoryGirl.create(
+      :simple_message,
+      sender: second_user,
+      recipients_to_save: { 'groups' => membered_group.slug },
+      subject: 'Membered Group Draft',
+      draft: true
+  )
+
+  owned_group_draft = FactoryGirl.create(
+      :simple_message,
+      sender: first_user,
+      recipients_to_save: { 'groups' => first_group.slug },
+      subject: 'Owned Group Draft',
+      draft: true
+  )
+
+  role_message = FactoryGirl.create(
+      :simple_message,
+      sender: superadmin_user,
+      recipients_to_save: { 'roles' => 'admin' },
+      subject: 'Role Conversation'
+  )
+
+  role_draft = FactoryGirl.create(
+      :simple_message,
+      sender: superadmin_user,
+      recipients_to_save: { 'roles' => 'admin' },
+      subject: 'Role Draft',
+      draft: true
   )
 
   draft_message = FactoryGirl.create(
@@ -80,7 +169,7 @@ after :users, :groups do
   )
   FactoryGirl.create_list(:simple_message, 11,
                           conversation: long_message.conversation,
-                          sender: second_user,
-                          recipients_to_save: { 'users' => first_user.slug }
+                          sender: first_user,
+                          recipients_to_save: { 'users' => second_user.slug }
   )
 end

data/spec/dummy/db/seeds/groups.seeds.rb

@@ -1,9 +1,36 @@
 after :users do
   first_user = User.friendly.find('first-user')
+  second_user = User.friendly.find('second-user')
+  third_user = User.friendly.find('third-user')
+  fourth_user = User.friendly.find('fourth-user')
+  unsubscribed_user = User.friendly.find('unsubscribed-user')
 
   first_group = FactoryGirl.create(
     :group,
     title: 'First Group',
     owner: first_user
   )
+  fourth_user.add_role(:member, first_group)
+  unsubscribed_user.add_role(:member, first_group)
+
+  membered_group = FactoryGirl.create(
+      :group,
+      title: 'Membered Group',
+      owner: second_user
+  )
+  first_user.add_role(:member, membered_group)
+  unsubscribed_user.add_role(:member, membered_group)
+
+  empty_group = FactoryGirl.create(
+      :group,
+      title: 'Empty Group',
+      owner: first_user
+  )
+
+  unsubscribed_group = FactoryGirl.create(
+      :group,
+      title: 'Unsubscribed Group',
+      owner: second_user
+  )
+  first_user.add_role(:member, unsubscribed_group)
 end

data/spec/dummy/db/seeds/test/attachments.seeds.rb

@@ -0,0 +1,4 @@
+after 'test:conversations' do
+  FactoryGirl.create(:attachment, attachment: File.open('spec/dummy/app/assets/files/message_train/attachments/image-sample.jpg'))
+  FactoryGirl.create(:attachment, attachment: File.open('spec/dummy/app/assets/files/message_train/attachments/pdf-sample.pdf'))
+end

data/spec/dummy/db/seeds/unsubscribes.seeds.rb

@@ -0,0 +1,12 @@
+after :groups do
+  first_user = User.friendly.find('first-user')
+  unsubscribed_user = User.friendly.find('unsubscribed-user')
+  first_group = Group.find_by_title('First Group')
+  membered_group = Group.find_by_title('Membered Group')
+  unsubscribed_group = Group.find_by_title('Unsubscribed Group')
+
+  unsubscribed_user.unsubscribe_from(first_group)
+  unsubscribed_user.unsubscribe_from(membered_group)
+  unsubscribed_user.unsubscribe_from(unsubscribed_user)
+  first_user.unsubscribe_from(unsubscribed_group)
+end

data/spec/dummy/db/seeds/users.seeds.rb

@@ -13,3 +13,30 @@ third_user = FactoryGirl.create(
     display_name: 'Third User',
     email: 'third.user@example.com'
 )
+fourth_user = FactoryGirl.create(
+    :user,
+    display_name: 'Fourth User',
+    email: 'fourth.user@example.com'
+)
+unsubscribed_user = FactoryGirl.create(
+    :user,
+    display_name: 'Unsubscribed User',
+    email: 'unsubscribed.user@example.com'
+)
+silent_user = FactoryGirl.create(
+    :user,
+    display_name: 'Silent User',
+    email: 'silent.user@example.com'
+)
+superadmin_user = FactoryGirl.create(
+    :user,
+    display_name: 'Superadmin User',
+    email: 'superadmin.user@example.com'
+)
+superadmin_user.add_role(:superadmin)
+admin_user = FactoryGirl.create(
+    :user,
+    display_name: 'Admin User',
+    email: 'admin.user@example.com'
+)
+admin_user.add_role(:admin)

data/spec/factories/group.rb

@@ -4,7 +4,7 @@ FactoryGirl.define do
     description { Faker::Lorem.paragraphs([*1..5].sample).join("\n\n") }
     transient do
       owner { User.order("RANDOM()").first }
-      member_count 5
+      # member_count 5
     end
     # the after(:create) yields two values; the user instance itself and the
     # evaluator, which stores all values from the factory, including transient
@@ -12,9 +12,9 @@ FactoryGirl.define do
     # to create and we make sure the user is associated properly to the post
     after(:create) do |group, evaluator|
       evaluator.owner.add_role(:owner, group)
-      User.where('id != ?', evaluator.owner.id).order("RANDOM()").limit(evaluator.member_count).find_each do |user|
-        user.add_role(:member, group)
-      end
+      # User.where('id != ?', evaluator.owner.id).order("RANDOM()").limit(evaluator.member_count).find_each do |user|
+      #   user.add_role(:member, group)
+      # end
     end
   end
 end

data/spec/factories/message.rb

@@ -7,14 +7,18 @@ FactoryGirl.define do
     }
     recipients_to_save { { 'users' =>
        User
-         .where('id != ?', sender.id)
+         .where
+         .not(id: sender.id)
+         .where
+         .not(slug: 'silent-user')
          .order('RANDOM()')
          .limit([*1..5].sample)
          .collect { |x| x.slug }
          .join(', ')
     } }
     subject { Faker::Lorem.sentence }
-    body { Faker::Lorem.paragraphs([*1..5].sample).join("\n\n") }
+    body { "<p>#{Faker::Lorem.paragraphs([*1..5].sample).join('</p><p>')}</p>" }
+
     transient do
       generate_attachment? { [*1..100].sample >= 80 }
       generate_response? { [*1..100].sample >= 50 }
@@ -53,7 +57,10 @@ FactoryGirl.define do
       sender {
         recipient_user_ids = recipients_to_save['users'].split(',').collect{ |x| User.friendly.find(x.strip).id }
         User
-          .where('NOT (id IN (?))', recipient_user_ids )
+          .where
+          .not(id: recipient_user_ids )
+          .where
+          .not(slug: 'silent-user')
           .order('RANDOM()')
           .first
       }