Unit Tests for memoryLimit override feature

[danielrsfreitas]

Purpose of this PR

This PR includes Unit Tests for the features developed in #2383.

Proposed changes:

• Unit test for convertJavaMemoryStringToK8sMemoryString
• Unit test for addMemoryLimit

Change Category

• Bugfix (non-breaking change which fixes an issue)
• Feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that could affect existing functionality)
• Documentation update

Rationale

The most upvoted issue in the SparkOperator repository relates to having the ability to define different memory and memoryLimits values. The feature has been covered in #2383. I am adding unit tests to it to increase the trust in the code.

Checklist

• I have conducted a self-review of my own code.
• I have updated documentation accordingly.
• I have added tests that prove my changes are effective or that my feature works.
• Existing unit tests pass locally with my changes.

Additional Notes

[google-oss-prow]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign mwielgus for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

• OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment