Changeset 13

Timestamp:

01/04/08 16:46:09 (9 months ago)

Author:

muness

Message:

site wide settings for campfire login

Files:

• incubator/cc_campfire_notifier/campfire_notifier.rb (modified) (2 diffs)
• incubator/cc_campfire_notifier/test/campfire_notifier_test.rb (modified) (1 diff)

incubator/cc_campfire_notifier/campfire_notifier.rb

@@ -2 +2 @@
-
-class CampfireNotifier
-
-
+
-
-  def initialize(project = nil)
-  end
-
+  cattr_accessor :settings
+  def room
+    return if room_name.nil?
+    CruiseControl::Log.event("Campfire Notifier configured with #{settings.inspect}", :debug)
+    campfire = Tinder::Campfire.new(settings[:subdomain], :ssl => settings[:use_ssl])
+    campfire.login settings[:login], settings[:password]
+    CruiseControl::Log.event("Logged in to campfire #{settings[:subdomain]} as #{settings[:login]}", :debug)
+    campfire.find_room_by_name(room_name)
+  rescue => e
+    CruiseControl::Log.event("Trouble initalizing campfire room #{room_name}", :error)
+    raise
+  end
+  
-  def build_finished(build)
-    build_text = "Build #{build.label}"
@@ -24 +36 @@
-  end
-
-  def room
-    campfire = Tinder::Campfire.new(subdomain, :ssl => use_ssl)
-    campfire.login login, password
-    CruiseControl::Log.event("Logged in to campfire #{subdomain} as #{login}", :debug)
-    campfire.find_room_by_name(room_name)
-  rescue => e
-    CruiseControl::Log.event("Trouble initalizing campfire room #{room_name}", :error)
-    raise
-  end
-end
-

incubator/cc_campfire_notifier/test/campfire_notifier_test.rb

@@ -27 +27 @@
-   notifier.build_finished(build)
- end
+ it "room is nil if no room_name is set" do
+   notifer = CampfireNotifier.new
+   notifer.room.should == nil
+ end
-end