message_train 0.1.7 → 0.2.0

data/message_train.gemspec

@@ -2,17 +2,17 @@
 # DO NOT EDIT THIS FILE DIRECTLY
 # Instead, edit Jeweler::Tasks in Rakefile, and run 'rake gemspec'
 # -*- encoding: utf-8 -*-
-# stub: message_train 0.1.7 ruby lib
+# stub: message_train 0.2.0 ruby lib
 
 Gem::Specification.new do |s|
   s.name = "message_train"
-  s.version = "0.1.7"
+  s.version = "0.2.0"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.require_paths = ["lib"]
   s.authors = ["Karen Lundgren"]
-  s.date = "2015-08-25"
-  s.description = "Private/public messaging for any object, such as Users or Groups"
+  s.date = "2015-11-24"
+  s.description = "Rails 4 Engine providing private/public messaging for any object, such as Users or Groups"
   s.email = "karen.e.lundgren@gmail.com"
   s.executables = ["rails"]
   s.extra_rdoc_files = [
@@ -24,6 +24,7 @@ Gem::Specification.new do |s|
     ".rspec",
     ".ruby-gemset",
     ".ruby-version",
+    ".simplecov",
     ".travis.yml",
     "Gemfile",
     "LICENSE.txt",
@@ -32,26 +33,42 @@ Gem::Specification.new do |s|
     "VERSION",
     "app/assets/javascripts/message_train.js",
     "app/assets/stylesheets/message_train.scss",
+    "app/controllers/concerns/message_train_support.rb",
     "app/controllers/message_train/application_controller.rb",
     "app/controllers/message_train/boxes_controller.rb",
     "app/controllers/message_train/conversations_controller.rb",
     "app/controllers/message_train/messages_controller.rb",
     "app/controllers/message_train/participants_controller.rb",
+    "app/controllers/message_train/unsubscribes_controller.rb",
+    "app/helpers/message_train/application_helper.rb",
+    "app/helpers/message_train/attachments_helper.rb",
     "app/helpers/message_train/boxes_helper.rb",
+    "app/helpers/message_train/collectives_helper.rb",
     "app/helpers/message_train/conversations_helper.rb",
     "app/helpers/message_train/messages_helper.rb",
+    "app/mailers/message_train/application_mailer.rb",
+    "app/mailers/message_train/previews/receipt_mailer_preview.rb",
+    "app/mailers/message_train/receipt_mailer.rb",
     "app/models/message_train/attachment.rb",
     "app/models/message_train/box.rb",
     "app/models/message_train/conversation.rb",
     "app/models/message_train/ignore.rb",
     "app/models/message_train/message.rb",
     "app/models/message_train/receipt.rb",
+    "app/models/message_train/unsubscribe.rb",
     "app/views/application/404.html.haml",
+    "app/views/layouts/mailer.html.haml",
+    "app/views/message_train/application/_attachment_fields.html.haml",
+    "app/views/message_train/application/_attachment_link.html.haml",
+    "app/views/message_train/application/_widget.html.haml",
     "app/views/message_train/application/results.json.jbuilder",
     "app/views/message_train/boxes/_dropdown_list.html.haml",
     "app/views/message_train/boxes/_list_item.html.haml",
     "app/views/message_train/boxes/_widget.html.haml",
     "app/views/message_train/boxes/show.html.haml",
+    "app/views/message_train/collectives/_dropdown_list.html.haml",
+    "app/views/message_train/collectives/_list_item.html.haml",
+    "app/views/message_train/collectives/_widget.html.haml",
     "app/views/message_train/conversations/_conversation.html.haml",
     "app/views/message_train/conversations/_deleted_toggle.html.haml",
     "app/views/message_train/conversations/_ignored_toggle.html.haml",
@@ -71,9 +88,13 @@ Gem::Specification.new do |s|
     "app/views/message_train/messages/show.json.jbuilder",
     "app/views/message_train/participants/_field.html.haml",
     "app/views/message_train/participants/_participant.json.jbuilder",
+    "app/views/message_train/participants/_prefilled_field.html.haml",
     "app/views/message_train/participants/index.json.jbuilder",
     "app/views/message_train/participants/show.json.jbuilder",
+    "app/views/message_train/receipt_mailer/notification_email.html.haml",
+    "app/views/message_train/unsubscribes/index.html.haml",
     "bin/rails",
+    "config/environment.rb",
     "config/initializers/bootstrap_pager_config.rb",
     "config/initializers/date_time.rb",
     "config/locales/en.yml",
@@ -83,6 +104,9 @@ Gem::Specification.new do |s|
     "db/migrate/20150721161144_create_message_train_attachments.rb",
     "db/migrate/20150721161940_create_message_train_receipts.rb",
     "db/migrate/20150721163838_create_message_train_ignores.rb",
+    "db/migrate/20150901183458_add_received_through_to_message_train_receipts.rb",
+    "db/migrate/20151004184347_add_unique_index_to_receipts.rb",
+    "db/migrate/20151124000820_create_message_train_unsubscribes.rb",
     "lib/generators/message_train/install/install_generator.rb",
     "lib/generators/message_train/install/templates/initializer.rb",
     "lib/generators/message_train/utils.rb",
@@ -95,12 +119,13 @@ Gem::Specification.new do |s|
     "lib/tasks/message_train_tasks.rake",
     "message_train.gemspec",
     "spec/controllers/message_train/boxes_controller_spec.rb",
+    "spec/controllers/message_train/concerns_spec.rb",
     "spec/controllers/message_train/conversations_controller_spec.rb",
     "spec/controllers/message_train/messages_controller_spec.rb",
     "spec/controllers/message_train/participants_controller_spec.rb",
+    "spec/controllers/message_train/unsubscribes_controller_spec.rb",
     "spec/dummy/.rspec",
     "spec/dummy/Rakefile",
-    "spec/dummy/app/assets/files/message_train/attachments/1917-Boys_Race_Above-Wiki.jpg",
     "spec/dummy/app/assets/files/message_train/attachments/ACD_Silverbarn's_Mayumi.jpg",
     "spec/dummy/app/assets/files/message_train/attachments/ACD_obedience.jpg",
     "spec/dummy/app/assets/files/message_train/attachments/A_Man_and_His_Dog.jpg",
@@ -120,7 +145,6 @@ Gem::Specification.new do |s|
     "spec/dummy/app/assets/files/message_train/attachments/Galgo_Spanish_male_brindle.jpg",
     "spec/dummy/app/assets/files/message_train/attachments/Golden-retriever-carlos-bei-der-dummyarbeit.jpg",
     "spec/dummy/app/assets/files/message_train/attachments/Habavk.jpg",
-    "spec/dummy/app/assets/files/message_train/attachments/Haie_rci.svg",
     "spec/dummy/app/assets/files/message_train/attachments/Hanging_18.jpg",
     "spec/dummy/app/assets/files/message_train/attachments/Hundewa.jpg",
     "spec/dummy/app/assets/files/message_train/attachments/K9ProSports_AttackOnHandler.jpg",
@@ -128,6 +152,7 @@ Gem::Specification.new do |s|
     "spec/dummy/app/assets/files/message_train/attachments/K9Pro_Civil_agitation.JPG",
     "spec/dummy/app/assets/files/message_train/attachments/TrainingSchutzhundRetrieveOverWall.jpg",
     "spec/dummy/app/assets/files/message_train/attachments/example.pdf",
+    "spec/dummy/app/assets/files/message_train/attachments/image-sample.jpg",
     "spec/dummy/app/assets/files/message_train/attachments/letterlegal5.doc",
     "spec/dummy/app/assets/files/message_train/attachments/pdf-sample.pdf",
     "spec/dummy/app/assets/files/message_train/attachments/sample.pdf",
@@ -145,6 +170,7 @@ Gem::Specification.new do |s|
     "spec/dummy/app/models/user.rb",
     "spec/dummy/app/views/layouts/_top_navigation.html.haml",
     "spec/dummy/app/views/layouts/application.html.haml",
+    "spec/dummy/app/views/pages/index.html.haml",
     "spec/dummy/bin/bundle",
     "spec/dummy/bin/rails",
     "spec/dummy/bin/rake",
@@ -163,6 +189,7 @@ Gem::Specification.new do |s|
     "spec/dummy/config/initializers/cookies_serializer.rb",
     "spec/dummy/config/initializers/devise.rb",
     "spec/dummy/config/initializers/filter_parameter_logging.rb",
+    "spec/dummy/config/initializers/high_voltage.rb",
     "spec/dummy/config/initializers/inflections.rb",
     "spec/dummy/config/initializers/message_train.rb",
     "spec/dummy/config/initializers/mime_types.rb",
@@ -174,22 +201,28 @@ Gem::Specification.new do |s|
     "spec/dummy/config/locales/en.yml",
     "spec/dummy/config/routes.rb",
     "spec/dummy/config/secrets.yml",
+    "spec/dummy/config/settings.yml",
     "spec/dummy/config/unused/temporary_i18n_debugging.rb",
     "spec/dummy/db/migrate/20150721140013_devise_create_users.rb",
     "spec/dummy/db/migrate/20150721141009_rolify_create_roles.rb",
     "spec/dummy/db/migrate/20150721141128_create_groups.rb",
     "spec/dummy/db/migrate/20150721150307_add_display_name_column_to_users.rb",
-    "spec/dummy/db/migrate/20150724142846_create_message_train_conversations.night_train.rb",
-    "spec/dummy/db/migrate/20150724142847_create_message_train_messages.night_train.rb",
-    "spec/dummy/db/migrate/20150724142848_create_message_train_attachments.night_train.rb",
-    "spec/dummy/db/migrate/20150724142849_create_message_train_receipts.night_train.rb",
-    "spec/dummy/db/migrate/20150724142850_create_message_train_ignores.night_train.rb",
+    "spec/dummy/db/migrate/20150901183629_create_message_train_conversations.message_train.rb",
+    "spec/dummy/db/migrate/20150901183630_create_message_train_messages.message_train.rb",
+    "spec/dummy/db/migrate/20150901183631_create_message_train_attachments.message_train.rb",
+    "spec/dummy/db/migrate/20150901183632_create_message_train_receipts.message_train.rb",
+    "spec/dummy/db/migrate/20150901183633_create_message_train_ignores.message_train.rb",
+    "spec/dummy/db/migrate/20150901183634_add_received_through_to_message_train_receipts.message_train.rb",
+    "spec/dummy/db/migrate/20151004184519_add_unique_index_to_receipts.message_train.rb",
+    "spec/dummy/db/migrate/20151124001417_create_message_train_unsubscribes.message_train.rb",
     "spec/dummy/db/schema.rb",
     "spec/dummy/db/seeds.rb",
     "spec/dummy/db/seeds/conversations.seeds.rb",
     "spec/dummy/db/seeds/development/conversations.seeds.rb",
     "spec/dummy/db/seeds/groups.seeds.rb",
+    "spec/dummy/db/seeds/test/attachments.seeds.rb",
     "spec/dummy/db/seeds/test/conversations.seeds.rb",
+    "spec/dummy/db/seeds/unsubscribes.seeds.rb",
     "spec/dummy/db/seeds/users.seeds.rb",
     "spec/dummy/db/test.sqlite3",
     "spec/dummy/lib/assets/.keep",
@@ -197,7 +230,6 @@ Gem::Specification.new do |s|
     "spec/dummy/public/404.html",
     "spec/dummy/public/422.html",
     "spec/dummy/public/500.html",
-    "spec/dummy/public/capybara.html",
     "spec/dummy/public/favicon.ico",
     "spec/factories/attachment.rb",
     "spec/factories/group.rb",
@@ -206,7 +238,13 @@ Gem::Specification.new do |s|
     "spec/features/boxes_spec.rb",
     "spec/features/conversations_spec.rb",
     "spec/features/messages_spec.rb",
+    "spec/features/unsubscribes_spec.rb",
+    "spec/helpers/message_train/application_helper_spec.rb",
+    "spec/helpers/message_train/attachment_helper_spec.rb",
     "spec/helpers/message_train/boxes_helper_spec.rb",
+    "spec/helpers/message_train/collectives_helper_spec.rb",
+    "spec/helpers/message_train/conversations_helper_spec.rb",
+    "spec/helpers/message_train/messages_helper_spec.rb",
     "spec/message_train_spec.rb",
     "spec/models/group_spec.rb",
     "spec/models/message_train/attachment_spec.rb",
@@ -215,9 +253,12 @@ Gem::Specification.new do |s|
     "spec/models/message_train/ignore_spec.rb",
     "spec/models/message_train/message_spec.rb",
     "spec/models/message_train/receipt_spec.rb",
+    "spec/models/message_train/unsubscribe_spec.rb",
+    "spec/models/role_spec.rb",
     "spec/models/user_spec.rb",
     "spec/rails_helper.rb",
     "spec/spec_helper.rb",
+    "spec/support/attachments.rb",
     "spec/support/controller_behaviors.rb",
     "spec/support/controller_macros.rb",
     "spec/support/conversations.rb",
@@ -225,16 +266,19 @@ Gem::Specification.new do |s|
     "spec/support/feature_behaviors.rb",
     "spec/support/groups.rb",
     "spec/support/loaded_site.rb",
+    "spec/support/messages.rb",
+    "spec/support/roles.rb",
     "spec/support/shared_connection.rb",
     "spec/support/users.rb",
     "spec/support/utilities.rb",
+    "spec/support/wysihtml5_helper.rb",
     "vendor/assets/javascripts/bootstrap-tags.js",
     "vendor/assets/stylesheets/bootstrap-tags.css"
   ]
   s.homepage = "http://www.gemvein.com/museum/cases/message_train"
   s.licenses = ["MIT"]
   s.rubygems_version = "2.4.5"
-  s.summary = "Messaging for any object"
+  s.summary = "Rails 4 Engine providing messaging for any object"
 
   if s.respond_to? :specification_version then
     s.specification_version = 4
@@ -252,6 +296,8 @@ Gem::Specification.new do |s|
       s.add_runtime_dependency(%q<bootstrap_leather>, ["~> 0.8"])
       s.add_runtime_dependency(%q<bootstrap_pager>, ["~> 0.10"])
       s.add_runtime_dependency(%q<bootstrap_form>, ["~> 2.3"])
+      s.add_runtime_dependency(%q<bootstrap-wysihtml5-rails>, ["> 0.3"])
+      s.add_runtime_dependency(%q<cocoon>, ["~> 1.2"])
       s.add_development_dependency(%q<rdoc>, ["~> 3.12"])
       s.add_development_dependency(%q<bundler>, ["~> 1.0"])
       s.add_development_dependency(%q<jeweler>, ["~> 2.0"])
@@ -265,6 +311,7 @@ Gem::Specification.new do |s|
       s.add_development_dependency(%q<seedbank>, ["~> 0.3"])
       s.add_development_dependency(%q<friendly_id>, ["~> 5"])
       s.add_development_dependency(%q<byebug>, ["~> 5"])
+      s.add_development_dependency(%q<high_voltage>, ["~> 2.4"])
     else
       s.add_dependency(%q<rails>, ["~> 4"])
       s.add_dependency(%q<paperclip>, ["~> 4.3"])
@@ -278,6 +325,8 @@ Gem::Specification.new do |s|
       s.add_dependency(%q<bootstrap_leather>, ["~> 0.8"])
       s.add_dependency(%q<bootstrap_pager>, ["~> 0.10"])
       s.add_dependency(%q<bootstrap_form>, ["~> 2.3"])
+      s.add_dependency(%q<bootstrap-wysihtml5-rails>, ["> 0.3"])
+      s.add_dependency(%q<cocoon>, ["~> 1.2"])
       s.add_dependency(%q<rdoc>, ["~> 3.12"])
       s.add_dependency(%q<bundler>, ["~> 1.0"])
       s.add_dependency(%q<jeweler>, ["~> 2.0"])
@@ -291,6 +340,7 @@ Gem::Specification.new do |s|
       s.add_dependency(%q<seedbank>, ["~> 0.3"])
       s.add_dependency(%q<friendly_id>, ["~> 5"])
       s.add_dependency(%q<byebug>, ["~> 5"])
+      s.add_dependency(%q<high_voltage>, ["~> 2.4"])
     end
   else
     s.add_dependency(%q<rails>, ["~> 4"])
@@ -305,6 +355,8 @@ Gem::Specification.new do |s|
     s.add_dependency(%q<bootstrap_leather>, ["~> 0.8"])
     s.add_dependency(%q<bootstrap_pager>, ["~> 0.10"])
     s.add_dependency(%q<bootstrap_form>, ["~> 2.3"])
+    s.add_dependency(%q<bootstrap-wysihtml5-rails>, ["> 0.3"])
+    s.add_dependency(%q<cocoon>, ["~> 1.2"])
     s.add_dependency(%q<rdoc>, ["~> 3.12"])
     s.add_dependency(%q<bundler>, ["~> 1.0"])
     s.add_dependency(%q<jeweler>, ["~> 2.0"])
@@ -318,6 +370,7 @@ Gem::Specification.new do |s|
     s.add_dependency(%q<seedbank>, ["~> 0.3"])
     s.add_dependency(%q<friendly_id>, ["~> 5"])
     s.add_dependency(%q<byebug>, ["~> 5"])
+    s.add_dependency(%q<high_voltage>, ["~> 2.4"])
   end
 end
 

data/spec/controllers/message_train/boxes_controller_spec.rb

@@ -7,7 +7,8 @@ describe MessageTrain::BoxesController do
 
   let(:valid_params) { { 'conversations' => {unread_conversation.id.to_s => unread_conversation.id} } }
   let(:invalid_params) { { 'conversations' => {'999' => 999} } }
-  
+  let(:disallowed_params) { { 'conversations' => {someone_elses_conversation.id.to_s => someone_elses_conversation.id} } }
+
   before do
     login_user first_user
   end
@@ -35,7 +36,13 @@ describe MessageTrain::BoxesController do
       before do
         put :update, division: 'in', mark_to_set: 'read', objects: invalid_params
       end
-      it_should_behave_like 'a page with an error message', "Conversation 999 not found in box"
+      it_should_behave_like 'a 404 Not Found error'
+    end
+    describe 'with disallowed params' do
+      before do
+        put :update, division: 'in', mark_to_set: 'read', objects: disallowed_params
+      end
+      it_should_behave_like 'a page with an error message matching', /Access to Conversation \d+ denied/
     end
     describe 'without params' do
       before do
@@ -56,7 +63,7 @@ describe MessageTrain::BoxesController do
       before do
         delete :destroy, division: 'in', mark_to_set: 'ignore', objects: invalid_params
       end
-      it_should_behave_like 'a page with an error message', "Conversation 999 not found in box"
+      it_should_behave_like 'a 404 Not Found error'
     end
     describe 'without params' do
       before do

data/spec/controllers/message_train/concerns_spec.rb

@@ -0,0 +1,40 @@
+require 'rails_helper'
+
+describe MessageTrain::BoxesController do
+  include_context 'loaded site'
+  include ControllerMacros
+  routes { MessageTrain::Engine.routes }
+
+  describe "GET #show"  do
+    describe 'when not logged in' do
+      before do
+        get :show, division: 'in'
+      end
+      it_should_behave_like 'a redirect with notice', '/users/sign_in', 'You must sign in or sign up to continue.'
+    end
+
+    describe 'when neither sending nor receiving is allowed' do
+      before do
+        login_user third_user
+        get :show, division: 'in', collective_id: 'groups:first-group'
+      end
+      it_should_behave_like 'a redirect with error', '/', 'Access to that box denied'
+    end
+
+    describe 'when only sending is allowed' do
+      before do
+        login_user first_user
+        get :show, division: 'in', collective_id: 'groups:first-group'
+      end
+      it_should_behave_like 'a redirect with error', '/collectives/groups:first-group/box/sent', 'Access to that box denied'
+    end
+
+    describe 'when only receiving is allowed' do
+      before do
+        login_user first_user
+        get :show, division: 'sent', collective_id: 'groups:membered-group'
+      end
+      it_should_behave_like 'a redirect with error', '/collectives/groups:membered-group/box/in', 'Access to that box denied'
+    end
+  end
+end

data/spec/controllers/message_train/conversations_controller_spec.rb

@@ -5,7 +5,7 @@ describe MessageTrain::ConversationsController do
   include ControllerMacros
   routes { MessageTrain::Engine.routes }
 
-  let(:valid_params) { { 'messages' => {unread_conversation.messages.first.id.to_s => unread_conversation.messages.first.id} } }
+  let(:valid_params) { { 'messages' => {unread_message.id.to_s => unread_message.id} } }
   let(:invalid_params) { { 'messages' => {'999999' => 999999} } }
 
   before do
@@ -30,7 +30,7 @@ describe MessageTrain::ConversationsController do
 
     context 'loads messages into @messages' do
       subject { assigns(:messages) }
-      it { should include unread_conversation.messages.first }
+      it { should include unread_message }
     end
   end
 
@@ -39,7 +39,7 @@ describe MessageTrain::ConversationsController do
       before do
         put :update, box_division: 'in', id: unread_conversation.id, mark_to_set: 'read', objects: invalid_params
       end
-      it_should_behave_like 'a page with an error message', "Message 999999 not found in box"
+      it_should_behave_like 'a 404 Not Found error'
     end
     describe 'without params' do
       before do

data/spec/controllers/message_train/messages_controller_spec.rb

@@ -20,15 +20,13 @@ describe MessageTrain::MessagesController do
     { subject: nil }
   }
 
-  let(:draft_message) { draft_conversation.messages.first }
-
   before do
     login_user first_user
   end
 
   describe "GET #show" do
     before do
-      get :show, box_division: 'in', id: unread_conversation.messages.first.id, format: :json
+      get :show, box_division: 'in', id: unread_message.id, format: :json
     end
     it_should_behave_like 'a successful page', which_renders: 'show'
 
@@ -39,13 +37,13 @@ describe MessageTrain::MessagesController do
 
     context 'loads message into @message' do
       subject { assigns(:message) }
-      it { should eq unread_conversation.messages.first }
+      it { should eq unread_message }
     end
   end
 
   describe "GET #new" do
     before do
-      get :new, box_division: 'in', conversation_id: unread_conversation.messages.first.conversation_id
+      get :new, box_division: 'in', conversation_id: unread_message.conversation_id
     end
     it_should_behave_like 'a successful page', which_renders: 'new'
 
@@ -64,14 +62,14 @@ describe MessageTrain::MessagesController do
   describe "GET #edit" do
     describe 'as someone other than the owner' do
       before do
-        get :edit, box_division: 'in', id: someone_elses_conversation.messages.first.id
+        get :edit, box_division: 'in', id: someone_elses_message.id
       end
       it_should_behave_like 'a 404 Not Found error'
     end
     describe 'as the owner' do
       describe 'when message is a not a draft' do
         before do
-          get :edit, box_division: 'in', id: unread_conversation.messages.first.id
+          get :edit, box_division: 'in', id: unread_message.id
         end
         it_should_behave_like 'a 404 Not Found error'
       end
@@ -117,9 +115,18 @@ describe MessageTrain::MessagesController do
       end
     end
     describe 'with valid attributes, counting' do
+      it 'results in a new conversation' do
+        expect { post :create, box_division: 'in', message: valid_attributes }.to change { MessageTrain::Conversation.count }.by(1)
+      end
       it 'results in a new message' do
         expect { post :create, box_division: 'in', message: valid_attributes }.to change { MessageTrain::Message.count }.by(1)
       end
+      it 'results in new receipts' do
+        expect { post :create, box_division: 'in', message: valid_attributes }.to change { MessageTrain::Receipt.count }.by(3)
+      end
+      it 'results in email notifications' do
+        expect { post :create, box_division: 'in', message: valid_attributes }.to change { ActionMailer::Base.deliveries.count }.by(2)
+      end
     end
     describe 'with invalid params' do
       before do
@@ -148,7 +155,7 @@ describe MessageTrain::MessagesController do
   describe "PUT #update" do
     describe 'as someone other than the owner' do
       before do
-        put :update, box_division: 'in', id: someone_elses_conversation.messages.first.id, message: valid_attributes
+        put :update, box_division: 'in', id: someone_elses_message.id, message: valid_attributes
       end
       it_should_behave_like 'a 404 Not Found error'
     end
@@ -177,30 +184,56 @@ describe MessageTrain::MessagesController do
       describe 'with valid params' do
         describe 'when not a draft' do
           before do
-            put :update, box_division: 'in', id: sent_conversation.messages.first.id, message: valid_attributes
+            put :update, box_division: 'in', id: sent_message.id, message: valid_attributes
           end
           it_should_behave_like 'a 404 Not Found error'
         end
         describe 'when a draft' do
-          before do
-            put :update, box_division: 'in', id: draft_message.id, message: valid_attributes
+          describe 'and updating to not draft' do
+
+            before do
+              put :update, box_division: 'in', id: draft_message.id, message: valid_attributes
+            end
+            it_should_behave_like 'a redirect to', '/box/sent'
+            it_should_behave_like 'a response without error'
+
+            context 'updates @message' do
+              subject { draft_message.reload }
+              its(:subject) { should eq 'Test Subject' }
+            end
+
+            context "sets no alert" do
+              subject { flash[:alert] }
+              it { should be nil }
+            end
+
+            context "sets the flash with a notice of the message's update" do
+              subject { flash[:notice] }
+              it { should eq 'Message sent.'}
+            end
           end
-          it_should_behave_like 'a redirect to', '/box/sent'
-          it_should_behave_like 'a response without error'
-
-          context 'updates @message' do
-            subject { draft_message.reload }
-            its(:subject) { should eq 'Test Subject' }
-          end
-
-          context "sets no alert" do
-            subject { flash[:alert] }
-            it { should be nil }
-          end
-
-          context "sets the flash with a notice of the message's update" do
-            subject { flash[:notice] }
-            it { should eq 'Message sent.'}
+          describe 'and updating but keeping as a draft' do
+            let(:edited_draft_message) { { id: draft_message.id, subject: 'Still a draft', draft: 'Save As Draft' } }
+            before do
+              put :update, box_division: 'in', id: draft_message.id, message: edited_draft_message
+            end
+            it_should_behave_like 'a redirect matching', %r(/box/in/conversations/\d+)
+            it_should_behave_like 'a response without error'
+
+            context 'updates @message' do
+              subject { draft_message.reload }
+              its(:subject) { should eq 'Still a draft' }
+            end
+
+            context "sets no notice" do
+              subject { flash[:notice] }
+              it { should be nil }
+            end
+
+            context "sets the flash with a alert of the message's update" do
+              subject { flash[:alert] }
+              it { should eq 'Message saved as draft.'}
+            end
           end
         end
       end

data/spec/controllers/message_train/participants_controller_spec.rb

@@ -10,14 +10,49 @@ describe MessageTrain::ParticipantsController do
   end
 
   describe "GET #index" do
-    before do
-      get :index, box_division: 'in', model: 'users', format: :json
+    describe 'with model to users' do
+      before do
+        get :index, box_division: 'in', model: 'users', format: :json
+      end
+      it_should_behave_like 'a successful page', which_renders: 'index'
+
+      context 'loads participants into @participants' do
+        subject { assigns(:participants) }
+        its(:first) { should be_a User }
+      end
     end
-    it_should_behave_like 'a successful page', which_renders: 'index'
+    describe 'with model set to groups' do
+      describe 'given model responds to fallback method' do
+        before do
+          get :index, box_division: 'in', model: 'groups', format: :json
+        end
+        it_should_behave_like 'a successful page', which_renders: 'index'
 
-    context 'loads participants into @participants' do
-      subject { assigns(:participants) }
-      its(:first) { should be_a User }
+        context 'loads participants into @participants' do
+          subject { assigns(:participants) }
+          its(:first) { should be_a Group }
+          it { should have_exactly(2).items }
+        end
+      end
+      describe 'given model does not fallback method' do
+        before do
+          MessageTrain.configuration.address_book_method = nil
+          get :index, box_division: 'in', model: 'groups', format: :json
+        end
+        it_should_behave_like 'a successful page', which_renders: 'index'
+
+        context 'loads participants into @participants' do
+          subject { assigns(:participants) }
+          its(:first) { should be_a Group }
+          it { should have_exactly(4).items }
+        end
+      end
+    end
+    describe 'with no model set' do
+      before do
+        get :index, box_division: 'in', model: '', format: :json
+      end
+      it_should_behave_like 'a 404 Not Found error'
     end
   end
 

data/spec/controllers/message_train/unsubscribes_controller_spec.rb

@@ -0,0 +1,56 @@
+require 'rails_helper'
+
+describe MessageTrain::UnsubscribesController do
+  include_context 'loaded site'
+  include ControllerMacros
+  routes { MessageTrain::Engine.routes }
+
+  let(:valid_attributes) {
+    { from_type: 'Group', from_id: membered_group.id }
+  }
+
+  let(:invalid_attributes) {
+    { from_type: 'Group', from_id: empty_group.id }
+  }
+
+  before do
+    login_user first_user
+  end
+
+  describe "GET #index" do
+    before do
+      get :index
+    end
+    it_should_behave_like 'a successful page', which_renders: 'index'
+
+    context 'loads subscriptions into @subscriptions' do
+      subject { assigns(:subscriptions) }
+      it { should be_an Array }
+      it { should have_at_least(3).items }
+    end
+  end
+
+  describe "POST #create" do
+    describe 'with invalid attributes' do
+      before do
+        post :create, unsubscribe: invalid_attributes
+      end
+      it_should_behave_like 'a 404 Not Found error'
+    end
+    describe 'with valid attributes' do
+      before do
+        post :create, unsubscribe: valid_attributes
+      end
+      it_should_behave_like 'a redirect with a message', '/unsubscribes', notice: 'You are now unsubscribed from Membered Group, which means that you will not be notified by email of any messages received by that Group.'
+      it_should_behave_like 'a response without error'
+    end
+  end
+
+  describe "DELETE #destroy" do
+    before do
+      delete :destroy, id: first_user.unsubscribes.where(from_type: 'Group', from_id: unsubscribed_group.id).first.id
+    end
+    it_should_behave_like 'a redirect with a message', '/unsubscribes', notice: 'You are no longer unsubscribed from Unsubscribed Group, which means that you will now be notified by email of any messages received in that Group.'
+  end
+
+end