Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

chore: Make integration tests tests run concurrently #2069

Merged
merged 5 commits into from
Nov 13, 2024
Merged

chore: Make integration tests tests run concurrently #2069

merged 5 commits into from
Nov 13, 2024

Conversation

0xivanov
Copy link
Contributor

@0xivanov 0xivanov commented Nov 6, 2024 •
edited
Loading

Description:

Related issue(s):

Fixes #2074

Notes for reviewer:

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

@0xivanov
chore: make tests run concurrently
b33796b 
Signed-off-by: Ivan Ivanov <ivanivanov.ii726@gmail.com>
@0xivanov 0xivanov self-assigned this Nov 6, 2024
@0xivanov
chore: revert some changes
2d533d0 
Signed-off-by: Ivan Ivanov <ivanivanov.ii726@gmail.com>
@0xivanov
Merge branch 'main' into make-tests-run-concurrently
14b5009
@0xivanov
add randomness to transacionID nanosecs
3c7d291 
Signed-off-by: Ivan Ivanov <ivanivanov.ii726@gmail.com>
@0xivanov
chore: remove comments
445ee13 
Signed-off-by: Ivan Ivanov <ivanivanov.ii726@gmail.com>
@0xivanov 0xivanov changed the title chore: Make tests run concurrently chore: Make integration tests tests run concurrently Nov 12, 2024
@0xivanov 0xivanov marked this pull request as ready for review November 12, 2024 11:35
@0xivanov 0xivanov requested review from a team as code owners November 12, 2024 11:35
@0xivanov 0xivanov added the enhancement New feature or request label Nov 12, 2024
andrewb1269hg
andrewb1269hg approved these changes Nov 12, 2024
View reviewed changes
Copy link

@andrewb1269hg andrewb1269hg left a comment

Choose a reason for hiding this comment

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

Reviewed gradle/plugins/src/main/kotlin/com.hedera.gradle.java.gradle.kts and approved.

agadzhalov
agadzhalov approved these changes Nov 13, 2024
View reviewed changes
Copy link
Contributor

@agadzhalov agadzhalov left a comment

Choose a reason for hiding this comment

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

LGTM

@0xivanov 0xivanov merged commit f451eee into main Nov 13, 2024
11 checks passed
@0xivanov 0xivanov deleted the make-tests-run-concurrently branch November 13, 2024 07:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@andrewb1269hg andrewb1269hg andrewb1269hg approved these changes

@agadzhalov agadzhalov agadzhalov approved these changes

@thenswan thenswan Awaiting requested review from thenswan thenswan is a code owner automatically assigned from hashgraph/hedera-sdk-java-maintainers

Assignees

@0xivanov 0xivanov

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Make integration tests run concurrently
3 participants
@0xivanov @agadzhalov @andrewb1269hg