Aalekh Jain requested to merge improve-code-coverage into master Sep 24, 2020

All Submissions:

[YES] I have added an explanation of what changes in this merge do and why we should include it?
[NO] I have updated the documentation accordingly.
[YES] I have added tests to cover my changes.
[YES] All new and existing tests passed.
[YES] My code follows the code style of this project.
[NO] I ran lint checks locally prior to submission.

What is the current behavior?

No Unit Tests and zero code coverage for legal-azure.

What is the new behavior?

Added Unit Tests to improve code coverage for legal-azure

Does this introduce a breaking change?

[NO]

cc: @kibattul @polavishnu @amaverma

Note

Code coverage excluding the Config classes is above 80%. This can be merged.

To exclude the Config classes from jacoco code coverage reports, add the following configuration in pom.xml for Jacoco plugin.

<configuration>
    <excludes>
        <exclude>org/opengroup/osdu/legal/azure/*/*Config*</exclude>
    </excludes>
</configuration>

Code coverage report -

Edited Nov 04, 2020 by Aalekh Jain