Added an all missing treatments exception

[hepcat72]

Summary Change Description

The excel file template creation is implemented such that if a user supplies an existing study sheet, it will add samples to that file. The samples haven't been added yet, but as a part of that template, it also pre-populates the treatments and tissues like we currently have in the previous template. When adding to an existing study doc however, there's the possibility that Tissues or treatments referenced in the Animals and Samples sheets might be missing. Thus, these exceptions were modified/added:

• MissingTissues (changed to MissingTissue)
• AllMissingTissues
• MissingTreatment
• AllMissingTreatments

Since the plan is to attach errors directly to cells in the spreadsheets in the form of comments, I changed how MissingTissues exceptions were structured. Instead of containing a summary of all tissues missing from 1 file, each exception is created for a single cell in a sheet of 1 file. And AllMissingTissues was changed to contain a list of those individual cell exceptions.

Treatments was handled in the same way.

This entire effort just took a couple hours this morning, including tests, so it was quick.

Affected Issues/Pull Requests

• Partially addresses #829
• Merges into PR #929
• Next PR: #931

Review Notes

See comments in-line.

Checklist

This pull request will be merged once the following requirements are met. The author and/or reviewers should uncheck any unmet requirements:

• Review requirements
  • Minimum approvals: 1
  • No changes requested
  • All blocking issues resolved by reviewers
  • Specific reviewers: @add_username_here
  • Review period: 2 days
• Associated issue/pull request requirements:
  • [x] All requirements in affected issues marked "resolved" are satisfied
  • [x] All required pull requests are merged (or none)
• Basic requirements
  • [x] All linters pass
  • [x] All tests pass
  • [x] All conflicts resolved
• Overhead requirements
  • [x] New/changed method tests implemented/updated (or no change)
  • [ ] Updated Unreleased section of changelog.md (or no change)
  • [x] Migrations created & committed (or no change)

[hepcat72]

While missing Tissues is certainly a problem (they will need to be added before data can be loaded), treatments are typically supplied along with the animal/sample sheet and aren't assumed to be in the database. I'm not sure I follow the logic of raising exceptions for them when we are adding samples to an existing sheet.

The mechanism for adding the samples will be the same. I'm thinking of it in 2 ways:

• The mechanism for transmitting missing tissues already exists. This is 3 birds, one stone - and making these changes was fast. I did the whole thing before leaving for work.
• Reviewing the details of the design for transmitting the Tissues re-familiarizes me with the design work I'd done already and allows me to gain insights into the solution for the samples. It's like I'm getting a head start and am directly benefitting from time already invested.

Yes, adding tissues is not the direct goal, but going through quickly adding them means that doing the same thing for samples and treatments will be nearly plug and play.