Add tests for MethodWrapper

[Lemeri123]

With this, the test coverage go above 60%

Successfully closes #81

[sonarqubecloud]

Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[Lemeri123]

@josdem help show me how I can refactor the lambda expression correctly

[josdem]

@josdem help show me how I can refactor the lambda expression correctly

Hi @Lemeri123 , the fix is much easier, here are some things you should notice:

1. MethodWrapper is an old class and it is not longer used in the code, therefore you can delete it.
2. With that said the previous test is not required
3. Finally seems like you do not have the latest version of the code, please notice these clases do not exist in the package dnd:

• TransparencyManager
• Jdk6u10TransparencyManager
• NullTransparencyManager

Let me know if any questions.

[Lemeri123]

Wait so meaning even the tests for the method wrapper isnt needed since the method wrapper is no longer going to be in use??

[josdem]

Wait so meaning even the tests for the method wrapper isnt needed since the method wrapper is no longer going to be in use??

Correct, seems like your branch does not have the latest changes from the main branch. Currently, we do not have any MethodWrapperusage in the project, therefore is perfectly right just to delete it. Good news, that will increase the coverage anyways in the util package.

[Lemeri123]

Ohh okay let me close this one then