-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
EUDI Interop #41
EUDI Interop #41
Conversation
This allows callers to request different credential schemes with the same instance of the class
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am unaware of any reals tests, to verify whether equality checks have as intended. If i overlooked them and we do, in fact have regression rests, please ignore my comments, otherwise, please provide tests
) { | ||
|
||
fun serialize() = jsonSerializer.encodeToString(this) | ||
|
||
override fun equals(other: Any?): Boolean { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
where are the regression tests? Serious question here: do we know equality checks still work?
issuer = issuer, | ||
issuanceDate = issuanceDate, | ||
expirationDate = issuanceDate + lifetime, | ||
credentialStatus = null, | ||
credentialSubject = credentialSubject, | ||
) | ||
|
||
override fun equals(other: Any?): Boolean { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
again: do we know equality checks still work?
@@ -38,40 +38,6 @@ data class VerifiableCredentialSdJwt( | |||
|
|||
fun serialize() = jsonSerializer.encodeToString(this) | |||
|
|||
override fun equals(other: Any?): Boolean { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same same
@@ -30,25 +30,4 @@ data class VerifiablePresentation( | |||
audience = audienceId, | |||
jwtId = id | |||
) | |||
|
|||
override fun equals(other: Any?): Boolean { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same same
val revokedVerifiableCredentials: Array<VerifiableCredentialJws> = arrayOf(), | ||
val invalidVerifiableCredentials: Array<String> = arrayOf(), | ||
) { | ||
override fun equals(other: Any?): Boolean { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same same
@@ -10,20 +10,5 @@ import kotlinx.serialization.Serializable | |||
@Serializable | |||
data class FormatContainerLdp( | |||
@SerialName("proof_type") | |||
val proofType: Array<String>, | |||
) { | |||
override fun equals(other: Any?): Boolean { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same same
constraints = constraints, | ||
) | ||
|
||
override fun equals(other: Any?): Boolean { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same same
formats: FormatHolder | ||
) : this(id = uuid4().toString(), inputDescriptors = inputDescriptors, formats = formats) | ||
|
||
fun serialize() = jsonSerializer.encodeToString(this) | ||
|
||
override fun equals(other: Any?): Boolean { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same same
@@ -14,28 +14,5 @@ data class PresentationSubmission( | |||
@SerialName("definition_id") | |||
val definitionId: String, | |||
@SerialName("descriptor_map") | |||
val descriptorMap: Array<PresentationSubmissionDescriptor>? = null, | |||
) { | |||
override fun equals(other: Any?): Boolean { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same same
@@ -24,38 +24,5 @@ data class SubmissionRequirement( | |||
@SerialName("from") | |||
val from: String? = null, | |||
@SerialName("from_nested") | |||
val fromNested: Array<SubmissionRequirement>? = null, | |||
) { | |||
override fun equals(other: Any?): Boolean { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same same
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am adding an issue wrt. equality tests. Hence, I am leaving my comments here for reference, just that we won't miss anything.
I everything else looks good
Work on interop with EUDI credentials