Test-More / Test2-Harness

Alternative to Test::Harness
Other
23 stars 26 forks source link

Fix permissions on temp dirs #189

Closed exodist closed 4 years ago

exodist commented 4 years ago

Fixes #185

rjbs commented 4 years ago

Short answer: It didn't solve the problem. I ended up with

Caught exception in subtest: Failed to start login handshake: 
$VAR1 = '500 Error in tempfile() using template /tmp/yath-229591-f2a_3Q/tmp/kKgGVq/XXXXXXXXXX: 
Could not create temp file /tmp/yath-229591-f2a_3Q/tmp/kKgGVq/6Unq1DqbCZ: Permission denied

I did not dig deeper than that, yet. The key thing to know is that yath is being run as root (not ideal, but necessary for now) and the individual test changes effective user as part of testing a service that runs as that user.

exodist commented 4 years ago

Hmm. I will probably still merge this as an improvement. But I will probably add an option to disable run/job specific temp dirs that you can use in cases like this. Probably also a directive you can put inside specific test files that need to use the system temp.