A failed java installation does not get noticed by Puppet

[otteydw]

I ran in to a situation where I used an incorrect $install_path format.

Example:
windows_java::jdk{"7u55":
install_path => "c:/java/jdk1.7.0_55",
build_number_hash => {
'8u66' => 'b17',
'8u31' => 'b13',
'7u55' => 'b13',
},
default => false,
}

Notice the forward slashes instead of backslashes.

When doing this, Puppet did not die or error in any way. It seemed like the install was successful and then moved on to the next block/section.

I'd like to have forward slashes be allowed (it would decrease my amount on Linux/Windows logic needed in my manifests), but more importantly I'd like this module to FAIL if the installation doesn't succeed. If the installation provides any error output, it should get displayed/captured by Puppet.