cartodb-org / cartodb

Location Intelligence & Data Visualization tool
http://cartodb.com
BSD 3-Clause "New" or "Revised" License
3 stars 2 forks source link

Changes unp_spec crawl test to use a unique temporary directory to av… #227

Closed acarrera42 closed 6 years ago

acarrera42 commented 6 years ago

…oid conflicts with other temp files

acarrera42 commented 6 years ago

The current directory /var/tmp can cause conflicts because systemd can add root owned files to this which will lead to permission denied errors.

acarrera42 commented 6 years ago

Updated to use Tempfile. However Tempfile will by default use the OS's default tmp directory which in this case is '/tmp'. This would cause the same conflict problems since unp.crawl() will try to crawl the entire directory yielding other temp files. Therefore we still need to specify a specific directory that is unique to this test for it to consistently work.