add CheckSemVer, CheckRootfsPath, CheckPlatform unit test

Signed-off-by: liangchenye <liangchenye@huawei.com>

Author: ababyshark

.travis.yml

@@ -11,8 +11,7 @@ before_install:
 install: true
 
 script:
-  - go vet -x ./...
-  - $HOME/gopath/bin/golint ./...
   - $HOME/gopath/bin/git-validation -run DCO,short-subject -v -range ${TRAVIS_COMMIT_RANGE}
-  - make
+  - make test
+  - make all
 

Makefile

@@ -37,7 +37,7 @@ $(RUNTIME_TOOLS_LINK):
 
 .PHONY: test .gofmt .govet .golint
 
-test: .gofmt .govet .golint
+test: .gofmt .govet .golint .gotest
 
 .gofmt:
 	go fmt ./...
@@ -48,3 +48,5 @@ test: .gofmt .govet .golint
 .golint:
 	golint ./...
 
+.gotest:
+	go test -v ./...

validate/validate_test.go

@@ -0,0 +1,82 @@
+package validate
+
+import (
+	"io/ioutil"
+	"os"
+	"path/filepath"
+	"strings"
+	"testing"
+
+	rspec "github.com/opencontainers/runtime-spec/specs-go"
+)
+
+func checkErrors(t *testing.T, title string, msgs []string, valid bool) {
+	if valid && len(msgs) > 0 {
+		t.Fatalf("%s: expected not to get error, but get %d errors:\n%s", title, len(msgs), strings.Join(msgs, "\n"))
+	} else if !valid && len(msgs) == 0 {
+		t.Fatalf("%s: expected to get error, but actually not", title)
+	}
+}
+
+func TestCheckRootfsPath(t *testing.T) {
+	tmpBundle, err := ioutil.TempDir("", "oci-check-rootfspath")
+	if err != nil {
+		t.Fatalf("Failed to create a TempDir in 'CheckRootfsPath'")
+	}
+	defer os.RemoveAll(tmpBundle)
+
+	rootfsDir := "rootfs"
+	rootfsNonDir := "rootfsfile"
+	rootfsNonExists := "rootfsnil"
+	os.MkdirAll(filepath.Join(tmpBundle, rootfsDir), 0700)
+	os.Create(filepath.Join(tmpBundle, rootfsNonDir))
+
+	cases := []struct {
+		val      string
+		expected bool
+	}{
+		{rootfsDir, true},
+		{rootfsNonDir, false},
+		{rootfsNonExists, false},
+		{filepath.Join(tmpBundle, rootfsDir), true},
+		{filepath.Join(tmpBundle, rootfsNonDir), false},
+		{filepath.Join(tmpBundle, rootfsNonExists), false},
+	}
+	for _, c := range cases {
+		v := NewValidator(&rspec.Spec{Root: rspec.Root{Path: c.val}}, tmpBundle, false)
+		checkErrors(t, "CheckRootfsPath "+c.val, v.CheckRootfsPath(), c.expected)
+	}
+}
+
+func TestCheckSemVer(t *testing.T) {
+	cases := []struct {
+		val      string
+		expected bool
+	}{
+		{rspec.Version, true},
+		//FIXME: validate currently only handles rpsec.Version
+		{"0.0.1", false},
+		{"invalid", false},
+	}
+
+	for _, c := range cases {
+		v := NewValidator(&rspec.Spec{Version: c.val}, "", false)
+		checkErrors(t, "CheckSemVer "+c.val, v.CheckSemVer(), c.expected)
+	}
+}
+
+func TestCheckPlatform(t *testing.T) {
+	cases := []struct {
+		val      rspec.Platform
+		expected bool
+	}{
+		{rspec.Platform{OS: "darwin", Arch: "arm64"}, true},
+		{rspec.Platform{OS: "hurd", Arch: "arm64"}, false},
+		{rspec.Platform{OS: "darwin", Arch: "ppc64"}, false},
+	}
+
+	for _, c := range cases {
+		v := NewValidator(&rspec.Spec{Platform: c.val}, "", false)
+		checkErrors(t, "CheckPlatform "+c.val.OS+" "+c.val.Arch, v.CheckPlatform(), c.expected)
+	}
+}