Skip to content

[INLONG-9883][Manager] Add unit test of HeartbeatController#9887

Closed
beatCao wants to merge 1 commit intoapache:masterfrom
beatCao:INLONG-9883
Closed

[INLONG-9883][Manager] Add unit test of HeartbeatController#9887
beatCao wants to merge 1 commit intoapache:masterfrom
beatCao:INLONG-9883

Conversation

@beatCao
Copy link
Copy Markdown
Contributor

@beatCao beatCao commented Mar 28, 2024

Title Name: [INLONG-9883] add unit test of HeartbeatController

#9883

Motivation
Explain here the context, and why you're making that change. What is the problem you're trying to solve.

Modifications
Describe the modifications you've done.

Copy link
Copy Markdown

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Hello @beatCao, thank you for submitting a PR to InLong 💖 We will respond as soon as possible ⏳
This seems to be your first PR 🌠 Please be sure to follow our Contribution Guidelines.
If you have any questions in the meantime, you can also ask us on the InLong Discussions 🔍

@dockerzhang
Copy link
Copy Markdown
Contributor

@beatCao please fix the failed UT, thanks.

@fuweng11
Copy link
Copy Markdown
Contributor

This UT cannot verify whether the API's functionality meets expectations.Please refer to StreamSourceServiceTest.testGetAndUpdate to verify whether the actual functionality of the API is effective.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants