fix(kubernetes): [CKV_K8S_140] Update ApiServerTlsCertAndKey.py to check RHS values

[nijdarshan]

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Description

The earlier code was only checking if the -tls-cert-file and --tls-private-key-file parameters are present but was not checking if the values are provided on the RHS.

Fixes

Optimised CKV_K8S_140 to check for the values and non-empty values on the RHS of the parameters.

Checklist:

• My code follows the style guidelines of this project
• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• I have added tests that prove my feature, policy, or fix is effective and works
• New and existing tests pass locally with my changes
• Any dependent changes have been merged and published in downstream modules

[gruebel]

nice catch 🚀

[gruebel]

nice, looks good 🚀 can you also add one failing test, where one of the flags is used, but without a value?

[nijdarshan]

Added a new fail check for the update

[nijdarshan]

nice, looks good 🚀 can you also add one failing test, where one of the flags is used, but without a value?

Hey, @gruebel could help me identify why the code fails in a few tests? I will rectify them accordingly!

[gruebel]

quite easy, if you add a new test case it also needs to be added to the actual test file. Additionally the object names should be unique, so we can be sure, the correct ones are failing/passing.