message_train 0.1.7 → 0.2.0

data/spec/features/boxes_spec.rb

@@ -2,47 +2,174 @@ require 'rails_helper'
 RSpec.feature 'Boxes' do
   include_context 'loaded site'
 
-  before do
-    login_as first_user
-  end
-  describe 'Showing' do
-    describe 'at /box/in' do
-      before do
-        visit '/box/in'
-      end
-      it_behaves_like 'a bootstrap page listing a collection of items', MessageTrain::Conversation, plural_title: 'Inbox', minimum: 2
+  describe 'When logged in as first-user' do
+    before do
+      login_as first_user
     end
-  end
-  describe 'Marking', js: true do
-    describe 'at /box/in' do
-      describe 'without checking anything' do
-        before do
-          visit '/box/in'
-          click_button 'Mark'
-          click_link 'mark-read'
-        end
-        it_behaves_like 'a bootstrap page with an alert', 'warning', 'Nothing to do'
-      end
-      describe 'after checking a box' do
-        describe 'Marking Read' do
+    describe 'Without a collective' do
+
+      describe 'Showing' do
+        describe 'at /box/in' do
           before do
             visit '/box/in'
             click_link 'Last'
-            check "objects_conversations_#{unread_conversation.id.to_s}"
-            click_button 'Mark'
-            click_link 'mark-read'
           end
-          it_behaves_like 'a bootstrap page with an alert', 'info', 'Update successful'
+          it_behaves_like 'a bootstrap page listing a collection of items', MessageTrain::Conversation, plural_title: 'Inbox', minimum: 2
+
+          describe 'does not list Roles dropdown' do
+            subject { page }
+            it { should_not have_css('li.dropdown a.dropdown-toggle', text: 'Roles') }
+          end
+
+          describe 'shows the attachment icon on the correct conversation' do
+            before do
+              if page.has_no_css?("#message_train_conversation_#{attachment_conversation.id.to_s}")
+                click_link 'Prev'
+              end
+            end
+            subject { page }
+            it { should have_css("#message_train_conversation_#{attachment_conversation.id.to_s} .glyphicon-paperclip")}
+          end
+
+          describe 'shows a link to compose' do
+            before do
+              within ".content" do
+                find('.compose').click
+              end
+            end
+            it_behaves_like 'a bootstrap page', title: 'New Message'
+          end
         end
-        describe 'Marking Ignored' do
+      end
+
+      describe 'Marking', js: true do
+        describe 'at /box/in' do
+          describe 'using the buttons on each conversation' do
+            before do
+              visit '/box/in'
+              click_link 'Last'
+              find("#message_train_conversation_#{unread_conversation.id.to_s}").hover
+              within "#message_train_conversation_#{unread_conversation.id.to_s}" do
+                accept_confirm do
+                  find('.trash-toggle').click
+                end
+              end
+            end
+            it_behaves_like 'a bootstrap page with an alert', 'info', 'Update successful'
+          end
+          describe 'without checking anything' do
+            before do
+              visit '/box/in'
+              click_button 'Mark'
+              click_link 'mark-read'
+            end
+            it_behaves_like 'a bootstrap page with an alert', 'warning', 'Nothing to do'
+          end
+          describe 'after checking a box' do
+            describe 'Marking Read' do
+              before do
+                visit '/box/in'
+                click_link 'Last'
+                check "objects_conversations_#{unread_conversation.id.to_s}"
+                click_button 'Mark'
+                click_link 'mark-read'
+              end
+              it_behaves_like 'a bootstrap page with an alert', 'info', 'Update successful'
+            end
+            describe 'Marking Ignored' do
+              before do
+                visit '/box/in'
+                click_link 'Last'
+                check "objects_conversations_#{unread_conversation.id.to_s}"
+                click_button 'Mark'
+                click_link 'mark-ignored'
+              end
+              it_behaves_like 'a bootstrap page with an alert', 'info', 'Update successful'
+            end
+          end
+        end
+
+      end
+    end
+
+    describe 'Within a collective' do
+
+      describe 'Showing' do
+        describe 'at /collectives/groups:membered-group/box/in' do
           before do
-            visit '/box/in'
-            click_link 'Last'
-            check "objects_conversations_#{unread_conversation.id.to_s}"
-            click_button 'Mark'
-            click_link 'mark-ignored'
+            visit '/collectives/groups:membered-group/box/in'
           end
-          it_behaves_like 'a bootstrap page with an alert', 'info', 'Update successful'
+          it_behaves_like 'a bootstrap page listing a collection of items', MessageTrain::Conversation, plural_title: 'Inbox', minimum: 1
+        end
+      end
+
+      describe 'Marking', js: true do
+        describe 'at /collectives/groups:membered-group/box/in' do
+          describe 'without checking anything' do
+            before do
+              visit '/collectives/groups:membered-group/box/in'
+              click_button 'Mark'
+              click_link 'mark-read'
+            end
+            it_behaves_like 'a bootstrap page with an alert', 'warning', 'Nothing to do'
+          end
+          describe 'after checking a box' do
+            describe 'Marking Read' do
+              before do
+                visit '/collectives/groups:membered-group/box/in'
+                check "objects_conversations_#{membered_group_conversation.id.to_s}"
+                click_button 'Mark'
+                click_link 'mark-read'
+              end
+              it_behaves_like 'a bootstrap page with an alert', 'info', 'Update successful'
+            end
+            describe 'Marking Ignored' do
+              before do
+                visit '/collectives/groups:membered-group/box/in'
+                check "objects_conversations_#{membered_group_conversation.id.to_s}"
+                click_button 'Mark'
+                click_link 'mark-ignored'
+              end
+              it_behaves_like 'a bootstrap page with an alert', 'info', 'Update successful'
+            end
+          end
+        end
+
+      end
+    end
+  end
+  describe 'When logged in as admin-user' do
+    before do
+      login_as admin_user
+    end
+
+    describe 'Showing' do
+      describe 'at /box/in' do
+        before do
+          visit '/box/in'
+        end
+
+        describe 'lists Roles dropdown' do
+          subject { page }
+          it { should have_css('li.dropdown a.dropdown-toggle', text: 'Roles') }
+        end
+      end
+    end
+  end
+  describe 'When logged in as superadmin-user' do
+    before do
+      login_as superadmin_user
+    end
+
+    describe 'Showing' do
+      describe 'at /box/in' do
+        before do
+          visit '/box/in'
+        end
+
+        describe 'lists Roles dropdown' do
+          subject { page }
+          it { should have_css('li.dropdown a.dropdown-toggle', text: 'Roles') }
         end
       end
     end

data/spec/features/conversations_spec.rb

@@ -8,23 +8,30 @@ RSpec.feature 'Conversations' do
   describe 'Showing' do
     describe 'at /box/in/conversations/:id' do
       before do
-        visit '/box/in/conversations/'+unread_conversation.id.to_s
+        visit '/box/in/conversations/' + unread_conversation.id.to_s
       end
       it_behaves_like 'a bootstrap page showing an item', MessageTrain::Conversation, 'Unread Conversation'
+
+      describe 'shows a reply link' do
+        before do
+          click_link 'Reply'
+        end
+        it_behaves_like 'a bootstrap page', title: 'Reply'
+      end
     end
   end
   describe 'Marking', js: true do
     describe 'at /box/in/conversations/:id' do
       describe 'Marking Read' do
         before do
-          visit '/box/in/conversations/'+unread_conversation.id.to_s
-          click_link "mark_read_#{unread_conversation.messages.first.id.to_s}"
+          visit '/box/in/conversations/' + unread_conversation.id.to_s
+          click_link "mark_read_#{unread_message.id.to_s}"
         end
         it_behaves_like 'a bootstrap page with an alert', 'info', 'Update successful'
       end
       describe 'Marking Ignored' do
         before do
-          visit '/box/in/conversations/'+unread_conversation.id.to_s
+          visit '/box/in/conversations/' + unread_conversation.id.to_s
           accept_confirm do
             click_link 'Mark as Ignored'
           end

data/spec/features/messages_spec.rb

@@ -1,6 +1,7 @@
 require 'rails_helper'
 RSpec.feature 'Messages' do
   include_context 'loaded site'
+  include Wysihtml5Helper
 
   before do
     login_as first_user
@@ -12,7 +13,7 @@ RSpec.feature 'Messages' do
         recipient_input = find(:css, "#message_recipients_to_save_users .tags-input")
         recipient_input.set("")
         fill_in 'Subject', with: 'This is a draft.'
-        fill_in 'Body', with: 'This is the body.'
+        fill_in_html 'Body', with: 'This is the body.'
         click_button 'Send'
       end
       it_behaves_like 'a bootstrap page with an alert', 'warning', 'Message saved as draft.'
@@ -23,11 +24,25 @@ RSpec.feature 'Messages' do
       before do
         visit '/box/in/messages/new/'
         recipient_input = find(:css, "#message_recipients_to_save_users .tags-input")
-        recipient_input.set("sec")
+        recipient_input.set("sec") # Should auto-complete to second-user with the following two key-presses
         recipient_input.native.send_keys :arrow_down
         recipient_input.native.send_keys :return
         fill_in 'Subject', with: 'This is the subject.'
-        fill_in 'Body', with: 'This is the body.'
+        fill_in_html 'Body', with: 'This is the body.'
+        click_link 'add-attachment'
+        click_link 'add-attachment'
+        click_link 'add-attachment'
+        within '#attachments .nested-fields:nth-child(1)' do
+          attach_file 'Attachment', File.absolute_path('./spec/dummy/app/assets/files/message_train/attachments/example.pdf')
+        end
+        within '#attachments .nested-fields:nth-child(2)' do
+          attach_file 'Attachment', File.absolute_path('./spec/dummy/app/assets/files/message_train/attachments/Bambisj.jpg')
+        end
+        within '#attachments .nested-fields:nth-child(3)' do
+          accept_confirm do
+            find('.remove_fields').click
+          end
+        end
         click_button 'Send'
       end
       it_behaves_like 'a bootstrap page with an alert', 'info', 'Message sent.'
@@ -38,12 +53,11 @@ RSpec.feature 'Messages' do
       before do
         visit "/box/in/conversations/#{draft_conversation.id}"
         recipient_input = find(:css, "#message_recipients_to_save_users .tags-input")
-        recipient_input.set("sec")
+        recipient_input.set("sec") # Should auto-complete to second-user with the following two key-presses
         recipient_input.native.send_keys :arrow_down
         recipient_input.native.send_keys :return
         fill_in 'Subject', with: 'This is the subject.'
-        fill_in 'Body', with: 'This is the body.'
-        find(:css, "#message_draft").set(false)
+        fill_in_html 'Body', with: 'This is the body.'
         click_button 'Send'
       end
       it_behaves_like 'a bootstrap page with an alert', 'info', 'Message sent.'

data/spec/features/unsubscribes_spec.rb

@@ -0,0 +1,38 @@
+require 'rails_helper'
+RSpec.feature 'Unsubscribes' do
+  include_context 'loaded site'
+
+  before do
+    login_as first_user
+  end
+  describe 'Index' do
+    describe 'at /unsubscribes' do
+      before do
+        visit '/unsubscribes'
+      end
+      it_behaves_like 'a bootstrap page', title: 'Manage Your Email Notifications'
+      context "displays a list of #subscriptions in .subscription" do
+        subject { page }
+        it { should have_css("#subscriptions .subscription", minimum: 3) }
+      end
+    end
+  end
+  describe 'Unsubscribing' do
+    describe 'at /unsubscribes' do
+      before do
+        visit '/unsubscribes'
+        find("#unsubscribe-group-#{membered_group.id.to_s}").click
+      end
+      it_behaves_like 'a bootstrap page with an alert', 'info', 'You are now unsubscribed from Membered Group, which means that you will not be notified by email of any messages received by that Group.'
+    end
+  end
+  describe 'Removing an Unsubscribe' do
+    describe 'at /unsubscribes' do
+      before do
+        visit '/unsubscribes'
+        find("#remove-unsubscribe-#{unsubscribed_group.id.to_s}").click
+      end
+      it_behaves_like 'a bootstrap page with an alert', 'info', '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
+end

data/spec/helpers/message_train/application_helper_spec.rb

@@ -0,0 +1,60 @@
+require 'rails_helper'
+
+describe MessageTrain::ApplicationHelper do
+  include_context 'loaded site'
+  include ControllerMacros
+  helper MessageTrain::BoxesHelper
+  helper MessageTrain::CollectivesHelper
+
+  before do
+    login_user first_user
+  end
+
+  describe '#message_train_widget' do
+    context 'when @collective is nil' do
+      before do
+        assign(:collective, nil)
+        assign(:box_user, first_user)
+        helper.message_train_widget
+      end
+      subject { helper.render_widgets }
+      it { should have_tag 'h3', text: /Messages/ }
+      it { should have_tag 'ul', with: { class: 'list-group' } }
+      it { should have_tag 'li', with: { class: 'list-group-item' }, count: 6 }
+    end
+    context 'when @collective is set' do
+      before do
+        assign(:collective, membered_group)
+        assign(:box_user, first_user)
+        helper.message_train_widget
+      end
+      subject { helper.render_widgets }
+      it { should have_tag 'h3', text: /Membered Group Messages/ }
+      it { should have_tag 'ul', with: { class: 'list-group' } }
+      it { should have_tag 'li', with: { class: 'list-group-item' }, count: 4 }
+    end
+  end
+
+  describe '#fuzzy_date' do
+    context 'when less than a minute ago' do
+      subject { helper.fuzzy_date(30.seconds.ago) }
+      it { should eq 'Just Now' }
+    end
+    context 'when less than an day ago' do
+      subject { helper.fuzzy_date(2.hours.ago) }
+      it { should eq 2.hours.ago.strftime('%l:%M %p') }
+    end
+    context 'when less than a week ago' do
+      subject { helper.fuzzy_date(2.days.ago) }
+      it { should eq 2.days.ago.strftime('%a %l:%M %p') }
+    end
+    context 'when less than a year ago' do
+      subject { helper.fuzzy_date(2.months.ago) }
+      it { should eq 2.months.ago.strftime('%b %-d') }
+    end
+    context 'when more than a year ago' do
+      subject { helper.fuzzy_date(2.years.ago) }
+      it { should eq 2.years.ago.strftime('%b %-d, %Y') }
+    end
+  end
+end

data/spec/helpers/message_train/attachment_helper_spec.rb

@@ -0,0 +1,35 @@
+require 'rails_helper'
+
+describe MessageTrain::AttachmentsHelper do
+  include_context 'loaded site'
+
+  describe '#attachment_icon' do
+    context 'when attachment is an image' do
+      subject { helper.attachment_icon(image_attachment) }
+      it { should have_tag :img, with: { src: image_attachment.attachment.url(:thumb) } }
+    end
+    context 'when attachment is a file' do
+      subject { helper.attachment_icon(pdf_attachment) }
+      it { should have_tag :span, with: { class: 'glyphicon-save-file' } }
+    end
+  end
+
+  describe '#attachment_link' do
+    context 'when attachment is an image' do
+      subject { helper.attachment_link(image_attachment) }
+      it { should have_tag :a, with: {
+                                 href: '#',
+                                 class: 'thumbnail',
+                                 'data-toggle' => 'modal',
+                                 'data-target' => '#attachment_preview',
+                                 'data-src' => image_attachment.attachment.url(:large),
+                                 'data-original' => image_attachment.attachment.url(:original),
+                                 'data-text' => 'Click for Original'
+                             } }
+    end
+    context 'when attachment is a file' do
+      subject { helper.attachment_link(pdf_attachment) }
+      it { should have_tag :a, with: { href: pdf_attachment.attachment.url, class: 'thumbnail' } }
+    end
+  end
+end

data/spec/helpers/message_train/boxes_helper_spec.rb

@@ -7,6 +7,7 @@ describe MessageTrain::BoxesHelper do
   before do
     login_user first_user
   end
+
   context '#box_nav_item' do
     subject { helper.box_nav_item(first_user.box(:in)) }
     it { should have_tag 'li', text: /Inbox/ }
@@ -20,16 +21,21 @@ describe MessageTrain::BoxesHelper do
   end
 
   context '#boxes_widget' do
-    subject { helper.boxes_widget }
+    subject { helper.boxes_widget(first_user) }
     it { should have_tag 'h3', text: /Messages/ }
     it { should have_tag 'ul', with: { class: 'list-group' } }
     it { should have_tag 'li', with: { class: 'list-group-item' }, count: 6 }
   end
 
-  context '#boxes_dropdown_list' do
-    subject { helper.boxes_dropdown_list }
-    it { should have_tag 'ul', with: { class: 'dropdown-menu' } }
-    it { should have_tag 'li', minimum: 5 }
+  describe '#boxes_dropdown_list' do
+    subject { helper.boxes_dropdown_list(first_user) }
+    it { should have_tag 'ul', with: { class: 'dropdown-menu' }, count: 1 }
+    it { should have_tag 'li', text: /^Inbox/ }
+    it { should have_tag 'li', text: /^Sent/ }
+    it { should have_tag 'li', text: /^All/ }
+    it { should have_tag 'li', text: /^Drafts/ }
+    it { should have_tag 'li', text: /^Trash/ }
+    it { should have_tag 'li', text: /^Ignored/ }
   end
 
   context '#box_participant_slug' do

data/spec/helpers/message_train/collectives_helper_spec.rb

@@ -0,0 +1,76 @@
+require 'rails_helper'
+
+describe MessageTrain::CollectivesHelper do
+  include_context 'loaded site'
+  include ControllerMacros
+
+  before do
+    login_user first_user
+  end
+
+  describe '#collective_boxes_widget' do
+    subject { helper.collective_boxes_widget(membered_group, first_user) }
+    it { should have_tag 'h3', text: /Membered Group Messages/ }
+    it { should have_tag 'ul', with: { class: 'list-group' } }
+    it { should have_tag 'li', with: { class: 'list-group-item' }, count: 4 }
+  end
+
+  describe '#collective_nav_item' do
+    context 'when user is not allowed' do
+      subject { helper.collective_nav_item(first_group.box(:in, third_user), third_user) }
+      it { should be nil }
+    end
+    context 'when no messages are unread' do
+      subject { helper.collective_nav_item(first_group.box(:in, first_user), first_user) }
+      it { should have_tag 'li', text: /^First Group/ }
+      it { should_not have_tag 'span', with: { class: 'badge' } }
+    end
+    context 'when there are unread messages' do
+      subject { helper.collective_nav_item(membered_group.box(:in, first_user), first_user) }
+      it { should have_tag 'li', text: /^Membered Group/ }
+      it { should have_tag 'span', text: /[0-9]+/, with: { class: 'badge' } }
+    end
+  end
+
+  describe '#collective_list_item' do
+    context 'when user is a valid sender and box is sent' do
+      subject { helper.collective_list_item(first_group.box(:sent, first_user)) }
+      it { should have_tag 'li', text: /^Sent/ }
+      it { should have_tag 'span', text: /[0-9]+/, with: { class: 'badge' } }
+    end
+    context 'when user is a valid recipient and box is in' do
+      subject { helper.collective_list_item(membered_group.box(:in, first_user)) }
+      it { should have_tag 'li', text: /^Inbox/ }
+      it { should have_tag 'span', text: /[0-9]+/, with: { class: 'badge' } }
+    end
+  end
+
+  describe '#collective_boxes_dropdown_list' do
+    describe 'passed a regular user' do
+      subject { helper.collective_boxes_dropdown_list(first_user) }
+      it { should have_tag 'ul', with: { class: 'dropdown-menu' }, count: 1 } # Groups and Roles
+      it { should have_tag 'li', text: /^First Group/ }
+      it { should have_tag 'li', text: /^Membered Group/ }
+    end
+    describe 'passed an admin user' do
+      subject { helper.collective_boxes_dropdown_list(admin_user) }
+      it { should have_tag 'ul', with: { class: 'dropdown-menu' }, count: 1 } # Groups and Roles
+      it { should have_tag 'li', text: /^Admin/ }
+    end
+    describe 'passed a superadmin user' do
+      subject { helper.collective_boxes_dropdown_list(superadmin_user) }
+      it { should have_tag 'ul', with: { class: 'dropdown-menu' }, count: 1 } # Groups and Roles
+      it { should have_tag 'li', text: /^Admin/ }
+    end
+  end
+
+  context '#collective_slug' do
+    subject { helper.collective_slug(first_group) }
+    it { should eq 'first-group' }
+  end
+
+  context '#collective_name' do
+    subject { helper.collective_name(first_group) }
+    it { should eq 'First Group' }
+  end
+end