lita-youtube-me 0.0.4 → 0.0.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 84e50b2bf564204b460ec3da6d9b267aab005701
-  data.tar.gz: 816c091089bbf1ad1eb6c816003b6f28894cde66
+  metadata.gz: b69c15d7418f23826ab51582155b6a0f5aeefea6
+  data.tar.gz: 85623f1c85cb60c6e29bfa91389f8479e706406d
 SHA512:
-  metadata.gz: 9c5b96f5cc3a162f7a02fdeb76260ca56d98f4ed44ebbdb5d1f64a9c687a2cd58599ece5c8d05fe3558826fc24a86bfca6852ff6e22f8b5328c175183ffbba6b
-  data.tar.gz: c9d9a3686daafd6057a7c6eba8adda27e5b3fc6bec140838ea8ac1aaf24835d0495071d94dfe4c2e17e74eee518373c8d966a9f5d9425078aa04f62a07aee5ec
+  metadata.gz: cbc9658f85ca27ce1c7aee9c4bf5b7de6c0b348e90a12edb4611d08ab331d7fc1a6385b74a907f86e6984eaee2adfefd895dd48f3e735d27b68d32ed18cbf030
+  data.tar.gz: 65a7a70f4f42160aa0a6b0de9699634d6069630262ffb3c2701b1bace3dda6ec695520b64d8f8f92644f112917ffeab3d83c3a4a899fc6f255832f3ca352b61c

data/README.md

@@ -23,11 +23,13 @@ end
 ### Optional attributes
 * `video_info` (boolean) - When set to `true`, Lita will return additional information (title, duration, etc.) about the video. Default: `false`
 * `detect_urls` (boolean) - When set to `true`, Lita will return additional information about any YouTube URLs it detects. Default: `false`
+* `top_result` (boolean) - When set to `true`, Lita will return the top result in response to a video search. Default: `false`, which returns a random result selected from the top 15.
 
 ``` ruby
 Lita.configure do |config|
   config.handlers.youtube_me.video_info = true
   config.handlers.youtube_me.detect_urls = true
+  config.handlers.youtube_me.top_result = true
 end
 ```
 

data/lib/lita/handlers/youtube_me.rb

@@ -9,6 +9,7 @@ module Lita
       config :api_key, type: String, required: true
       config :video_info, types: [TrueClass, FalseClass], default: false
       config :detect_urls, types: [TrueClass, FalseClass], default: false
+      config :top_result, types: [TrueClass, FalseClass], default: false
 
       route(/^(?:youtube|yt)(?: me)?\s+(.*)/i, :find_video, command: true, help: {
         "youtube (me) QUERY" => "Gets a YouTube video."
@@ -28,8 +29,8 @@ module Lita
           key: config.api_key
         )
         return if http_response.status != 200
-        videos = MultiJson.load(http_response.body)["items"]
-        video = videos.sample
+        videos = MultiJson.load(http_response.body)["items"].select { |v| v["id"].key?("videoId") }
+        video = config.top_result ? videos.first : videos.sample
         id = video["id"]["videoId"]
         response.reply "https://www.youtube.com/watch?v=#{id}"
         if config.video_info

data/lita-youtube-me.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |spec|
   spec.name          = "lita-youtube-me"
-  spec.version       = "0.0.4"
+  spec.version       = "0.0.5"
   spec.authors       = ["Taylor Lapeyre"]
   spec.email         = ["taylorlapeyre@gmail.com"]
   spec.description   = %q{A Lita handler that replies with a youtube URL when given a query.}

data/spec/lita/handlers/youtube_me_spec.rb

@@ -30,6 +30,13 @@ describe Lita::Handlers::YoutubeMe, lita_handler: true do
     expect(replies.last).to match(/youtube\.com/)
   end
 
+  it "does not attempt to return a playlist" do
+    send_command("youtube babbletron birds")
+    expect(replies.count).to eq 1
+    expect(replies.last).to_not be_nil
+    expect(replies.last).not_to match(/^https:\/\/www\.youtube\.com\/watch\?v=$/)
+  end
+
   it "displays info for a requested video when the video_info config variable is true" do
     registry.config.handlers.youtube_me.video_info = true
     send_command("yt soccer")
@@ -68,6 +75,12 @@ describe Lita::Handlers::YoutubeMe, lita_handler: true do
     expect(replies.count).to eq 0
   end
 
+  it "returns the top video in response to a query when the top_result config variable is true" do
+    registry.config.handlers.youtube_me.top_result = true
+    send_command("yt polica lay your cards out")
+    expect(replies.first).to match(/Rl03afAqeFQ/)
+  end
+
   it "does not return a video in response to a query when the API key is invalid" do
     registry.config.handlers.youtube_me.api_key = "this key doesn't work"
     send_command("youtube me soccer")

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: lita-youtube-me
 version: !ruby/object:Gem::Version
-  version: 0.0.4
+  version: 0.0.5
 platform: ruby
 authors:
 - Taylor Lapeyre
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-05-09 00:00:00.000000000 Z
+date: 2015-06-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: lita