[visheshupadhyay]

Fixes #301

[dxnnystyles]

nice addition with the early null guard! one case missing from the new tests is verifyChain(null, ...). since this changes a NullPointerException into INVALID_CHAIN_LENGTH, an assertion for that exact input would help protect the new behavior :)

[visheshupadhyay]

Thanks for catching that, @dxnnystyles. I’ve added explicit coverage for verifyChain(null, ...) to verify that it returns INVALID_CHAIN_LENGTH rather than throwing a NullPointerException.

I also ran the focused ChainVerifierTest suite successfully.

[dxnnystyles]

thanks for adding this! I checked the updated test, and it now covers verifyChain(null, ...) with the expected INVALID_CHAIN_LENGTH status. the early rejection also prevents certificate parsing for invalid chain lengths. looks good!