producer-core 0.1.3 → 0.1.4

checksums.yaml

@@ -0,0 +1,7 @@
+---
+SHA1:
+  metadata.gz: c9dca497744fe5fa26c09753548310eb42414a25
+  data.tar.gz: 212301a43272758c9624d5e0d2556ced790c17b3
+SHA512:
+  metadata.gz: 7f456c2225b377db0d58df63d3207fce98a103d18501a0d30969fc75c243c094888ba667dfa109cfb1aef878243fa161a5e800dfe0941bfda7c98dd760bce8ee
+  data.tar.gz: e4eb9ce9cc707064a9318b0971971231ef9a42279b5f17d9038bb99d2e1d6ae9ec84eccd1b1e4d67486163eb86e817a24bddeeb756bf51376d30b9badf662585

data/features/ssh/config.feature

@@ -0,0 +1,22 @@
+Feature: SSH settings
+
+  Background:
+    Given a recipe with:
+      """
+      target 'some_host.example'
+
+      puts env.remote.user_name
+      """
+
+  Scenario: uses current user login name as SSH user name by default
+    When I successfully execute the recipe
+    Then the output must contain my current login name
+
+  Scenario: uses configured SSH user name for a given host
+    Given an SSH config with:
+      """
+      Host some_host.example
+        User some_user
+      """
+    When I successfully execute the recipe
+    Then the output must contain "some_user"

data/features/steps/etc_steps.rb

@@ -0,0 +1,5 @@
+# FIXME: our monkey patch currently prevent us from using `must' in step
+# definitions.
+Then(/^the output should contain my current login name$/) do
+  assert_partial_output(Etc.getlogin, all_output)
+end

data/features/steps/ssh_steps.rb

@@ -0,0 +1,6 @@
+# FIXME: current home directory shouldn't be changed here, maybe we should use
+# a tag for features needing a fake home directory.
+Given(/^an SSH config with:$/) do |config|
+  ENV['HOME'] = File.expand_path current_dir
+  write_file '.ssh/config', config
+end

data/lib/producer/core/remote.rb

@@ -11,7 +11,15 @@ module Producer
       end
 
       def session
-        @session ||= Net::SSH.start(@hostname, Etc.getlogin)
+        @session ||= Net::SSH.start(@hostname, user_name)
+      end
+
+      def config
+        @config ||= Net::SSH::Config.for(@hostname)
+      end
+
+      def user_name
+        config[:user] || Etc.getlogin
       end
 
       def fs

data/lib/producer/core/version.rb

@@ -1,5 +1,5 @@
 module Producer
   module Core
-    VERSION = '0.1.3'
+    VERSION = '0.1.4'
   end
 end

data/spec/producer/core/remote/fs_spec.rb

@@ -66,7 +66,7 @@ module Producer::Core
 
       context 'when querying the path raises a Net::SFTP::StatusException' do
         before do
-          response = double('response', code: '42', message: '…')
+          response = double('response', code: '42', message: 'some message')
           ex = Net::SFTP::StatusException.new(response)
           allow(stat).to receive(:file?).and_raise(ex)
         end

data/spec/producer/core/remote_spec.rb

@@ -12,8 +12,8 @@ module Producer::Core
     end
 
     describe '#session' do
-      it 'builds a new SSH session to the remote host' do
-        expect(Net::SSH).to receive(:start).with(hostname, Etc.getlogin)
+      it 'builds a new SSH session to the remote host with #user_name' do
+        expect(Net::SSH).to receive(:start).with(hostname, remote.user_name)
         remote.session
       end
 
@@ -29,6 +29,44 @@ module Producer::Core
       end
     end
 
+    describe '#config' do
+      it 'builds a config for current host name' do
+        expect(Net::SSH::Config).to receive(:for).with(hostname)
+        remote.config
+      end
+
+      it 'returns the config' do
+        ssh_config = double('ssh config')
+        allow(Net::SSH::Config).to receive(:for) { ssh_config }
+        expect(remote.config).to be ssh_config
+      end
+
+      it 'memoizes the config' do
+        allow(Net::SSH::Config).to receive(:for) { Object.new }
+        expect(remote.config).to be remote.config
+      end
+    end
+
+    describe '#user_name' do
+      context 'ssh config has an entry for user' do
+        let(:config_user_name) { 'my_user_name' }
+
+        before do
+          allow(Net::SSH::Config).to receive(:for) { { user: config_user_name } }
+        end
+
+        it 'returns the configured value' do
+          expect(remote.user_name).to eq config_user_name
+        end
+      end
+
+      context 'ssh config has no entry for user' do
+        it 'returns the name of the user currently logged in' do
+          expect(remote.user_name).to eq Etc.getlogin
+        end
+      end
+    end
+
     describe '#fs' do
       it 'builds a new FS' do
         expect(Remote::FS).to receive(:new)

metadata

@@ -1,122 +1,108 @@
 --- !ruby/object:Gem::Specification
 name: producer-core
 version: !ruby/object:Gem::Version
-  version: 0.1.3
-  prerelease: 
+  version: 0.1.4
 platform: ruby
 authors:
 - Thibault Jouan
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-09-26 00:00:00.000000000 Z
+date: 2015-04-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: net-ssh
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: net-sftp
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: cucumber
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: aruba
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-description: ! 'Software provisioning tool, including a DSL to write "recipes".
-
-'
+description: |
+  Software provisioning tool, including a DSL to write "recipes".
 email: tj@a13.fr
 executables:
 - producer
 extensions: []
 extra_rdoc_files: []
 files:
-- .gitignore
+- ".gitignore"
 - Gemfile
 - Guardfile
 - LICENSE
@@ -129,8 +115,11 @@ files:
 - features/recipes/evaluation.feature
 - features/recipes/source.feature
 - features/recipes/target.feature
+- features/ssh/config.feature
+- features/steps/etc_steps.rb
 - features/steps/recipe_steps.rb
 - features/steps/remote_steps.rb
+- features/steps/ssh_steps.rb
 - features/support/env_aruba.rb
 - features/support/env_cucumber-doc_string.rb
 - features/support/ssh.rb
@@ -187,28 +176,27 @@ files:
 - spec/support/tests_helpers.rb
 homepage: https://rubygems.org/gems/producer-core
 licenses: []
+metadata: {}
 post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
-  - - ! '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
-  - - ! '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 1.8.23
+rubygems_version: 2.4.5
 signing_key: 
-specification_version: 3
-summary: producer-core-0.1.3
+specification_version: 4
+summary: producer-core-0.1.4
 test_files:
 - features/actions/echo.feature
 - features/actions/sh.feature
@@ -217,8 +205,11 @@ test_files:
 - features/recipes/evaluation.feature
 - features/recipes/source.feature
 - features/recipes/target.feature
+- features/ssh/config.feature
+- features/steps/etc_steps.rb
 - features/steps/recipe_steps.rb
 - features/steps/remote_steps.rb
+- features/steps/ssh_steps.rb
 - features/support/env_aruba.rb
 - features/support/env_cucumber-doc_string.rb
 - features/support/ssh.rb