fix(cli): check run sources exists
* Changed the name of run.isLocal func to run.isLocalAndExists as the function does both things
* Added a default switch case to report either the file doesn't exist or the URI scheme cannot be parsed
* Added some unit test to verify error messages expected
Fixes #1911
diff --git a/pkg/cmd/run.go b/pkg/cmd/run.go
index 5904272..695f01f 100644
--- a/pkg/cmd/run.go
+++ b/pkg/cmd/run.go
@@ -380,7 +380,7 @@
files = append(files, o.OpenAPIs...)
for _, s := range files {
- if isLocal(s) {
+ if isLocalAndFileExists(s) {
changes, err := sync.File(o.Context, s)
if err != nil {
return err
@@ -645,7 +645,7 @@
return nil
}
-func isLocal(fileName string) bool {
+func isLocalAndFileExists(fileName string) bool {
info, err := os.Stat(fileName)
if os.IsNotExist(err) {
return false
diff --git a/pkg/cmd/run_test.go b/pkg/cmd/run_test.go
index 9db4193..ce9527f 100644
--- a/pkg/cmd/run_test.go
+++ b/pkg/cmd/run_test.go
@@ -416,6 +416,23 @@
assert.NotNil(t, err)
}
+func TestRunValidateArgs(t *testing.T) {
+ runCmdOptions, rootCmd, _ := initializeRunCmdOptions(t)
+ args := []string{}
+ err := runCmdOptions.validateArgs(rootCmd, args)
+ assert.NotNil(t, err)
+ assert.Equal(t, "run expects at least 1 argument, received 0", err.Error())
+
+ args = []string{"run_test.go"}
+ err = runCmdOptions.validateArgs(rootCmd, args)
+ assert.Nil(t, err)
+
+ args = []string{"missing_file"}
+ err = runCmdOptions.validateArgs(rootCmd, args)
+ assert.NotNil(t, err)
+ assert.Equal(t, "One of the provided sources is not reachable: Missing file or unsupported scheme in missing_file", err.Error())
+}
+
//
// This test does work when running as single test but fails
// otherwise as we are using a global viper instance
diff --git a/pkg/cmd/util_content.go b/pkg/cmd/util_content.go
index 75d9c08..c210870 100644
--- a/pkg/cmd/util_content.go
+++ b/pkg/cmd/util_content.go
@@ -30,7 +30,7 @@
var content []byte
var err error
- if isLocal(source) {
+ if isLocalAndFileExists(source) {
content, err = ioutil.ReadFile(source)
} else {
u, err := url.Parse(source)
diff --git a/pkg/cmd/util_sources.go b/pkg/cmd/util_sources.go
index 50a0838..f539705 100644
--- a/pkg/cmd/util_sources.go
+++ b/pkg/cmd/util_sources.go
@@ -65,7 +65,7 @@
sources := make([]Source, 0, len(locations))
for _, location := range locations {
- if isLocal(location) {
+ if isLocalAndFileExists(location) {
answer, err := ResolveLocalSource(location, compress)
if err != nil {
return sources, err
@@ -176,6 +176,8 @@
return sources, err
}
sources = append(sources, answer)
+ default:
+ return sources, fmt.Errorf("Missing file or unsupported scheme in %s", location)
}
}
}
