producer-core 0.1.5 → 0.1.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 2806c20d86e7d0aa08f72a3005b5ee5456cad346
-  data.tar.gz: 4c75c67cd9815e30849302aeb9021dcbaf4721a7
+  metadata.gz: 69bc1d881cfd85eea00037b60c396d9b09bf7bdd
+  data.tar.gz: 6d440b7bd0beb137b073d00c2096222bb7f2b58a
 SHA512:
-  metadata.gz: 4cddfc09e41207a5d9f01a9e04bac5c7ec7f841e89ce1e225d073923e5555460461b92f93b2978508c1003d8bdd24c92f9f106e947226190ca165ef78ae10b75
-  data.tar.gz: 3bace5c3bbba48e3727e380f96d5f4a31f11fd29e9a319a7c09b2e284c400c2b105352bca4b3ae4ebfcabb5159faba99b9d88fa4e47153dfd4ff747b92b70d92
+  metadata.gz: 837737ed2a5bc77959a9c0e7c8295acdc7a997884a63fd286035cb9d811191ac7097d93ed5fb3e24c0c8c0afe22db4d05ca6b45afefcc4f5a69af34da3595e74
+  data.tar.gz: 098654fa4620affd4eac4b89fb8906a57537c13b134b9ab8c1a47e48cff83cce96d48b8cdbeed4cd50df2de4e95fbdfa7cd9589c4f5be04dedee3810630cc26d

data/features/actions/file_write.feature

@@ -0,0 +1,15 @@
+@sshd
+Feature: `file_write' task action
+
+  Scenario: writes given data to given file path
+    Given a recipe with:
+      """
+      target 'some_host.test'
+
+      task :write_some_data do
+        file_write 'some_file', 'some_content'
+      end
+      """
+    When I execute the recipe
+    Then the exit status must be 0
+    And the remote file "some_file" must contain "some_content"

data/features/steps/remote_steps.rb

@@ -1,3 +1,7 @@
-Given(/^a remote file named "(.*?)"$/) do |file_name|
+Given /^a remote file named "([^"]+)"$/ do |file_name|
   write_file file_name, ''
 end
+
+Then /^the remote file "([^"]+)" should contain "([^"]+)"/ do |path, content|
+  check_file_content path, content, true
+end

data/lib/producer/core/actions/file_writer.rb

@@ -0,0 +1,19 @@
+module Producer
+  module Core
+    module Actions
+      class FileWriter < Action
+        def apply
+          env.remote.fs.file_write path, content
+        end
+
+        def path
+          arguments[0]
+        end
+
+        def content
+          arguments[1]
+        end
+      end
+    end
+  end
+end

data/lib/producer/core/remote/fs.rb

@@ -17,6 +17,12 @@ module Producer
         rescue Net::SFTP::StatusException
           false
         end
+
+        def file_write(path, content)
+          sftp.file.open path, 'w' do |f|
+            f.write content
+          end
+        end
       end
     end
   end

data/lib/producer/core/task/dsl.rb

@@ -19,6 +19,8 @@ module Producer
         define_action :echo,  Actions::Echo
         define_action :sh,    Actions::ShellCommand
 
+        define_action :file_write,  Actions::FileWriter
+
         attr_accessor :actions
 
         def initialize(&block)

data/lib/producer/core/version.rb

@@ -1,5 +1,5 @@
 module Producer
   module Core
-    VERSION = '0.1.5'
+    VERSION = '0.1.6'
   end
 end

data/lib/producer/core.rb

@@ -2,6 +2,7 @@
 require 'producer/core/action'
 require 'producer/core/actions/echo'
 require 'producer/core/actions/shell_command'
+require 'producer/core/actions/file_writer'
 
 # condition tests (need to be defined before the condition DSL)
 require 'producer/core/test'

data/spec/producer/core/actions/file_writer_spec.rb

@@ -0,0 +1,29 @@
+require 'spec_helper'
+
+module Producer::Core
+  describe Actions::FileWriter do
+    let(:env)           { Env.new }
+    let(:path)          { 'some_path' }
+    let(:content)       { 'some_content' }
+    subject(:writer)    { Actions::FileWriter.new(env, path, content) }
+
+    describe '#apply' do
+      it 'delegates the call to env.remote.fs.file_write method' do
+        expect(env.remote.fs).to receive(:file_write).with(path, content)
+        writer.apply
+      end
+    end
+
+    describe '#path' do
+      it 'returns the path' do
+        expect(writer.path).to eq path
+      end
+    end
+
+    describe '#content' do
+      it 'returns the content' do
+        expect(writer.content).to eq content
+      end
+    end
+  end
+end

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

@@ -34,7 +34,7 @@ module Producer::Core
     # FIXME: We rely a lot on mocking net-sftp heavily, while we already use a
     # part of net-ssh story helpers, which are more close to integration tests.
     describe '#has_file?', :ssh do
-      let(:file_path) { "some_file_path" }
+      let(:file_path) { 'some_file_path' }
       let(:stat)      { double 'stat' }
 
       before do
@@ -72,5 +72,30 @@ module Producer::Core
         end
       end
     end
+
+    describe '#file_write' do
+      let(:sftp)    { double('sftp').as_null_object }
+      let(:file)    { double('sftp').as_null_object }
+      let(:path)    { 'some_file_path' }
+      let(:content) { 'some_content' }
+
+      before do
+        allow(fs).to receive(:sftp) { sftp }
+        allow(sftp).to receive(:file) { file }
+      end
+
+      it 'opens the file' do
+        expect(file).to receive(:open).with(path, 'w')
+        fs.file_write path, content
+      end
+
+      it 'writes the content' do
+        expect(file).to receive(:open).with(any_args) do |&b|
+          expect(file).to receive(:write).with(content)
+          b.call file
+        end
+        fs.file_write path, content
+      end
+    end
   end
 end

data/spec/producer/core/remote_spec.rb

@@ -94,8 +94,7 @@ module Producer::Core
           ch.sends_exec command
           ch.gets_data arguments
         end
-        remote.execute command
-        expect(story_completed?).to be
+        expect_story_completed { remote.execute command }
       end
 
       it 'returns the output' do

data/spec/producer/core/task/dsl_spec.rb

@@ -6,7 +6,7 @@ module Producer::Core
     let(:env)     { double 'env' }
     subject(:dsl) { Task::DSL.new(&block) }
 
-    %w[echo sh].each do |action|
+    %w[echo sh file_write].each do |action|
       it "has `#{action}' action defined" do
         expect(dsl).to respond_to action.to_sym
       end

data/spec/support/net_ssh_story_helpers.rb

@@ -29,10 +29,6 @@ module NetSSHStoryHelpers
     end
   end
 
-  def story_completed?
-    socket.script.events.empty?
-  end
-
   def sftp_story
     story do |session|
       ch = session.opens_channel
@@ -48,4 +44,11 @@ module NetSSHStoryHelpers
       yield ch if block_given?
     end
   end
+
+  def expect_story_completed
+    raise 'there is no story to expect' if socket.script.events.empty?
+    yield
+    expect(socket.script.events)
+      .to be_empty, "#{socket.script.events.count} story events still pending"
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: producer-core
 version: !ruby/object:Gem::Version
-  version: 0.1.5
+  version: 0.1.6
 platform: ruby
 authors:
 - Thibault Jouan
@@ -109,6 +109,7 @@ files:
 - Rakefile
 - bin/producer
 - features/actions/echo.feature
+- features/actions/file_write.feature
 - features/actions/sh.feature
 - features/cli/usage.feature
 - features/recipes/env.feature
@@ -131,6 +132,7 @@ files:
 - lib/producer/core.rb
 - lib/producer/core/action.rb
 - lib/producer/core/actions/echo.rb
+- lib/producer/core/actions/file_writer.rb
 - lib/producer/core/actions/shell_command.rb
 - lib/producer/core/cli.rb
 - lib/producer/core/condition.rb
@@ -154,6 +156,7 @@ files:
 - spec/fixtures/recipes/throw.rb
 - spec/producer/core/action_spec.rb
 - spec/producer/core/actions/echo_spec.rb
+- spec/producer/core/actions/file_writer_spec.rb
 - spec/producer/core/actions/shell_command_spec.rb
 - spec/producer/core/cli_spec.rb
 - spec/producer/core/condition/dsl_spec.rb
@@ -197,9 +200,10 @@ rubyforge_project:
 rubygems_version: 2.4.5
 signing_key: 
 specification_version: 4
-summary: producer-core-0.1.5
+summary: producer-core-0.1.6
 test_files:
 - features/actions/echo.feature
+- features/actions/file_write.feature
 - features/actions/sh.feature
 - features/cli/usage.feature
 - features/recipes/env.feature
@@ -223,6 +227,7 @@ test_files:
 - spec/fixtures/recipes/throw.rb
 - spec/producer/core/action_spec.rb
 - spec/producer/core/actions/echo_spec.rb
+- spec/producer/core/actions/file_writer_spec.rb
 - spec/producer/core/actions/shell_command_spec.rb
 - spec/producer/core/cli_spec.rb
 - spec/producer/core/condition/dsl_spec.rb