-
Notifications
You must be signed in to change notification settings - Fork 25
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
Unit testing utilities #35
Comments
Hey @eladb can I work on this issue? I'm really interested in contributing to this project. |
Yes, absolutely. You can find inspiration in the how the AWS CDK testing works and the @aws-cdk/assert module. I was wondering if perhaps we should actually write this as a general purpose JSON assertion library instead of cdk8s specific. Would you be able to write a quick RFC that plots your plan for this so we will make sure we are on the same page? |
Is anyone can share detail implementation direction we can go? this feature and CDK pipeline feature can make difference between the helm and programming language. I am join to contriube on it. |
This issue is now marked as stale because it hasn't seen activity for a while. Add a comment or it will be closed soon. |
This issue is now marked as stale because it hasn't seen activity for a while. Add a comment or it will be closed soon. |
Closing this issue as it hasn't seen activity for a while. Please add a comment @mentioning a maintainer to reopen. |
keep |
This issue is now marked as stale because it hasn't seen activity for a while. Add a comment or it will be closed soon. If you wish to exclude this issue from being marked as stale, add the "backlog" label. |
keep |
We would like to offer utilities for unit testing in the
cdk8s.Testing
class.Here are some examples of useful assertions:
apiVersion
, a subset of properties (or exact match)These functions should be testing-framework agnostic (and naturally jsii-able) so they can be used in all languages. If people want to create wrappers/plugins for specific frameworks, it should be easy to wrap these utilities.
The text was updated successfully, but these errors were encountered: