Test deps not required on nightly

[fredrikekre]

e.g. https://travis-ci.org/JuliaGraphs/LightGraphs.jl/builds/519328333

[KristofferC]

I'm guessing the LOAD_PATH is no longer correctly set on the spawned process?

[fredrikekre]

Also happens for build https://discourse.julialang.org/t/travis-ci-configuration-with-new-project-toml-file/23942

[00vareladavid]

Can you elaborate on the issue? The original isolation is not working anymore?

[fredrikekre]

Probably caused by #1058 which we backported to 1.0.4 and 1.1.1 so not this bug is there too 🎉

[fredrikekre]

Another example; https://travis-ci.org/libAtoms/ASE.jl/builds/543072251

[KristofferC]

I think what happens is that recursive dependencies gets put into the temporary project file but those should only be put into the manifest file.

So if Test somehow ends up as a recursive dependency on the package tested, this will happen.

[fredrikekre]

Yea, I was trying to fix that, but I think the reason we add them to the project is that the resolver can only keep project-deps fixed on a version. I tried just pinning them in the manifest, but seems like it is beeing ignored.