FEATURE: Add NodeIdentities Role Attribute

[marietta-a]

Description

• Created NodeIdentity class
• Added NodeIdentities property to Token and Role classes
• Added Node Identities to the Constructor of Token and Role classes
• Created Role and Token tests for Node Identities

Related Issues

Addresses issue #531

Additional Context

Checklist

Please make sure to check the following before submitting your pull request:

• Did you read the contributing guidelines?
• Did you update the docs?
• Did you write any tests to validate this change?

[marietta-a]

Hi @marcin-krystianc

[marcin-krystianc]

Hi @marietta-a,

The v1.17.14 tests are failing. The reason is that we explicitly intend to skip serializing NodeIdentities when they are null, but elsewhere we default-initialize them to an empty value.

I think the initialization in the constructor is unnecessary, and the fix for the CI failures should be to revert the Token and Role constructor changes entirely.

[marietta-a]

Noted with thanks

[marietta-a]

Hi @marcin-krystianc, I removed NodeIdentities initialization from the constructor

[marcin-krystianc]

We just need to update the cut-off version, looks good otherwise!

[marcin-krystianc]

@marietta-a Awesome, thank you!