lita-cleverbot 1.0.0 → 1.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 104f6a69de73936a7ad72725a5e86425601cd48a
-  data.tar.gz: 2f6e1ae0f2665e27da0c1ad9b2241e15289216c8
+  metadata.gz: 4c839053ab529353ba99813bd1a1018577b48ab2
+  data.tar.gz: f432e3756c809132f8990ac23a5d61ec6aaaa8e1
 SHA512:
-  metadata.gz: afe6d44dc2cc7e3ae012107ff6c8d683409edb89e0a2aba1b42079b7a8d4a0812a0d65fff76a2c04ea247210f52ff03584912617d5af735c67d3dc7f589488ea
-  data.tar.gz: 4d4444124ffa2efb24a4c0d10b5193aa51dc82efe60deba6e75c652988c59b7fe7b48b9c84a23bd7230f67dd3fdb10cd5701b12a8070cf01be2d3e2bf2017651
+  metadata.gz: cab5580c284a58dd8ce73945ea8685be920d78d69552b6319040f72f99f56801d116269196bf180ecc23f8beeb75a14cd03e3cae566d071bba1af3b3e88debab
+  data.tar.gz: 5a3f72825ecf161cc8783979dec6d8b181d19ad88a9d677a3c7c1b134c2dd150be87cfd514640ea11bcffecd88c307edfaa7b233fe032a6142eb635761c073e6

data/.travis.yml

@@ -6,3 +6,5 @@ before_install:
   - gem update --system
 services:
   - redis-server
+notifications:
+  email: false

data/README.md

@@ -1,7 +1,7 @@
 # lita-cleverbot
 
 [![Build Status](https://travis-ci.org/cristianbica/lita-cleverbot.png?branch=master)](https://travis-ci.org/cristianbica/lita-cleverbot)
-[![Coverage Status](https://coveralls.io/repos/cristianbica/lita-cleverbot/badge.png)](https://coveralls.io/r/cristianbica/lita-cleverbot)
+[![Coverage Status](https://coveralls.io/repos/github/cristianbica/lita-cleverbot/badge.svg?branch=master)](https://coveralls.io/github/cristianbica/lita-cleverbot?branch=master)
 
 This Lita handler will send unhadled messages directed to Lita to Cleverbot and reply with clever messages.
 

data/lib/lita/handlers/cleverbot.rb

@@ -24,7 +24,19 @@ module Lita
       end
 
       def should_reply?(message)
-        message.command? || message.body =~ /#{aliases.join('|')}/i
+        !sent_by_robot?(message) && ( command?(message) || robot_mentioned?(message) )
+      end
+
+      def command?(message)
+        message.command?
+      end
+
+      def sent_by_robot?(message)
+        message.user.mention_name == robot.mention_name
+      end
+
+      def robot_mentioned?(message)
+        message.body =~ /#{aliases.join('|')}/i
       end
 
       def build_response(message)

data/lita-cleverbot.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |spec|
   spec.name          = "lita-cleverbot"
-  spec.version       = "1.0.0"
+  spec.version       = "1.0.1"
   spec.authors       = ["Cristian Bica"]
   spec.email         = ["cristian.bica@gmail.com"]
   spec.description   = "Lita AI via cleverbot.io"

data/spec/lita/handlers/cleverbot_spec.rb

@@ -29,6 +29,12 @@ describe Lita::Handlers::Cleverbot, lita: true, lita_handler: true, additional_l
       send_message("test", from: room)
     end
 
+    it "shouldn't reply to messages sent by the bot" do
+      expect_any_instance_of(Lita::Handlers::Cleverbot).to receive(:reply).never
+      robot_user = Lita::User.new(100, name: robot.mention_name)
+      send_message("Lita: test", as: robot_user, from: room)
+    end
+
     it "should try to reply to messages addressed to the robot" do
       expect_any_instance_of(Lita::Handlers::Cleverbot).to receive(:reply).once
       send_message("hey Lita, who am I?", from: room)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: lita-cleverbot
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.0.1
 platform: ruby
 authors:
 - Cristian Bica