Add unit tests for builders #543

[onewhl]

Description

Added unit tests for SimpleNameBuilder, SimpleCommentBuilder, SimpleClusterCommentBuilder.
Unfortunately, most of the methods are private and kotlin properties not always could be mocked, so the number of tests is small.

Fixes#543

Type of Change

Please delete options that are not relevant.

• New tests

How Has This Been Tested?

Automated Testing

Run tests in PR.

Checklist (remove irrelevant options):

This is the author self-check list

• The change followed the style guidelines of the UTBot project
• Self-review of the code is passed
• The change contains enough commentaries, particularly in hard-to-understand areas
• New documentation is provided or existed one is altered
• No new warnings
• New tests have been added
• All tests pass locally with my changes

[amandelpie]

It's a good question, should we test private methods or not, I suggest to use reflection or change the visibility of a few methods to the package-private or internal and test private methods without mockito but after moving from Soot to JCDB

If you wish, you could play with changing visibility at this PR for 1-2 method for the SimpleNameBuilder

Please add a few tests if possible, fix the broken test and fix the issue with the mockito verision

[amandelpie]

This test fails on my side with

expected: <<pre>
Test throws Throwable </pre>> but was: < >

[onewhl]

i didn't expect these two builders have different behavior, but now we know. I fixed test

[amandelpie]

@onewhl LGTM for me