-
Notifications
You must be signed in to change notification settings - Fork 676
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
Create golden standards to test against json schema #34
Comments
+1 on that. It would be nice to have it test both the markdown embedded and examples but that is more of a nice to have. |
So we could add the files from the embedded markdown examples as files, and have a test target against them. I'm not really sure of a clean way to work with the examples while they're embedded in the markdown though. |
On Sat, Apr 16, 2016 at 07:57:13AM -0700, Vincent Batts wrote:
Probably just make sure all embedded examples are subsets of the |
@stevvooe Are you going to work on this? |
I can look into it, but I'm not very familiar with JSONSchema. If anyone could suggest tools or an approach, that would be helpful. I'll probably just use a Go package and a Makefile, barring common tools above. |
On Wed, Apr 20, 2016 at 03:56:44PM -0700, Stephen Day wrote:
#18 and #32 already landed the JSON Schema and Go tooling for You will need to add the example JSON manifests, etc. that are used to |
@wking Thanks! validate.go should be sufficient to get started. Do we want to consider the specification to be the source of all golden standards? |
I like it. On Thu, Apr 21, 2016 at 8:42 PM, Stephen Day notifications@github.com
|
Also, it would dictate that a new item for the On Fri, Apr 22, 2016 at 10:39 AM, Vincent Batts vbatts@hashbangbash.com
|
It's covered this since opencontainers/go-digest@4ca13015 (disallow upper characters (/A-F/) in hex-encoded portion, 2017-05-12, opencontainers#34), so we don't need to do it locally. Also, use a more specific check for ignoring unrecognized algorithms, because we don't want to return nil when go-digest notices an invalid encoding, etc. Signed-off-by: W. Trevor King <wking@tremily.us>
It's covered this since opencontainers/go-digest@4ca13015 (disallow upper characters (/A-F/) in hex-encoded portion, 2017-05-12, opencontainers#34), so we don't need to do it locally. Also, use a more specific check for ignoring unrecognized algorithms, because we don't want to return nil when go-digest notices an invalid encoding, etc. Signed-off-by: W. Trevor King <wking@tremily.us>
We need a few "golden standards" to test against the json schema specs to catch unintentional changes. It will also help in having them as part of PRs that show the schema change and how it affects the "golden standard".
@vbatts Thoughts?
The text was updated successfully, but these errors were encountered: