campfire-bot-absentee-camper 0.1.0 → 0.2.0

data/CHANGELOG.md

@@ -0,0 +1,7 @@
+# 0.2.0
+
+* Message now prefixed with "_user_ says:"
+
+# 0.1.0
+
+* Initial release

data/Gemfile

@@ -8,6 +8,5 @@ group :development, :test do
   gem 'guard'
   gem 'guard-rspec'
   gem 'rake'
-  gem 'ruby-debug19'
   gem 'growl'
 end

data/lib/campfire_bot/absentee_camper.rb

@@ -29,7 +29,7 @@ module CampfireBot
 
             # If the user isn't in the room, fire off a notification
             unless room.users.map { |u| u['id'] }.include? user_id_from_config(mentioned)
-              NotificationManager.new(room, plugin_config['users'][mentioned]).send_notifications body
+              NotificationManager.new(msg, plugin_config['users'][mentioned]).send_notifications
               room.speak("[Notified #{mentioned}]")
             end
           end

data/lib/campfire_bot/absentee_camper/notification/notification_manager.rb

@@ -3,7 +3,10 @@ module CampfireBot
     module Notification
       class NotificationManager
 
-        def initialize(room, user_config)
+        def initialize(message, user_config)
+          @message = message
+          room = message[:room]
+
           if user_config.is_a?(Hash) and user_config['notification_methods']
             user_config['notification_methods'].each do |notifier, initialization_info|
               add_notifier Notification.const_get("#{notifier}Notifier".to_sym).new(room, initialization_info)
@@ -17,12 +20,16 @@ module CampfireBot
           end
         end
 
-        def send_notifications(message)
-          @notifiers.each { |notifier| notifier.notify message }
+        def send_notifications
+          @notifiers.each { |notifier| notifier.notify format_message(@message) }
         end
 
         private
 
+        def format_message(message)
+          "#{message[:person]} says: #{message[:message]}"
+        end
+
         def add_notifier(notifier)
           @notifiers ||= []
           @notifiers << notifier

data/lib/campfire_bot/absentee_camper/version.rb

@@ -1,5 +1,5 @@
 module CampfireBot
   module AbsenteeCamper
-    VERSION = "0.1.0"
+    VERSION = "0.2.0"
   end
 end

data/spec/lib/campfire_bot/absentee_camper/notification/email_notifier_spec.rb

@@ -5,7 +5,6 @@ module CampfireBot
     module Notification
       describe EmailNotifier do
 
-        let(:email_address) { "johnny@test.com" }
         let(:uri) { 'http://www.google.com' }
         let(:body) do
           body = <<-BODY
@@ -16,7 +15,7 @@ Come back to the campfire!  We're having a good time telling ghost stories!  Her
 #{uri}
 BODY
         end
-        let(:message) { "this is a test" }
+        let(:message) { "Jon Levin says: this is a test" }
         let(:user_id) { 1 }
         let(:email_address) { 'jon.levin@shredders-r-us.com' }
         let(:room) do
@@ -41,7 +40,7 @@ BODY
           it "logs a message indicating the message is being sent" do
             Pony.stub(:mail)
             Logger.instance.should_receive(:debug).with("sending email to #{email_address}")
-            subject.notify(message)
+            subject.notify message
           end
 
           it "sends the email with the correct information" do
@@ -50,7 +49,7 @@ BODY
               :body => body
             }.merge(pony_options))
 
-            subject.notify(message)
+            subject.notify message
           end
         end
       end

data/spec/lib/campfire_bot/absentee_camper/notification/notification_manager_spec.rb

@@ -6,6 +6,13 @@ module CampfireBot
       describe NotificationManager do
         let(:room) { double('room').as_null_object }
         let(:user_id) { 1 }
+        let(:message) do
+          {
+            :person => "Chad Boyd",
+            :message => "test",
+            :room => room
+          }
+        end
 
         before(:all) do
           NotificationManager.send :public, :add_notifier
@@ -103,12 +110,12 @@ module CampfireBot
 
               it "logs a message" do
                 Logger.instance.should_receive(:debug).with(log_message)
-                NotificationManager.new(room, user_info)
+                NotificationManager.new(message, user_info)
               end
 
               it "creates an EmailNotifier by default" do
                 EmailNotifier.should_receive(:new).with(room, user_info)
-                NotificationManager.new(room, user_info)
+                NotificationManager.new(message, user_info)
               end
             end
           end
@@ -118,7 +125,8 @@ module CampfireBot
           let(:user_info) do
             { 'id' => user_id }
           end
-          let(:message) { "test" }
+          let(:user_name) { "Chad" }
+          let(:formatted_message) { "#{message[:person]} says: #{message[:message]}" }
 
           context "when there is only one notifier" do
             before do
@@ -127,8 +135,8 @@ module CampfireBot
 
             it "only sends one notification" do
               ProwlNotifier.any_instance.should_receive(:notify)
-              manager = NotificationManager.new(room, user_info)
-              manager.send_notifications message
+              manager = NotificationManager.new(message, user_info)
+              manager.send_notifications
             end
           end
 
@@ -143,15 +151,15 @@ module CampfireBot
             it "only sends one notification" do
               ProwlNotifier.any_instance.should_receive(:notify)
               EmailNotifier.any_instance.should_receive(:notify)
-              manager = NotificationManager.new(room, user_info)
-              manager.send_notifications message
+              manager = NotificationManager.new(message, user_info)
+              manager.send_notifications
             end
           end
 
           it "sends the correct message" do
-            EmailNotifier.any_instance.should_receive(:notify).with(message)
-            manager = NotificationManager.new(room, user_id)
-            manager.send_notifications message
+            EmailNotifier.any_instance.should_receive(:notify).with(formatted_message)
+            manager = NotificationManager.new(message, user_id)
+            manager.send_notifications
           end
         end
       end

data/spec/lib/campfire_bot/absentee_camper_spec.rb

@@ -91,11 +91,11 @@ module CampfireBot
 
               NotificationManager
                 .should_receive(:new)
-                .with(room, users_in_config[name])
+                .with(message, users_in_config[name])
                 .and_return(notification_manager)
             end
 
-            notification_manager.should_receive(:send_notifications).with(message['body']).exactly(times).times
+            notification_manager.should_receive(:send_notifications).exactly(times).times
             subject.role_call message
           end
         end
@@ -130,10 +130,10 @@ module CampfireBot
         def sends_notification_to(name)
           NotificationManager
             .should_receive(:new)
-            .with(room, users_in_config[name])
+            .with(message, users_in_config[name])
             .and_return(notification_manager)
 
-          notification_manager.should_receive(:send_notifications).with(message['body'])
+          notification_manager.should_receive(:send_notifications)
         end
 
         def does_not_send_notification

data/spec/spec_helper.rb

@@ -1,7 +1,6 @@
 require "rubygems"
 require "bundler/setup"
 Bundler.require(:default, :test)
-require 'ruby-debug'
 
 BOT_ROOT = File.join(FileUtils.pwd, 'spec')
 BOT_ENVIRONMENT = 'test'
@@ -11,6 +10,9 @@ require 'absentee_camper'
 RSpec.configure do |config|
   config.include CampfireBot::AbsenteeCamper::Notification
   config.include CampfireBot::AbsenteeCamper::Config
+  config.treat_symbols_as_metadata_keys_with_true_values = true
+  config.filter_run focus: true
+  config.run_all_when_everything_filtered = true
 
   config.before do
     CampfireBot::AbsenteeCamper::Logger.stub(:instance).and_return(double('logger').as_null_object)

metadata

@@ -1,60 +1,80 @@
---- !ruby/object:Gem::Specification
+--- !ruby/object:Gem::Specification 
 name: campfire-bot-absentee-camper
-version: !ruby/object:Gem::Version
-  version: 0.1.0
-  prerelease: 
+version: !ruby/object:Gem::Version 
+  prerelease: false
+  segments: 
+  - 0
+  - 2
+  - 0
+  version: 0.2.0
 platform: ruby
-authors:
+authors: 
 - hoverlover
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2011-10-13 00:00:00.000000000Z
-dependencies:
-- !ruby/object:Gem::Dependency
+
+date: 2011-12-07 00:00:00 -06:00
+default_executable: 
+dependencies: 
+- !ruby/object:Gem::Dependency 
   name: campfire-bot
-  requirement: &70221517964260 !ruby/object:Gem::Requirement
+  prerelease: false
+  requirement: &id001 !ruby/object:Gem::Requirement 
     none: false
-    requirements:
+    requirements: 
     - - ~>
-      - !ruby/object:Gem::Version
+      - !ruby/object:Gem::Version 
+        segments: 
+        - 0
+        - 0
+        - 1
         version: 0.0.1
   type: :runtime
-  prerelease: false
-  version_requirements: *70221517964260
-- !ruby/object:Gem::Dependency
+  version_requirements: *id001
+- !ruby/object:Gem::Dependency 
   name: pony
-  requirement: &70221517963420 !ruby/object:Gem::Requirement
+  prerelease: false
+  requirement: &id002 !ruby/object:Gem::Requirement 
     none: false
-    requirements:
+    requirements: 
     - - ~>
-      - !ruby/object:Gem::Version
-        version: '1.3'
+      - !ruby/object:Gem::Version 
+        segments: 
+        - 1
+        - 3
+        version: "1.3"
   type: :runtime
-  prerelease: false
-  version_requirements: *70221517963420
-- !ruby/object:Gem::Dependency
+  version_requirements: *id002
+- !ruby/object:Gem::Dependency 
   name: prowl
-  requirement: &70221517962640 !ruby/object:Gem::Requirement
+  prerelease: false
+  requirement: &id003 !ruby/object:Gem::Requirement 
     none: false
-    requirements:
+    requirements: 
     - - ~>
-      - !ruby/object:Gem::Version
+      - !ruby/object:Gem::Version 
+        segments: 
+        - 0
+        - 1
+        - 3
         version: 0.1.3
   type: :runtime
-  prerelease: false
-  version_requirements: *70221517962640
-description: Plugin for campfire-bot that monitors incoming messages and notifies
-  a user that is @mentioned if they aren't present.
-email:
+  version_requirements: *id003
+description: Plugin for campfire-bot that monitors incoming messages and notifies a user that is @mentioned if they aren't present.
+email: 
 - hoverlover@gmail.com
 executables: []
+
 extensions: []
+
 extra_rdoc_files: []
-files:
+
+files: 
 - .gitignore
 - .rspec
 - .travis.yml
+- CHANGELOG.md
 - Gemfile
 - Guardfile
 - README.md
@@ -75,32 +95,39 @@ files:
 - spec/lib/campfire_bot/absentee_camper/notification/prowl_notifier_spec.rb
 - spec/lib/campfire_bot/absentee_camper_spec.rb
 - spec/spec_helper.rb
-homepage: ''
+has_rdoc: true
+homepage: ""
 licenses: []
+
 post_install_message: 
 rdoc_options: []
-require_paths:
+
+require_paths: 
 - lib/campfire_bot
-required_ruby_version: !ruby/object:Gem::Requirement
+required_ruby_version: !ruby/object:Gem::Requirement 
   none: false
-  requirements:
-  - - ! '>='
-    - !ruby/object:Gem::Version
-      version: '0'
-required_rubygems_version: !ruby/object:Gem::Requirement
+  requirements: 
+  - - ">="
+    - !ruby/object:Gem::Version 
+      segments: 
+      - 0
+      version: "0"
+required_rubygems_version: !ruby/object:Gem::Requirement 
   none: false
-  requirements:
-  - - ! '>='
-    - !ruby/object:Gem::Version
-      version: '0'
+  requirements: 
+  - - ">="
+    - !ruby/object:Gem::Version 
+      segments: 
+      - 0
+      version: "0"
 requirements: []
+
 rubyforge_project: campfire-bot-absentee-camper
-rubygems_version: 1.8.10
+rubygems_version: 1.3.7
 signing_key: 
 specification_version: 3
-summary: Plugin for campfire-bot that monitors incoming messages and notifies a user
-  that is @mentioned if they aren't present.
-test_files:
+summary: Plugin for campfire-bot that monitors incoming messages and notifies a user that is @mentioned if they aren't present.
+test_files: 
 - spec/absentee-camper-config.yml
 - spec/config.yml
 - spec/lib/campfire_bot/absentee_camper/notification/email_notifier_spec.rb