Skip to content

Commit

Permalink
Disable rm
Browse files Browse the repository at this point in the history
  • Loading branch information
@cstjean
cstjean committed May 12, 2017
1 parent 95a05ef commit d745f50
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion test/runtests.jl
View file
Original file line number Diff line number Diff line change
Expand Up @@ -19,5 +19,6 @@ try
save(tmp, "x", obj)
@test load(tmp, "x") == obj
finally
rm(dir, force=true, recursive=true)
# Disabled because it fails on Travis-Linux
#rm(dir, force=true, recursive=true)
end

3 comments on commit d745f50

@tkelman
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you're trying to delete the entirety of /tmp ! should put these in a mktemp() do ... end block, which will automatically clean up for you

@cstjean
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The problem is that FileIO.load "dispatches" based on the file extension, and mktemp() cannot create a .jld file.

@tkelman
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ah fair enough

Please sign in to comment.