kubernetes-client addedIsBlank/isNotBlank

Description

Added Utils.isBlank() and Utils.isNotBlank() fixes #4057

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] Feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change
[ ] Chore (non-breaking change which doesn't affect codebase; test, version modification, documentation, etc.)

Checklist

[x] Code contributed by me aligns with current project license: Apache 2.0
[ ] I Added CHANGELOG entry regarding this change
[ ] I have implemented unit tests to cover my changes
[ ] I have added/updated the javadocs and other documentation accordingly
[ ] No new bugs, code smells, etc. in SonarCloud report
[ ] I tested my code in Kubernetes
[ ] I tested my code in OpenShift

Apr 21 '23 19:04 aditya-32

Hi @manusa @rohanKanojia can you please review the PR

May 16 '23 03:05 aditya-32

@aditya-32 : Could you please link the issue which this PR intends to fix?

May 17 '23 04:05 rohanKanojia

Hi @rohanKanojia fixes issue: #4057 updated the description

May 17 '23 10:05 aditya-32

I see. Could you please write some unit tests for this? You would also need to check and update other places where we're using == null/ != null etc checks.

May 17 '23 10:05 rohanKanojia

Hi @rohanKanojia can you please review it , i have added unit test cases and replaced instances of ==null and != null with the use of Utils.isBlank() and Utils.isNotBlank() . Also please suggest any changes if needed or any where you see i can replace null check with utils.isBlank().

May 18 '23 05:05 aditya-32

SonarCloud Quality Gate failed.