(PUP-3678) skip non-directories on the module path

This avoids pointless warnings about invalid modules that started with puppetlabs/puppet#3261.
This should probably be a `Puppet::FileSystem.directory?` call (also assuming that `fullpath` is a string at this point; this is currently the source of the Travis failures).