griffithlab / civic-server

Backend Server for CIViC Project
MIT License
39 stars 32 forks source link

Fix code that assumes that the disease field is required on evidence items #674

Closed susannasiebert closed 3 years ago

susannasiebert commented 3 years ago

Functional evidence items no longer allow a disease to be set but in different parts of the codebase we assume that all evidence items will have a disease assigned to it. This PR fixes the places I found where that is the case.

Closes https://github.com/griffithlab/civic-client/issues/1589

coveralls commented 3 years ago

Pull Request Test Coverage Report for Build 578693676


Changes Missing Coverage Covered Lines Changed/Added Lines %
app/presenters/minimal_evidence_grid_presenter.rb 0 1 0.0%
app/presenters/evidence_item_tsv_presenter.rb 0 2 0.0%
<!-- Total: 3 6 50.0% -->
Totals Coverage Status
Change from base Build 558880154: 0.01%
Covered Lines: 2137
Relevant Lines: 13340

💛 - Coveralls