Y25-286 - Separate accession eligibility from validity

[StephenHulme]

Merges into #5491

Changes proposed in this pull request

• Do not raise InternalValidationErrors if a sample should not be accessioned
• Replace scope for_sample_accessioning with Railsy model methods
• Refactor validatation of studies for accessioning
• Removes error messages and does not accession these samples, but records a reason as a rails log instead
  • Sample is linked to a study that does not require accessioning.
  • Sample is not linked to any studies but must be linked to exactly one study.
  • Removes a lot of email spam...
• Update tests

Instructions for Reviewers

[All PRs] - Confirm PR template filled
[Feature Branches] - Review code
[Production Merges to main]
    - Check story numbers included
    - Check for debug code
    - Check version

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.28%. Comparing base (8851d11) to head (1b63341).

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #5486      +/-   ##
===========================================
- Coverage    87.39%   87.28%   -0.12%     
===========================================
  Files         1457     1457              
  Lines        32705    32703       -2     
  Branches      3423     3422       -1     
===========================================
- Hits         28584    28545      -39     
- Misses        4106     4143      +37     
  Partials        15       15              

☔ 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.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[StephenHulme]

Moved later developments into #5499

[BenTopping]

Couple questions, looks good.