Add Gonorrhea support to test card

[mpbrown]

FRONTEND PULL REQUEST

Related Issue

• Resolves Test card supports Gonorrhea test results #7434

Changes Proposed

• Adds support for Gonorrhea to the test card
• Converts the HepatitisCAoeForm component into STIGenericAoeForm that receives the disease specific data as props. This component is used for Hep C AOE and Gonorrhea AOE

Additional Information

• This also adds support for Gonorrhea on the dashboard and (most) of the result page besides result specific guidance and some conditional logic for showing genders of sexual partners on view details
• Gonorrhea symptoms list may not necessarily be final (see this Slack thread)

Testing

• Deployed on dev5
• Gonorrhea test device has been created on Boban's Org and boban 1 facility

Screenshots

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
82.6% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[emyl3]

LGTM! Thank you for renaming the shared component to STIGenericAOEForm! 🎉

[mehansen]

tangentially related to this change: do you think we need to make it known somewhere (e.g. warn Jayna, add a validation to add device page?) that we can't support devices that test for multiple STIs currently without changing this part of the code? it's technically possible to create that device in SR so I can see it happening where Jayna creates a ticket to add a device, we refine it and don't think about it, and then end up creating it and the AOE don't work properly. but maybe I'm being paranoid about my memory

[mpbrown]

Hmm good point, we should make sure everyone on the team knows we can't support multi-STI devices yet. I don't think we necessarily need to add separate validation since the multiple device test card redesign is one of the big upcoming features that will likely require us to deal with those AOE questions in a more comprehensive manner

[mehansen]

agree! seems like this won't be the case for long

[mehansen]

this name did confuse me a little because I thought we'd be using it for all the STIs instead of just 2 of them. but I don't have a better suggestion 😆

[mpbrown]

Yeah ideally we could have a children slot for this component so syphilis AOE form can extend it simply by passing in the component for the syphilis history question but there's a risk of the form management getting harder to follow and it seems like AOE questions will get a large revamp once we start doing multiple devices per test card

[mehansen]

there's a risk of the form management getting harder to follow and it seems like AOE questions will get a large revamp once we start doing multiple devices per test card

also agree!