MHV-56069-Update-rxRfRecords-structure

[agravell047]

Summary

Updated the rxRfRecords field to return an array of strings.

Related issue(s)

https://jira.devops.va.gov/browse/MHV-56069

AC1: fix rxRfRecords data structure on vets-api to return an array of refill history data

Testing done

• Tested locally
• Ran unit tests

What areas of the site does it impact?

MHV Medications app

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

[dcloud]

Overall, I think this is great since it adds test coverage that was missing. I left a comment about the conditional in the test for 'rx_rf_records', where I think we want to assert some control over our factory instance for testing presence/absence of those records

[dcloud]

Thanks for addressing my feedback. I think this is good to go! 🚢