Adding JUnit. Fixes #3874

[ayush0407]

What's the purpose of this PR

This PR is for JUnit fix and increased coverage and logic test.

XXXXX

Which issue(s) this PR fixes:

Fixes #3874

Brief changelog

XXXXX

Follow this checklist to help us incorporate your contribution quickly and easily:

• Read the Contributing Guide before making this pull request.
• Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
• Write necessary unit tests to verify the code.
• Run mvn clean test to make sure this pull request doesn't break anything.
• Update the CHANGES log.

[nobodyiam]

Thanks for the contribution, would you please follow the code style to format these 2 files?

BTW, the new file need to have the copyright comment in its header.

[codecov-commenter]

Codecov Report

Merging #4271 (5032560) into master (672320f) will increase coverage by 0.02%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #4271      +/-   ##
============================================
+ Coverage     52.56%   52.58%   +0.02%     
- Complexity     2635     2641       +6     
============================================
  Files           488      488              
  Lines         15253    15253              
  Branches       1577     1577              
============================================
+ Hits           8017     8021       +4     
+ Misses         6682     6681       -1     
+ Partials        554      551       -3     

Impacted Files	Coverage Δ
...ervice/service/ReleaseMessageServiceWithCache.java	85.88% <0.00%> (-1.18%)	⬇️
...ework/apollo/spring/util/BeanRegistrationUtil.java	91.66% <0.00%> (+4.16%)	⬆️
...k/apollo/biz/utils/ConfigChangeContentBuilder.java	100.00% <0.00%> (+10.52%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 672320f...5032560. Read the comment docs.

[ayush0407]

Thanks for the contribution, would you please follow the code style to format these 2 files?

BTW, the new file need to have the copyright comment in its header.

Sure, I'll fix that and check in the code.

[ayush0407]

Hi there, I added the copyright statement in the new file but can you please help me with the code format. I didn't get that part?

[nobodyiam]

@ayush0407

According to the Code Conventions, you need to import the code style files into your IDE, and use that to format your code changes.
If you don't use any IDE, then you may manually format the code changes, e.g. indent size is 2, tab size is 2, use space to replace tab, etc.

dismiss

[zouyx]

Fix test case pls

[ayush0407]

Fix test case pls

Which one specifically? Is any of the test breaking?

[nobodyiam]

LGTM