-
Notifications
You must be signed in to change notification settings - Fork 30
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
🌱 Add File and RuleSet API tests #410
Conversation
Moving File and RuleSet-related functions from addon package to binding package and adding basic API tests for both. Related to konveyor/go-konveyor-tests#2 Signed-off-by: Marek Aufart <maufart@redhat.com>
Signed-off-by: Marek Aufart <maufart@redhat.com>
addon/adapter.go
Outdated
// | ||
// Setting client in binding package via setter method. | ||
adapter.File.SetClient(client) | ||
adapter.RuleSet.SetClient(client) |
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.
Since client
is not available outside the package (binding in this case), I added SetClient methods to allow set the client from addon package. Expecting its deletion when addon is migrated to binding.
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.
In the binding package, we could just make the client
in all of the endpoint handlers public instead?
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.
Sure, we can, will update.
Signed-off-by: Marek Aufart <maufart@redhat.com>
Signed-off-by: Marek Aufart <maufart@redhat.com>
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.
LGTM
Moving File and RuleSet-related functions from addon package to binding package and adding basic API tests for both.
Related to konveyor/go-konveyor-tests#2