Raise error for missing input file (#104)

* Raise error for missing input file
* Fix testcase issue seen in test pipeline

Author: apoorvdeshmukh

cmd/sqlcmd/main.go

@@ -266,6 +266,8 @@ func run(vars *sqlcmd.Variables, args *SQLCmdArguments) (int, error) {
 	} else {
 		for f := range args.InputFile {
 			if err = s.IncludeFile(args.InputFile[f], true); err != nil {
+				_, _ = os.Stderr.Write([]byte(err.Error() + sqlcmd.SqlcmdEol))
+				s.Exitcode = 1
 				break
 			}
 		}

cmd/sqlcmd/main_test.go

@@ -310,6 +310,23 @@ func TestAzureAuth(t *testing.T) {
 	}
 }
 
+func TestMissingInputFile(t *testing.T) {
+	args = newArguments()
+	args.InputFile = []string{"testdata/missingFile.sql"}
+
+	if canTestAzureAuth() {
+		args.UseAad = true
+	}
+
+	vars := sqlcmd.InitializeVariables(!args.DisableCmdAndWarn)
+	setVars(vars, &args)
+
+	exitCode, err := run(vars, &args)
+	assert.Error(t, err, "run")
+	assert.Contains(t, err.Error(), "Error occurred while opening or operating on file", "Unexpected error: "+err.Error())
+	assert.Equal(t, 1, exitCode, "exitCode")
+}
+
 // Assuming public Azure, use AAD when SQLCMDUSER environment variable is not set
 func canTestAzureAuth() bool {
 	server := os.Getenv(sqlcmd.SQLCMDSERVER)

pkg/sqlcmd/sqlcmd.go

@@ -286,7 +286,7 @@ func (s *Sqlcmd) promptPassword() (string, error) {
 func (s *Sqlcmd) IncludeFile(path string, processAll bool) error {
 	f, err := os.Open(path)
 	if err != nil {
-		return err
+		return InvalidFileError(err, path)
 	}
 	defer f.Close()
 	b := s.batch.batchline