Fix various usages of MOI.delete

[odow]

Closes #122

[codecov]

Codecov Report

Attention: Patch coverage is 44.44444% with 5 lines in your changes missing coverage. Please review.

Project coverage is 98.34%. Comparing base (8ab9e07) to head (a66bf0a).
Report is 1 commits behind head on master.

Files with missing lines	Patch %	Lines
src/algorithms/TambyVanderpooten.jl	0.00%	3 Missing ⚠️
src/algorithms/KirlikSayin.jl	50.00%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #123      +/-   ##
==========================================
- Coverage   98.85%   98.34%   -0.52%     
==========================================
  Files           9        9              
  Lines         960      966       +6     
==========================================
+ Hits          949      950       +1     
- Misses         11       16       +5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[kofgokhan]

TV is not supposed to try to solve infeasible problems and KS is not supposed to as well in the second stage if the first stage is feasible

[odow]

Sure, we don't expect these to be infeasible. But the solvers can fail for any number of reasons. Iteration limits, numerical error, network license validation errors.

[kofgokhan]

I agree they should be there. I cannot think of a test case that will enable coverage, though.

[odow]

I cannot think of a test case that will enable coverage, though.

Yeah, I need to come up with something like "a solver that errors on the 5th solve"...

[odow]

Merged as an improvement. I'll think about how to test solve failures