Skip to content

Fix expired test tokens and remove external URL dependencies (5.x) - #3495

Merged
Ignacio Inglese (iNinja) merged 1 commit into
dev5xfrom
iinglese/fix-5x-test-infrastructure
May 20, 2026
Merged

Ignacio Inglese (iNinja) merged 1 commit into
dev5xfrom
iinglese/fix-5x-test-infrastructure

Conversation

@iNinja

Copy link
Copy Markdown
Contributor

Updates test infrastructure on the 5.x servicing branch to restore CI green state.

Changes

  • Add ValidateLifetime = false to test theory data where tests are not exercising lifetime validation (expired Default.ExpiresString from March 2025)
  • Update inline hardcoded expiry dates in UseSecurityTokenDescriptorProperties test cases to 2040
  • Replace external URL dependencies (someaddress.com, httpstat.us/429) in ConfigurationManagerTests.GetConfiguration with a local ExceptionThrowingDocumentRetriever to eliminate flaky network-dependent failures

Test results

  • 4,587 tests passing, 0 failures across all TFMs (net45, net452, net461)

Update test infrastructure to handle hardcoded token expiry dates that
are now in the past. Add ValidateLifetime = false where tests are not
exercising lifetime validation. Replace external URL dependencies in
ConfigurationManager tests with a local document retriever to prevent
flaky failures from network-dependent assertions.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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.

4 participants