Skip to content

Commit

Permalink
Updates for Lita 4.
Browse files Browse the repository at this point in the history
  • Loading branch information
jimmycuadra committed Oct 23, 2014
1 parent 0928f4d commit cf766d2
Show file tree
Hide file tree
Showing 4 changed files with 19 additions and 19 deletions.
22 changes: 10 additions & 12 deletions lib/lita/handlers/google_images.rb
Original file line number Diff line number Diff line change
Expand Up @@ -4,23 +4,28 @@ module Lita
module Handlers
class GoogleImages < Handler
URL = "https://ajax.googleapis.com/ajax/services/search/images"
VALID_SAFE_VALUES = %w(active moderate off)

config :safe_search, types: [String, Symbol], default: :active do
validate do |value|
unless VALID_SAFE_VALUES.include?(value.to_s.strip)
"valid values are :active, :moderate, or :off"
end
end
end

route(/(?:image|img)(?:\s+me)? (.+)/, :fetch, command: true, help: {
"image QUERY" => "Displays a random image from Google Images matching the query."
})

def self.default_config(handler_config)
handler_config.safe_search = :active
end

def fetch(response)
query = response.matches[0][0]

http_response = http.get(
URL,
v: "1.0",
q: query,
safe: safe_value,
safe: config.safe_search,
rsz: 8
)

Expand Down Expand Up @@ -50,13 +55,6 @@ def ensure_extension(url)
"#{url}#.png"
end
end

def safe_value
safe = Lita.config.handlers.google_images.safe_search || "active"
safe = safe.to_s.downcase
safe = "active" unless ["active", "moderate", "off"].include?(safe)
safe
end
end

Lita.register_handler(GoogleImages)
Expand Down
6 changes: 3 additions & 3 deletions lita-google-images.gemspec
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
Gem::Specification.new do |spec|
spec.name = "lita-google-images"
spec.version = "1.0.5"
spec.version = "2.0.0"
spec.authors = ["Jimmy Cuadra"]
spec.email = ["[email protected]"]
spec.description = %q{A Lita handler for fetching images from Google.}
Expand All @@ -14,11 +14,11 @@ Gem::Specification.new do |spec|
spec.test_files = spec.files.grep(%r{^(test|spec|features)/})
spec.require_paths = ["lib"]

spec.add_runtime_dependency "lita", ">= 2.0"
spec.add_runtime_dependency "lita", ">= 4.0"

spec.add_development_dependency "bundler", "~> 1.3"
spec.add_development_dependency "rake"
spec.add_development_dependency "rspec", ">= 3.0.0.beta2"
spec.add_development_dependency "rspec", ">= 3.0.0"
spec.add_development_dependency "simplecov"
spec.add_development_dependency "coveralls"
end
8 changes: 4 additions & 4 deletions spec/lita/handlers/google_images_spec.rb
Original file line number Diff line number Diff line change
@@ -1,10 +1,10 @@
require "spec_helper"

describe Lita::Handlers::GoogleImages, lita_handler: true do
it { routes_command("image me foo").to(:fetch) }
it { routes_command("image foo").to(:fetch) }
it { routes_command("img foo").to(:fetch) }
it { routes_command("img me foo").to(:fetch) }
it { is_expected.to route_command("image me foo").to(:fetch) }
it { is_expected.to route_command("image foo").to(:fetch) }
it { is_expected.to route_command("img foo").to(:fetch) }
it { is_expected.to route_command("img me foo").to(:fetch) }

describe "#foo" do
let(:response) { double("Faraday::Response") }
Expand Down
2 changes: 2 additions & 0 deletions spec/spec_helper.rb
Original file line number Diff line number Diff line change
Expand Up @@ -8,3 +8,5 @@

require "lita-google-images"
require "lita/rspec"

Lita.version_3_compatibility_mode = false

0 comments on commit cf766d2

Please sign in to comment.