Fix for failing test’s

Removed a test file that was missed by my first sweep
Updated if statement to account for differences between the now removed built in dirname function and the stdlib version

Author: david22swan

Diff for: manifests/server/config.pp

@@ -44,11 +44,16 @@
     # on mariadb systems, $includedir is not defined, but /etc/my.cnf.d has
     # to be managed to place the server.cnf there
     $configparentdir = dirname($mysql::server::config_file)
-    if $configparentdir != '/' and $configparentdir != '/etc' and $configparentdir
-        != $includedir and $configparentdir != dirname($includedir) {
-      file { $configparentdir:
-        ensure => directory,
-        mode   => '0755',
+    # Before setting $configparentdir we first check to make sure that it's value is valid
+    if $configparentdir != '/' and $configparentdir != '/etc' {
+      # We then check that the value of $includedir is either undefined or that different from $configparentdir
+      # We first check that it is undefined due to dirname throwing an error when given undef/empty strings
+      if $includedir == undef or $includedir == '' or
+        ($configparentdir != $includedir and $configparentdir != dirname($includedir)) {
+        file { $configparentdir:
+          ensure => directory,
+          mode   => '0755',
+        }
       }
     }
   }