diff options
author | Gavin Reynolds <gavin@chef.io> | 2019-02-07 14:59:11 +0100 |
---|---|---|
committer | Tim Smith <tsmith@chef.io> | 2019-03-06 20:48:45 -0800 |
commit | 8103e0a2d8a612226976fe8fda64c769a0ed5283 (patch) | |
tree | 10c1fa59fd08b90559913a1ec674d27014e0b00c | |
parent | a0fa852130aec281caa0001b09fbdc92447b9c36 (diff) | |
download | chef-8103e0a2d8a612226976fe8fda64c769a0ed5283.tar.gz |
Ensure that a new blank line `\n\n` is added when appending a new mount point for AIX.aix_mount_13
Add existing mount to enable spec
Signed-off-by: Gavin Reynolds <gavin@chef.io>
-rw-r--r-- | lib/chef/provider/mount/aix.rb | 2 | ||||
-rw-r--r-- | spec/unit/provider/mount/aix_spec.rb | 10 |
2 files changed, 10 insertions, 2 deletions
diff --git a/lib/chef/provider/mount/aix.rb b/lib/chef/provider/mount/aix.rb index b5641ee246..a63649f470 100644 --- a/lib/chef/provider/mount/aix.rb +++ b/lib/chef/provider/mount/aix.rb @@ -164,7 +164,7 @@ class Chef disable_fs end ::File.open("/etc/filesystems", "a") do |fstab| - fstab.puts("\n#{@new_resource.mount_point}:") + fstab.puts("\n\n#{@new_resource.mount_point}:") if network_device? device_details = device_fstab.split(":") fstab.puts("\tdev\t\t= #{device_details[1]}") diff --git a/spec/unit/provider/mount/aix_spec.rb b/spec/unit/provider/mount/aix_spec.rb index 615f3c3304..5b7c48e3d7 100644 --- a/spec/unit/provider/mount/aix_spec.rb +++ b/spec/unit/provider/mount/aix_spec.rb @@ -202,12 +202,20 @@ WRONG it "should enable mount if it is mounted and not enabled" do @new_resource.options("nodev,rw") stub_mounted_enabled(@provider, @mounted_output, "") + # Add existing mount to test enable action appends additional mount with seperating blank line filesystems = StringIO.new + filesystems.puts <<~ETCFILESYSTEMS + /tmp/abc: + dev = /dev/sdz2 + vfs = jfs2 + mount = true + options = rw + ETCFILESYSTEMS allow(::File).to receive(:open).with("/etc/filesystems", "a").and_yield(filesystems) @provider.run_action(:enable) - expect(filesystems.string).to match(%r{^/tmp/foo:\n\tdev\t\t= /dev/sdz1\n\tvfs\t\t= jfs2\n\tmount\t\t= false\n\toptions\t\t= nodev,rw\n$}) + expect(filesystems.string).to match(%r{^\n\n/tmp/foo:\n\tdev\t\t= /dev/sdz1\n\tvfs\t\t= jfs2\n\tmount\t\t= false\n\toptions\t\t= nodev,rw\n$}) end it "should not enable mount if it is mounted and already enabled and mount options are unchanged" do |