Update endpoint for testing fhir server connections

[katyasoup]

PULL REQUEST

Summary

Updates the "test connection" on the FHIR server config page to check the /metadata endpoint, rather than /Patient. This endpoint returns a Capability Statement that does not require Patient identifiers to be present on the request.

Related Issue

Fixes #670

Additional Information

Anything else the review team should know?

Checklist

• Descriptive Pull Request title
• Link to relevant issues
• Provide necessary context for design reviewers
• Ensure test coverage is above agreed upon threshold
• Update documentation

[linear]

QUE-307 Allow empty patient identifiers when testing server connections

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Project coverage is 64.73%. Comparing base (a7bb221) to head (bc5f377).

Files with missing lines	Patch %	Lines
src/app/shared/fhirClient.ts	0.00%	1 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #671   +/-   ##
=======================================
  Coverage   64.73%   64.73%           
=======================================
  Files         113      113           
  Lines        5521     5521           
  Branches     1308     1331   +23     
=======================================
  Hits         3574     3574           
  Misses       1935     1935           
  Partials       12       12           

☔ 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.

[nickclyde]

Okay so this is a tricky one, because we want to be able to test a proper authentication with "Test connection", but I think typically /metadata is an unauthenticated endpoint. Let me look around to see if there is a better option of an endpoint to hit, open to suggestions as well.

read Nick's comment

[katyasoup]

@nickclyde @fzhao99 I added back the /Patient endpoint, but threw in a dummy string for the name param - expected result is bundle response with 0 resources. Should all be working now!