lita-consul 0.0.2 → 0.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 4d9d08f566174b54aff99adc47f1b5ed4005f3ae
-  data.tar.gz: 7c38b9852453a70228a1883847ef46f442733ba9
+  metadata.gz: 844421003794d177006fb04861b3bac4d30db932
+  data.tar.gz: d93e3f1d03baf6ad5c1b05eae68ec71707ab4193
 SHA512:
-  metadata.gz: 0742652f2f71c7b74fc220251efc2d28a1300799399f41fd7fef900c1f2b68ba71f9504e4ad6f443a3db7ac836b06567c2ea580f4d0576a001396087b9c072fb
-  data.tar.gz: 8cdfe7ec5ffb5bd7f851a2ff7e0de3771ef6716541407b62918ab8d69434b2c35678ecb29ff69f2111423635b9857ad373b02da91237996ac16aa1015c5ae63f
+  metadata.gz: a42a1bf02b78009ecdc219c1cc75a34d6f5110b6acf30e035893aca154279f12f08bbe23568e4c203f2bcf05bbbdce19fa6982854d90f58f335375471d17a38e
+  data.tar.gz: 8d9c85237ff3efa1ac2c3238bd9f70692043496a713ec980932792cd69a8d36f3a93234fb361e245c3d30a34af6a5d568f41892da7a5c49151a810e733647b97

data/.travis.yml

@@ -1,8 +1,10 @@
 language: ruby
+cache: bundler
 rvm:
   - 2.0.0
 script: bundle exec rake
 before_install:
   - gem update --system
+  - gem install bundler
 services:
   - redis-server

data/README.md

@@ -1,9 +1,9 @@
 # lita-consul
 
 [![Build Status](https://travis-ci.org/dpires/lita-consul.png?branch=master)](https://travis-ci.org/dpires/lita-consul)
-[![Coverage Status](https://coveralls.io/repos/dpires/lita-consul/badge.png)](https://coveralls.io/r/dpires/lita-consul)
+[![Coverage Status](https://coveralls.io/repos/dpires/lita-consul/badge.svg?branch=master&service=github)](https://coveralls.io/github/dpires/lita-consul?branch=master)
 
-***lita-consul*** is a handler for [Lita](https://github.com/litaio/lita) for interacting with [Consul](https://github.com/hashicorp/consul).
+**lita-consul** is a handler for [Lita](https://github.com/litaio/lita) for interacting with [Consul](https://github.com/hashicorp/consul).
 
 ## Installation
 

data/lib/lita/handlers/consul.rb

@@ -14,6 +14,19 @@ module Lita
       route /^consul set ([a-zA-Z0-9\-\/_]+) ([a-zA-Z0-9\-\/_]+)/, :consul_set, command: true, help: {
         "consul set <key> <value>" => "Set <value> for <key>"
       }
+  
+      route /^consul members/, :consul_members, command: true, help: {
+        "consul members" => "Return consul nodes"
+      }
+
+      def consul_members(response)
+          resp = http.get("#{api_url}/catalog/nodes/")
+          replies = []
+          MultiJson.load(resp.body).each do | node |
+            replies << "#{node['Node']} - #{node['Address']}"
+          end 
+          response.reply replies.join("\n")
+      end
 
       def consul_get(response)
         key = response.matches.first.first
@@ -21,21 +34,6 @@ module Lita
         value = get_key_value(key)
         response.reply "#{key} = #{value}"
       end
-=begin
-        begin
-          resp = http.get("#{api_url}/kv/#{key}")
-          obj = MultiJson.load(resp.body)
-          unless obj[0]["Value"].nil?
-            value = Base64.decode64(obj[0]["Value"])
-            response.reply "#{key} = #{value}"
-          else
-            response.reply "#{key} = null"
-          end
-        rescue Faraday::ConnectionFailed=> e
-          response.reply e.to_s
-        end
-      end
-=end
 
       def consul_set(response)
         key = response.matches.first.first

data/lita-consul.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |spec|
   spec.name          = "lita-consul"
-  spec.version       = "0.0.2"
+  spec.version       = "0.0.3"
   spec.authors       = ["David Pires"]
   spec.email         = ["david.pires@gmail.com"]
   spec.description   = "Lita handler for Consul"

data/spec/lita/handlers/consul_spec.rb

@@ -4,10 +4,12 @@ describe Lita::Handlers::Consul, lita_handler: true do
   describe 'lita routes' do
     it { is_expected.to route_command('consul get mykey').to(:consul_get) }
     it { is_expected.to route_command('consul set mykey myvalue').to(:consul_set) }
+    it { is_expected.to route_command('consul members').to(:consul_members) }
   end
 
   before do
     allow_any_instance_of(Faraday::Connection).to receive(:get).and_return(response)
+    allow_any_instance_of(Faraday::Connection).to receive(:put).and_return(response)
   end
     
   let(:response) { double("Faraday::Response") }
@@ -56,12 +58,30 @@ describe Lita::Handlers::Consul, lita_handler: true do
     }
   }
 
+  let(:members_response) {
+    %{
+      [
+        {"Node":"node1.node.consul","Address":"192.168.0.33"},
+        {"Node":"node2.node.consul","Address":"192.168.0.34"}
+      ]
+    }
+  }
+
+  describe '#consul members' do
+    it 'should list member nodes' do
+      allow(response).to receive(:body).and_return(members_response)
+      send_command('consul members')
+      expect(replies.last).to eq("node1.node.consul - 192.168.0.33\nnode2.node.consul - 192.168.0.34")
+    end
+  end
+
   describe '#consul get' do
     it 'return value for key' do
       allow(response).to receive(:body).and_return(single_key_response)
       send_command('consul get mykey')
       expect(replies.last).to eq("mykey = testing")
     end
+
     it 'return null value for key' do
       allow(response).to receive(:body).and_return(null_value_response)
       send_command('consul get mykey')
@@ -72,6 +92,7 @@ describe Lita::Handlers::Consul, lita_handler: true do
   describe '#consul set' do
     it 'set and return value for key' do
       allow(response).to receive(:body).and_return(new_key_response)
+      allow(response).to receive(:status).and_return(200)
       send_command('consul set myapp/config/url www.test.com')
       expect(replies.last).to eq("myapp/config/url = www.test.com")
     end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: lita-consul
 version: !ruby/object:Gem::Version
-  version: 0.0.2
+  version: 0.0.3
 platform: ruby
 authors:
 - David Pires