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

[Unit Test] Unit test for ThreadUtils #4540

Closed
2 of 3 tasks
harshithasudhakar opened this issue Nov 5, 2023 · 8 comments · Fixed by #5111
Closed
2 of 3 tasks

[Unit Test] Unit test for ThreadUtils #4540

harshithasudhakar opened this issue Nov 5, 2023 · 8 comments · Fixed by #5111
Labels
good first issue Issues for first-time contributors testing

Comments

@harshithasudhakar
Copy link
Member

harshithasudhakar commented Nov 5, 2023 •
edited by Pil0tXia
Loading

Search before asking

  • I had searched in the issues and found no similar issues.

Read the unit testing guidelines

  • I have read.

Unit test request

Add Unit Test for ThreadUtils.java

Describe the unit tests you want to do

Module name: eventmesh-common
Located at: eventmesh/eventmesh-common/src/main/java/org/apache/eventmesh/common/utils/ThreadUtils.java

Are you willing to submit PR?

  • Yes I am willing to submit a PR!
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Nov 5, 2023

Welcome to the Apache EventMesh community!!
We are glad that you are contributing by opening this issue. :D

Please make sure to include all the relevant context.
We will be here shortly.

If you are interested in contributing to our project, please let us know!
You can check out our contributing guide on contributing to EventMesh.

Want to get closer to the community?

WeChat Assistant WeChat Public Account Slack
Join Slack Chat

Mailing Lists:

Name Description Subscribe Unsubscribe Archive
Users User support and questions mailing list Subscribe Unsubscribe Mail Archives
Development Development related discussions Subscribe Unsubscribe Mail Archives
Commits All commits to repositories Subscribe Unsubscribe Mail Archives
Issues Issues or PRs comments and reviews Subscribe Unsubscribe Mail Archives

@VishalMCF
Copy link
Contributor

VishalMCF commented Nov 16, 2023 •
edited
Loading

Is anyone working on this issue??

@harshithasudhakar
Copy link
Member Author

@VishalMCF yeah, I'm working on it.

@harshithasudhakar harshithasudhakar self-assigned this Nov 16, 2023
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Apr 6, 2024

It has been 90 days since the last activity on this issue. Apache EventMesh values the voices of the community. Please don't hesitate to share your latest insights on this matter at any time, as the community is more than willing to engage in discussions regarding the development and optimization directions of this feature.

If you feel that your issue has been resolved, please feel free to close it. Should you have any additional information to share, you are welcome to reopen this issue.

@github-actions github-actions bot added the Stale label Apr 6, 2024
@harshithasudhakar harshithasudhakar removed their assignment Apr 6, 2024
@github-actions github-actions bot removed the Stale label Apr 7, 2024
@Pil0tXia Pil0tXia mentioned this issue Apr 14, 2024
Closed
@Pil0tXia Pil0tXia added the good first issue Issues for first-time contributors label Apr 15, 2024
@Pil0tXia Pil0tXia mentioned this issue Apr 15, 2024
Open
@github-actions GitHub Actions
Copy link
Contributor

It has been 90 days since the last activity on this issue. Apache EventMesh values the voices of the community. Please don't hesitate to share your latest insights on this matter at any time, as the community is more than willing to engage in discussions regarding the development and optimization directions of this feature.

If you feel that your issue has been resolved, please feel free to close it. Should you have any additional information to share, you are welcome to reopen this issue.

@github-actions github-actions bot added the Stale label Jul 14, 2024
@melisolmez
Copy link
Contributor

Hello, I want to work on this isuue. Could I take on this task and contribute?

@harshithasudhakar
Copy link
Member Author

Sure, please go ahead :)

@github-actions github-actions bot removed the Stale label Sep 9, 2024
This was referenced Sep 11, 2024
Closed
Closed
@melisolmez
Copy link
Contributor

Hİ, @harshithasudhakar, Could you review my pr please

@melisolmez melisolmez mentioned this issue Sep 30, 2024
Merged
@mxsm mxsm closed this as completed in #5111 Oct 7, 2024
mxsm pushed a commit that referenced this issue Oct 7, 2024
@melisolmez
[ISSUE #4540] Add unit test for ThreadUtils (#5111)
a4e29d8
KrispauI pushed a commit to KrispauI/eventmesh that referenced this issue Oct 8, 2024
@melisolmez @KrispauI
[ISSUE apache#4540] Add unit test for ThreadUtils (apache#5111)
386c7a6
harshithasudhakar pushed a commit that referenced this issue Oct 8, 2024
@KrispauI @dependabot @melisolmez
[ISSUE #4990] Add unit test for HttpConvertsUtils.java (#5110)
7abe42a 
* add unit test for HttpConvertsUtils.java

* fix package error

* Bump com.gradle.develocity from 3.17.5 to 3.18.1 (#5121)

Bumps com.gradle.develocity from 3.17.5 to 3.18.1.

---
updated-dependencies:
- dependency-name: com.gradle.develocity
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump software.amazon.awssdk:s3 from 2.27.17 to 2.28.12 (#5120)

Bumps software.amazon.awssdk:s3 from 2.27.17 to 2.28.12.

---
updated-dependencies:
- dependency-name: software.amazon.awssdk:s3
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* [ISSUE #4540] Add unit test for ThreadUtils (#5111)

* Bump com.rabbitmq:amqp-client from 5.21.0 to 5.22.0 (#5119)

Bumps [com.rabbitmq:amqp-client](https://github.com/rabbitmq/rabbitmq-java-client) from 5.21.0 to 5.22.0.
- [Release notes](https://github.com/rabbitmq/rabbitmq-java-client/releases)
- [Commits](rabbitmq/rabbitmq-java-client@v5.21.0...v5.22.0)

---
updated-dependencies:
- dependency-name: com.rabbitmq:amqp-client
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* fix CI errors

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Melis Ölmez <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue Issues for first-time contributors testing
Projects
None yet
Milestone
No milestone
4 participants
@Pil0tXia @VishalMCF @melisolmez @harshithasudhakar