Skip to content

Update TestFhirServer initialization#5481

Merged
jestradaMS merged 3 commits intomainfrom
users/jestrada/fixtestfhirserver
Apr 3, 2026
Merged

Update TestFhirServer initialization#5481
jestradaMS merged 3 commits intomainfrom
users/jestrada/fixtestfhirserver

Conversation

@jestradaMS
Copy link
Copy Markdown
Contributor

@jestradaMS jestradaMS commented Apr 3, 2026

Description

This pull request refactors and improves the robustness of the FHIR server startup and security configuration logic in the end-to-end test suite, and also updates deployment configuration to enhance application warmup behavior. The main focus is on making the test server's availability and metadata retrieval more reliable and maintainable, with clearer retry logic and error handling.

Deployment configuration enhancements:

  • Updated default-azuredeploy-docker.json to add new environment variables (WEBSITE_WARMUP_PATH, WEBSITE_WARMUP_STATUSES, and WEBSITES_CONTAINER_START_TIME_LIMIT) to the FHIR server configuration, improving the application's warmup and readiness checks in Azure environments.

Related issues

Addresses AB#181187.

Testing

Tested directly on CI runs and updated the CI environments with the same setting sad defined in this PR to stabilize startup of containers. Verified via app and program logs for the containers.

Describe how this change was tested.

FHIR Team Checklist

  • Update the title of the PR to be succinct and less than 65 characters
  • Add a milestone to the PR for the sprint that it is merged (i.e. add S47)
  • Tag the PR with the type of update: Bug, Build, Dependencies, Enhancement, New-Feature or Documentation
  • Tag the PR with Open source, Azure API for FHIR (CosmosDB or common code) or Azure Healthcare APIs (SQL or common code) to specify where this change is intended to be released.
  • Tag the PR with Schema Version backward compatible or Schema Version backward incompatible or Schema Version unchanged if this adds or updates Sql script which is/is not backward compatible with the code.
  • When changing or adding behavior, if your code modifies the system design or changes design assumptions, please create and include an ADR.
  • CI is green before merge Build Status
  • Review squash-merge requirements

Semver Change (docs)

Patch|Skip|Feature|Breaking (reason)

@jestradaMS jestradaMS requested a review from a team as a code owner April 3, 2026 18:14
@jestradaMS jestradaMS added this to the FY26\Q3\2Wk\2Wk20 milestone Apr 3, 2026
@jestradaMS jestradaMS added Enhancement-Test Enhancement on tests. Azure API for FHIR Label denotes that the issue or PR is relevant to the Azure API for FHIR Azure Healthcare APIs Label denotes that the issue or PR is relevant to the FHIR service in the Azure Healthcare APIs OSS-specific This is category of issues that are specific to OSS. This category does not impact managed service. No-PaaS-breaking-change No-ADR ADR not needed Open source This change is only relevant to the OSS code or release. and removed Azure API for FHIR Label denotes that the issue or PR is relevant to the Azure API for FHIR Azure Healthcare APIs Label denotes that the issue or PR is relevant to the FHIR service in the Azure Healthcare APIs labels Apr 3, 2026
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (main@7b910d0). Learn more about missing BASE report.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #5481   +/-   ##
=======================================
  Coverage        ?   76.90%           
=======================================
  Files           ?      978           
  Lines           ?    35680           
  Branches        ?     5356           
=======================================
  Hits            ?    27440           
  Misses          ?     6907           
  Partials        ?     1333           
🚀 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.

@jestradaMS jestradaMS enabled auto-merge (squash) April 3, 2026 20:53
@jestradaMS jestradaMS merged commit ea948f8 into main Apr 3, 2026
109 checks passed
@jestradaMS jestradaMS deleted the users/jestrada/fixtestfhirserver branch April 3, 2026 20:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Enhancement-Test Enhancement on tests. No-ADR ADR not needed No-PaaS-breaking-change Open source This change is only relevant to the OSS code or release. OSS-specific This is category of issues that are specific to OSS. This category does not impact managed service.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants