99555 add specs

[balexandr]

ALL module code covered by unit tests.
Add a group in spec/spec_helper.rb for 'Burials'

Summary

• Create tests for files in the new Burial module
• Delete conflicting modules/classes outside of module
• Port over existing tests into module
• Add Burials to spec_helper to appear in coverage as its own category

Related issue(s)

• [Burial Module] all module code covered by unit tests va.gov-team#99555

Testing done

• 100% text coverage

Screenshots

What areas of the site does it impact?

Burial module

[github-actions]

Error: A file (or its parent directories) was deleted but its reference still exists in CODEOWNERS. Please update the .github/CODEOWNERS file and delete the entry for the Offending file: lib/burials

[github-actions]

1 Warning

⚠️

This PR changes 401 LoC (not counting whitespace/newlines). In order to ensure each PR receives the proper attention it deserves, we recommend not exceeding 200. Expect some delays getting reviews. File Summary Files .github/CODEOWNERS (+0/-2) lib/burials/monitor.rb (+0/-119) lib/burials/notification_callback.rb (+0/-10) lib/burials/notification_email.rb (+0/-33) {spec => modules/burials/spec}/lib/burials/notification_email_spec.rb (+4/-8) spec/lib/burials/monitor_spec.rb (+0/-223) spec/spec_helper.rb (+2/-0) Note: We exclude files matching the following when considering PR size: *.csv, *.json, *.tsv, *.txt, *.md, Gemfile.lock, app/swagger, modules/mobile/docs, spec/fixtures/, spec/support/vcr_cassettes/, modules/mobile/spec/support/vcr_cassettes/, db/seeds, modules/vaos/app/docs, modules/meb_api/app/docs, modules/appeals_api/app/swagger/, *.bru, *.pdf Big PRs are difficult to review, often become stale, and cause delays.

Generated by 🚫 Danger

[github-actions]

Error: A file (or its parent directories) was deleted but its reference still exists in CODEOWNERS. Please update the .github/CODEOWNERS file and delete the entry for the Offending file: lib/burials

[TaiWilkin]

LGTM