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.

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

【UT】add unit test for collector system info #18717

Merged

Conversation

lengrongfu
Copy link
Contributor

@lengrongfu lengrongfu commented May 23, 2023

Thank you for contributing to Harbor!

Comprehensive Summary of your change

add unit test for collector system info.

Issue being fixed

Fixes #(issue)

/label release-note/enhancement

Please indicate you've done the following:

  • Well Written Title and Summary of the PR
  • Label the PR as needed. "release-note/ignore-for-release, release-note/new-feature, release-note/update, release-note/enhancement, release-note/community, release-note/breaking-change, release-note/docs, release-note/infra, release-note/deprecation"
  • Accepted the DCO. Commits without the DCO will delay acceptance.
  • Made sure tests are passing and test coverage is added if needed.
  • Considered the docs impact and opened a new docs issue or PR with docs changes if needed in website repository.

@lengrongfu lengrongfu requested a review from a team as a code owner May 23, 2023 07:29
@codecov
Copy link

codecov bot commented May 23, 2023

Codecov Report

Merging #18717 (32060c6) into main (2f51daf) will increase coverage by 22.65%.
The diff coverage is n/a.

❗ Current head 32060c6 differs from pull request most recent head 2e045e8. Consider uploading reports for the commit 2e045e8 to get more accurate results

Impacted file tree graph

@@             Coverage Diff             @@
##             main   #18717       +/-   ##
===========================================
+ Coverage   44.77%   67.43%   +22.65%     
===========================================
  Files         235      980      +745     
  Lines       13067   106774    +93707     
  Branches     2665     2665               
===========================================
+ Hits         5851    72005    +66154     
- Misses       6921    30899    +23978     
- Partials      295     3870     +3575     
Flag Coverage Δ
unittests 67.43% <ø> (+22.65%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 749 files with indirect coverage changes

@lengrongfu lengrongfu force-pushed the test/add_test_systeminfocontroller branch 3 times, most recently from 6a253c8 to 3125665 Compare May 29, 2023 03:42
@lengrongfu
Copy link
Contributor Author

@Vad1mo @OrlinVasilev Hi, PTAL.

@AllForNothing AllForNothing requested review from stonezdj and chlins May 29, 2023 08:38
@AllForNothing AllForNothing added the release-note/enhancement Label to mark PR to be added under release notes as enhancement label May 29, 2023
@AllForNothing AllForNothing assigned stonezdj and chlins and unassigned Vad1mo, zyyw and OrlinVasilev May 29, 2023
@AllForNothing AllForNothing force-pushed the test/add_test_systeminfocontroller branch from 3125665 to 3e5a156 Compare May 29, 2023 08:39
@wy65701436
Copy link
Contributor

thanks @lengrongfu for your contribution, please add description of the PR according to this.

Copy link
Member

@chlins chlins left a comment

Choose a reason for hiding this comment

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

The change looks good to me, but please complete the description for this PR, then we can move forward it to merge, thanks.

@lengrongfu lengrongfu changed the title 【UT】add unit test for system collector test 【UT】add unit test for collector system info Jun 2, 2023
@lengrongfu
Copy link
Contributor Author

@wy65701436 @chlins add this pr description, PTAL.

@chlins chlins force-pushed the test/add_test_systeminfocontroller branch from 32060c6 to 2e045e8 Compare June 8, 2023 01:01
Copy link
Member

@chlins chlins left a comment

Choose a reason for hiding this comment

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

lgtm

@stonezdj stonezdj merged commit 8251fd2 into goharbor:main Jun 8, 2023
WilfredAlmeida pushed a commit to WilfredAlmeida/harbor that referenced this pull request Jul 8, 2023
add unit test for system collector test

Signed-off-by: lengrongfu <[email protected]>
Signed-off-by: Wilfred Almeida <[email protected]>
WilfredAlmeida pushed a commit to WilfredAlmeida/harbor that referenced this pull request Jul 8, 2023
add unit test for system collector test

Signed-off-by: lengrongfu <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note/enhancement Label to mark PR to be added under release notes as enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants