easymon 1.3 → 1.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 6d0e28f8e997050c2331556bc5590cc3bb6d5bea
-  data.tar.gz: 252312ef1045ab3b6acdd7893628b676b631e304
+SHA256:
+  metadata.gz: 995c7954f5c5d720f9840c0f93b1d673de2e6a6682c7a134336cac6d21f2a280
+  data.tar.gz: 7cbce9feb1e3a4f4fc7c661a5597314fb2107e3ac36d5ee84cfa2810c218dc75
 SHA512:
-  metadata.gz: c6883cf5fb8c234f358b50253d5eb0f38ae8456112b15ef8d73ea8a4f880f17211642f5fc3ffdedfc25f800c5e747c187633dc9f08fe16cbc62da05d802072c7
-  data.tar.gz: 06e78f0fc445aca9a0b671d3eb8aaedc5f5846c078dc00acf41c0c14acda2987763cc2e1b6c260b892e098378fae62e5ffacb16610e108fb2df6319192d3995f
+  metadata.gz: c805aec0b3a45ac5e289ffe5fe7f17e85abb250dbd31d09787e1e3912e11d4040a3a7aa9c04018fc8dfe4febc3e0aaeb638f4ef4e4a65eea8d0250d2fb60b090
+  data.tar.gz: 2763483c3faf6714ccfbc3ea055bb985a9ed0a125527e41ec9e72e25946a85979794590c6886fa42259c9e1fa6dbac99b441ac575ef89a3bfbebdf7c8d8372b0

data/README.md

@@ -173,7 +173,7 @@ check = Easymon::SemaphoreCheck.new("config/redis.yml")
 
 This is mainly a check that gets subclassed by the next check.
 
-###Traffic Enabled
+### Traffic Enabled
 `Easymon::TrafficEnabledCheck` is fairly specific, but when we want a server to
 accept traffic, we can place a file in the Rails.root, and the load balancers
 can use the result of this check to help decide whether or not to send traffic
@@ -184,7 +184,7 @@ Easymon::TrafficEnabledCheck.new("enable-traffic")
 ````
 This is a subclass of the Semaphore check mentioned above.
 
-###Split ActiveRecord
+### Split ActiveRecord
 `Easymon::SplitActiveRecordCheck` is the most complicated check, as it's not
 something you can use out of the gate. Here we pass a block so we get a fresh
 instance of `ActiveRecord::Base` or whatever other class we might be using to

data/lib/easymon/checks/redis_check.rb

@@ -22,7 +22,7 @@ module Easymon
       def redis_up?
         redis = Redis.new(@config)
         reply = redis.ping == 'PONG'
-        redis.client.disconnect
+        redis.close
         reply
       rescue
         false

data/lib/easymon/version.rb

@@ -1,3 +1,3 @@
 module Easymon
-  VERSION = "1.3"
+  VERSION = "1.4"
 end

data/test/controllers/easymon/checks_controller_test.rb

@@ -61,14 +61,14 @@ module Easymon
 
     test "show when the check passes" do
       Easymon::Repository.add("database", Easymon::ActiveRecordCheck.new(ActiveRecord::Base))
-      get :show, check: "database"
+      get :show, :params => { check: "database" }
       assert_response :success
       assert response.body.include?("Up"), "Response should include message text, got #{response.body}"
     end
 
     test "show json when the check passes" do
       Easymon::Repository.add("database", Easymon::ActiveRecordCheck.new(ActiveRecord::Base))
-      get :show, :check => "database", :format => :json
+      get :show, :params => { :check => "database", :format => :json }
 
       json = JSON.parse(response.body)
 
@@ -80,7 +80,7 @@ module Easymon
       Easymon::Repository.add("database", Easymon::ActiveRecordCheck.new(ActiveRecord::Base))
       ActiveRecord::Base.connection.stubs(:active?).raises("boom")
 
-      get :show, :check => "database"
+      get :show, :params => { :check => "database" }
 
       assert_response :service_unavailable
       assert response.body.include?("Down"), "Response should include failure text, got #{response.body}"
@@ -89,7 +89,7 @@ module Easymon
     test "show if the check is not found" do
       Easymon::Repository.names.each {|name| Easymon::Repository.remove(name)}
 
-      get :show, :check => "database"
+      get :show, :params => { :check => "database" }
       assert_response :not_found
     end
 
@@ -99,7 +99,7 @@ module Easymon
       get :index
       assert_response :forbidden
 
-      get :show, :check => "database"
+      get :show, :params => { :check => "database" }
       assert_response :forbidden
     end
   end