[SDK-3231] Added support for multiple checks on a single claim

[poovamraj]

Changes

• We have added option where multiple claim checks can be done for same claim.
• This causes breaking change to the existing behaviour where only one check can be done over a claim.
• This also overrides the behaviour where if null is passed to a claim check, the check will get removed. Instead, now it will check for null value for that claim

Before this code removes the check for claim with 123
After the code checks for claim to have value 123 and then checks for value null

JWT.require(Algorithm.HMAC256("secret"))
                .withClaim("claim", 123)
                .withClaim("claim", (Integer) null)
                .build();

Before this code overrides the check for claim with check1 to check2
After the code checks for claim to have value check1 and check2

JWT.require(Algorithm.HMAC256("secret"))
                .withClaim("claim", "check1")
                .withClaim("claim", "check2")
                .build();

Before this code doesn't check for the claim value
After the code checks for claim to have null

JWT.require(Algorithm.HMAC256("secret"))
                .withClaim("claim", (Integer) null)
                .build();

References

#558

Testing

• We have added tests to ensure null values are checked properly in the claim

• We have tested that multiple claim checks are added

• Improved branch coverage to ~98%

• This change adds test coverage

• This change has been tested on the latest version of Java or why not

[jimmyjames]

👍