chef 18.8.11 → 18.8.54

data/spec/unit/provider/mount/windows_spec.rb

@@ -1,150 +0,0 @@
-#
-# Author:: Doug MacEachern (<dougm@vmware.com>)
-# Copyright:: Copyright 2010-2016, VMware, Inc.
-# License:: Apache License, Version 2.0
-#
-# Licensed under the Apache License, Version 2.0 (the "License");
-# you may not use this file except in compliance with the License.
-# You may obtain a copy of the License at
-#
-#     http://www.apache.org/licenses/LICENSE-2.0
-#
-# Unless required by applicable law or agreed to in writing, software
-# distributed under the License is distributed on an "AS IS" BASIS,
-# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-# See the License for the specific language governing permissions and
-# limitations under the License.
-#
-
-require "spec_helper"
-
-class Chef
-  class Util
-    class Windows
-      class NetUse
-      end
-
-      class Volume
-      end
-    end
-  end
-end
-
-GUID = "\\\\?\\Volume{578e72b5-6e70-11df-b5c5-000c29d4a7d9}\\".freeze
-REMOTE = "\\\\server-name\\path".freeze
-
-describe Chef::Provider::Mount::Windows do
-  before(:each) do
-    @node = Chef::Node.new
-    @events = Chef::EventDispatch::Dispatcher.new
-    @run_context = Chef::RunContext.new(@node, {}, @events)
-    @new_resource = Chef::Resource::Mount.new("X:")
-    @new_resource.device GUID
-    @current_resource = Chef::Resource::Mount.new("X:")
-    allow(Chef::Resource::Mount).to receive(:new).and_return(@current_resource)
-
-    @net_use = double("Chef::Util::Windows::NetUse")
-    allow(Chef::Util::Windows::NetUse).to receive(:new).and_return(@net_use)
-    @vol = double("Chef::Util::Windows::Volume")
-    allow(Chef::Util::Windows::Volume).to receive(:new).and_return(@vol)
-
-    @provider = Chef::Provider::Mount::Windows.new(@new_resource, @run_context)
-    @provider.current_resource = @current_resource
-  end
-
-  describe "when loading the current resource" do
-    it "should set mounted true if the mount point is found" do
-      allow(@vol).to receive(:device).and_return(@new_resource.device)
-      expect(@current_resource).to receive(:mounted).with(true)
-      @provider.load_current_resource
-    end
-
-    it "should set mounted false if the mount point is not found" do
-      allow(@vol).to receive(:device).and_raise(ArgumentError)
-      expect(@current_resource).to receive(:mounted).with(false)
-      @provider.load_current_resource
-    end
-
-    describe "with a local device" do
-      before do
-        @new_resource.device GUID
-        allow(@vol).to receive(:device).and_return(@new_resource.device)
-        allow(@net_use).to receive(:device).and_raise(ArgumentError)
-      end
-
-      it "should determine the device is a volume GUID" do
-        expect(@provider).to receive(:is_volume).with(@new_resource.device).and_return(true)
-        @provider.load_current_resource
-      end
-    end
-
-    describe "with a remote device" do
-      before do
-        @new_resource.device REMOTE
-        allow(@net_use).to receive(:device).and_return(@new_resource.device)
-        allow(@vol).to receive(:device).and_raise(ArgumentError)
-      end
-
-      it "should determine the device is remote" do
-        expect(@provider).to receive(:is_volume).with(@new_resource.device).and_return(false)
-        @provider.load_current_resource
-      end
-    end
-
-    describe "when mounting a file system" do
-      before do
-        @new_resource.device GUID
-        allow(@vol).to receive(:add)
-        allow(@vol).to receive(:device).and_raise(ArgumentError)
-        @provider.load_current_resource
-      end
-
-      it "should mount the filesystem if it is not mounted" do
-        expect(@vol).to receive(:add).with(remote: @new_resource.device,
-                                           username: @new_resource.username,
-                                           domainname: @new_resource.domain,
-                                           password: @new_resource.password)
-        @provider.mount_fs
-      end
-
-      it "should not mount the filesystem if it is mounted" do
-        expect(@vol).not_to receive(:add)
-        allow(@current_resource).to receive(:mounted).and_return(true)
-        @provider.mount_fs
-      end
-
-      context "mount_options_unchanged?" do
-        it "should return true if mounted device is the same" do
-          allow(@current_resource).to receive(:device).and_return(GUID)
-          expect(@provider.send :mount_options_unchanged?).to be true
-        end
-
-        it "should return false if mounted device has changed" do
-          allow(@current_resource).to receive(:device).and_return("XXXX")
-          expect(@provider.send :mount_options_unchanged?).to be false
-        end
-      end
-    end
-
-    describe "when unmounting a file system" do
-      before do
-        @new_resource.device GUID
-        allow(@vol).to receive(:delete)
-        allow(@vol).to receive(:device).and_raise(ArgumentError)
-        @provider.load_current_resource
-      end
-
-      it "should umount the filesystem if it is mounted" do
-        allow(@current_resource).to receive(:mounted).and_return(true)
-        expect(@vol).to receive(:delete)
-        @provider.umount_fs
-      end
-
-      it "should not umount the filesystem if it is not mounted" do
-        allow(@current_resource).to receive(:mounted).and_return(false)
-        expect(@vol).not_to receive(:delete)
-        @provider.umount_fs
-      end
-    end
-  end
-end

data/spec/unit/provider/mount_spec.rb

@@ -1,222 +0,0 @@
-#
-# Author:: Joshua Timberman (<joshua@chef.io>)
-# Author:: Lamont Granquist (<lamont@chef.io>)
-# Copyright:: Copyright (c) Chef Software Inc.
-# License:: Apache License, Version 2.0
-#
-# Licensed under the Apache License, Version 2.0 (the "License");
-# you may not use this file except in compliance with the License.
-# You may obtain a copy of the License at
-#
-#     http://www.apache.org/licenses/LICENSE-2.0
-#
-# Unless required by applicable law or agreed to in writing, software
-# distributed under the License is distributed on an "AS IS" BASIS,
-# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-# See the License for the specific language governing permissions and
-# limitations under the License.
-#
-
-require "spec_helper"
-
-describe Chef::Provider::Mount do
-
-  let(:node) { Chef::Node.new }
-
-  let(:events) { Chef::EventDispatch::Dispatcher.new }
-
-  let(:run_context) { Chef::RunContext.new(node, {}, events) }
-
-  let(:new_resource) do
-    new_resource = Chef::Resource::Mount.new("/tmp/foo")
-    new_resource.device      "/dev/sdz1"
-    new_resource.name        "/tmp/foo"
-    new_resource.mount_point "/tmp/foo"
-    new_resource.fstype      "ext3"
-    new_resource
-  end
-
-  let(:current_resource) do
-    # this abstract superclass has no load_current_resource to call
-    current_resource = Chef::Resource::Mount.new("/tmp/foo")
-    current_resource.device      "/dev/sdz1"
-    current_resource.name        "/tmp/foo"
-    current_resource.mount_point "/tmp/foo"
-    current_resource.fstype      "ext3"
-    current_resource
-  end
-
-  let(:provider) do
-    provider = Chef::Provider::Mount.new(new_resource, run_context)
-    provider.current_resource = current_resource
-    provider
-  end
-
-  describe "when the target state is a mounted filesystem" do
-
-    it "should mount the filesystem if it isn't mounted" do
-      allow(current_resource).to receive(:mounted).and_return(false)
-      expect(provider).to receive(:mount_fs).and_return(true)
-      provider.run_action(:mount)
-      expect(new_resource).to be_updated_by_last_action
-    end
-  end
-
-  describe "when the target state is an unmounted filesystem" do
-    it "should umount the filesystem if it is mounted" do
-      allow(current_resource).to receive(:mounted).and_return(true)
-      expect(provider).to receive(:umount_fs).and_return(true)
-      provider.run_action(:umount)
-      expect(new_resource).to be_updated_by_last_action
-    end
-
-    it "should unmount the filesystem if it is mounted" do
-      allow(current_resource).to receive(:mounted).and_return(true)
-      expect(provider).to receive(:umount_fs).and_return(true)
-      provider.run_action(:unmount)
-      expect(new_resource).to be_updated_by_last_action
-    end
-
-    it "should not umount the filesystem if it is not mounted" do
-      allow(current_resource).to receive(:mounted).and_return(false)
-      expect(provider).not_to receive(:umount_fs)
-      provider.run_action(:umount)
-      expect(new_resource).not_to be_updated_by_last_action
-    end
-  end
-
-  describe "when the filesystem should be remounted and the resource supports remounting" do
-    before do
-      new_resource.supports[:remount] = true
-    end
-
-    it "should remount the filesystem if it is mounted" do
-      allow(current_resource).to receive(:mounted).and_return(true)
-      expect(provider).to receive(:remount_fs).and_return(true)
-      provider.run_action(:remount)
-      expect(new_resource).to be_updated_by_last_action
-    end
-
-    it "should not remount the filesystem if it is not mounted" do
-      allow(current_resource).to receive(:mounted).and_return(false)
-      expect(provider).not_to receive(:remount_fs)
-      provider.run_action(:remount)
-      expect(new_resource).not_to be_updated_by_last_action
-    end
-  end
-
-  describe "when the filesystem should be remounted and the resource does not support remounting" do
-    before do
-      new_resource.supports[:remount] = false
-      allow(current_resource).to receive(:mounted).and_return(true)
-    end
-
-    it "should try a umount/remount of the filesystem" do
-      expect(provider).to receive(:umount_fs)
-      expect(provider).to receive(:mounted?).and_return(true, false)
-      expect(provider).to receive(:mount_fs)
-      provider.run_action(:remount)
-      expect(new_resource).to be_updated_by_last_action
-    end
-
-    it "should fail when it runs out of remounts" do
-      provider.unmount_retries = 1
-      expect(provider).to receive(:umount_fs)
-      expect(provider).to receive(:mounted?).and_return(true, true)
-      expect { provider.run_action(:remount) }.to raise_error(Chef::Exceptions::Mount)
-    end
-  end
-
-  describe "when enabling the filesystem to be mounted" do
-    it "should enable the mount if it isn't enable" do
-      allow(current_resource).to receive(:enabled).and_return(false)
-      expect(provider).not_to receive(:mount_options_unchanged?)
-      expect(provider).to receive(:enable_fs).and_return(true)
-      provider.run_action(:enable)
-      expect(new_resource).to be_updated_by_last_action
-    end
-
-    it "should enable the mount if it is enabled and mount options have changed" do
-      allow(current_resource).to receive(:enabled).and_return(true)
-      expect(provider).to receive(:mount_options_unchanged?).and_return(false)
-      expect(provider).to receive(:enable_fs).and_return(true)
-      provider.run_action(:enable)
-      expect(new_resource).to be_updated_by_last_action
-    end
-
-    it "should not enable the mount if it is enabled and mount options have not changed" do
-      allow(current_resource).to receive(:enabled).and_return(true)
-      expect(provider).to receive(:mount_options_unchanged?).and_return(true)
-      expect(provider).not_to receive(:enable_fs)
-      provider.run_action(:enable)
-      expect(new_resource).not_to be_updated_by_last_action
-    end
-
-    it "should enable the mount if device changed" do
-      allow(current_resource).to receive(:enabled).and_return(true)
-      expect(provider).to receive(:mount_options_unchanged?).and_return(true)
-      expect(provider).to receive(:device_unchanged?).and_return(false)
-      expect(provider).to receive(:enable_fs).and_return(true)
-      provider.run_action(:enable)
-      expect(new_resource).to be_updated_by_last_action
-    end
-
-    it "should not enable the mount if device not changed" do
-      allow(current_resource).to receive(:enabled).and_return(true)
-      expect(provider).to receive(:mount_options_unchanged?).and_return(true)
-      expect(provider).to receive(:device_unchanged?).and_return(true)
-      expect(provider).not_to receive(:enable_fs)
-      provider.run_action(:enable)
-      expect(new_resource).not_to be_updated_by_last_action
-    end
-  end
-
-  describe "when the target state is to disable the mount" do
-    it "should disable the mount if it is enabled" do
-      allow(current_resource).to receive(:enabled).and_return(true)
-      expect(provider).to receive(:disable_fs).and_return(true)
-      provider.run_action(:disable)
-      expect(new_resource).to be_updated_by_last_action
-    end
-
-    it "should not disable the mount if it isn't enabled" do
-      allow(current_resource).to receive(:enabled).and_return(false)
-      expect(provider).not_to receive(:disable_fs)
-      provider.run_action(:disable)
-      expect(new_resource).not_to be_updated_by_last_action
-    end
-  end
-
-  it "should delegates the mount implementation to subclasses" do
-    expect { provider.mount_fs }.to raise_error(Chef::Exceptions::UnsupportedAction)
-  end
-
-  it "should delegates the umount implementation to subclasses" do
-    expect { provider.umount_fs }.to raise_error(Chef::Exceptions::UnsupportedAction)
-  end
-
-  it "should delegates the remount implementation to subclasses" do
-    expect { provider.remount_fs }.to raise_error(Chef::Exceptions::UnsupportedAction)
-  end
-
-  it "should delegates the enable implementation to subclasses" do
-    expect { provider.enable_fs }.to raise_error(Chef::Exceptions::UnsupportedAction)
-  end
-
-  it "should delegates the disable implementation to subclasses" do
-    expect { provider.disable_fs }.to raise_error(Chef::Exceptions::UnsupportedAction)
-  end
-
-  # Not supported on solaris because it can't cope with a LABEL device type.
-  describe "#device_unchanged?", :not_supported_on_solaris do
-    it "should be true when device_type not changed" do
-      expect(provider.device_unchanged?).to be_truthy
-    end
-
-    it "should be false when device_type changed" do
-      new_resource.device_type :label
-      current_resource.device_type :device
-      expect(provider.device_unchanged?).to be_falsey
-    end
-  end
-end