Skip to content

chore: fix ut ExceptionCircuitBreakerTest - #3600

Merged
LearningGp merged 2 commits into
alibaba:1.8from
uuuyuqi:uuuyuqi/fix-1.8-ut
Mar 3, 2026
Merged

chore: fix ut ExceptionCircuitBreakerTest#3600
LearningGp merged 2 commits into
alibaba:1.8from
uuuyuqi:uuuyuqi/fix-1.8-ut

Conversation

@uuuyuqi

@uuuyuqi uuuyuqi commented Mar 2, 2026

Copy link
Copy Markdown
Collaborator

Does this pull request fix one issue?

closes #3599

The test method was missing the MockedStatic<TimeUtil> parameter that
the updated AbstractTimeBasedTest requires after migrating from PowerMock
to Mockito.

Change-Id: Ib2807cdd888e2347b2ea7f9d608f3dcbe9de3f59
Co-developed-by: Claude <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@uuuyuqi
uuuyuqi requested a review from LearningGp March 2, 2026 11:10
Change-Id: Ib1441354aba3c45b09a72881692812a1f267b2af

@LearningGp LearningGp left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@LearningGp
LearningGp merged commit 52eac45 into alibaba:1.8 Mar 3, 2026
8 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] ExceptionCircuitBreakerTest ut need be fixed.

2 participants