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

This avoids pointless warnings about invalid modules that started with puppetlabs/puppet#3261.
How do you see the `modulepath` method being used in this situation?