Skip to content

Refactor OpTimerTest#2927

Merged
DomGarguilo merged 1 commit intoapache:mainfrom
DomGarguilo:updateOpTimerTest
Sep 13, 2022
Merged

Refactor OpTimerTest#2927
DomGarguilo merged 1 commit intoapache:mainfrom
DomGarguilo:updateOpTimerTest

Conversation

@DomGarguilo
Copy link
Copy Markdown
Member

Fixes #2886

  • remove try-catch blocks
  • adds messages to assert statements
  • make Logger final
  • add descriptive variables for numbers used for conversion

@DomGarguilo DomGarguilo merged commit 1aeffcf into apache:main Sep 13, 2022
@DomGarguilo DomGarguilo deleted the updateOpTimerTest branch September 13, 2022 12:45
@ctubbsii ctubbsii added this to the 2.1.0 milestone Jul 12, 2024
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.

Refactor OpTimerTest.verifyExceptionCallingStartWhenRunning to improve the test experience

3 participants