Skip to content
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

examples: validate markdown examples #41

Merged
merged 2 commits into from
Apr 29, 2016
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
22 changes: 22 additions & 0 deletions Makefile
Original file line number Diff line number Diff line change
@@ -0,0 +1,22 @@

default: help

help:
@echo "Usage: make <target>"
@echo
@echo " * 'fmt' - format the json with indentation"
@echo " * 'validate' - build the validation tool"

fmt:
for i in *.json ; do jq --indent 2 -M . "$${i}" > xx && cat xx > "$${i}" && rm xx ; done

.PHONY: validate-examples
validate-examples: oci-validate-examples
./oci-validate-examples < manifest.md

oci-validate-json: validate.go
go build ./cmd/oci-validate-json

oci-validate-examples: cmd/oci-validate-examples/main.go
go build ./cmd/oci-validate-examples

208 changes: 208 additions & 0 deletions cmd/oci-validate-examples/main.go
Original file line number Diff line number Diff line change
@@ -0,0 +1,208 @@
package main

import (
"bytes"
"errors"
"flag"
"fmt"
"io"
"io/ioutil"
"log"
"net/url"
"os"
"path/filepath"
"strings"

"github.com/russross/blackfriday"
"github.com/xeipuuv/gojsonschema"
)

var (
verbose bool
skipped bool
schemaPath string
)

func init() {
flag.BoolVar(&verbose, "verbose", false, "display examples no matter what")
flag.BoolVar(&skipped, "skipped", false, "show skipped examples")
flag.StringVar(&schemaPath, "schema", "./schema", "specify location of schema directory")
}

func main() {
flag.Parse()

examples, err := extractExamples(os.Stdin)
if err != nil {
log.Fatalln(err)
}
var fail bool
for _, example := range examples {
if example.Err != nil {
printFields("error", example.Mediatype, example.Title, example.Err)
fail = true
continue
}

schema, err := schemaByMediatype(schemaPath, example.Mediatype)
if err != nil {
if err == errSchemaNotFound {
if skipped {
printFields("skip", example.Mediatype, example.Title)

if verbose {
fmt.Println(example.Body, "---")
}
}
continue
}
}

// BUG(stevvooe): Recursive validation is not working. Need to
// investigate. Will use this code as information for bug.
document := gojsonschema.NewStringLoader(example.Body)
result, err := gojsonschema.Validate(schema, document)

if err != nil {
printFields("error", example.Mediatype, example.Title, err)
fmt.Println(example.Body, "---")
fail = true
continue
}

if !result.Valid() {
// TOOD(stevvooe): This is nearly useless without file, line no.
printFields("invalid", example.Mediatype, example.Title)
for _, desc := range result.Errors() {
printFields("reason", example.Mediatype, example.Title, desc)
}
fmt.Println(example.Body, "---")
fail = true
continue
}

printFields("ok", example.Mediatype, example.Title)
if verbose {
fmt.Println(example.Body, "---")
}
}

if fail {
os.Exit(1)
}
}

var (
specsByMediaType = map[string]string{
"application/vnd.oci.image.manifest.v1+json": "image-manifest-schema.json",
"application/vnd.oci.image.manifest.list.v1+json": "manifest-list-schema.json",
}

errSchemaNotFound = errors.New("schema: not found")
errFormatInvalid = errors.New("format: invalid")
)

func schemaByMediatype(root, mediatype string) (gojsonschema.JSONLoader, error) {
name, ok := specsByMediaType[mediatype]
if !ok {
return nil, errSchemaNotFound
}

if !filepath.IsAbs(root) {
wd, err := os.Getwd()
if err != nil {
return nil, err
}
root = filepath.Join(wd, root)
}

// lookup path
path := filepath.Join(root, name)
return gojsonschema.NewReferenceLoader("file://" + path), nil
}

// renderer allows one to incercept fenced blocks in markdown documents.
type renderer struct {
blackfriday.Renderer
fn func(text []byte, lang string)
}

func (r *renderer) BlockCode(out *bytes.Buffer, text []byte, lang string) {
r.fn(text, lang)
r.Renderer.BlockCode(out, text, lang)
}

type example struct {
Lang string // gets raw "lang" field
Title string
Mediatype string
Body string
Err error

// TODO(stevvooe): Figure out how to keep track of revision, file, line so
// that we can trace back verification output.
}

// parseExample treats the field as a syntax,attribute tuple separated by a comma.
// Attributes are encoded as a url values.
//
// An example of this is `json,title=Foo%20Bar&mediatype=application/json. We
// get that the "lang" is json, the title is "Foo Bar" and the mediatype is
// "application/json".
//
// This preserves syntax highlighting and lets us tag examples with further
// metadata.
func parseExample(lang, body string) (e example) {
e.Lang = lang
e.Body = body

parts := strings.SplitN(lang, ",", 2)
if len(parts) < 2 {
e.Err = errFormatInvalid
return
}

m, err := url.ParseQuery(parts[1])
if err != nil {
e.Err = err
return
}

e.Mediatype = m.Get("mediatype")
e.Title = m.Get("title")
return
}

func extractExamples(rd io.Reader) ([]example, error) {
p, err := ioutil.ReadAll(rd)
if err != nil {
return nil, err
}

var examples []example
renderer := &renderer{
Renderer: blackfriday.HtmlRenderer(0, "test test", ""),
fn: func(text []byte, lang string) {
examples = append(examples, parseExample(lang, string(text)))
},
}

// just pass over the markdown and ignore the rendered result. We just want
// the side-effect of calling back for each code block.
// TODO(stevvooe): Consider just parsing these with a scanner. It will be
// faster and we can retain file, line no.
blackfriday.MarkdownOptions(p, renderer, blackfriday.Options{
Extensions: blackfriday.EXTENSION_FENCED_CODE,
})

return examples, nil
}

// printFields prints each value tab separated.
func printFields(vs ...interface{}) {
var ss []string
for _, f := range vs {
ss = append(ss, fmt.Sprint(f))
}
fmt.Println(strings.Join(ss, "\t"))
}
File renamed without changes.
4 changes: 2 additions & 2 deletions manifest.md
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,7 @@ A client will distinguish a manifest list from an image manifest based on the Co
## Example Manifest List

*Example showing a simple manifest list pointing to image manifests for two platforms:*
```json
```json,title=Manifest%20List&mediatype=application/vnd.oci.image.manifest.list.v1%2Bjson
{
"schemaVersion": 2,
"mediaType": "application/vnd.oci.image.manifest.list.v1+json",
Expand Down Expand Up @@ -185,7 +185,7 @@ The image manifest provides a configuration and a set of layers for a container
## Example Image Manifest

*Example showing an image manifest:*
```json
```json,title=Manifest&mediatype=application/vnd.oci.image.manifest.v1%2Bjson
{
"schemaVersion": 2,
"mediaType": "application/vnd.oci.image.manifest.v1+json",
Expand Down
15 changes: 0 additions & 15 deletions schema/Makefile

This file was deleted.