modifié : spec/acceptance/basic_samba_spec.rb

modifié : spec/defines/samba__server__share_spec.rb
ajjahn · Mar 15, 2017 · 9448bb8 · 9448bb8
1 parent bcfa15b
commit 9448bb8
Show file tree

Hide file tree

Showing 2 changed files with 23 additions and 22 deletions.
diff --git a/spec/acceptance/basic_samba_spec.rb b/spec/acceptance/basic_samba_spec.rb
@@ -11,21 +11,22 @@ class { 'samba::server':
       }
 
       samba::server::share {'example-share':
-        comment              => 'Example Share',
-        path                 => '/path/to/share',
-        guest_only           => true,
-        guest_ok             => true,
-        guest_account        => 'guest',
-        browsable            => false,
-        create_mask          => 0777,
-        force_create_mask    => 0777,
-        directory_mask       => 0777,
-        force_directory_mode => 0777,
-        force_group          => 'group',
-        force_user           => 'user',
-        hide_dot_files       => false,
-        msdfs_root           => true,
-        hosts_allow          => '127.0.0.1'
+        comment                   => 'Example Share',
+        path                      => '/path/to/share',
+        guest_only                => true,
+        guest_ok                  => true,
+        guest_account             => 'guest',
+        browsable                 => false,
+        create_mask               => 0777,
+        force_create_mask         => 0777,
+        directory_mask            => 0777,
+        force_directory_mode      => 0777,
+        force_group               => 'group',
+        force_user                => 'user',
+        hide_dot_files            => false,
+        msdfs_root                => true,
+        hosts_allow               => '127.0.0.1',
+        acl_allow_execute_always  => true,
       }
     "}
 

diff --git a/spec/defines/samba__server__share_spec.rb b/spec/defines/samba__server__share_spec.rb
@@ -679,7 +679,7 @@
       context 'with guest_account set to "killing trees"' do
         include_examples "default share"
         let(:params) {{
-          :ensure       => 'present',
+          :ensure        => 'present',
           :guest_account => 'someone',
         }}
         let(:change_set) { default_changes.with("guest account", "'someone'") }
@@ -689,18 +689,18 @@
         include_examples "default share"
         let(:params) {{
           :ensure       => 'present',
-          :hosts_allow => '127.0.0.1',
+          :hosts_allow  => '127.0.0.1',
         }}
-        let(:change_set) { default_changes.with("hosts allow", "127.0.0.1") }
+        let(:change_set) { default_changes.with("hosts allow", "'127.0.0.1'") }
       end
 
-      context 'with acl_allow_execute_always to "true"' do
+      context 'with acl_allow_execute_always to "yes"' do
         include_examples "default share"
         let(:params) {{
-          :ensure       => 'present',
-          :acl_allow_execute_always => true,
+          :ensure                   => 'present',
+          :acl_allow_execute_always => 'yes',
         }}
-        let(:change_set) { default_changes.with("acl allow execute always", "true") }
+        let(:change_set) { default_changes.with("acl allow execute always", "yes") }
       end
     end
   end