Skip to content

Release sync: merge develop into main for v1.24.0 test fixes#87

Merged
homotechsual merged 4 commits into
mainfrom
develop
Jun 22, 2026
Merged

Release sync: merge develop into main for v1.24.0 test fixes#87
homotechsual merged 4 commits into
mainfrom
develop

Conversation

@homotechsual

Copy link
Copy Markdown
Owner

Summary

  • sync develop into main to include the post-release unit test fixes
  • ensure main includes the same Connect-HaloAPI Key Vault test updates currently on develop
  • keep release/tag history aligned with the passing test state for v1.24.0

Included changes

  • update Connect-HaloAPI unit tests for renamed parameter and Key Vault behavior
  • no additional functional cmdlet changes in this PR beyond syncing branch state

Validation

  • pwsh -File .\DevOps\Quality\test.ps1 -IncludeVSCodeMarker (pass)

Issues

Copilot AI review requested due to automatic review settings June 22, 2026 20:47

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR syncs develop into main for the v1.24.0 release state, primarily to align main with the updated Connect-HaloAPI Key Vault unit tests (managed identity switch + Key Vault secret name/value behavior) so release history reflects a passing test baseline.

Changes:

  • Update Connect-HaloAPI unit tests to use -UseManagedIdentity (instead of the prior -Identity argument pattern).
  • Update Key Vault secret naming in tests from underscore-delimited to hyphen-delimited (e.g., halo-URL).
  • Update Key Vault secret mock returns to match Get-AzKeyVaultSecret -AsPlainText behavior (plain text values).

Comment thread Tests/HaloAPI.Unit.Tests.ps1
Comment thread Tests/HaloAPI.Unit.Tests.ps1
@homotechsual homotechsual merged commit 459ca4b into main Jun 22, 2026
8 of 14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Connect-HaloAPI Azure Key Vault integration does not work Invoke-HaloRequest should have a Fragment option Adding CustomTable Data Endpoint

2 participants