Use Source#private_message? to determine message targets.

Author: jimmycuadra

lib/lita/adapters/irc.rb

@@ -26,12 +26,12 @@ def run
       end
 
       def send_messages(target, strings)
-        if target.room
-          channel = Cinch::Channel.new(target.room, cinch)
-          strings.each { |s| channel.msg(s) }
-        else
+        if target.private_message?
           user = Cinch::User.new(target.user.name, cinch)
           strings.each { |s| user.msg(s) }
+        else
+          channel = Cinch::Channel.new(target.room, cinch)
+          strings.each { |s| channel.msg(s) }
         end
       end
 

spec/lita/adapters/irc_spec.rb

@@ -59,7 +59,7 @@
 
   describe "#send_messages" do
     it "sends messages to rooms" do
-      source = double("Lita::Source", room: "#foo")
+      source = double("Lita::Source", room: "#foo", private_message?: false)
       channel = double("Cinch::Channel")
       allow(Cinch::Channel).to receive(:new).with(
         "#foo",
@@ -72,7 +72,7 @@
 
     it "sends messages to users" do
       user = double("Lita::User", name: "Carl")
-      source = double("Lita::Source", room: nil, user: user)
+      source = double("Lita::Source", user: user, private_message?: true)
       user = double("Cinch::User")
       allow(Cinch::User).to receive(:new).with(
         "Carl",