Merge pull request #507 from q384566678/validate-platform-runtimetest

Ma Shimiao · web-flow · commit 8a6d284dff53 · 2017-11-17T16:34:16.000+08:00
runtimetest: add host platform validation
diff --git a/cmd/runtimetest/main.go b/cmd/runtimetest/main.go
@@ -778,14 +778,17 @@ func run(context *cli.Context) error {
 	}
 	logrus.SetLevel(logLevel)
 
+	platform := runtime.GOOS
+	if platform != "linux" && platform != "solaris" && platform != "windows" {
+		return fmt.Errorf("runtime-tools has not implemented testing for your platform %q, because the spec has nothing to say about it", platform)
+	}
+
 	inputPath := context.String("path")
 	spec, err := loadSpecConfig(inputPath)
 	if err != nil {
 		return err
 	}
 
-	platform := runtime.GOOS
-
 	defaultValidations := []validation{
 		{
 			test:        validateRootFS,

Original file line number	Diff line number	Diff line change
`@@ -778,14 +778,17 @@ func run(context *cli.Context) error {`
`778`	`778`	`}`
`779`	`779`	`logrus.SetLevel(logLevel)`
`780`	`780`
	`781`	`+ platform := runtime.GOOS`
	`782`	`+ if platform != "linux" && platform != "solaris" && platform != "windows" {`
	`783`	`+ return fmt.Errorf("runtime-tools has not implemented testing for your platform %q, because the spec has nothing to say about it", platform)`
	`784`	`+ }`
	`785`	`+`
`781`	`786`	`inputPath := context.String("path")`
`782`	`787`	`spec, err := loadSpecConfig(inputPath)`
`783`	`788`	`if err != nil {`
`784`	`789`	`return err`
`785`	`790`	`}`
`786`	`791`
`787`		`- platform := runtime.GOOS`
`788`		`-`
`789`	`792`	`defaultValidations := []validation{`
`790`	`793`	`{`
`791`	`794`	`test: validateRootFS,`