producer-core 0.2.2 → 0.2.3

data/features/actions/file_write.feature

@@ -11,4 +11,18 @@ Feature: `file_write' task action
       end
       """
     When I successfully execute the recipe
-    And the remote file "some_file" must contain "some_content"
+    Then the remote file "some_file" must contain "some_content"
+
+  Scenario: creates file with given permissions
+    Given a recipe with:
+      """
+      target 'some_host.test'
+
+      task :write_some_data do
+        file_write 'some_file_0600', 'some_content', 0600
+        file_write 'some_file_0700', 'some_content', 0700
+      end
+      """
+    When I successfully execute the recipe
+    Then the remote file "some_file_0600" must have 0600 mode
+    And  the remote file "some_file_0700" must have 0700 mode

data/features/actions/mkdir.feature

@@ -12,3 +12,17 @@ Feature: `mkdir' task action
       """
     When I successfully execute the recipe
     Then the remote directory "some_directory" must exists
+
+  Scenario: creates directory with given permissions
+    Given a recipe with:
+      """
+      target 'some_host.test'
+
+      task :create_some_dir do
+        mkdir '0700_directory', 0700
+        mkdir '0500_directory', 0500
+      end
+      """
+    When I successfully execute the recipe
+    Then the remote directory "0700_directory" must have 0700 mode
+    And  the remote directory "0500_directory" must have 0500 mode

data/features/steps/remote_steps.rb

@@ -21,3 +21,17 @@ end
 Then /^the remote file "([^"]+)" must contain exactly "([^"]+)"$/ do |path, content|
   check_exact_file_content path, content
 end
+
+def stat_mode(path)
+  in_current_dir do
+    ('%o' % [File::Stat.new(path).mode])[-4, 4]
+  end
+end
+
+Then /^the remote file "([^"]+)" must have (\d+) mode$/ do |path, mode|
+  expect(stat_mode path).to eq mode
+end
+
+Then /^the remote directory "([^"]+)" must have (\d+) mode$/ do |path, mode|
+  expect(stat_mode path).to eq mode
+end

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

@@ -3,7 +3,12 @@ module Producer
     module Actions
       class FileWriter < Action
         def apply
-          fs.file_write path, content
+          case arguments.size
+          when 2
+            fs.file_write path, content
+          when 3
+            fs.file_write path, content, mode
+          end
         end
 
         def path
@@ -13,6 +18,10 @@ module Producer
         def content
           arguments[1]
         end
+
+        def mode
+          arguments[2]
+        end
       end
     end
   end

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

@@ -3,12 +3,21 @@ module Producer
     module Actions
       class Mkdir < Action
         def apply
-          fs.mkdir path
+          case arguments.size
+          when 1
+            fs.mkdir path
+          when 2
+            fs.mkdir path, mode
+          end
         end
 
         def path
           arguments.first
         end
+
+        def mode
+          arguments[1]
+        end
       end
     end
   end

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

@@ -20,8 +20,9 @@ module Producer
           false
         end
 
-        def mkdir(path)
-          sftp.mkdir! path
+        def mkdir(path, mode = nil)
+          options = mode ? { permissions: mode } : {}
+          sftp.mkdir! path, options
         end
 
         def file_read(path)
@@ -30,8 +31,8 @@ module Producer
           nil
         end
 
-        def file_write(path, content)
-          sftp.file.open path, 'w' do |f|
+        def file_write(path, content, mode = nil)
+          sftp.file.open path, 'w', mode do |f|
             f.write content
           end
         end

data/lib/producer/core/version.rb

@@ -1,5 +1,5 @@
 module Producer
   module Core
-    VERSION = '0.2.2'
+    VERSION = '0.2.3'
   end
 end

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

@@ -14,6 +14,16 @@ module Producer::Core
           expect(remote_fs).to receive(:file_write).with(path, content)
           writer.apply
         end
+
+        context 'when a mode was given' do
+          subject(:writer) { FileWriter.new(env, path, content, 0600) }
+
+          it 'specifies the given mode' do
+            expect(remote_fs)
+              .to receive(:file_write).with(anything, anything, 0600)
+            writer.apply
+          end
+        end
       end
 
       describe '#path' do
@@ -27,6 +37,20 @@ module Producer::Core
           expect(writer.content).to eq content
         end
       end
+
+      describe '#mode' do
+        it 'returns nil' do
+          expect(writer.mode).to be nil
+        end
+
+        context 'when a mode was given' do
+          subject(:writer) { FileWriter.new(env, path, content, 0600) }
+
+          it 'returns the mode' do
+            expect(writer.mode).to eq 0600
+          end
+        end
+      end
     end
   end
 end

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

@@ -13,6 +13,15 @@ module Producer::Core
           expect(remote_fs).to receive(:mkdir).with(path)
           mkdir.apply
         end
+
+        context 'when a mode was given' do
+          subject(:mkdir) { Mkdir.new(env, path, 0700) }
+
+          it 'creates the directory with given mode' do
+            expect(remote_fs).to receive(:mkdir).with(anything, 0700)
+            mkdir.apply
+          end
+        end
       end
 
       describe '#path' do
@@ -20,6 +29,20 @@ module Producer::Core
           expect(mkdir.path).to eq path
         end
       end
+
+      describe '#mode' do
+        it 'returns nil' do
+          expect(mkdir.mode).to be nil
+        end
+
+        context 'when a mode was given' do
+          subject(:mkdir) { Mkdir.new(env, path, 0700) }
+
+          it 'returns the mode' do
+            expect(mkdir.mode).to be 0700
+          end
+        end
+      end
     end
   end
 end

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

@@ -89,9 +89,16 @@ module Producer::Core
         let(:path) { 'some_directory_path' }
 
         it 'creates the directory' do
-          expect(sftp).to receive(:mkdir!).with(path)
+          expect(sftp).to receive(:mkdir!).with(path, anything)
           fs.mkdir path
         end
+
+        it 'specifies permissions from optional mode argument' do
+          expect(sftp).to receive(:mkdir!) do |_, options|
+            expect(options[:permissions]).to eq 0700
+          end
+          fs.mkdir path, 0700
+        end
       end
 
       describe '#file_read' do
@@ -126,7 +133,7 @@ module Producer::Core
         let(:content) { 'some_content' }
 
         it 'opens the file' do
-          expect(sftp_file).to receive(:open).with(path, 'w')
+          expect(sftp_file).to receive(:open).with(path, 'w', anything)
           fs.file_write path, content
         end
 
@@ -137,6 +144,11 @@ module Producer::Core
           end
           fs.file_write path, content
         end
+
+        it 'accepts an optional mode argument' do
+          expect(sftp_file).to receive(:open).with(anything, anything, 0600)
+          fs.file_write path, content, 0600
+        end
       end
     end
   end

metadata

@@ -1,18 +1,20 @@
 --- !ruby/object:Gem::Specification
 name: producer-core
 version: !ruby/object:Gem::Version
-  version: 0.2.2
+  version: 0.2.3
+  prerelease: 
 platform: ruby
 authors:
 - Thibault Jouan
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-04-03 00:00:00.000000000 Z
+date: 2014-05-09 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
@@ -20,6 +22,7 @@ dependencies:
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
@@ -27,6 +30,7 @@ dependencies:
 - !ruby/object:Gem::Dependency
   name: net-sftp
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
@@ -34,6 +38,7 @@ dependencies:
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
@@ -41,6 +46,7 @@ dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
@@ -48,6 +54,7 @@ dependencies:
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
@@ -55,6 +62,7 @@ dependencies:
 - !ruby/object:Gem::Dependency
   name: cucumber
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
@@ -62,6 +70,7 @@ dependencies:
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
@@ -69,6 +78,7 @@ dependencies:
 - !ruby/object:Gem::Dependency
   name: aruba
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
@@ -76,6 +86,7 @@ dependencies:
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
@@ -83,6 +94,7 @@ dependencies:
 - !ruby/object:Gem::Dependency
   name: cucumber-sshd
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
@@ -90,6 +102,7 @@ dependencies:
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
@@ -97,6 +110,7 @@ dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
@@ -104,6 +118,7 @@ dependencies:
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
@@ -228,26 +243,27 @@ files:
 - spec/support/test_env_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: 2.4.5
+rubygems_version: 1.8.29
 signing_key: 
-specification_version: 4
+specification_version: 3
 summary: Provisioning tool
 test_files:
 - features/actions/echo.feature

checksums.yaml

@@ -1,7 +0,0 @@
----
-SHA1:
-  metadata.gz: b0948086dee94e6a1363c83af78cbc8fdec0c6d8
-  data.tar.gz: 6994469ae9fce4b8c7184545b453b3304555f648
-SHA512:
-  metadata.gz: e783aa0c29560a4c5f6f13c6ea5ab4367ca5031c2f5b766d351b8087bd64b0815deea04d355c0eb70a86a95db0b68076e2a9fb1aae405ca8e5a630635c547d90
-  data.tar.gz: c38e3b3445bbd59f58c0b8467fabf64c2a14c79b20aa9cbc6ca9586e1ba69d41bf84e3dc3e6bebb63d07e46ba5a8d301612318c7c93ae8443ee36d03e8793f22