Add test case for nullSafeToString with Double array

[skonda33]

Add test case for nullSafeToString with Double array

I hereby agree to the terms of the JUnit Contributor License Agreement.

---

Definition of Done

• There are no TODOs left in the code
• Method preconditions are checked and documented in the method's Javadoc
• Coding conventions (e.g. for logging) have been followed
• Change is covered by automated tests including corner cases, errors, and exception handling
• Public API has Javadoc and @API annotations
• Change is documented in the User Guide and Release Notes

[sbrannen]

Hi @skonda33,

Thanks for the PR.

Unfortunately, I don't think this adds much benefit.

The nullSafeToStringChecks() test method is not intended to cover an exhaustive set of use cases. Rather, it's only intended to check a minimal subset of use cases.

For example, the assertion for nullSafeToString(new Integer[] { 34, 42 }) is there to ensure that a one-dimensional array for a wrapper (Integer, Double, Float, etc.) is supported.

In light of that, I don't think we need an additional assertion for a Double array, and I'm closing this PR.

In the future, I suggest that you ask the team if certain kinds of tests or assertions are desired before spending time on creating and submitting a PR. For example, you could create an issue to discuss your proposal or start a discussion.

Cheers,

Sam