lita-server_status 0.0.1 → 0.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 908cea291aafe36dd2a564bf5d0a4a34e8af1e0e
-  data.tar.gz: dd97aabd137f443e9fc26bce5bca142f4f180502
+  metadata.gz: eaa823e27040eeab4be70ce6477dca2838fb6046
+  data.tar.gz: 122209a480a1c0774bf8c23a8538fa3164cbe6bd
 SHA512:
-  metadata.gz: 59bf87e4b46b7ca0434dd3ee5128f3866f47302bea1f23d79795f52eafcc44e78be717c54b21ac808cdf2b505167d9f1b561b9b154c321c8918ab89cf1184a4a
-  data.tar.gz: 5e3119cd40ec6cf42092c73f538c24c5133a12077a15665531d8e56435a03d8913d357ea1cdad8077a4d4f80b0b1b6f956918a634b32742ecd8be4353b1312f2
+  metadata.gz: d0d62697d4c3fecb4c8f31fa92051dd9b0d47f4b6aff92de8e0495ba4ceb2ded8c5813954630437fb7eb415335a4404db0d3e9bbaeba4d6ceff7122b9aeb6fc2
+  data.tar.gz: f85fbf9e74fb9bea5b456b00a480cc7ae6060ae644ad991130e356e7a1076f4ed759355ab905f336b11cb8bacc35cdfce8c246c5fcc02e8ed785895b0eaed877

data/lib/lita/handlers/server_status.rb

@@ -13,18 +13,23 @@ module Lita
         user, application, branch, environment = message.match(MESSAGE_REGEX).captures
 
         apply_status = { id: "#{application}:#{environment}",
-                         message: "#{application} #{environment}: #{branch} (#{user} @ #{Time.now.to_s})" }
+                         message: "#{application} #{environment}: #{branch} (#{user} @ #{formatted_time})" }
 
         redis.set("server_status:#{apply_status[:id]}", apply_status[:message])
       end
 
       def list_statuses(response)
-        message = []
-        redis.keys("server_status*").each do |key|
-          message << redis.get(key)
-        end
+        response.reply status_message
+      end
+
+      def status_message
+        messages = redis.keys("server_status*").map { |key| redis.get(key) }
+        messages << "I don't know what state the servers are in." if messages.empty?
+        messages.join("\n")
+      end
 
-        response.reply message.join("\n")
+      def formatted_time
+        Time.now.strftime("%Y-%m-%d %H:%M")
       end
     end
 

data/lita-server_status.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |spec|
   spec.name          = "lita-server_status"
-  spec.version       = "0.0.1"
+  spec.version       = "0.0.2"
   spec.authors       = ["Michael van den Beuken", "Ruben Estevez", "Jordan Babe", "Mathieu Gilbert", "Ryan Jones", "Darko Dosenovic"]
   spec.email         = ["michael.beuken@gmail.com", "ruben.a.estevez@gmail.com", "jorbabe@gmail.com", "mathieu.gilbert@ama.ab.ca", "ryan.michael.jones@gmail.com", "darko.dosenovic@ama.ab.ca"]
   spec.description   = %q{Store and list out the statuses of applications}

data/spec/lita/handlers/server_status_spec.rb

@@ -2,6 +2,12 @@ require "spec_helper"
 require "timecop"
 
 describe Lita::Handlers::ServerStatus, lita_handler: true do
+  let(:formatted_local_time) { Time.now.strftime("%Y-%m-%d %H:%M") }
+
+  before(:each) do
+    allow_any_instance_of(Lita::Handlers::ServerStatus).to receive(:formatted_time).and_return(formatted_local_time)
+  end
+
   it { routes("Waffle McRib is starting deploy of 'APPNAME' from branch 'MASTER' to PRODUCTION").to(:save_status) }
   it { routes_command("server status").to(:list_statuses) }
 
@@ -11,24 +17,29 @@ describe Lita::Handlers::ServerStatus, lita_handler: true do
   end
 
   it "update the server status if it changes" do
-    Timecop.freeze do
+    Timecop.freeze(Time.now) do
       send_message(%{Waffle McRib is starting deploy of 'FAKEAPP' from branch 'MASTER' to PRODUCTION})
       send_command("server status")
-      expect(replies.last).to include("FAKEAPP PRODUCTION: MASTER (Waffle McRib @ #{Time.now.to_s})")
+      expect(replies.last).to include("FAKEAPP PRODUCTION: MASTER (Waffle McRib @ #{formatted_local_time})")
 
       send_message(%{Waffle McRib is starting deploy of 'FAKEAPP' from branch 'WAFFLE' to PRODUCTION})
       send_command("server status")
-      expect(replies.last).to include("FAKEAPP PRODUCTION: WAFFLE (Waffle McRib @ #{Time.now.to_s})")
+      expect(replies.last).to include("FAKEAPP PRODUCTION: WAFFLE (Waffle McRib @ #{formatted_local_time})")
     end
   end
 
   it "should list the current server statuses" do
-    Timecop.freeze do
+    Timecop.freeze(Time.now) do
       send_message(%{Waffle McRib is starting deploy of 'FAKEAPP' from branch 'MASTER' to PRODUCTION})
       send_message(%{Waffle McRib is starting deploy of 'BATMAN' from branch 'AWESOME' to STAGING})
       send_command("server status")
-      expect(replies.last).to include("FAKEAPP PRODUCTION: MASTER (Waffle McRib @ #{Time.now.to_s})")
-      expect(replies.last).to include("BATMAN STAGING: AWESOME (Waffle McRib @ #{Time.now.to_s})")
+      expect(replies.last).to include("FAKEAPP PRODUCTION: MASTER (Waffle McRib @ #{formatted_local_time})")
+      expect(replies.last).to include("BATMAN STAGING: AWESOME (Waffle McRib @ #{formatted_local_time})")
     end
   end
+
+  it "admits when there are no statuses" do
+    send_command("server status")
+    expect(replies.last).to eq("I don't know what state the servers are in.")
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: lita-server_status
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.0.2
 platform: ruby
 authors:
 - Michael van den Beuken
@@ -13,7 +13,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-07-25 00:00:00.000000000 Z
+date: 2014-08-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: lita
@@ -175,7 +175,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.2.2
+rubygems_version: 2.4.1
 signing_key: 
 specification_version: 4
 summary: Store and list out the statuses of applications