delete-root-analysis-artifact

[antgonza]

No description provided.

[codecov-io]

Codecov Report

Merging #2244 into dev will increase coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##              dev    #2244      +/-   ##
==========================================
+ Coverage   92.81%   92.82%   +0.01%     
==========================================
  Files         171      171              
  Lines       18905    18906       +1     
==========================================
+ Hits        17546    17549       +3     
+ Misses       1359     1357       -2

Impacted Files	Coverage Δ
qiita_db/test/test_artifact.py	99.84% <100%> (ø)	⬆️
qiita_db/artifact.py	98.54% <100%> (ø)	⬆️
...t/handlers/api_proxy/tests/test_sample_template.py	98.66% <0%> (+0.88%)	⬆️

---

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 db0a4b6...3e17876. Read the comment docs.

[josenavas]

If I understand correctly, if the user deletes the last artifact there will be an empty analysis (i.e. no artifacts) so the user can delete it. Did you test how the interface behaves with this case? I'm unsure if it will just go smooth or it will fail somewhere else...

[antgonza]

Yes, that's correct and it works fine ... I guess you didn't tested when you wrote it 😄