-
Notifications
You must be signed in to change notification settings - Fork 6
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
Java SDK tests against examples from the spec repo #63
Conversation
@@ -1,22 +1,42 @@ | |||
package dev.cdevents.constants; | |||
|
|||
import dev.cdevents.events.*; |
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.
Does this pass linting?
Maybe a comment on why the import ...* i.e we need all the events
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.
Linting doesn't stop this
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.
Seems fine to me
createdEvent.setSource(URI.create("/event/source/123")); | ||
createdEvent.setSubjectId("pkg:golang/mygit.com/myorg/myapp@234fd47e07d1004f0aed9c"); | ||
createdEvent.setSubjectSource(URI.create("/event/source/123")); | ||
createdEvent.setSubjectChangeId("myChange123"); | ||
createdEvent.setSubjectChangeSource("my-git.example/an-org/a-repo"); |
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.
Except for this section, these tests can also be generated or generalised eventually
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.
Will do a refactor of these tests in a separate PR
cf1a64c
to
ef4aeeb
Compare
update unit testing for new events add unit testing for CDEventClass generated for each schema file Signed-off-by: Jalander Ramagiri <jalander.ramagiri@est.tech>
Signed-off-by: Jalander Ramagiri <jalander.ramagiri@est.tech>
Signed-off-by: Jalander Ramagiri <jalander.ramagiri@est.tech>
ef4aeeb
to
f7f24f4
Compare
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.
Thank you for this! lgtm
This PR includes the changes to,