ssh-short 0.3.0 → 0.3.1

checksums.yaml

@@ -1,7 +1,7 @@
 --- 
 SHA512: 
-  data.tar.gz: 013894983677178843ed84e84de179db2646b94851fbb8352bec508d392d0bef9f4473178cf6bb9170f8882ba2dc6173f45cf1673236be8e7794631e81d565cd
-  metadata.gz: f0950a8ffdcdbcbe3f69fe3b92d34d9be9f69aa713d76d5459f36794c2a87ab43ececc0660b4de31632d5645cb57bfa030e7eb17c650d10038590baa224a4c45
+  metadata.gz: 8a9a320436c6e0b8a938afdd293d1858ccd464d573d9b00ed1107968aae483f2d90c0de3951cf9be001aed63d99061a3fd00f8be52ab09cc0330afb35ef42308
+  data.tar.gz: 463417fcb7a6d501490505394025a3622d3280a9170012804c99cf791662e7a948ab9f75adbe81c4cda04df55fc65c15e64bd8a24e3eb92f85b64088f7832711
 SHA1: 
-  data.tar.gz: 31394bed9beb69c61f3cfc15142abe3ed88d1e7f
-  metadata.gz: 9ae918535b15ed6c1ea250f7b12d9902bdb12820
+  metadata.gz: 640dc503747f8c948f83342d6f29f8d803a618b6
+  data.tar.gz: caef154ae31d2e1818976f4837ccc4ceb28e83d3

data/lib/ssh_short/nodemapper.rb

@@ -22,8 +22,7 @@ module SshShort
     end
 
     def update_node(node)
-      alias_changed = alias_changed? node
-      if alias_changed
+      if alias_changed? node
         existing_node_with_alias = get_node_by_alias(node[:alias])
         if existing_node_with_alias
           puts "Moving alias #{node[:alias]} from #{existing_node_with_alias[:host]} to #{node[:host]}"
@@ -55,6 +54,7 @@ module SshShort
     private
 
     def alias_changed?(node)
+      return false unless node[:alias]
       old_node = get_node_by_host node[:host]
       old_node ? old_node[:alias] != node[:alias] : true
     end

data/lib/ssh_short/version.rb

@@ -1,3 +1,3 @@
 module SshShort
-  VERSION = '0.3.0'
+  VERSION = '0.3.1'
 end

data/spec/ssh_short/nodemapper_spec.rb

@@ -141,6 +141,23 @@ describe SshShort::NodeMapper do
 
     end
 
+    context 'when alias is nil' do
+
+      let(:node) { {:host => '10.0.0.6', :key => 'key_for_6.pem'} }
+      let(:nodemap) {
+        [
+            {:host => '10.0.0.1', :alias => 'alice', :key => 'key_for_alice.pem'},
+            {:host => '10.0.0.2', :key => 'key_for_2.pem'},
+        ]
+      }
+
+      it 'does not move the empty alias' do
+        expect(nodemapper).to_not receive(:puts).with(/Moving alias/)
+        nodemapper.update_node(node)
+      end
+
+    end
+
   end
 
   describe 'get_aliases' do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: ssh-short
 version: !ruby/object:Gem::Version 
-  version: 0.3.0
+  version: 0.3.1
 platform: ruby
 authors: 
 - Tom Poulton
@@ -9,7 +9,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2016-03-15 00:00:00 Z
+date: 2016-04-26 00:00:00 Z
 dependencies: []
 
 description: Easy ssh