Datastore provider supports setting basepath.

* acceptance test included as example.

Author: sebastiansucker

lib/puppet/provider/onedatastore/cli.rb

@@ -61,7 +61,12 @@ def exists?
   def self.get_datastore(xml)
     datastore_hash = Hash.new
     get_attributes.each do |node|
-      if node == :type
+      if node == :base_path
+        ## removes a char, one or more digits from the end
+        #  required to match basepath /tmp vs /tmp/102
+        text = xml.css("#{node.to_s.upcase}").first.text.sub!(/.\d+\z/, '')
+        datastore_hash[node] = text
+      elsif node == :type
         case xml.css("#{node.to_s.upcase}").first.text
           when '0' then text = 'IMAGE_DS'
           when '1' then text = 'SYSTEM_DS'

spec/acceptance/onedatastore_spec.rb

@@ -55,6 +55,23 @@ class { 'one':
       end
     end
 
+    context "with custom basepath" do
+      it 'should idempotently run' do
+        pp = <<-EOS
+        onedatastore { 'nfs_ds':
+          ensure    => present,
+          type      => 'system_ds',
+          tm_mad    => 'shared',
+          driver    => 'raw',
+          disk_type => 'file',
+          base_path => '/tmp',
+        }
+        EOS
+
+        apply_manifest(pp, :catch_failures => true)
+        apply_manifest(pp, :catch_changes => true)
+      end
+    end
   end
 
   describe 'when creating a File datastore' do