Fixed possible cyclic dep when used with other modules

[pulquero]

"cwd => '/opt'" implies that /opt should exist, so there should be an explicit file {'opt':}. This fixed some cyclic dep hell for me.

[carlossg]

I don't think it is needed, from https://docs.puppetlabs.com/references/latest/type.html#exec

If Puppet is managing an exec’s cwd or the executable file used in an exec’s command, the exec resource will autorequire those files

but it doesn't say that the dir needs to be defined, and the tests are passing without it

I'm reluctant to add new dir creation because even inside ! defined the order will matter

[jyaworski]

Possibly simpler solution:

Can we just use /tmp as the cwd, and mkdir -p if we really need it to be /opt?