Skip to content

chore: migrate to kotlin Duration usage for polling, timeout units for test framework (WPB-21886)#4783

Merged
yamilmedina merged 6 commits into
developfrom
chore/improve-test-frameworkapis
May 5, 2026
Merged

chore: migrate to kotlin Duration usage for polling, timeout units for test framework (WPB-21886)#4783
yamilmedina merged 6 commits into
developfrom
chore/improve-test-frameworkapis

Conversation

@yamilmedina
Copy link
Copy Markdown
Contributor

@yamilmedina yamilmedina commented May 5, 2026

https://wearezeta.atlassian.net/browse/WPB-21886


PR Submission Checklist for internal contributors

  • The PR Title

    • conforms to the style of semantic commits messages¹ supported in Wire's Github Workflow²
    • contains a reference JIRA issue number like SQPIT-764
    • answers the question: If merged, this PR will: ... ³
  • The PR Description

    • is free of optional paragraphs and you have filled the relevant parts to the best of your ability

What's new in this PR?

Follow up on:

In order to have a better API for devs using the framework and make the testing API stable we are improving the code and refactoring to reflect a less verbose and reduce duplicated code

Solutions

Deprecate time unit representations in numeric value and use Kotlin Duration.

Testing

Test Coverage (Optional)

  • I have added automated test to this contribution

PR Post Submission Checklist for internal contributors (Optional)

  • Wire's Github Workflow has automatically linked the PR to a JIRA issue

PR Post Merge Checklist for internal contributors

  • If any soft of configuration variable was introduced by this PR, it has been added to the relevant documents and the CI jobs have been updated.

References
  1. https://sparkbox.com/foundry/semantic_commit_messages
  2. https://github.com/wireapp/.github#usage
  3. E.g. feat(conversation-list): Sort conversations by most emojis in the title #SQPIT-764.

@yamilmedina yamilmedina requested a review from emmaoke-w May 5, 2026 12:46
@yamilmedina yamilmedina changed the title chore: migrate to kotkin Duration usage for polling, timeout units test framework (WPB-21886) chore: migrate to kotkin Duration usage for polling, timeout units for test framework (WPB-21886) May 5, 2026
@yamilmedina yamilmedina changed the title chore: migrate to kotkin Duration usage for polling, timeout units for test framework (WPB-21886) chore: migrate to kotlin Duration usage for polling, timeout units for test framework (WPB-21886) May 5, 2026
yamilmedina and others added 2 commits May 5, 2026 15:56
….2 (#4706)

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Oussama Hassine <oussama.has100@gmail.com>
@yamilmedina yamilmedina requested a review from a team as a code owner May 5, 2026 13:57
@yamilmedina yamilmedina requested review from Garzas, alexandreferris, saleniuk and typfel and removed request for a team May 5, 2026 13:57
@pull-request-size pull-request-size Bot added size/L and removed size/M labels May 5, 2026
@pull-request-size pull-request-size Bot added size/M and removed size/L labels May 5, 2026
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented May 5, 2026

Copy link
Copy Markdown
Contributor

@emmaoke-w emmaoke-w left a comment

Choose a reason for hiding this comment

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

Thanks for the refactoring.

@yamilmedina yamilmedina added this pull request to the merge queue May 5, 2026
Merged via the queue into develop with commit 5a01bb1 May 5, 2026
18 checks passed
@yamilmedina yamilmedina deleted the chore/improve-test-frameworkapis branch May 5, 2026 14:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants