[1115] poc: add logic to verify a PDF was properly stamped

[pennja]

Summary

• This work is behind a feature toggle (flipper): NO
• Added a lambda method which verifies when a stamp has successfully been applied to a pdf
• Added test coverage surrounding new logic
• Updated existing test coverage for the pdf_stamper class to align with RSpec patterns

Related issue(s)

• Link to ticket created in va.gov-team repo OR screenshot of Jira ticket if your team uses Jira
  department-of-veterans-affairs/VA.gov-team-forms#1115

Testing done

• New code is covered by unit tests

Screenshots

Note: Optional

What areas of the site does it impact?

• PDF Stamping and Filling

Acceptance criteria

• I fixed|updated|added unit tests and integration tests for each feature (if applicable).
• No error nor warning in the console.
• Events are being sent to the appropriate logging solution
• Documentation has been updated (link to documentation)
• No sensitive information (i.e. PII/credentials/internal URLs/etc.) is captured in logging, hardcoded, or specs
• Feature/bug has a monitor built into Datadog or Grafana (if applicable)
• If app impacted requires authentication, did you login to a local build and verify all authenticated routes work as expected
• I added a screenshot of the developed feature

Requested Feedback

(OPTIONAL)What should the reviewers know in addition to the above. Is there anything specific you wish the reviewer to assist with. Do you have any concerns with this PR, why?

[Thrillberg]

One small comment and I think tests are not passing quite yet. Otherwise I'd approve! Great work!

[github-actions]

1 Warning

⚠️

This PR changes 205 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 modules/simple_forms_api/app/services/simple_forms_api/pdf_stamper.rb (+33/-18) modules/simple_forms_api/spec/services/pdf_filler_spec.rb (+3/-0) modules/simple_forms_api/spec/services/pdf_stamper_spec.rb (+138/-13) Note: We exclude files matching the following when considering PR size: *.csv, *.json, *.tsv, *.txt, 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 Big PRs are difficult to review, often become stale, and cause delays.

Generated by 🚫 Danger

[Thrillberg]

Looks great! Thanks for updating all those fixtures. I'm really excited to roll this out and see it in action! 😃