Delete untested files in /test

[odow]

Closes #227

@ccoffrin not sure what you want to do with these.

There's a circular dependency with Juniper and MINLPTests. For now I've removed the MINLPTests.jl file from Juniper because it wasn't being tested in CI.

We could remove the Juniper test from MINLPTests.jl, and add a MINLPTest here instead?

[codecov[bot]]

Codecov Report

Merging #235 (49fed10) into master (bfb7125) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #235   +/-   ##
=======================================
  Coverage   91.81%   91.81%           
=======================================
  Files          20       20           
  Lines        2015     2015           
=======================================
  Hits         1850     1850           
  Misses        165      165           

---

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update bfb7125...49fed10. Read the comment docs.

[ccoffrin]

How is the circular dependency managed with Ipopt.jl?

[odow]

Ipopt doesn't run MINLPTests

[ccoffrin]

In that case, I say we drop MINLPTests from Juniper to avoid the circular dep. We can try to improve MOI-tests if we find issues, right?

Re the other tests:

• We can drop the PowerModels ones, these are from a bygone era when JuMP testing infrastructure was not so nice
• current_118.jl looks like a test for the function registration interface. Which should now be covered by FP: no linear

@Wikunia any objections in dropping current_118.jl?

[odow]

Judging by the syntax, it doesn't seem like the current_118 file has ever really been tested. And yes, the registration stuff is now covered by other tests.

[odow]

Merging for now. We can re-add the current_118 if @Wikunia objects.

[ccoffrin]

Sounds good to me!